-
Notifications
You must be signed in to change notification settings - Fork 123
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
Tests: Various Minor Improvements #2401
The head ref may contain hidden characters: "\u{1F9EA}"
Conversation
# Mount yamlcpp plugin to cascading namespace `/tests/yamlcpp` | ||
sudo kdb mount config.yaml /tests/yamlcpp yamlcpp | ||
# Mount yamlcpp plugin to `user/tests/yamlcpp` | ||
sudo kdb mount config.yaml user/tests/yamlcpp yamlcpp |
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.
Why to use user instead of cascading?
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.
I used the explicit namespace, since otherwise the output of the command kdb ls
in line 86
would depend on the (root or non-root) user that executes the test.
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.
Thanks for the tip! Maybe we should write that somewhere in the shell recorder tutorial?
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.
I do not think that is necessary. The output of failing Markdown Shell Recorder tests on FreeBSD should already show the issue to someone who might not expect that behavior.
Thank you! |
No description provided.