-
-
Notifications
You must be signed in to change notification settings - Fork 398
[skip-changelog] Migrate tests from test_compile_part_2.py
to compile_part_2_test.go
#1897
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_compile_part_2.py
to compile_part_2_test.go
#1897
Conversation
Codecov ReportBase: 36.67% // Head: 36.67% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #1897 +/- ##
=======================================
Coverage 36.67% 36.67%
=======================================
Files 231 231
Lines 19684 19684
=======================================
Hits 7219 7219
Misses 11635 11635
Partials 830 830
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. |
4d473dc
to
b16c327
Compare
…ompile_part_2_test.go
… to compile_part_2_test.go
… compile_part_2_test.go
…ile_part_2_test.go
…py to compile_part_2_test.go
to compile_part_2_test.go
… compile_part_2_test.go
…py to compile_part_2_test.go
…d delete test_compile_part_2.py
b16c327
to
b36daf2
Compare
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_compile_part_2.py
have been rewritten incompile_part_2_test.go
, without changing their original purpose.The tests are the following:
Does this PR introduce a breaking change, and is titled accordingly?
No
See how to contribute