Skip to content

Comments

Improve test summary to show both passed and failed packages#148

Merged
tamurashingo merged 2 commits intodevelopfrom
feature/improve-test-summary
Feb 8, 2026
Merged

Improve test summary to show both passed and failed packages#148
tamurashingo merged 2 commits intodevelopfrom
feature/improve-test-summary

Conversation

@tamurashingo
Copy link
Owner

Summary

Improved the test runner summary to display both passed and failed packages separately. This makes it clear which tests were executed even when all tests pass successfully.

Background

Previously, only failed packages were displayed in the summary. When all tests passed, there was no indication of which packages were actually tested without reviewing the full log output.

Changes

  • Added passed-package-tests variable to track successful packages
  • Changed test result evaluation from unless to if to record both success and failure
  • Split summary display into two sections: "Passed Packages" and "Failed Packages"
  • Display package count for each section along with the list of packages

Test Results

Before

When all tests passed, no summary section was displayed at all, making it unclear which packages were actually tested.

When there were failures:

=== Failed Tests Summary ===

Package: PACKAGE-C

After

=== Test Results Summary ===

Passed Packages (3):
  TODOAPP-TEST/SAMPLE
  TODOAPP-TEST/MODELS/TODO
  TODOAPP-TEST/CONTROLLERS/TODO-CONTROLLER

When there are failed packages:

=== Test Results Summary ===

Passed Packages (2):
  PACKAGE-A
  PACKAGE-B

Failed Packages (1):
  PACKAGE-C

Verification

Verified with E2E tests:

make e2e.test

Confirmed that all tests pass and executed packages are clearly displayed in the summary.

- Add passed-package-tests variable to track successful packages
- Separate test results into 'Passed Packages' and 'Failed Packages' sections
- Display package count for each section
- Make it clear which tests were executed even when all tests pass
@tamurashingo tamurashingo merged commit 939bb07 into develop Feb 8, 2026
2 checks passed
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.

1 participant