Skip to content

Add metaclass inheritance test #139

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

Merged
merged 2 commits into from
Nov 17, 2016

Conversation

rpkilby
Copy link
Collaborator

@rpkilby rpkilby commented Nov 12, 2016

Fixes #132, closes #133.

@codecov-io
Copy link

codecov-io commented Nov 12, 2016

Current coverage is 96.84% (diff: 100%)

Merging #139 into master will not change coverage

@@             master       #139   diff @@
==========================================
  Files             6          6          
  Lines           222        222          
  Methods           0          0          
  Messages          0          0          
  Branches         45         45          
==========================================
  Hits            215        215          
  Misses            3          3          
  Partials          4          4          

Powered by Codecov. Last update 7119eaa...a57789f

@rpkilby
Copy link
Collaborator Author

rpkilby commented Nov 12, 2016

Hi @detonavomek, I've tried to add a test case for #132, but can't reproduce the failing behavior. Can you provide details?

Edit: The original test case was not taking into account a related filterset. Calling get_subset() on the base filterset was not sufficient.

@rpkilby rpkilby force-pushed the fix-metaclass-inheritance branch from d5b53be to bc9b62b Compare November 17, 2016 18:12
@rpkilby rpkilby force-pushed the fix-metaclass-inheritance branch from 75abf67 to a57789f Compare November 17, 2016 18:46
@rpkilby rpkilby merged commit 062c092 into philipn:master Nov 17, 2016
@rpkilby rpkilby deleted the fix-metaclass-inheritance branch November 17, 2016 18:56
@rpkilby rpkilby modified the milestone: v1.0.0 Nov 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inheriting FilterSet with Metaclass
3 participants