-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#836 Test categories matrix sorting #874
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems you took changes from different branch
# @todo #822:60m Create tests for the catalog matrix. | ||
# This cases are would be good to test: | ||
# - Matrix sorting by page position | ||
# - Block items limiting |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Block items limiting
It seems you still haven't test this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
y, did it intentionally. Updated PR body about it
e54e251
to
0841eee
Compare
sorry, i should wait for rebase on #850 task's code merging |
@artemiy312 , it's ready for review |
shopelectro/tests/tests_views.py
Outdated
class CategoriesMatrix(BaseCatalogTestCase): | ||
|
||
fixtures = ['dump.json'] | ||
URL = reverse_lazy('custom_page', kwargs={'page': 'catalog'}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can move this to get_page
and use reverse
instead of reverse_lazy
Closes #836
Didn't test block items limiting. The test should be too complex, for a such minor limiting feature