feat: delete versions (#109)

Author: bobbyg603

src/versions/versions-api-client/versions-api-client.e2e.ts

@@ -44,6 +44,17 @@ describe('VersionsApiClient', () => {
         });
     });
 
+    describe('deleteVersions', () => {
+        it('should return 200 for delete with valid database, application and version', async () => {
+            const response = await client.deleteVersions(
+                database,
+                [{ application, version }]
+            );
+
+            expect(response.status).toEqual(200);
+        });
+    });
+
     describe('putFullDumps', () => {
         it('should return 403 because account is not licensed', async () => {
             const result = await client.putFullDumps(

src/versions/versions-api-client/versions-api-client.spec.ts

@@ -77,6 +77,38 @@ describe('VersionsApiClient', () => {
         });
     });
 
+    describe('deleteVersions', () => {
+        it('should call delete with route containing database and appVersions', async () => {
+            const appVersions = [
+                { application: 'myConsoleCrasher', version: '2022.4.20.0' },
+                { application: 'myConsoleCrasher', version: '2022.4.20.1' }
+            ];
+            await versionsApiClient.deleteVersions(database, appVersions);
+
+            expect(fakeBugSplatApiClient.fetch).toHaveBeenCalledWith(
+                `/api/versions?database=${database}&appVersions=${appVersions[0].application},${appVersions[0].version},${appVersions[1].application},${appVersions[1].version}`,
+                jasmine.anything()
+            );
+        });
+
+        describe('error', () => {
+            it('should throw if response is not 200', async () => {
+                const fakeErrorResponse = createFakeResponseBody(400);
+                fakeBugSplatApiClient.fetch.and.resolveTo(fakeErrorResponse);
+
+                await expectAsync(versionsApiClient.deleteVersions(database, [])).toBeRejectedWithError(`Error deleting symbols for ${database}- status 400`);
+            });
+
+            it('should throw if response json Status is \'Failed\'', async () => {
+                const message = '🤮';
+                const fakeErrorResponse = createFakeResponseBody(200, { Status: 'Failed', Error: message });
+                fakeBugSplatApiClient.fetch.and.resolveTo(fakeErrorResponse);
+
+                await expectAsync(versionsApiClient.deleteVersions(database, [])).toBeRejectedWithError(message);
+            });
+        });
+    });
+
     describe('putFullDumps', () => {
         const fullDumps = true;
         let result;

src/versions/versions-api-client/versions-api-client.ts

@@ -28,6 +28,29 @@ export class VersionsApiClient {
         };
     }
 
+    async deleteVersions(database: string, appVersions: Array<{ application: string, version: string }>): Promise<BugSplatResponse> {
+        const appVersionsParam = appVersions.reduce((prev, curr) => [...prev, curr.application, curr.version], [] as Array<string>).join(',');
+        const route = `${this.route}?database=${database}&appVersions=${appVersionsParam}`;
+        const request = {
+            method: 'DELETE',
+            cache: 'no-cache',
+            credentials: 'include',
+            redirect: 'follow'
+        } as RequestInit;
+
+        const response = await this._client.fetch(route, request);
+        if (response.status !== 200) {
+            throw new Error(`Error deleting symbols for ${database}-${appVersionsParam} status ${response.status}`);
+        }
+
+        const json = await response.json() as Response;
+        if (json.Status === 'Failed') {
+            throw new Error(json.Error);
+        }
+
+        return response;
+    }
+
     async putFullDumps(
         database: string,
         application: string,