fix(测试跟踪): 修复测试计划报告连接分享，接口用例查出的是最新执行结果的问题

--bug=1022575 --user=宋天阳【测试跟踪】github#21600，定时执行的测试计划在执行失败后将报告分享至钉钉群，但是访问钉钉群的链接发现，执行结果取的是最新执行的结果，而我们是想看当时执行失败时的接口执行信息 https://www.tapd.cn/55049933/s/1348250
2023-03-10 17:49:17 +08:00 · 2023-03-10 17:49:17 +08:00 · 73378e3bba
parent ce0e3f4559
commit 73378e3bba
3 changed files with 185 additions and 128 deletions
--- a/api-test/backend/src/main/java/io/metersphere/controller/plan/ShareTestPlanApiReportController.java
+++ b/api-test/backend/src/main/java/io/metersphere/controller/plan/ShareTestPlanApiReportController.java
@ -23,4 +23,10 @@ public class ShareTestPlanApiReportController {
        shareInfoService.validate(shareId);
        return apiDefinitionService.getDbResult(testId);
    }
+
+    @GetMapping("/api/definition/report/by/id/{reportId}")
+    public ApiReportResult getApiReport(@PathVariable String reportId) {
+        return apiDefinitionService.getReportById(reportId);
+    }
+
 }
