-
-
Notifications
You must be signed in to change notification settings - Fork 393
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
[skip-changelog] Migrate tests from test_config.py
to config_test.go
#1898
[skip-changelog] Migrate tests from test_config.py
to config_test.go
#1898
Conversation
Codecov ReportBase: 36.65% // Head: 36.61% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #1898 +/- ##
==========================================
- Coverage 36.65% 36.61% -0.05%
==========================================
Files 231 231
Lines 19683 19683
==========================================
- Hits 7215 7206 -9
- Misses 11640 11648 +8
- Partials 828 829 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
9c04d90
to
7e055fd
Compare
Converting this PR back to drafting stage since |
7e055fd
to
4e3aec2
Compare
The content of PR #1902 has been migrated. |
Please check if the PR fulfills these requirements
before creating one)
our contributing guidelines
UPGRADING.md
has been updated with a migration guide (for breaking changes)What kind of change does this PR introduce?
Infrastructure enhancement
What is the new behavior?
This PR is a followup to PR #1806
Integration tests that were previously present in
test_config.py
have been rewritten inconfig_test.go
, without changing their original purpose.More informations on the tests can be found checking the single commits.
Does this PR introduce a breaking change, and is titled accordingly?
No
See how to contribute