microsoft/accessibility-insights-web

Quick Assess HTML report groups requirements incorrectly

madalynrose opened this issue · 8 comments

Describe the bug

The generated HTML report for Quick Assess currently uses the same summary model as Assessment. The assessment summary model groups requirements by Assessment. This means that the test details here display the Assessment title, not requirement titles. It also means that if any requirements are part of the same Assessment, they appear together and that the order of requirements is not respected in this list.

Expected behavior

Test details list requirements by name and show one requirement per title. It should instead use the Quick Assess summary model, as used on the Overview page for Quick Assess.

Screenshots

quick assess report with a red circle around "Headings", which has two circles indicating requirements

Are you willing to submit a PR?

Yes

Did you search for similar existing issues?

N/A

Additional context

N/A

@madalynrose, I ma interested in this repo, please assign this to me.

Hello there!

I'm interested in working on this issue. Could you please assign it to me?

@thesharifi we've assigned the bug to you. If you're no longer interested, let us know and we'll assign it to the other user who showed interest.

Implementation note: Please don't change the report package in any way, since it's used by other packages that we create. The optimal path forward may be to leverage the code from get-quick-assess-summary-model.tx from AssessmentsProviderImpl so that the results are properly grouped when running from the QuickAssess. Once the results are correctly grouped, that should be automatically reflected in the report package.

@thesharifi, any update on this? We might want to assign to someone else if it's not going to happen anytime soon

@shanmukhasaisai, are you still interested in working on this? The first person to volunteer seems to have lost interest

Yes, iam interested to work

@shanmukhasaisai, Sorry for the delay in getting back to you. If you're still available, please take this and run with it. Thanks!