-
Notifications
You must be signed in to change notification settings - Fork 74
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 #21175: Impossible to put a Rule in a subcategory #4355
Fixes #21175: Impossible to put a Rule in a subcategory #4355
Conversation
f393067
to
c186fdf
Compare
Commit modified |
1 similar comment
Commit modified |
c186fdf
to
cc047fc
Compare
Commit modified |
cc047fc
to
3c8254c
Compare
) | ||
restTestSetUp.ruleApiService14.listCategoriesId(root) shouldEqual(listCatIds) | ||
} | ||
|
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.
you should add a test that you only get cat4 and children if starting from cat4 (locality) and that you correctly get nothing if non existing cat and that you only get one cat starting from cat1
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.
LGTM modulo missing tests, you can merge after adding them
3c8254c
to
84fb42e
Compare
Commit modified |
OK, merging this PR |
https://issues.rudder.io/issues/21175