-
Notifications
You must be signed in to change notification settings - Fork 1.8k
chore(ci): add xunit output to evergreen config #2596
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -103,7 +103,7 @@ functions: | |
params: | ||
file: mo-expansion.yml | ||
|
||
bootstrap mongohoused: | ||
"bootstrap mongohoused": | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is it possible to make this change without all the whitespace (quotation mark) changes in this file? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah, I just added quotes to a few that were missing them for consistency - it makes the titles show up in a different color on syntax highlighting, which I thought was nice, but I can revert that. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @mbroadst Just confirming, should I revert the quotation mark changes I made to improve consistency? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No that's fine, I just wondered what was going on |
||
- command: shell.exec | ||
params: | ||
script: | | ||
|
@@ -242,14 +242,16 @@ functions: | |
script: | | ||
MONGODB_URI='${plain_auth_mongodb_uri}' NODE_LTS_NAME='${NODE_LTS_NAME}' \ | ||
bash ${PROJECT_DIRECTORY}/.evergreen/run-ldap-tests.sh | ||
run data lake tests: | ||
|
||
"run data lake tests": | ||
- command: shell.exec | ||
type: test | ||
params: | ||
working_dir: src | ||
script: | | ||
MONGODB_URI='mongodb://mhuser:pencil@localhost' NODE_LTS_NAME='${NODE_LTS_NAME}' \ | ||
bash ${PROJECT_DIRECTORY}/.evergreen/run-data-lake-tests.sh | ||
|
||
"run tls tests": | ||
- command: shell.exec | ||
type: test | ||
|
@@ -286,7 +288,7 @@ functions: | |
} | ||
EOF | ||
|
||
run aws auth test with regular aws credentials: | ||
"run aws auth test with regular aws credentials": | ||
- command: shell.exec | ||
type: test | ||
params: | ||
|
@@ -315,7 +317,7 @@ functions: | |
${PREPARE_SHELL} | ||
${PROJECT_DIRECTORY}/.evergreen/run-mongodb-aws-test.sh | ||
|
||
run aws auth test with assume role credentials: | ||
"run aws auth test with assume role credentials": | ||
- command: shell.exec | ||
type: test | ||
params: | ||
|
@@ -348,7 +350,7 @@ functions: | |
${PREPARE_SHELL} | ||
${PROJECT_DIRECTORY}/.evergreen/run-mongodb-aws-test.sh | ||
|
||
run aws auth test with aws EC2 credentials: | ||
"run aws auth test with aws EC2 credentials": | ||
- command: shell.exec | ||
type: test | ||
params: | ||
|
@@ -365,7 +367,7 @@ functions: | |
${PREPARE_SHELL} | ||
${PROJECT_DIRECTORY}/.evergreen/run-mongodb-aws-test.sh | ||
|
||
run aws auth test with aws credentials as environment variables: | ||
"run aws auth test with aws credentials as environment variables": | ||
- command: shell.exec | ||
type: test | ||
params: | ||
|
@@ -385,7 +387,7 @@ functions: | |
${PREPARE_SHELL} | ||
${PROJECT_DIRECTORY}/.evergreen/run-mongodb-aws-test.sh | ||
|
||
run aws auth test with aws credentials and session token as environment variables: | ||
"run aws auth test with aws credentials and session token as environment variables": | ||
- command: shell.exec | ||
type: test | ||
params: | ||
|
@@ -406,7 +408,7 @@ functions: | |
${PREPARE_SHELL} | ||
${PROJECT_DIRECTORY}/.evergreen/run-mongodb-aws-test.sh | ||
|
||
run aws ECS auth test: | ||
"run aws ECS auth test": | ||
- command: shell.exec | ||
type: test | ||
params: | ||
|
@@ -433,7 +435,7 @@ functions: | |
cat setup.js | ||
mongo --nodb setup.js aws_e2e_ecs.js | ||
|
||
run-ocsp-test: | ||
"run-ocsp-test": | ||
- command: shell.exec | ||
type: test | ||
params: | ||
|
@@ -446,7 +448,7 @@ functions: | |
OCSP_TLS_SHOULD_SUCCEED="${OCSP_TLS_SHOULD_SUCCEED}" \ | ||
bash ${PROJECT_DIRECTORY}/.evergreen/run-ocsp-tests.sh | ||
|
||
run-valid-ocsp-server: | ||
"run-valid-ocsp-server": | ||
- command: shell.exec | ||
params: | ||
script: | | ||
|
@@ -464,7 +466,7 @@ functions: | |
--ocsp_responder_key rsa/ca.key \ | ||
-p 8100 -v | ||
|
||
run-revoked-ocsp-server: | ||
"run-revoked-ocsp-server": | ||
- command: shell.exec | ||
params: | ||
script: | | ||
|
@@ -484,6 +486,11 @@ functions: | |
-v \ | ||
--fault revoked | ||
|
||
"upload test results": | ||
# Upload the xunit-format test results. | ||
- command: attach.xunit_results | ||
params: | ||
file: "src/xunit.xml" | ||
|
||
pre: | ||
- func: "fetch source" | ||
|
@@ -493,4 +500,5 @@ pre: | |
- func: "make files executable" | ||
|
||
post: | ||
- func: "upload test results" | ||
- func: "cleanup" |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,6 +31,9 @@ manual_tests/ | |
docs/build | ||
docs/Makefile | ||
|
||
# xunit test output for CI | ||
xunit.xml | ||
|
||
# Directory for dbs | ||
db | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,4 @@ | ||
{ | ||
"require": "ts-node/register" | ||
"require": "ts-node/register", | ||
"reporter": "spec-xunit-file" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. hm this is specified in the script above, can we reduce it to one place? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I didn't add There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can't you add this same setting to |
||
} |
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 there a reason this was removed?
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'm guessing it was a change that was made directly to the
config.yml
and not toconfig.yml.in
, so it was lost when regenerating the file. I can take a closer look.