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.

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

testing: fix up various aspects that are breaking tests (bug 1887042) #48

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

zzzeid
Copy link
Collaborator

@zzzeid zzzeid commented Apr 3, 2024

  • skip: test_auth, test_dockerflow, test_notifications, test_diff_warnings
  • skip parts of test_landing_job
  • remove test_health module (mostly deprecated)
  • remove test_py3_code_quality (already replaced elsewhere)
  • convert headers in tests from a list to a dict (bug 1887043)
  • use django test client in lieu of FlaskClient (bug 1887043)
  • remove disable_migrations fixture
  • remove celery_app fixture (bug bug 1870105)
  • temporarily comment out cache decorators
  • adapt app fixture to new platform
  • remove flask celery functionality (bug 1870105)
  • add base celery support (bug bug 1870105)
  • add db to github workflow (postgres)
  • add rust toolchain
  • various fix ups to get tests to run
  • various other missed fixes
  • various additions to the settings module
  • consolidate migrations

@zzzeid zzzeid force-pushed the zeid/bug-1887042-port-testing branch from 38c2d3c to f6a1a98 Compare April 3, 2024 15:13
@zzzeid zzzeid force-pushed the zeid/bug-1887013-implement-support-placeholders branch 2 times, most recently from 18b180e to eb6f15c Compare April 10, 2024 15:22
@zzzeid zzzeid force-pushed the zeid/bug-1887042-port-testing branch 6 times, most recently from 24b23e7 to 97f8b4e Compare April 16, 2024 19:14
@zzzeid zzzeid force-pushed the zeid/bug-1887042-port-testing branch 6 times, most recently from 9874df0 to 155b3d9 Compare April 19, 2024 17:22
@zzzeid zzzeid force-pushed the zeid/bug-1887013-implement-support-placeholders branch 2 times, most recently from ed647b2 to 9d6c381 Compare April 19, 2024 17:37
Base automatically changed from zeid/bug-1887013-implement-support-placeholders to main April 19, 2024 17:38
@zzzeid zzzeid force-pushed the zeid/bug-1887042-port-testing branch 3 times, most recently from 0f6f024 to 1d48eaf Compare April 24, 2024 20:19
- skip: test_auth, test_dockerflow, test_notifications, test_diff_warnings
- skip parts of test_landing_job
- remove test_health module (mostly deprecated)
- remove test_py3_code_quality (already replaced elsewhere)
- convert headers in tests from a list to a dict (bug 1887043)
- use django test client in lieu of FlaskClient (bug 1887043)
- remove disable_migrations fixture
- remove celery_app fixture
- temporarily comment out cache decorators
- adapt app fixture to new platform
- remove flask celery functionality
- add base celery support
- add boilerplate django celery implementation
- add db to github workflow (postgres)
- add rust toolchain
- various fix ups to get tests to run
- various other missed fixes
- various additions to the settings module
- consolidate migrations
@zzzeid zzzeid force-pushed the zeid/bug-1887042-port-testing branch from 1d48eaf to 5457ef0 Compare April 25, 2024 13:15
@zzzeid zzzeid merged commit b32363f into main Apr 25, 2024
1 check failed
@zzzeid zzzeid deleted the zeid/bug-1887042-port-testing branch April 25, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant