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

test: Prevent excessive errors during tests #2360

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

taratatach
Copy link
Member

@taratatach taratatach commented Nov 5, 2024

Some tests are the cause of errors (while not turning red) as they
start background processings which are not stopped at the end of the
tests and end up trying to access a destroyed database.

Stopping these processings will prevent those errors and might improve
test performance.

Please make sure the following boxes are checked:

  • PR is not too big
  • it improves UX & DX in some way
  • it includes unit tests matching the implementation changes
  • it includes scenarios matching a new behaviour or has been manually tested
  • it includes relevant documentation

@taratatach taratatach self-assigned this Nov 5, 2024
@taratatach taratatach force-pushed the feat/add-suspend-resume-actions-to-watchers branch from 2f30015 to 1c38a40 Compare November 5, 2024 16:55
@taratatach taratatach force-pushed the test/prevent-excessive-errors branch from 6ffc29f to 090aeff Compare November 5, 2024 16:55
@taratatach taratatach force-pushed the feat/add-suspend-resume-actions-to-watchers branch from 1c38a40 to 10c7343 Compare November 15, 2024 16:13
  Some tests are the cause of errors (while not turning red) as they
  start background processings which are not stopped at the end of the
  tests and end up trying to access a destroyed database.

  Stopping these processings will prevent those errors and might improve
  test performance.
@taratatach taratatach force-pushed the test/prevent-excessive-errors branch from 090aeff to 54f5cce Compare December 10, 2024 17:44
@taratatach taratatach changed the base branch from feat/add-suspend-resume-actions-to-watchers to master December 10, 2024 17:45
@taratatach taratatach merged commit 0af2bc4 into master Dec 11, 2024
12 of 16 checks passed
@taratatach taratatach deleted the test/prevent-excessive-errors branch December 11, 2024 12:48
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