Skip to content

Conversation

@Vincent4486
Copy link

Added a condition in the target test-status in build.xml to print "None" when there are no tests skipped or failed. This patch uses a new approach to displaying the list of skipped/failed tests by adding a condition before the print to make the string display "None" when it is empty.

Bug 69890

Before:

test-status:
   [concat] Testsuites with skipped tests: 
   [concat] Testsuites with failed tests: 

After:

test-status:
   [concat] Testsuites with skipped tests: 
   [concat] None
   [concat] Testsuites with failed tests: 
   [concat] None

Copilot AI review requested due to automatic review settings December 11, 2025 13:00
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the test-status target in the Ant build script to display "None" when there are no skipped or failed tests, rather than leaving the output empty. This improves the clarity of test result reporting.

Key Changes

  • Added resource counting to determine if any tests were skipped or failed
  • Implemented conditional logic to display "None" when counts are zero
  • Converted the existing toString approach to use pathconvert with explicit properties

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Vincent4486 and others added 2 commits December 11, 2025 21:42
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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