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

Add test for ModuleNotFoundError fix #7947

Merged
merged 1 commit into from
Dec 15, 2022
Merged

Conversation

living180
Copy link
Contributor

Type
βœ“ πŸ› Bug fix

This PR is a follow-up to #7940 to add a regression test for the issue fixed in that PR. It uses a simple plugin that mimics what happens in the pylint_django plugin. I verified that the test failed before the fix and passes after. I had to make my best guess as to which test module to use to add the test, so if there is a better place to put it, let me know.

Refs #7938

@coveralls
Copy link

Pull Request Test Coverage Report for Build 3703581595

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.476%

Totals Coverage Status
Change from base Build 3699236593: 0.0%
Covered Lines: 17663
Relevant Lines: 18500

πŸ’› - Coveralls

@Pierre-Sassoulas Pierre-Sassoulas added Maintenance Discussion or action around maintaining pylint or the dev workflow Skip news πŸ”‡ This change does not require a changelog entry labels Dec 15, 2022
@DanielNoord DanielNoord merged commit b1d3f5a into pylint-dev:main Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance Discussion or action around maintaining pylint or the dev workflow Skip news πŸ”‡ This change does not require a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants