-
Notifications
You must be signed in to change notification settings - Fork 28
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 #28904 - make REMOTE_USER_ENVIRON_NAME configurable #61
Conversation
49cfbe3
to
1b6b7c6
Compare
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.
Could you write a test for this? I think something like this probably works if you add it to spec/classes/pulpcore_spec.rb
:
it { is_expected.to contain_concat__fragment('base').without_content(/REMOTE_USER_ENVIRON_NAME/) }
Of course you can also add a test case where you do pass it in to ensure it does show up then.
1f35968
to
af05c1b
Compare
af05c1b
to
33e6580
Compare
33e6580
to
f573c67
Compare
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.
Looks great. Thanks @synkd !
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.
Looks good. If you want to go for bonus points, then you can add a positive test where you set it and expect the content to be there.
Fixes #28904