This may require some changes in WPA, but after a job is done, I'd like to be able to aggregate those results into some sort of summary report and then maybe find a way to not only display it, but potentially setup email and/or slack integration.

I'd imagine this could be done with some sort of event subscriber. More research would need to be done, but this is not a blocker for release.

Comments

WidgetsBurritos created an issue. See original summary.

WidgetsBurritos’s picture

WidgetsBurritos’s picture

Status: Active » Needs work
StatusFileSize
new2.66 KB

This is a POC patch based on the work done in #3042687: Dispatch event upon job completion. Obviously we'd want to do something more meaningful with the report, but this shows the event subscriber will work.

For example, I just generated the following output:

{"total":26,"pass":0,"fail":26,"num_of_errors":52,"num_of_likely_problems":26,"num_of_potential_problems":104,"guidelines":{"BITV 1.0 (Level 2)":"BITV 1.0 (Level 2)"}

WidgetsBurritos’s picture

Okay #3042687: Dispatch event upon job completion has been merged in so we're good to proceed with this task further.

WidgetsBurritos’s picture

StatusFileSize
new11.93 KB
new11.71 KB

Still a work in progress but just uploading the latest version of the patch for visibility. This has obvious issues around code styling and things of that nature, but it does the necessary work of storing the results into the DB and provides a simple method for retrieving results.

The next step would be to take these results and output them somewhere. Maybe with a chart (although that might be a separate issue).

We may also want to think about notification services as well (probably should be handled in a separate issue).

WidgetsBurritos’s picture

StatusFileSize
new8.53 KB
new20.92 KB

This continues to be a work in progress, but here's the next iteration of this functionality. There's still a bit of room for cleanup and testing here, but just want to get the latest patch uploaded.

WidgetsBurritos’s picture

Status: Needs work » Needs review
StatusFileSize
new28.22 KB
new34.16 KB

I provided an interdiff, but honestly it's long enough, reading the patch itself may just be the easier way of reviewing this.

This is a heavy change, but it should mostly be pretty straightforward.

Status: Needs review » Needs work

The last submitted patch, 7: 3041509-summary-report-7.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

WidgetsBurritos’s picture

Status: Needs work » Needs review
StatusFileSize
new352 bytes
new34.56 KB

So apparently WPA 8.x-2.7 was tagged, but never officially released. I just released the version, but given that dependency I'm gonna mandate WPA 2.7+ for this module in case anyone decides to use it who already has WPA installed.

WidgetsBurritos’s picture

StatusFileSize
new601.06 KB
new700.69 KB

New summary mode

History charts

WidgetsBurritos’s picture

StatusFileSize
new34.57 KB
new3.12 KB

Resolving a few issues found with @vessel_adrift during a crash session on this patch.

vessel_adrift’s picture

Status: Needs review » Reviewed & tested by the community

+1 rtbc

  • WidgetsBurritos committed 4841e03 on 8.x-1.x
    Issue #3041509 by WidgetsBurritos, vessel_adrift: Job Summary Report
    
WidgetsBurritos’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.