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

--bug=1022575 --user=宋天阳
【测试跟踪】github#21600，定时执行的测试计划在执行失败后将报告分享至钉钉群，但是访问钉钉群的链接发现，执行结果取的是最新执行的结果，而我们是想看当时执行失败时的接口执行信息
https://www.tapd.cn/55049933/s/1348250

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);
+    }
+
 }

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) {
+  if (herfUrl.indexOf("shareId=") > -1) {
-    let shareId = '';
+    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);
 }

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"
+          <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">
+            :data="apiCases"
-
+          >
-            <ms-table-column
+            <ms-table-column :width="100" :label="$t('commons.id')" prop="num">
-              :width="100"
-              :label="$t('commons.id')"
-              prop="num">
             </ms-table-column>
 
-            <ms-table-column
+            <ms-table-column :label="$t('commons.name')" prop="name">
-              :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"
+        <micro-app
+          v-if="!isTemplate"
           route-name="ApiReportView"
           service="api"
           :route-params="{
             isTestPlan: showResponse,
-                      response
+            response,
-                    }"/>
+          }"
+        />
         <el-card v-else>
-          <ms-request-result-tail :response="response"
+          <ms-request-result-tail
+            :response="response"
             :is-test-plan="showResponse"
-                                  ref="debugResult"/>
+            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 { getShareApiReport, getShareApiReportByReportId } from "@/api/share";
-import {apiDefinitionReportGet, apiDefinitionReportGetDb} from "@/api/remote/api/api-definition";
+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,
+    MsTableColumn,
-    MsRequestResultTail
+    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,40 +161,46 @@ 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)
+          getSharePlanApiErrorReportCase(this.shareId, this.planId).then(
-            .then((r) => {
+            (r) => {
               this.loading = false;
               this.apiCases = r.data;
-            });
+            }
+          );
         } else if (this.isUnExecute) {
           this.loading = true;
-          getSharePlanApiUnExecuteCase(this.shareId, this.planId)
+          getSharePlanApiUnExecuteCase(this.shareId, this.planId).then((r) => {
-            .then((r) => {
             this.loading = false;
             this.apiCases = r.data;
           });
         } else if (this.isAll) {
           this.loading = true;
-          getSharePlanApiAllCase(this.shareId, this.planId)
+          getSharePlanApiAllCase(this.shareId, this.planId).then((r) => {
-            .then((r) => {
             this.loading = false;
             this.apiCases = r.data;
           });
         } else {
           this.loading = true;
-          getSharePlanApiFailureCase(this.shareId, this.planId)
+          getSharePlanApiFailureCase(this.shareId, this.planId).then((r) => {
-            .then((r) => {
             this.loading = false;
             this.apiCases = r.data;
           });
@@ -183,59 +208,34 @@ export default {
       } else {
         if (this.isErrorReport) {
           this.loading = true;
-          getPlanApiErrorReportCase(this.planId)
+          getPlanApiErrorReportCase(this.planId).then((r) => {
-            .then((r) => {
             this.loading = false;
             this.apiCases = r.data;
           });
         } else if (this.isUnExecute) {
           this.loading = true;
-          getPlanApiUnExecuteCase(this.planId)
+          getPlanApiUnExecuteCase(this.planId).then((r) => {
-            .then((r) => {
             this.loading = false;
             this.apiCases = r.data;
           });
         } else if (this.isAll) {
           this.loading = true;
-          getPlanApiAllCase(this.planId)
+          getPlanApiAllCase(this.planId).then((r) => {
-            .then((r) => {
             this.loading = false;
             this.apiCases = r.data;
           });
         } else {
           this.loading = true;
-          getPlanApiFailureCase(this.planId)
+          getPlanApiFailureCase(this.planId).then((r) => {
-            .then((r) => {
             this.loading = false;
             this.apiCases = r.data;
           });
         }
       }
     },
-    rowClick(row) {
+    selectReportContentByReportId(reportId) {
-      this.showResponse = false;
-      this.$nextTick(() => {
-        if (this.isTemplate) {
-          if (row.response) {
-            this.showResponse = true;
-            this.response = JSON.parse(row.response);
-          }
-        } else if (this.isShare) {
       this.responseLoading = true;
-          getShareApiReport(this.shareId, row.id)
+      apiDefinitionReportGet(reportId).then((response) => {
-            .then((r) => {
-              this.responseLoading = false;
-              let data = r.data;
-              if (data && data.content) {
-                this.showResponse = true;
-                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;
@@ -249,10 +249,52 @@ export default {
           }
         }
       });
+    },
+    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(() => {
+        if (this.isTemplate) {
+          if (row.response) {
+            this.showResponse = true;
+            this.response = JSON.parse(row.response);
+          }
+        } else if (this.isShare) {
+          if (row.reportId) {
+            this.selectShareReportContentByReportId(row.reportId);
           } else {
             this.responseLoading = true;
-            apiDefinitionReportGetDb(row.id)
+            getShareApiReport(this.shareId, row.id).then((r) => {
-              .then((r) => {
+              this.responseLoading = false;
+              let data = r.data;
+              if (data && data.content) {
+                this.showResponse = true;
+                this.response = JSON.parse(data.content);
+              }
+            });
+          }
+        } else {
+          if (row.reportId) {
+            this.selectReportContentByReportId(row.reportId);
+          } else {
+            this.responseLoading = true;
+            apiDefinitionReportGetDb(row.id).then((r) => {
               this.responseLoading = false;
               let data = r.data;
               if (data && data.content) {
@@ -267,13 +309,12 @@ export default {
           }
         }
       });
-    }
+    },
-  }
+  },
-}
+};
 </script>
 
 <style scoped>
-
 .el-card :deep(.el-card__body) {
   height: 550px;
 }