--- a/test-track/frontend/src/api/share.js
+++ b/test-track/frontend/src/api/share.js
@ -1,4 +1,8 @@
-import {get, post, generateShareUrl} from "metersphere-frontend/src/plugins/request"
+import {
+  generateShareUrl,
+  get,
+  post,
+} from "metersphere-frontend/src/plugins/request";

 export function generateApiDocumentShareInfo(param) {
  return post("/share/generate/api/document", param);
@ -18,20 +22,26 @@ export function getShareContent(shareId, stepId) {
 }

 export function getShareApiReport(shareId, testId) {
-  return get('/share/api/definition/report/getReport/' + shareId + '/' + testId);
+  return get(
+    "/share/api/definition/report/getReport/" + shareId + "/" + testId
+  );
+}
+
+export function getShareApiReportByReportId(reportId) {
+  return get("/share/api/definition/report/by/id/" + reportId);
 }

 export function getShareInfo(id) {
-  return get('/share/info/get/' + id);
+  return get("/share/info/get/" + id);
 }

 export function getShareId() {
  //let herfUrl = 'http://localhost:8080/sharePlanReport?shareId=bf9496ac-8577-46b4-adf9-9c7e93dd06a8';
  let herfUrl = window.location.href;
-  if (herfUrl.indexOf('shareId=') > -1) {
-    let shareId = '';
+  if (herfUrl.indexOf("shareId=") > -1) {
+    let shareId = "";
    new URL(herfUrl).searchParams.forEach((value, key) => {
-      if (key === 'shareId') {
+      if (key === "shareId") {
        shareId = value;
      }
    });
@ -52,6 +62,6 @@ export function getShareId() {
 }

 export function getShareRedirectUrl(data) {
-  let name = '/share-plan-report';
+  let name = "/share-plan-report";
  return generateShareUrl(name, data.shareUrl);
 }
--- a/test-track/frontend/src/business/plan/view/comonents/report/detail/component/ApiCaseFailureResult.vue
+++ b/test-track/frontend/src/business/plan/view/comonents/report/detail/component/ApiCaseFailureResult.vue
@ -1,47 +1,53 @@
 <template>
  <el-container>
-    <ms-aside-container width="500px" :default-hidden-bottom-top="200" :enable-auto-height="true">
+    <ms-aside-container
+      width="500px"
+      :default-hidden-bottom-top="200"
+      :enable-auto-height="true"
+    >
      <el-card>
        <el-scrollbar>
-          <ms-table v-loading="loading"
-                    :show-select-all="false"
-                    :screen-height="null"
-                    :enable-selection="false"
-                    :highlight-current-row="true"
-                    @refresh="getScenarioApiCase"
-                    @handleRowClick="rowClick"
-                    :data="apiCases">
-
-            <ms-table-column
-              :width="100"
-              :label="$t('commons.id')"
-              prop="num">
+          <ms-table
+            v-loading="loading"
+            :show-select-all="false"
+            :screen-height="null"
+            :enable-selection="false"
+            :highlight-current-row="true"
+            @refresh="getScenarioApiCase"
+            @handleRowClick="rowClick"
+            :data="apiCases"
+          >
+            <ms-table-column :width="100" :label="$t('commons.id')" prop="num">
            </ms-table-column>

-            <ms-table-column
-              :label="$t('commons.name')"
-              prop="name">
+            <ms-table-column :label="$t('commons.name')" prop="name">
            </ms-table-column>

            <el-table-column
              prop="principalName"
-              :label="$t('test_track.plan.plan_principal')"/>
+              :label="$t('test_track.plan.plan_principal')"
+            />

            <ms-table-column
              :label="$t('test_track.case.priority')"
              :width="80"
-              prop="priority">
+              prop="priority"
+            >
              <template v-slot:default="scope">
-                <priority-table-item :value="scope.row.priority" ref="priority"/>
+                <priority-table-item
+                  :value="scope.row.priority"
+                  ref="priority"
+                />
              </template>
            </ms-table-column>

            <ms-table-column
              :width="80"
              :label="$t('test_track.plan_view.execute_result')"
-              prop="lastResult">
+              prop="lastResult"
+            >
              <template v-slot:default="scope">
-                <ms-test-plan-api-status :status="scope.row.execResult"/>
+                <ms-test-plan-api-status :status="scope.row.execResult" />
              </template>
            </ms-table-column>
          </ms-table>
@ -50,22 +56,27 @@
    </ms-aside-container>
    <ms-main-container v-loading="responseLoading">
      <div v-if="showResponse">
-        <micro-app v-if="!isTemplate"
-                   route-name="ApiReportView"
-                   service="api"
-                   :route-params="{
-                      isTestPlan: showResponse,
-                      response
-                    }"/>
+        <micro-app
+          v-if="!isTemplate"
+          route-name="ApiReportView"
+          service="api"
+          :route-params="{
+            isTestPlan: showResponse,
+            response,
+          }"
+        />
        <el-card v-else>
-          <ms-request-result-tail :response="response"
-                                  :is-test-plan="showResponse"
-                                  ref="debugResult"/>
-
+          <ms-request-result-tail
+            :response="response"
+            :is-test-plan="showResponse"
+            ref="debugResult"
+          />
        </el-card>
      </div>

-      <div class="empty" v-else>{{ $t('test_track.plan.load_case.content_empty') }}</div>
+      <div class="empty" v-else>
+        {{ $t("test_track.plan.load_case.content_empty") }}
+      </div>
    </ms-main-container>
  </el-container>
 </template>
@ -85,14 +96,17 @@ import {
  getSharePlanApiAllCase,
  getSharePlanApiErrorReportCase,
  getSharePlanApiFailureCase,
-  getSharePlanApiUnExecuteCase
+  getSharePlanApiUnExecuteCase,
 } from "@/api/remote/plan/test-plan";
 import MsTable from "metersphere-frontend/src/components/table/MsTable";
 import MsTableColumn from "metersphere-frontend/src/components/table/MsTableColumn";
 import MsAsideContainer from "metersphere-frontend/src/components/MsAsideContainer";
 import MsMainContainer from "metersphere-frontend/src/components/MsMainContainer";
-import {getShareApiReport} from "@/api/share";
-import {apiDefinitionReportGet, apiDefinitionReportGetDb} from "@/api/remote/api/api-definition";
+import { getShareApiReport, getShareApiReportByReportId } from "@/api/share";
+import {
+  apiDefinitionReportGet,
+  apiDefinitionReportGetDb,
+} from "@/api/remote/api/api-definition";
 import MicroApp from "metersphere-frontend/src/components/MicroApp";
 import MsTestPlanApiStatus from "@/business/plan/view/comonents/api/TestPlanApiStatus";

@ -103,8 +117,13 @@ export default {
    MsMainContainer,
    MsAsideContainer,
    MicroApp,
-    MsTableColumn, MsTable, StatusTableItem, MethodTableItem, TypeTableItem, PriorityTableItem,
-    MsRequestResultTail
+    MsTableColumn,
+    MsTable,
+    StatusTableItem,
+    MethodTableItem,
+    TypeTableItem,
+    PriorityTableItem,
+    MsRequestResultTail,
  },
  props: {
    planId: String,
@ -115,7 +134,7 @@ export default {
    isAll: Boolean,
    isErrorReport: Boolean,
    isUnExecute: Boolean,
-    isDb: Boolean
+    isDb: Boolean,
  },
  data() {
    return {
@ -123,17 +142,17 @@ export default {
      responseLoading: false,
      loading: false,
      response: {},
-      showResponse: false
-    }
+      showResponse: false,
+    };
  },
  watch: {
    apiCases() {
      if (this.apiCases) {
-        this.$emit('setSize', this.apiCases.length);
+        this.$emit("setSize", this.apiCases.length);
      } else {
        this.apiCases = [];
      }
-    }
+    },
  },
  mounted() {
    this.getScenarioApiCase();
@ -142,76 +161,112 @@ export default {
    getScenarioApiCase() {
      if (this.isTemplate || this.isDb) {
        if (this.isErrorReport) {
-          this.apiCases = this.report.errorReportCases ? this.report.errorReportCases : [];
+          this.apiCases = this.report.errorReportCases
+            ? this.report.errorReportCases
+            : [];
        } else if (this.isUnExecute) {
-          this.apiCases = this.report.unExecuteCases ? this.report.unExecuteCases : [];
+          this.apiCases = this.report.unExecuteCases
+            ? this.report.unExecuteCases
+            : [];
        } else if (this.isAll) {
-          this.apiCases = this.report.apiAllCases ? this.report.apiAllCases : [];
+          this.apiCases = this.report.apiAllCases
+            ? this.report.apiAllCases
+            : [];
        } else {
-          this.apiCases = this.report.apiFailureCases ? this.report.apiFailureCases : [];
+          this.apiCases = this.report.apiFailureCases
+            ? this.report.apiFailureCases
+            : [];
        }
      } else if (this.isShare) {
        if (this.isErrorReport) {
          this.loading = true;
-          getSharePlanApiErrorReportCase(this.shareId, this.planId)
-            .then((r) => {
+          getSharePlanApiErrorReportCase(this.shareId, this.planId).then(
+            (r) => {
              this.loading = false;
              this.apiCases = r.data;
-            });
+            }
+          );
        } else if (this.isUnExecute) {
          this.loading = true;
-          getSharePlanApiUnExecuteCase(this.shareId, this.planId)
-            .then((r) => {
-              this.loading = false;
-              this.apiCases = r.data;
-            });
+          getSharePlanApiUnExecuteCase(this.shareId, this.planId).then((r) => {
+            this.loading = false;
+            this.apiCases = r.data;
+          });
        } else if (this.isAll) {
          this.loading = true;
-          getSharePlanApiAllCase(this.shareId, this.planId)
-            .then((r) => {
-              this.loading = false;
-              this.apiCases = r.data;
-            });
+          getSharePlanApiAllCase(this.shareId, this.planId).then((r) => {
+            this.loading = false;
+            this.apiCases = r.data;
+          });
        } else {
          this.loading = true;
-          getSharePlanApiFailureCase(this.shareId, this.planId)
-            .then((r) => {
-              this.loading = false;
-              this.apiCases = r.data;
-            });
+          getSharePlanApiFailureCase(this.shareId, this.planId).then((r) => {
+            this.loading = false;
+            this.apiCases = r.data;
+          });
        }
      } else {
        if (this.isErrorReport) {
          this.loading = true;
-          getPlanApiErrorReportCase(this.planId)
-            .then((r) => {
-              this.loading = false;
-              this.apiCases = r.data;
-            });
+          getPlanApiErrorReportCase(this.planId).then((r) => {
+            this.loading = false;
+            this.apiCases = r.data;
+          });
        } else if (this.isUnExecute) {
          this.loading = true;
-          getPlanApiUnExecuteCase(this.planId)
-            .then((r) => {
-              this.loading = false;
-              this.apiCases = r.data;
-            });
+          getPlanApiUnExecuteCase(this.planId).then((r) => {
+            this.loading = false;
+            this.apiCases = r.data;
+          });
        } else if (this.isAll) {
          this.loading = true;
-          getPlanApiAllCase(this.planId)
-            .then((r) => {
-              this.loading = false;
-              this.apiCases = r.data;
-            });
+          getPlanApiAllCase(this.planId).then((r) => {
+            this.loading = false;
+            this.apiCases = r.data;
+          });
        } else {
          this.loading = true;
-          getPlanApiFailureCase(this.planId)
-            .then((r) => {
-              this.loading = false;
-              this.apiCases = r.data;
-            });
+          getPlanApiFailureCase(this.planId).then((r) => {
+            this.loading = false;
+            this.apiCases = r.data;
+          });
        }
      }
    },
+    selectReportContentByReportId(reportId) {
+      this.responseLoading = true;
+      apiDefinitionReportGet(reportId).then((response) => {
+        this.responseLoading = false;
+        if (response.data) {
+          let data = response.data;
+          if (data) {
+            this.showResponse = true;
+            try {
+              this.response = JSON.parse(data.content);
+            } catch (e) {
+              this.response = {};
+            }
+          }
+        }
+      });
+    },
+    selectShareReportContentByReportId(reportId) {
+      this.responseLoading = true;
+      getShareApiReportByReportId(reportId).then((response) => {
+        this.responseLoading = false;
+        if (response.data) {
+          let data = response.data;
+          if (data) {
+            this.showResponse = true;
+            try {
+              this.response = JSON.parse(data.content);
+            } catch (e) {
+              this.response = {};
+            }
+          }
+        }
+      });
+    },
    rowClick(row) {
      this.showResponse = false;
      this.$nextTick(() => {
@ -221,9 +276,11 @@ export default {
            this.response = JSON.parse(row.response);
          }
        } else if (this.isShare) {
-          this.responseLoading = true;
-          getShareApiReport(this.shareId, row.id)
-            .then((r) => {
+          if (row.reportId) {
+            this.selectShareReportContentByReportId(row.reportId);
+          } else {
+            this.responseLoading = true;
+            getShareApiReport(this.shareId, row.id).then((r) => {
              this.responseLoading = false;
              let data = r.data;
              if (data && data.content) {
@ -231,49 +288,33 @@ export default {
                this.response = JSON.parse(data.content);
              }
            });
+          }
        } else {
          if (row.reportId) {
-            this.responseLoading = true;
-            apiDefinitionReportGet(row.reportId)
-              .then(response => {
-                this.responseLoading = false;
-                if (response.data) {
-                  let data = response.data;
-                  if (data) {
-                    this.showResponse = true;
-                    try {
-                      this.response = JSON.parse(data.content);
-                    } catch (e) {
-                      this.response = {};
-                    }
-                  }
-                }
-              });
+            this.selectReportContentByReportId(row.reportId);
          } else {
            this.responseLoading = true;
-            apiDefinitionReportGetDb(row.id)
-              .then((r) => {
-                this.responseLoading = false;
-                let data = r.data;
-                if (data && data.content) {
-                  this.showResponse = true;
-                  try {
-                    this.response = JSON.parse(data.content);
-                  } catch (e) {
-                    this.response = {};
-                  }
+            apiDefinitionReportGetDb(row.id).then((r) => {
+              this.responseLoading = false;
+              let data = r.data;
+              if (data && data.content) {
+                this.showResponse = true;
+                try {
+                  this.response = JSON.parse(data.content);
+                } catch (e) {
+                  this.response = {};
                }
-              });
+              }
+            });
          }
        }
      });
-    }
-  }
-}
+    },
+  },
+};
 </script>

 <style scoped>
-
 .el-card :deep(.el-card__body) {
  height: 550px;
 }