Enhance Dashboard component to display Network and Data test results

[alexandair]

Add feature: Enhance Dashboard component to display Network and Data test results

• Removed "peer" property from various dependencies in package-lock.json.
• Updated Dashboard component to conditionally render Network test results alongside Data test results.
• Adjusted labels and calculations to ensure clarity in the displayed metrics.

[Copilot]

Pull request overview

This PR enhances the Dashboard component to conditionally display Network and Data test results alongside existing Identity and Devices test results. The changes enable preview mode functionality where optional test pillars can be shown when available.

Key Changes:

• Added conditional rendering for Data and Network test result sections
• Updated chart configuration to include Network pillar with color mapping
• Removed "peer" property from multiple dependencies in package-lock.json

Reviewed changes

Copilot reviewed 1 out of 3 changed files in this pull request and generated 20 comments.

File	Description
src/report/src/pages/Dashboard.tsx	Added conditional rendering logic for Data and Network test results in both the summary display and radial chart, including proper chart configuration for the new Network pillar
src/report/package-lock.json	Removed "peer" property metadata from various dependencies (build tooling cleanup)

Files not reviewed (1)

• src/report/package-lock.json: Language not supported

Comments suppressed due to low confidence (1)

src/report/src/pages/Dashboard.tsx:370

• The code checks if DataTotal is undefined but doesn't handle the case where it equals zero. This could cause a division by zero error when calculating the percentage value. Consider adding a check to ensure DataTotal is greater than zero, or handle the zero case by setting the value to 0.

                                        ...(reportData.TestResultSummary.DataPassed !== undefined && reportData.TestResultSummary.DataTotal !== undefined
                                            ? [{
                                                activity: "data",
                                                value: (reportData.TestResultSummary.DataPassed / reportData.TestResultSummary.DataTotal) * 100,
                                                fill: "var(--color-stand)",
                                            }]
                                            : []),

---

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

[Copilot]

The code checks if NetworkTotal is undefined but doesn't handle the case where it equals zero. This could cause a division by zero error when calculating the percentage value. Consider adding a check to ensure NetworkTotal is greater than zero, or handle the zero case by setting the value to 0.

[Copilot]

The className includes 'flex-1' which is inconsistent with the Identity and Devices sections above (lines 285 and 294) that use 'grid auto-rows-min gap-0.5' without 'flex-1'. This inconsistency may cause unintended layout differences. Consider using the same className pattern as the other sections for consistency.

[Copilot]

The className includes 'flex-1' which is inconsistent with the Identity and Devices sections above (lines 285 and 294) that use 'grid auto-rows-min gap-0.5' without 'flex-1'. This inconsistency may cause unintended layout differences. Consider using the same className pattern as the other sections for consistency.

[Copilot]

This guard always evaluates to true.

[Copilot]

Unused variable MenuSubProvider.

[Copilot]

Avoid automated semicolon insertion (95% of all statements in the enclosing function have an explicit semicolon).

[Copilot]

Avoid automated semicolon insertion (90% of all statements in the enclosing function have an explicit semicolon).

[Copilot]

Avoid automated semicolon insertion (90% of all statements in the enclosing function have an explicit semicolon).

[Copilot]

Avoid automated semicolon insertion (94% of all statements in the enclosing function have an explicit semicolon).

[Copilot]

This statement is unreachable.