Skip to content
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

Include test case stdout and stderr in annotation details #358

Merged
merged 9 commits into from Oct 14, 2022

Conversation

EnricoMi
Copy link
Owner

@EnricoMi EnricoMi commented Oct 4, 2022

Test cases provide stdout and stderror, which provide useful information. These are added to the annotation details.

Fixes #354.

finally:
if sys.platform == 'win32':
os.unlink(file.name)
settings = get_settings(dict(
Copy link

Choose a reason for hiding this comment

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

hardcoded_password_funcarg: Possible hardcoded password: '********'


ℹ️ Learn about @sonatype-lift commands

You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.

Command Usage
@sonatype-lift ignore Leave out the above finding from this PR
@sonatype-lift ignoreall Leave out all the existing findings from this PR
@sonatype-lift exclude <file|issue|path|tool> Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file

Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.


Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (setup-python)

       63 files  ±       0         63 suites  ±0   28m 59s ⏱️ + 4m 52s
     374 tests +     70       374 ✔️ +     70      0 💤 ±0  0 ±0 
23 562 runs  +4 410  23 058 ✔️ +4 410  504 💤 ±0  0 ±0 

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (Test File)

97 tests  ±0   80 ✔️ ±0   3m 25s ⏱️ ±0s
  1 suites ±0   17 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (Dockerfile)

       63 files  ±       0         63 suites  ±0   28m 59s ⏱️ + 4m 52s
     374 tests +     70       374 ✔️ +     70      0 💤 ±0  0 ±0 
23 562 runs  +4 410  23 058 ✔️ +4 410  504 💤 ±0  0 ±0 

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (macOS python 3.6)

       63 files         63 suites   28m 59s ⏱️
     374 tests      374 ✔️     0 💤 0
23 562 runs  23 058 ✔️ 504 💤 0

Results for commit 4b6a1b4.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (Test Files)

     67 files  +1         6 errors  358 suites  +1   1h 6m 6s ⏱️ +2s
1 682 tests +1  1 396 ✔️ ±0    71 💤 ±0  208 +1  7 🔥 ±0 
1 929 runs  +1  1 589 ✔️ ±0  106 💤 ±0  226 +1  8 🔥 ±0 

For more details on these parsing errors, failures and errors, see this check.

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (Docker Image)

       63 files  ±       0         63 suites  ±0   28m 59s ⏱️ + 4m 52s
     374 tests +     70       374 ✔️ +     70      0 💤 ±0  0 ±0 
23 562 runs  +4 410  23 058 ✔️ +4 410  504 💤 ±0  0 ±0 

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (macOS python installed)

       63 files  ±       0         63 suites  ±0   28m 59s ⏱️ + 4m 52s
     374 tests +     70       374 ✔️ +     70      0 💤 ±0  0 ±0 
23 562 runs  +4 410  23 058 ✔️ +4 410  504 💤 ±0  0 ±0 

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (Linux python 3.6)

       63 files  ±       0         63 suites  ±0   28m 59s ⏱️ + 4m 52s
     374 tests +     70       374 ✔️ +     70      0 💤 ±0  0 ±0 
23 562 runs  +4 410  23 058 ✔️ +4 410  504 💤 ±0  0 ±0 

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (macOS 12 python installed)

       63 files  ±       0         63 suites  ±0   28m 59s ⏱️ + 4m 52s
     374 tests +     70       374 ✔️ +     70      0 💤 ±0  0 ±0 
23 562 runs  +4 410  23 058 ✔️ +4 410  504 💤 ±0  0 ±0 

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (Linux python installed)

       63 files  ±       0         63 suites  ±0   28m 59s ⏱️ + 4m 52s
     374 tests +     70       374 ✔️ +     70      0 💤 ±0  0 ±0 
23 562 runs  +4 410  23 058 ✔️ +4 410  504 💤 ±0  0 ±0 

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (Linux 22.04 python installed)

       63 files  ±       0         63 suites  ±0   28m 59s ⏱️ + 4m 52s
     374 tests +     70       374 ✔️ +     70      0 💤 ±0  0 ±0 
23 562 runs  +4 410  23 058 ✔️ +4 410  504 💤 ±0  0 ±0 

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (Windows python installed)

       63 files  ±       0         63 suites  ±0   28m 59s ⏱️ + 4m 52s
     374 tests +     70       374 ✔️ +     70      0 💤 ±0  0 ±0 
23 562 runs  +4 410  23 058 ✔️ +4 410  504 💤 ±0  0 ±0 

Results for commit 4b6a1b4. ± Comparison against base commit 338606e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2022

Test Results (reference)

       63 files         63 suites   28m 59s ⏱️
     374 tests      374 ✔️     0 💤 0
23 562 runs  23 058 ✔️ 504 💤 0

Results for commit 4b6a1b4.

♻️ This comment has been updated with latest results.

@EnricoMi EnricoMi force-pushed the branch-stdout-and-stderr branch 2 times, most recently from 2f4826e to 8bd6b21 Compare October 4, 2022 15:49
@EnricoMi EnricoMi marked this pull request as ready for review October 14, 2022 13:21
@EnricoMi EnricoMi merged commit 0928be3 into master Oct 14, 2022
@EnricoMi EnricoMi deleted the branch-stdout-and-stderr branch October 14, 2022 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include test output in the check
1 participant