From 056bfc7b7850842f7ebbf1a6ee7f4d7be7547970 Mon Sep 17 00:00:00 2001
From: Sourabh Mehta <73165318+soumeh01@users.noreply.github.com>
Date: Tue, 25 Jun 2024 19:47:16 +0200
Subject: [PATCH] [Fix]Consolidate E2E test report

---
 .github/workflows/e2e_test.yaml | 5 +++--
 .github/workflows/nightly.yml   | 2 +-
 test/lib/execution_summary.py   | 4 ++--
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/e2e_test.yaml b/.github/workflows/e2e_test.yaml
index 91ae178d..2c7c9b75 100644
--- a/.github/workflows/e2e_test.yaml
+++ b/.github/workflows/e2e_test.yaml
@@ -72,7 +72,8 @@ jobs:
       - name: Run Test
         shell: bash
         run: |
-          python -m robot --outputdir reports-${{ matrix.target }}-${{ matrix.arch }} --settag ${{ matrix.target }}-${{ matrix.arch }} --name ${{ matrix.target }}-${{ matrix.arch }} ./test
+          robot --outputdir reports-${{ matrix.target }}-${{ matrix.arch }} --settag ${{ matrix.target }}-${{ matrix.arch }} --name ${{ matrix.target }}-${{ matrix.arch }} ./test
+        continue-on-error: true
 
       - name: Archieve test results
         if: always()
@@ -117,7 +118,7 @@ jobs:
     - name: Generate Summary report
       if: always()
       run: |
-        python ./test/lib/execution_summary.py artifacts/collective_robot_results/output.xml
+        python ./test/lib/execution_summary.py artifacts -o artifacts/collective_robot_results/output.xml -m summary_report.md
 
     - name: Print E2E Report
       if: always()
diff --git a/.github/workflows/nightly.yml b/.github/workflows/nightly.yml
index fa2cb38f..fdbb2d48 100644
--- a/.github/workflows/nightly.yml
+++ b/.github/workflows/nightly.yml
@@ -276,7 +276,7 @@ jobs:
     - name: Generate Summary report
       if: always()
       run: |
-        python ./test/lib/execution_summary.py artifacts artifacts/collective_robot_results/output.xml summary_report.md
+        python ./test/lib/execution_summary.py artifacts -o artifacts/collective_robot_results/output.xml -m summary_report.md
 
     - name: Print E2E Report
       if: always()
diff --git a/test/lib/execution_summary.py b/test/lib/execution_summary.py
index 4681b2ee..d4a7f77e 100644
--- a/test/lib/execution_summary.py
+++ b/test/lib/execution_summary.py
@@ -113,8 +113,8 @@ def __write_test_section(self, file, test_dict, section_header, table_header):
 def main():
     parser = argparse.ArgumentParser(description='Consolidate test summary report')
     parser.add_argument('test_env_files_path', type=str, help='Path to the test environment files')
-    parser.add_argument('output_file', type=str, help='Path to output xml file')
-    parser.add_argument('markdown_file', type=str, help='Path to consolidated summary markdown file')
+    parser.add_argument('-o', '--output_file', type=str, nargs='?', default='output.xml', help='Path to output xml file')
+    parser.add_argument('-m', '--markdown_file', type=str, nargs='?', default='summary_report.md', help='Path to consolidated summary markdown file')
     args = parser.parse_args()
 
     test_env_files_path = args.test_env_files_path