Update flare-ui/src/app/components/test/test.component.ts

flare-ui/src/app/components/test/test.component.ts

@@ -416,7 +416,7 @@ export class TestComponent implements OnInit {
           }
         }).toPromise();
     
-        // 2. Create test project with version that uses the API
+        // 2. Create test project
         const testProjectName = `test_project_${Date.now()}`;
         const createProjectResponse = await this.apiService.createProject({
           name: testProjectName,
@@ -435,13 +435,14 @@ export class TestComponent implements OnInit {
     
         testProjectId = createProjectResponse.id;
     
-        // 3. Update the version to add an intent that uses our API
+        // 3. Get the first version
         const version = createProjectResponse.versions[0];
         if (!version) {
           throw new Error('No version found in created project');
         }
     
-        await this.apiService.updateVersion(testProjectId!, version.id, {
+        // 4. Update the version to add an intent that uses our API
+        await this.apiService.updateVersion(testProjectId, version.id, {
           caption: version.caption,
           general_prompt: 'Test prompt',
           llm: version.llm,
@@ -452,18 +453,18 @@ export class TestComponent implements OnInit {
             detection_prompt: 'Test detection',
             examples: ['test example'],
             parameters: [],
-            action: testApiName,  // This is where we use the API
+            action: testApiName,
             fallback_timeout_prompt: 'Timeout',
             fallback_error_prompt: 'Error'
           }],
           last_update_date: version.last_update_date
         }).toPromise();
     
-        // 4. Try to delete the API - this should fail
+        // 5. Try to delete the API - this should fail with 400
         try {
           await this.apiService.deleteAPI(testApiName).toPromise();
           
-          // If we reach here, deletion succeeded when it shouldn't have
+          // If deletion succeeded, test failed
           return {
             name: 'API used in intent cannot be deleted',
             status: 'FAIL',
@@ -471,9 +472,18 @@ export class TestComponent implements OnInit {
             duration_ms: Date.now() - start
           };
         } catch (deleteError: any) {
-          // Check if we got the expected error
+          // Check if we got the expected 400 error
+          const errorMessage = deleteError.error?.detail || deleteError.message || '';
           const isExpectedError = deleteError.status === 400 && 
-                                 deleteError.error?.detail?.includes('API is used');
+                                 errorMessage.includes('API is used');
+          
+          if (!isExpectedError) {
+            console.error('Delete API Error Details:', {
+              status: deleteError.status,
+              error: deleteError.error,
+              message: errorMessage
+            });
+          }
           
           return {
             name: 'API used in intent cannot be deleted',
@@ -481,29 +491,29 @@ export class TestComponent implements OnInit {
             duration_ms: Date.now() - start,
             details: isExpectedError 
               ? 'Correctly prevented deletion of API in use' 
-              : `Unexpected error: Status ${deleteError.status}, Detail: ${deleteError.error?.detail || deleteError.message}`
+              : `Unexpected error: Status ${deleteError.status}, Message: ${errorMessage}`
           };
         }
-      } catch (error: any) {
+      } catch (setupError: any) {
         return {
           name: 'API used in intent cannot be deleted',
           status: 'FAIL',
-          error: `Test setup failed: ${error.message}`,
+          error: `Test setup failed: ${setupError.message || setupError}`,
           duration_ms: Date.now() - start
         };
       } finally {
-        // Cleanup
+        // Cleanup: first delete project, then API
         try {
           if (testProjectId !== undefined) {
             await this.apiService.deleteProject(testProjectId).toPromise();
           }
+        } catch {}
+        
+        try {
           if (testApiName) {
-            // Now we can delete the API since the project is gone
             await this.apiService.deleteAPI(testApiName).toPromise();
           }
-        } catch (cleanupError) {
-          // Ignore cleanup errors
-        }
+        } catch {}
       }
     });