Skip to content
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.

Sign up for GitHub

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

Jump to bottom

tests_selenium.py:604-608: Csrf problems with OrderPage... #610

Closed
0pdd opened this issue Oct 22, 2018 · 1 comment
Closed

tests_selenium.py:604-608: Csrf problems with OrderPage... #610

0pdd opened this issue Oct 22, 2018 · 1 comment
Assignees
@ArtemijRodionov
Labels
1 burning issue 2 hours some big and monolith issue with hard decisions, discussions or bug with strong research cleanup everything that do project stronger, flexible, reusable hard issue requires strong architecture skills, management skills, project domain area, etc report Put it to completed tasks temporary list for business

Comments

@0pdd
Copy link
Collaborator

0pdd commented Oct 22, 2018

The puzzle rf182-d8bdf334 from #rf182 has to be resolved:

shopelectro/shopelectro/tests/tests_selenium.py

Lines 604 to 608 in 8a10969

# @todo #rf182:120m Csrf problems with OrderPage test.
# The fourth test (test_empty_cart here) produces error for csrf token:
# `Forbidden (CSRF cookie not set.): /shop/cart-add/`
# Problem does not occur if we start test `test_empty_cart` as single.
# See and launch `test_csrf_problem` method of this class.

The puzzle was created by duker33 on 22-Oct-18.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.
@0pdd
Copy link
Collaborator Author

0pdd commented Oct 22, 2018

@artemiyds2 @duker33 please pay attention to this new issue.

0pdd referenced this issue Oct 22, 2018
@duker33
rf#182 Move prepare products to context (#602)
8a10969 
- Remove get_active method
- Review #1 fix. Use raw page_is_active for excluded model
- Review#2 fix. Rm ActiveManagers in favour of QS.active()
- Review#2 fix. Minor syntax fix
- Draft for prepare prods removing
- Actualize views code to the new refarm's context interface
- Change link on main page
- Grade refarm dep to 0.4.5
- Fix price category manager using
- Mass rename `Category.objects` to `Category.objects_`
- Fix failed tests
- Move load_more to new PageImages context
- Move product view to context manager processing
- Reuse ProductImages context in Index and Product pages
- Skip tests with csrf problems
- Skip failed OrderPage tests
- Ignore temporary code doubling
@duker33 duker33 mentioned this issue Oct 23, 2018
Closed
@duker33 duker33 added 2 hours some big and monolith issue with hard decisions, discussions or bug with strong research 1 burning issue take it any teammate can take this issue and start working with it hard issue requires strong architecture skills, management skills, project domain area, etc cleanup everything that do project stronger, flexible, reusable labels Oct 23, 2018
@ArtemijRodionov ArtemijRodionov self-assigned this Nov 15, 2018
@ArtemijRodionov ArtemijRodionov removed the take it any teammate can take this issue and start working with it label Nov 15, 2018
This was referenced Nov 27, 2018
Merged
Closed
ArtemijRodionov added a commit that referenced this issue Nov 28, 2018
@ArtemijRodionov
#610 Remove delete_all_cookies (#646)
afdec40
@ArtemijRodionov ArtemijRodionov added the report Put it to completed tasks temporary list for business label Dec 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ArtemijRodionov ArtemijRodionov

Labels
1 burning issue 2 hours some big and monolith issue with hard decisions, discussions or bug with strong research cleanup everything that do project stronger, flexible, reusable hard issue requires strong architecture skills, management skills, project domain area, etc report Put it to completed tasks temporary list for business
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@duker33 @ArtemijRodionov @0pdd