-
Notifications
You must be signed in to change notification settings - Fork 987
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
Fixes #32029 - test coverage for taxed_and_untaxed #8370
Conversation
Add scoping for current search tests to divide those and newly added tax and permission tests.
Issues: #32029 |
bcb7204
to
afa0135
Compare
One of the unit tests failed, and integration tests stalled so there is probably something wrong here |
It seems to be my simple code improvement fault. I've removed it, it didn't belong here. We are green, but testing the bugged version. (current subtaxonomy allows to see parent records). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please open an issue to fix the wrong behavior?
I've created two, because two is more than one! https://projects.theforeman.org/issues/32053 |
Adding test coverage for
taxed_and_untaxed
scope so we have better confidency over its refactorings.Note I've splitted to two commits, so the additions are more readable.
So for easier review just take a look at the last commit.