Skip to content

ARC 1.1.0 reports#221

Open
Famuda1234 wants to merge 5 commits into
mosip:masterfrom
Famuda1234:master
Open

ARC 1.1.0 reports#221
Famuda1234 wants to merge 5 commits into
mosip:masterfrom
Famuda1234:master

Conversation

@Famuda1234

@Famuda1234 Famuda1234 commented Jun 9, 2026

Copy link
Copy Markdown

Added the ARC 1.1.0 Reports

Summary by CodeRabbit

Release Notes

  • Tests
    • Added comprehensive test execution reports documenting test results, including summary statistics, scenario breakdowns, and execution timelines.

Famuda1234 and others added 5 commits June 9, 2026 17:01
Signed-off-by: Famuda1234 <144789326+Famuda1234@users.noreply.github.com>
Signed-off-by: Famuda1234 <144789326+Famuda1234@users.noreply.github.com>
Signed-off-by: Famuda1234 <144789326+Famuda1234@users.noreply.github.com>
Signed-off-by: Famuda1234 <144789326+Famuda1234@users.noreply.github.com>
Signed-off-by: Famuda1234 <famuda.m@cyberpwn.com>
@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

Five static TestNG HTML report artifacts are added to the test-management repository, capturing test execution results from AndroidRegClient test runs. The reports include environment metadata, test summary metrics, scenario listings with timings, and detailed exception stack traces for failed and skipped tests.

Changes

TestNG Test Result Reports

Layer / File(s) Summary
Test result report artifacts
ARC/ARC 1.1.0/Automation ARC 1.1.0_Reports/AndroidRegClient-null-run-*-report_*.html (5 files)
Five static TestNG HTML reports added: four reports displaying all-passing test scenarios (8, 9, 2, and 2 passed) with summary tables and execution timings, and one report capturing mixed outcomes (3 passed, 1 skipped, 2 failed) with detailed exception headers and formatted stack traces for failures.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • Raginikrishnamurthy

Poem

🐰 Five report scrolls arrive today,
With tests that passed in every way,
(And one with failures we display),
The AndroidRegClient's here to stay! 📋✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'ARC 1.1.0 reports' directly matches the PR's objective to introduce ARC 1.1.0 reports, clearly summarizing the main change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Warning

⚠️ This pull request might be slop. It has been flagged by CodeRabbit slop detection and should be reviewed carefully.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@ARC/ARC` 1.1.0/Automation ARC
1.1.0_Reports/AndroidRegClient-null-run-1779367864271-report_T-6_P-3_S-1_F-2.html:
- Around line 183-184: The committed HTML test report contains sensitive
internal hostnames and full stack traces (look for the table header "Use Cases
Test Report" and occurrences of "qa21.mosip.net" and stacktrace lines with
class/method/line numbers); remove this raw report from the commit and replace
it with either (a) a sanitized version that redacts hostnames and stack traces
(replace class/method/line details with "[REDACTED]" or summary messages) or (b)
omit the file entirely and upload the original to CI artifact storage instead;
ensure any retained artifact in the repo only contains non-sensitive summary
metadata and no full stack traces.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4d8b9377-79bc-4eb0-a370-76d3cda6afb4

📥 Commits

Reviewing files that changed from the base of the PR and between a70d9a0 and a3a0a51.

⛔ Files ignored due to path filters (2)
  • ARC/ARC 1.1.0/ARC 1.1.0_QA Functional Report.docx is excluded by !**/*.docx
  • ARC/ARC 1.1.0/ARC_1.1.0_Verification_Report.xlsx is excluded by !**/*.xlsx
📒 Files selected for processing (5)
  • ARC/ARC 1.1.0/Automation ARC 1.1.0_Reports/AndroidRegClient-null-run-1779257202405-report_T-8_P-8_S-0_F-0.html
  • ARC/ARC 1.1.0/Automation ARC 1.1.0_Reports/AndroidRegClient-null-run-1779274074258-report_T-9_P-9_S-0_F-0.html
  • ARC/ARC 1.1.0/Automation ARC 1.1.0_Reports/AndroidRegClient-null-run-1779365082880-report_T-2_P-2_S-0_F-0.html
  • ARC/ARC 1.1.0/Automation ARC 1.1.0_Reports/AndroidRegClient-null-run-1779367864271-report_T-6_P-3_S-1_F-2.html
  • ARC/ARC 1.1.0/Automation ARC 1.1.0_Reports/AndroidRegClient-null-run-1779422418166-report_T-2_P-2_S-0_F-0.html

Comment on lines +183 to +184
<th colspan='7'>Use Cases Test Report ---- Report Date: Thursday 21 May 19:15:02 ---- Tested Environment:
qa21.mosip.net</th>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Do not commit raw test report artifacts with environment and stacktrace details.

This file publishes internal hostnames and full failure stack traces (class names, methods, and line numbers). That leaks internal topology and implementation detail into the repo history.

✅ Suggested remediation
- ARC/ARC 1.1.0/Automation ARC 1.1.0_Reports/AndroidRegClient-null-run-1779367864271-report_T-6_P-3_S-1_F-2.html
+ (remove generated HTML reports from source control)
+# .gitignore
+ARC/**/Automation ARC 1.1.0_Reports/*.html

If these reports must be retained, publish sanitized artifacts in CI storage (with hostnames and stack traces redacted) instead of committing them.

Also applies to: 281-327

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@ARC/ARC` 1.1.0/Automation ARC
1.1.0_Reports/AndroidRegClient-null-run-1779367864271-report_T-6_P-3_S-1_F-2.html
around lines 183 - 184, The committed HTML test report contains sensitive
internal hostnames and full stack traces (look for the table header "Use Cases
Test Report" and occurrences of "qa21.mosip.net" and stacktrace lines with
class/method/line numbers); remove this raw report from the commit and replace
it with either (a) a sanitized version that redacts hostnames and stack traces
(replace class/method/line details with "[REDACTED]" or summary messages) or (b)
omit the file entirely and upload the original to CI artifact storage instead;
ensure any retained artifact in the repo only contains non-sensitive summary
metadata and no full stack traces.

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