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.

Sign up for GitHub

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

Jump to bottom

test(core): Log amazonq/toolkit tests at the bottom of the logs #6288

Draft
wants to merge 17 commits into
base: master
Choose a base branch
from
Draft

test(core): Log amazonq/toolkit tests at the bottom of the logs #6288

wants to merge 17 commits into from
+283 −9

Conversation

jpinkney-aws
Copy link
Contributor

Problem

  • It's really annoying to have to scroll up the logs whenever amazon q tests fails

Solution

  • Print out everything at the bottom of the logs

Based off of: #6261

  • Treat all work as PUBLIC. Private feature/x branches will not be squash-merged at release time.
  • Your code changes must meet the guidelines in CONTRIBUTING.md.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jpinkney-aws
test(core): merge all package test reports
0b2aed9 
Problem:
- CI test reporting only captures the last package's results
- Running `npm run testE2E` only preserves toolkit's report.xml, losing other package results

Solution:
- Generate individual report.xml files per subproject
- Consolidate all existing package reports into root .test-reports/report.xml
@jpinkney-aws
fixup
081c031
@jpinkney-aws
test failures
5ef73cf
@jpinkney-aws
failure
dfe2892
@jpinkney-aws
test
5273b20
@jpinkney-aws
test
5b6e145
@jpinkney-aws
test
adf5ca6
@jpinkney-aws
test
4e5655f
@jpinkney-aws
fixup
bb0bf30
@jpinkney-aws
fixup
b7e94e0
@jpinkney-aws
fixup
9d89d5b
@jpinkney-aws
fixup
155f4c5
@jpinkney-aws
fixup
d03a757
@jpinkney-aws
fixup
924b90e
@jpinkney-aws
test
5d3ade7
@jpinkney-aws
test(core): Log amazonq/toolkit tests at the bottom of the logs
c0ef81a
@github-actions GitHub Actions
Copy link

  • This pull request modifies code in src/* but no tests were added/updated.
    • Confirm whether tests should be added or ensure the PR description explains why tests are not required.

@jpinkney-aws
Copy link
Contributor Author

/runIntegrationTests

@jpinkney-aws jpinkney-aws changed the title Joined log test(core): Log amazonq/toolkit tests at the bottom of the logs Dec 23, 2024
@jpinkney-aws jpinkney-aws reopened this Dec 23, 2024
@jpinkney-aws
Copy link
Contributor Author

/runIntegrationTests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jpinkney-aws