Skip to content

Conversation

v-deepaktn
Copy link
Contributor

@v-deepaktn v-deepaktn commented Aug 14, 2025

Details

The report has been updated to include a new section titled "Incomplete Checks", which was previously not shown with report package.
This section now provides comprehensive information for each incomplete entry.
Relevant test cases have been added or updated to ensure these enhancements are thoroughly validated.

Added a separate function automatedChecksConversionForReportPackage in scan-results-to-unified-results.ts which will be called only for report package.

Screenshot
image

Motivation

Issue: https://github.com/1ES-microsoft/accessibility-insights-agent/issues/911
Sub Issue: https://github.com/1ES-microsoft/accessibility-insights-agent/issues/928

Context

This contains only the code changes for adding incomplete checks in HTML report, we have another sub issue for adding details to JSON report which will handled with different PR in agent repo.

Pull request checklist

  • Addresses an existing issue: https://github.com/1ES-microsoft/accessibility-insights-agent/issues/928
  • Ran yarn fastpass
  • Added/updated relevant unit test(s) (and ran yarn test)
  • Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
  • PR title AND final merge commit title both start with a semantic tag (fix:, chore:, feat(feature-name):, refactor:). See CONTRIBUTING.md.
  • (UI changes only) Added screenshots/GIFs to description above
  • (UI changes only) Verified usability with NVDA/JAWS

@v-deepaktn
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Microsoft"

@v-deepaktn v-deepaktn changed the title Incomplete Checks to be visible in Agent HTML report feat(ai-report): added incomplete checks section for report package Aug 19, 2025
@v-viyada v-viyada marked this pull request as ready for review August 20, 2025 19:14
@v-viyada v-viyada requested a review from a team as a code owner August 20, 2025 19:14
Copy link
Contributor

@homitc homitc left a comment

Choose a reason for hiding this comment

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

LGTM!

@v-viyada v-viyada merged commit 0aab769 into main Aug 21, 2025
12 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.

4 participants