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
(FM-6141) Add test-tiering, and README to spec directory #233
Conversation
|
||
# Get database name | ||
db_name = ("DB" + SecureRandom.hex(4)).upcase | ||
db_name = ('DB' + SecureRandom.hex(4)).upcase |
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 assume all the double to single quote conversions are just cleanup?
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.
Yeah, just cleanup.
Rakefile
Outdated
desc 'test tiering' | ||
RSpec::Core::RakeTask.new(:test_tier) do |t| | ||
|
||
sh 'beaker-hostgenerator windows2012r2-64sql_host%2Cdefault.a%7Bsql_version=2012%7D-redhat7-64mdca --hypervisor abs > spec/acceptance/nodesets/hosts.yml' |
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.
Is this supposed to be here?
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.
Nope, I'll get rid of it.
@@ -0,0 +1,82 @@ | |||
# sqlserver testing |
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.
A different question. This SQL Server Module is localized. Does this mean this file should be too?
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 don't know the scope of localization as it pertains to test execution. I'll ping Jean.
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.
Pending localization question.
All acceptance tests have been marked as low risk for the purposes of test tiering and acceptance efficiency.