fix: lack of error msg of download binary file

Author: LinuxSuRen

console/atest-ui/src/views/TestCase.vue

@@ -128,11 +128,8 @@ const handleTestResult = (e: any): void => {
   if (!isHistoryTestCase.value) {
     handleTestResultError(e)
   }
-  const isFilePath = e.body.startsWith("isFilePath-")
-
-  if(isFilePath){
-    isResponseFile.value = true
-  } else if(e.body !== ''){
+  isResponseFile.value = e.body.startsWith("isFilePath-")
+  if(!isResponseFile.value && e.body !== ''){
     testResult.value.bodyLength = e.body.length
     try {
       // Try to parse as JSON, fallback to plain text if parsing fails
@@ -149,11 +146,11 @@ const handleTestResult = (e: any): void => {
     }
   }
 
-    Cache.SetTestCaseResponseCache(suite + '-' + name, {
-      body: testResult.value.bodyObject,
-      output: e.output,
-      statusCode: testResult.value.statusCode
-    } as TestCaseResponse)
+  Cache.SetTestCaseResponseCache(suite + '-' + name, {
+    body: testResult.value.bodyObject,
+    output: e.output,
+    statusCode: testResult.value.statusCode
+  } as TestCaseResponse)
 
   parameters.value = [];
 }
@@ -436,7 +433,9 @@ function downloadResponseFile(){
       } else {
         console.error('No data to download.');
       }
-    })
+    }, (e) => {
+      UIAPI.ErrorTip(e);
+    });
 }
 
 function setDefaultValues(e) {

console/atest-ui/src/views/net.ts

@@ -67,7 +67,7 @@ async function DefaultResponseProcess(response: any): Promise<any> {
 
     // For OK responses, handle content based on Content-Type
     const contentType = response.headers.get('Content-Type') ?? '';
-    if (contentType.startsWith('text/plain')) {
+    if (isTextReadableExpectJSON(contentType)) {
         // For text/plain, directly return the text
         return await response.text();
     }
@@ -90,6 +90,11 @@ async function DefaultResponseProcess(response: any): Promise<any> {
     }
 }
 
+const isTextReadableExpectJSON = (contentType: string): boolean => {
+    // Check if the content type is text-based
+    return contentType.startsWith('text/') || contentType === 'application/javascript';
+}
+
 interface AppVersion {
     version: string
     commit: string

console/atest-ui/src/views/types.ts

@@ -37,7 +37,7 @@ export interface Suite {
 
 export interface TestResult {
   body: string
-  bodyObject: {}
+  bodyObject: {} | null
   bodyText: string
   bodyLength: number
   output: string
@@ -46,7 +46,7 @@ export interface TestResult {
   header: Pair[]
 
   // inner fields
-  originBodyObject:{}
+  originBodyObject:{} | null
 }
 
 export interface Pair {