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

Use in-memory celery for test suite instead of mocking #23

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Use in-memory celery for test suite instead of mocking #23

wants to merge 6 commits into from

Conversation

dato
Copy link
Owner

@dato dato commented Jul 28, 2024 •
edited
Loading

This builds on top of bookwyrm-social#3151 to actually remove most @patch decorators in the test suite, letting tests hit in-mem celery instead.

  • tests/activitypub
  • tests/activitystreams
  • tests/connectors
  • tests/importers
  • tests/lists_stream
  • tests/management
  • tests/models
  • tests/templatetags
  • tests/views/*.py
  • tests/views/admin
  • tests/views/books
  • tests/views/imports
  • tests/views/inbox
  • tests/views/landing
  • tests/views/lists
  • tests/views/preferences
  • tests/views/shelf

@dato dato changed the title Test suite: use in-memory celery for tasks instead of patching out Use in-memory celery for test suite tasks instead of patching out Jul 28, 2024
@dato dato mentioned this pull request Jul 28, 2024
Merged
4 tasks
@dato dato changed the title Use in-memory celery for test suite tasks instead of patching out Use in-memory celery for test suite instead of patching out tasks Jul 28, 2024
@dato dato force-pushed the test_suite_celery_inmem branch 2 times, most recently from 5d9bf35 to c313fbc Compare July 28, 2024 21:47
@dato dato changed the title Use in-memory celery for test suite instead of patching out tasks Do not patch Use in-memory celery for test suite instead of mocking Jul 28, 2024
@dato dato changed the title Do not patch Use in-memory celery for test suite instead of mocking Use in-memory celery for test suite instead of mocking Jul 28, 2024
@dato dato force-pushed the test_suite_celery_inmem branch from c313fbc to 850a080 Compare July 28, 2024 22:21
@dato dato force-pushed the test_suite_celery_inmem branch 2 times, most recently from ddd39f2 to e014333 Compare July 28, 2024 23:23
@dato dato force-pushed the test_suite_celery_inmem branch from e014333 to c8873bd Compare July 28, 2024 23:46
@dato dato force-pushed the test_suite_celery_inmem branch 2 times, most recently from 0e6bfb0 to d0212c5 Compare July 29, 2024 01:26
@dato dato force-pushed the test_suite_celery_inmem branch from d0212c5 to d65d0ef Compare July 29, 2024 01:29
@dato dato force-pushed the test_suite_celery_inmem branch from 40aa764 to f63dede Compare July 29, 2024 02:15
@dato dato force-pushed the test_suite_celery_inmem branch from ff81625 to e18cc7c Compare July 29, 2024 02:52
@dato dato marked this pull request as ready for review July 29, 2024 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dato