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 #16366 - Add SCAP content validations to policy #208
Conversation
assert p.errors[:scap_content_id].include?("can't be blank") | ||
end | ||
|
||
test "should not create policy with monthly period" do |
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.
Typo while copying... Will change once tests are finished
23d5afd
to
222b1be
Compare
Looks good, but there is no error message when I click 'next' without changing the dropdowns. Could you have another look please? |
222b1be
to
538d244
Compare
@xprazak2 - fixed |
Nice. Thanks! |
Note: I see JSON style hashes, so if all tests pass, then our rubocop is not running. |
Oh, I thought it was changed (noticed it has changed in core) |
I knew there was a rubocop discussion, but I did not notice they actually changed it in core. It is still on in our rubocop.yaml. There was also outdated rule when I tried running Seems like we can merge this and fix rubocop separately if there are no objections. |
Sounds like a plan. Merging, thanks! |
No description provided.