Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Aug 22, 2025

This PR adds a new include_skipped input parameter that allows users to control whether skipped tests appear in the detailed summary tables, addressing a common use case where users want to show passed tests while hiding skipped tests.

Problem

Users previously could not achieve the combination of showing passed tests (include_passed: true) while excluding skipped tests from the detailed summary table. The existing include_passed parameter only controlled passed test visibility, but there was no equivalent control for skipped tests.

As shown in the issue, users were seeing summary tables with many skipped tests cluttering the detailed view, making it harder to focus on the relevant passed and failed tests.

Solution

Added a new include_skipped input parameter that:

  • Defaults to true - maintains backward compatibility with existing behavior
  • Works independently of the include_passed parameter
  • Filters skipped tests from both detailed summary tables and flaky test tables when set to false

Usage Examples

# Show only passed and failed tests (hide skipped)
- uses: mikepenz/action-junit-report@v5
  with:
    include_passed: true
    include_skipped: false
    detailed_summary: true

# Show only failed tests (hide passed and skipped) 
- uses: mikepenz/action-junit-report@v5
  with:
    include_passed: false
    include_skipped: false
    detailed_summary: true

# Show all tests (default behavior)
- uses: mikepenz/action-junit-report@v5
  with:
    include_passed: true
    include_skipped: true
    detailed_summary: true

Implementation Details

  • Added include_skipped parameter to action.yml
  • Updated filtering logic in buildSummaryTables() and appendDetailsTable() functions
  • Applied filtering to both regular detailed tables and flaky test summaries
  • Added comprehensive test coverage for the new functionality
  • Updated documentation in README.md

Testing

  • All existing tests continue to pass (55/55)
  • Added specific test case verifying skipped tests are properly filtered
  • Verified backward compatibility with default parameter values

Fixes #1410.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@Copilot Copilot AI changed the title [WIP] Include passed but exclude skipped in summary table Add include_skipped parameter to control skipped test visibility in summary tables Aug 22, 2025
@Copilot Copilot AI requested a review from mikepenz August 22, 2025 11:35
Copilot finished work on behalf of mikepenz August 22, 2025 11:35
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 passed but exclude skipped in summary table
2 participants