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

Port query-oriented tests from v1 to v2 #888

Closed
1 task done
iand opened this issue Sep 8, 2023 · 6 comments
Closed
1 task done

Port query-oriented tests from v1 to v2 #888

iand opened this issue Sep 8, 2023 · 6 comments
Assignees
Labels
v2 All issues related to the v2 rewrite

Comments

@iand
Copy link

iand commented Sep 8, 2023

Port all the tests that rely on query behaviour to v2 to ensure we follow existing behaviours as closely as possible.

Tasks

  1. v2
    iand
@iand iand added the v2 All issues related to the v2 rewrite label Sep 8, 2023
@iand iand self-assigned this Sep 8, 2023
@guillaumemichel
Copy link
Contributor

We need to be careful not to transfer the flaky tests.

@BigLep
Copy link

BigLep commented Sep 18, 2023

Do we have metrics to see what our test coverage is?

@dennis-tra
Copy link
Contributor

Current stats of v2-develop all files in the v2 subdirectory:

  • 220 tests
  • 93.8% of files tested
  • 69.3% of statements tested

@BigLep
Copy link

BigLep commented Sep 18, 2023

Thanks @dennis-tra for the stats. I should have been more clear to make sure we have a mechanism in place to ensure good test coverage, something along the lines of:

  1. Measurements are automatically produced
  2. They're included in a PR
  3. We have gates that requires new code to have at least X% coverage
    etc.

@dennis-tra
Copy link
Contributor

@BigLep ah sorry, yup, that's already happening! Here's an example from a recent PR:

image

However, the metrics are included in the PR as a comment. Also, we don't have a gate that requires a certain coverage percentage. I would also a be bit skeptical about the merit of such a gate.

@iand
Copy link
Author

iand commented Oct 2, 2023

The majority of these have now been done as part of implementation of the routing.Routing interface methods

@iand iand closed this as completed Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2 All issues related to the v2 rewrite
Projects
None yet
Development

No branches or pull requests

4 participants