-
Notifications
You must be signed in to change notification settings - Fork 348
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
feat: add test results output #949
Closed
Closed
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
9104c25
Test summary output
mihaisee 86ba246
Removed debug logs
mihaisee 212e2ff
Added some documentation for the testResultsObject output
mihaisee 1e424a0
Test cypress testResultsObject output in example-recording.yml
mihaisee 8188a63
Test cypress testResultsObject output in example-recording.yml
mihaisee 63fc380
Skip parsing testResultsObject if empty
mihaisee 3c0abeb
Update example-recording.yml
mihaisee 53a3e26
Merge branch 'cypress-io:master' into feat/add-test-reults-output
mihaisee f9794b0
Update .gitignore
mihaisee 06567f7
Merge branch 'master' into feat/add-test-reults-output
mihaisee b116a60
Merge branch 'master' into feat/add-test-reults-output
mihaisee ab2c53e
Removed the calculated success status from testResultsObject
mihaisee 087d18b
Update docs and workflow to reflect removal of success property from …
mihaisee 3a8d645
Updated testResultsObject to testResults default
mihaisee 9624c3f
Merge branch 'master' into feat/add-test-reults-output
mihaisee 4d47501
Updated testResultsObject to testResults default - update workflow
mihaisee f4402c5
Merge branch 'feat/add-test-reults-output' of github.com:mihaisee/git…
mihaisee 7421400
Updated testResultsObject to testResults default - update workflow
mihaisee 84f0d2a
Update .github/workflows/example-recording.yml
mihaisee 18af1c0
Merge branch 'master' into feat/add-test-reults-output
mihaisee 185d2ef
Update README.md
mihaisee 7b06979
Merge branch 'master' into feat/add-test-reults-output
mihaisee 4f33b71
Update action.yml
mihaisee d144906
updating to runResults
mschile 6f3e59e
updating to runResults
mschile efff8c9
Merge branch 'master' into feat/add-test-reults-output
emilyrohrbough File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Discussed with the team. For simplicity, we would like to scope the exposed runResults stringified object to
version
,totalDuration
,totalFailed
,totalPassed
,totalPending
,totalSkipped
,totalSuites
andtotalTests
. The results object can get quite large depending on the size of your test suite and if more information is needed, it can be parsed and save in theafterRun
hook.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.
@emilyrohrbough can logging the entire results object be an optional input parameter instead? e.g.
This way, by default, a summary is output but still leaves us the option to get the entire object. This way one does not need to go in and change all the test files to log an object (only needs a version update and addition of a parameter)
CC @mihaisee