tests: ensuring an error is thrown when no containers have been found

containers · Jan 24, 2024 · 7117692 · 7117692
1 parent c98fa5c
commit 7117692
Show file tree

Hide file tree

Showing 2 changed files with 43 additions and 0 deletions.
diff --git a/packages/backend/src/managers/applicationManager.spec.ts b/packages/backend/src/managers/applicationManager.spec.ts
@@ -199,4 +199,46 @@ describe('pullApplication', () => {
     expect(cloneRepositoryMock).not.toHaveBeenCalled();
     expect(downloadModelMainSpy).not.toHaveBeenCalled();
   });
+
+  test('pullApplication should mark the loading config as error if not container are found', async () => {
+    mockForPullApplication({
+      recipeFolderExists: true,
+    });
+    getLocalModelsSpy.mockReturnValue([
+      {
+        id: 'model1',
+        file: 'model1.file',
+      },
+    ]);
+
+    const recipe: Recipe = {
+      id: 'recipe1',
+      name: 'Recipe 1',
+      categories: [],
+      description: '',
+      readme: '',
+      repository: 'repo',
+    };
+    const model: ModelInfo = {
+      id: 'model1',
+      description: '',
+      hw: '',
+      license: '',
+      name: 'Model 1',
+      popularity: 1,
+      registry: '',
+      url: '',
+    };
+
+    mocks.parseYamlMock.mockReturnValue({
+      application: {
+        containers: [],
+      },
+    });
+
+    await expect(manager.pullApplication(recipe, model)).rejects.toThrowError('No containers available.');
+
+    expect(cloneRepositoryMock).not.toHaveBeenCalled();
+    expect(downloadModelMainSpy).not.toHaveBeenCalled();
+  });
 });
diff --git a/packages/backend/src/managers/applicationManager.ts b/packages/backend/src/managers/applicationManager.ts
@@ -139,6 +139,7 @@ export class ApplicationManager {
       // Mark as failure.
       loadingConfiguration.state = 'error';
       taskUtil.setTask(loadingConfiguration);
+      throw new Error('No containers available.');
     }
 
     const localModels = this.modelsManager.getLocalModels();