Skip to content
This repository has been archived by the owner on Sep 6, 2023. It is now read-only.

More sp to root cause changes #7

Merged
merged 1 commit into from
Aug 26, 2020
Merged

More sp to root cause changes #7

merged 1 commit into from
Aug 26, 2020

Conversation

Bnaya
Copy link
Contributor

@Bnaya Bnaya commented Aug 26, 2020

No description provided.

@Bnaya Bnaya marked this pull request as ready for review August 26, 2020 07:41
@Bnaya Bnaya requested a review from benjamingr August 26, 2020 07:41
</testsuites>"
`;

exports[`the jUnit reporter for cloud executions generates a report with one test if the execution contains a single test 1`] = `
"<?xml version=\\"1.0\\" encoding=\\"UTF-8\\" standalone=\\"yes\\"?><testsuites disabled='0'> <testsuite name=\\"some test\\" tests=\\"0\\" failure=\\"0\\" failures=\\"0\\" timestamp=\\"Thu, 01 Jan 1970 00:00:00 GMT\\">

</testsuite>
<system-out>Suite Run URL: https://app.testim.io/#/project/brave-dog/branch/master/screenplay/run/1</system-out>
<system-out>Suite Run URL: https://app.testim.io/#/project/brave-dog/branch/master/root-cause/run/1</system-out>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this also fixed in the URL outputted when persisting?

Copy link
Contributor

@benjamingr benjamingr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please only land and release after releasing clickim

@@ -79,7 +79,7 @@ export function generateJunitReport({
const actualTests = Object.values(testimExecution.execution).filter(test => !test.isTestsContainer);
const dateValue = new Date(testimExecution.startTime).toUTCString();
const files = Object.values(testimExecution.execution).filter(test => test.isTestsContainer);
const baseUrl = `https://app.testim.io/#/project/${projectId}/branch/master/screenplay/run/${testimExecution.runId}`;
const baseUrl = `https://app.testim.io/#/project/${projectId}/branch/master/root-cause/run/${testimExecution.runId}`;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the persist fix

@Bnaya Bnaya merged commit eb61d7a into main Aug 26, 2020
@Bnaya Bnaya deleted the more-sp-to-rc-changes branch August 26, 2020 13:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants