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

Source Sentry: enable high test strictness level in SAT #19073

Closed
alafanechere opened this issue Nov 7, 2022 · 0 comments · Fixed by #21459
Closed

Source Sentry: enable high test strictness level in SAT #19073

alafanechere opened this issue Nov 7, 2022 · 0 comments · Fixed by #21459
Assignees

Comments

@alafanechere
Copy link
Contributor

What

A test_strictness_level field was introduced to Source Acceptance Tests (SAT).
Sentry is a generally_available connector, we want it to have a high test strictness level.

This will help:

  • maximize the SAT coverage on this connector.
  • document its potential weaknesses in term of test coverage.

How

  1. Migrate the existing acceptance-test-config.yml file to the latest configuration format. (See instructions here)
  2. Enable high test strictness level in acceptance-test-config.yml. (See instructions here)
  3. Commit changes on acceptance-test-config.yml and open a PR.
  4. Run SAT with the /test command on the branch.
  5. If tests are failing please fix the failing test or use bypass_reason fields to explain why a specific test can't be run.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants