Thanks, that looks good and its great that this could get added quickly (and I like the test!).
Unfortunately the test fails for me with:
$ PYTHONPATH=. python tests/gtk3/test_catview.py RecommendationsTestCase
...
======================================================================
ERROR: test_implicit_recommender_feedback_on_item_viewed (__main__.RecommendationsTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
File "tests/gtk3/test_catview.py", line 276, in test_implicit_recommender_feedback_on_item_viewed
self._populate_recommended_for_you_panel()
AttributeError: 'RecommendationsTestCase' object has no attribute '_populate_recommended_for_you_panel'
======================================================================
FAIL: test_recommended_for_you_spinner_display (__main__.RecommendationsTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
File "tests/gtk3/test_catview.py", line 162, in test_recommended_for_you_spinner_display
self.rec_panel.recommended_for_you_content.get_property("visible"))
AssertionError: False is not true
----------------------------------------------------------------------
Ran 10 tests in 20.199s
One of the failures is probably releated to the lp:~gary-lasker/software-center/recommender-test-fix that I merged earlier and trivial to update. I have no idea about the other one, but it could be something that
is simply a side effect of the first.
When this branch needs to get touched anyway it would be nice to add a small comment to "on_application_activated" like "# send viewed items to the recommender in addition to installed ones" or similar. For a moment I was confused when looking at the code as it looks similar to the part in _on_transaction_finished (i.e. the difference of the two blocks is only visible when looking at the last part that has either "RecommenderFeedbackActions.INSTALLED" or RecommenderFeedbackActions.VIEWED). Or maybe extracting it into a common function?
Thanks, that looks good and its great that this could get added quickly (and I like the test!).
Unfortunately the test fails for me with: test_catview. py Recommendations TestCase ======= ======= ======= ======= ======= ======= ======= ======= ======= recommender_ feedback_ on_item_ viewed (__main_ _.Recommendatio nsTestCase) ------- ------- ------- ------- ------- ------- ------- ------- ------- gtk3/test_ catview. py", line 276, in test_implicit_ recommender_ feedback_ on_item_ viewed _populate_ recommended_ for_you_ panel() sTestCase' object has no attribute '_populate_ recommended_ for_you_ panel'
$ PYTHONPATH=. python tests/gtk3/
...
=======
ERROR: test_implicit_
-------
Traceback (most recent call last):
File "tests/
self.
AttributeError: 'Recommendation
======= ======= ======= ======= ======= ======= ======= ======= ======= ======= d_for_you_ spinner_ display (__main_ _.Recommendatio nsTestCase) ------- ------- ------- ------- ------- ------- ------- ------- ------- gtk3/test_ catview. py", line 162, in test_recommende d_for_you_ spinner_ display rec_panel. recommended_ for_you_ content. get_property( "visible" ))
FAIL: test_recommende
-------
Traceback (most recent call last):
File "tests/
self.
AssertionError: False is not true
------- ------- ------- ------- ------- ------- ------- ------- ------- -------
Ran 10 tests in 20.199s
One of the failures is probably releated to the lp:~gary-lasker/software-center/recommender-test-fix that I merged earlier and trivial to update. I have no idea about the other one, but it could be something that
is simply a side effect of the first.
When this branch needs to get touched anyway it would be nice to add a small comment to "on_application _activated" like "# send viewed items to the recommender in addition to installed ones" or similar. For a moment I was confused when looking at the code as it looks similar to the part in _on_transaction _finished (i.e. the difference of the two blocks is only visible when looking at the last part that has either "RecommenderFee dbackActions. INSTALLED" or RecommenderFeed backActions. VIEWED) . Or maybe extracting it into a common function?