lightfm icon indicating copy to clipboard operation
lightfm copied to clipboard

Add new unit test for type checking logic

Open markdouthwaite opened this issue 3 years ago • 0 comments

Hey again!

In response @maciejkula's request on #588 I've added a test to check all combinations of valid input types for user_ids and item_ids (i.e. tuple, list, ndarray) pass successfully.

To do this, I've added a conftest.py file that includes some handy fixtures for the new test_predict_user_item_inputs_with_valid_types test.

As an aside, I think some of these fixtures might be useful in other tests (not addressed in this PR) too. I'd be happy to look into how existing tests could be streamlined if that would be useful?

markdouthwaite avatar Feb 08 '21 19:02 markdouthwaite