Fix warnings in test
See original GitHub issue🚀 Feature
Currently, we have several warnings raised during the tests from our side but not catched with pytest.warns
:
- https://travis-ci.org/github/pytorch/ignite/jobs/689725736#L3577
- https://travis-ci.org/github/pytorch/ignite/jobs/689725736#L3589
- https://travis-ci.org/github/pytorch/ignite/jobs/689725736#L3593
- etc
We need to improve the code of tests by introducing
with pytest.warns(...):
# code that raises a warning
Issue Analytics
- State:
- Created 3 years ago
- Comments:8 (4 by maintainers)
Top Results From Across the Web
Fix warnings inside our tests #2196 - storybookjs/storybook
When running tests for cra-kitchen-sink, we have lots of warnings and messages thrown for react things. Some of them are simply fixing the ......
Read more >Catch warnings in Jest tests | Ben Ilegbodu
There are many reasons why a developer will leave and not fix these warnings: time pressure, unfamiliarity with the codebase, laziness, ...
Read more >Fix deprecation warnings in test suite [#3326253] | Drupal.org
When running the tests using Drupal 9 (and 10) a bunch of deprecation warnings are triggered. Things like type hinting, and using assert...
Read more >Fix the "not wrapped in act(...)" warning when testing Custom ...
Any time your test interacts with a React component in a way that results in a state update, you need to make sure...
Read more >How to make ng test to fail on warnings - Stack Overflow
I've just finished fixing several warnings on my unit/component tests of my Angular app, that run using ng test (with Karma/Jasmine).
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
It sounds really doable. I’ll open a PR by tommorow
@InCogNiTo124 yes, sounds good !