fix nightly test report (#414)

Author: sunmou99

scripts/gha/it_workflow.py

@@ -78,9 +78,6 @@
 _BUILD_STAGES_REPORT = "report"
 _BUILD_STAGES = [_BUILD_STAGES_START, _BUILD_STAGES_PROGRESS, _BUILD_STAGES_END, _BUILD_STAGES_REPORT]
 
-_BUILD_AGAINST_SDK = "sdk"
-_BUILD_AGAINST_REPO = "repo"
-
 FLAGS = flags.FLAGS
 
 flags.DEFINE_string(
@@ -112,10 +109,6 @@
     "Use a different token to remove the \"in-progress\" label,"
     "to allow the removal to trigger the \"Check Labels\" workflow.")   
 
-flags.DEFINE_string(
-    "build_against", None,
-    "Integration testapps could either build against packaged SDK or repo")
-
 
 def test_start(token, issue_number, actor, commit, run_id):
   """In PR, when start testing, add comment and label \"tests: in-progress\""""
@@ -181,29 +174,26 @@ def test_end(token, issue_number, actor, commit, run_id, new_token):
   github.delete_label(new_token, issue_number, _LABEL_PROGRESS)
 
 
-def test_report(token, actor, commit, run_id, build_against):
+def test_report(token, actor, commit, run_id):
   """Update (create if not exist) a Daily Report in Issue. 
   The Issue with title _REPORT_TITLE and label _REPORT_LABEL:
   https://github.com/firebase/firebase-unity-sdk/issues?q=is%3Aissue+label%3Anightly-testing
   """
   issue_number = _get_issue_number(token, _REPORT_TITLE, _REPORT_LABEL)
-  previous_comment = github.get_issue_body(token, issue_number)
-  [previous_comment_repo, previous_comment_sdk] = previous_comment.split(_COMMENT_SUFFIX)
   success_or_only_flakiness, log_summary = _get_summary_table(token, run_id)
-  if success_or_only_flakiness and not log_summary:
-    # succeeded (without flakiness)
-    title = _COMMENT_TITLE_SUCCEED
-    comment = title + _get_description(actor, commit, run_id)
+  if success_or_only_flakiness:
+    if not log_summary:
+      # succeeded (without flakiness)
+      title = _COMMENT_TITLE_SUCCEED
+      comment = title + _get_description(actor, commit, run_id)
+    else:
+      title = _COMMENT_TITLE_FLAKY
+      comment = title + _get_description(actor, commit, run_id) + log_summary
   else:
-    title = _COMMENT_TITLE_FLAKY
+    title = _COMMENT_TITLE_FAIL
     comment = title + _get_description(actor, commit, run_id) + log_summary
-  
-  if build_against==_BUILD_AGAINST_REPO:
-    comment = comment + _COMMENT_SUFFIX + previous_comment_sdk
-  else:
-    comment = previous_comment_repo + _COMMENT_SUFFIX + comment
 
-  if _COMMENT_TITLE_SUCCEED:
+  if title == _COMMENT_TITLE_SUCCEED:
     github.close_issue(token, issue_number)
   else:
     github.open_issue(token, issue_number)
@@ -272,7 +262,7 @@ def main(argv):
   elif FLAGS.stage == _BUILD_STAGES_END:
     test_end(FLAGS.token, FLAGS.issue_number, FLAGS.actor, FLAGS.commit, FLAGS.run_id, FLAGS.new_token)
   elif FLAGS.stage == _BUILD_STAGES_REPORT:
-    test_report(FLAGS.token, FLAGS.actor, FLAGS.commit, FLAGS.run_id, FLAGS.build_against)
+    test_report(FLAGS.token, FLAGS.actor, FLAGS.commit, FLAGS.run_id)
   else:
     print("Invalid stage value. Valid value: " + ",".join(_BUILD_STAGES))