Kanan/test template fix (#3308)

* Fix test_template logic

* Rewrite test_template logic to make it's intention more clear

Author: kananb

src/cli/onefuzz/debug.py

@@ -822,29 +822,26 @@ def test_template(
     ) -> responses.NotificationTestResponse:
         """Test a notification template"""
 
-        if task_id is None and report is None:
-            raise Exception("must specify either task_id or report")
-
-        endpoint = Endpoint(self.onefuzz)
         if task_id is not None:
             task = self.onefuzz.tasks.get(task_id)
-            input_blob_ref = BlobRef(
-                account="dummy-storage-account",
-                container="test-notification-crashes",
-                name="fake-crash-sample",
-            )
-
-        if report is None:
-            report = self._create_report(
-                task.job_id, task.task_id, "fake_target.exe", input_blob_ref
-            )
-
-        if task is not None:
-            report.task_id = task.task_id
-            report.job_id = task.job_id
+            if report is None:
+                input_blob_ref = BlobRef(
+                    account="dummy-storage-account",
+                    container="test-notification-crashes",
+                    name="fake-crash-sample",
+                )
+                report = self._create_report(
+                    task.job_id, task.task_id, "fake_target.exe", input_blob_ref
+                )
+            else:
+                report.task_id = task.task_id
+                report.job_id = task.job_id
+        elif report is None:
+            raise Exception("must specify either task_id or report")
 
         report.report_url = "https://dummy-container.blob.core.windows.net/dummy-reports/dummy-report.json"
 
+        endpoint = Endpoint(self.onefuzz)
         return endpoint._req_model(
             "POST",
             responses.NotificationTestResponse,