sped up rating tool

rtjord · Dec 8, 2024 · 5a8eac4 · 5a8eac4
1 parent 231138f
commit 5a8eac4
Show file tree

Hide file tree

Showing 16 changed files with 165 additions and 141 deletions.
diff --git a/backend/src/common/__tests__/zip.test.ts b/backend/src/common/__tests__/zip.test.ts
@@ -35,14 +35,14 @@ describe("File Operations and Compression Functions", () => {
 
     describe("cloneAndZipRepository", () => {
         it("should clone a repository and zip it", async () => {
-            const repoUrl = "https://github.com/isomorphic-git/isomorphic-git"; // Use a real repo URL
+            const repoUrl = "https://github.com/thejoshwolfe/yazl"; // Use a real repo URL
             const zipBuffer = await cloneAndZipRepository(repoUrl);
 
             const zip = await JSZip.loadAsync(zipBuffer);
             const files = Object.keys(zip.files);
 
             expect(files).toContain("README.md");
-        });
+        }, 60000);
     });
 
     describe("zipDirectory", () => {

diff --git a/backend/src/handlers/RegistryReset/index.ts b/backend/src/handlers/RegistryReset/index.ts
@@ -3,6 +3,7 @@ import { APIGatewayProxyHandler } from "aws-lambda";
 import { S3Client } from "@aws-sdk/client-s3";
 import { DynamoDBClient } from "@aws-sdk/client-dynamodb";
 import { DynamoDBDocumentClient } from "@aws-sdk/lib-dynamodb";
+
 const commonPath = process.env.COMMON_PATH || '/opt/nodejs/common';
 const { getEnvVariable } = require(`${commonPath}/utils`);
 const { clearDynamoDBTable } = require(`${commonPath}/dynamodb`);

diff --git a/backend/src/handlers/__tests__/end_to_end/getCost.test.ts b/backend/src/handlers/__tests__/end_to_end/getCost.test.ts
@@ -12,7 +12,7 @@ describe("E2E Test for Get Cost Endpoint", () => {
     let circular2_id: string;
     beforeAll(async () => {
         // Reset the registry before running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
 
         // Upload yazl to the registry
         const requestBody: PackageData = {
@@ -59,7 +59,7 @@ describe("E2E Test for Get Cost Endpoint", () => {
     }, 90000);
     afterAll(async () => {
         // Reset the registry after running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, timeout);
 
     it("should only return total cost if dependency is false", async () => {

diff --git a/backend/src/handlers/__tests__/end_to_end/packageByName.test.ts b/backend/src/handlers/__tests__/end_to_end/packageByName.test.ts
@@ -8,7 +8,7 @@ describe("E2E Test for Package By Name Endpoint", () => {
     let id: string;
     beforeAll(async () => {
         // Reset the registry before running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
 
         // Upload a package with Content to the registry
         const requestBody: PackageData = {
@@ -43,7 +43,7 @@ describe("E2E Test for Package By Name Endpoint", () => {
     }, 90000);
     afterAll(async () => {
         // Reset the registry after running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, 90000);
 
     it("should return a 200 status for a package that exists", async () => {

diff --git a/backend/src/handlers/__tests__/end_to_end/packageByRegex.test.ts b/backend/src/handlers/__tests__/end_to_end/packageByRegex.test.ts
@@ -10,7 +10,7 @@ describe("E2E Test for PackageByRegEx Endpoint", () => {
     let url_id: string;
     beforeAll(async () => {
         // Reset the registry before running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
 
         // Upload a package with Content to the registry
         const requestBody: PackageData = {
@@ -41,7 +41,7 @@ describe("E2E Test for PackageByRegEx Endpoint", () => {
     }, 90000);
     afterAll(async () => {
         // Reset the registry after running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, 90000);
 
     it("should return a 200 status for a package that exists", async () => {

diff --git a/backend/src/handlers/__tests__/end_to_end/packageCreate.test.ts b/backend/src/handlers/__tests__/end_to_end/packageCreate.test.ts
@@ -9,11 +9,11 @@ describe("E2E Test for Package Create Endpoint", () => {
 
     beforeAll(async () => {
         // Reset the registry before running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, timeout);
     afterAll(async () => {
         // Reset the registry after running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, timeout);
     it("should return a 201 status for a package with Content set", async () => {
         const requestBody: PackageData = {

diff --git a/backend/src/handlers/__tests__/end_to_end/packageDelete.test.ts b/backend/src/handlers/__tests__/end_to_end/packageDelete.test.ts
@@ -10,7 +10,7 @@ describe("E2E Test for Package Delete Endpoint", () => {
     let url_id: string;
     beforeAll(async () => {
         // Reset the registry before running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
 
         // Upload a package with Content to the registry
         const requestBody: PackageData = {
@@ -39,7 +39,7 @@ describe("E2E Test for Package Delete Endpoint", () => {
     }, 90000);
     afterAll(async () => {
         // Reset the registry after running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, timeout);
 
     it("should return a 200 status for a package uploaded via Content", async () => {

diff --git a/backend/src/handlers/__tests__/end_to_end/packageRate.test.ts b/backend/src/handlers/__tests__/end_to_end/packageRate.test.ts
@@ -10,7 +10,7 @@ describe("E2E Test for Package Rate Endpoint", () => {
     let url_id: string;
     beforeAll(async () => {
         // Reset the registry before running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
 
         // Upload a package with Content to the registry
         const requestBody: PackageData = {
@@ -39,7 +39,7 @@ describe("E2E Test for Package Rate Endpoint", () => {
     }, 90000);
     afterAll(async () => {
         // Reset the registry after running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, timeout);
 
     it("should return a 200 status for a package with a rating", async () => {

diff --git a/backend/src/handlers/__tests__/end_to_end/packageRecommend.test.ts b/backend/src/handlers/__tests__/end_to_end/packageRecommend.test.ts
@@ -10,18 +10,17 @@ async function upload(name: string, url: string) {
         URL: url,
         debloat: false
     };
-
+    console.log("Uploading package: ", name);
     await axios.post(`${baseUrl}/package`, requestBody);
 }
 
 describe("E2E Test for PackageRecommend Endpoint", () => {
     beforeAll(async () => {
         // Reset the registry before running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
 
         // Upload packages to the registry
         await upload("yazl", "https://www.npmjs.com/package/yazl");
-        await upload("express", "https://www.npmjs.com/package/express");
         await upload("debug", "https://www.npmjs.com/package/debug");
         await upload("inversify", "https://www.npmjs.com/package/inversify");
         await upload("tslib", "https://www.npmjs.com/package/tslib");
@@ -33,7 +32,7 @@ describe("E2E Test for PackageRecommend Endpoint", () => {
     }, 90000);
     afterAll(async () => {
         // Reset the registry after running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, 90000);
 
     it("should return a list of 5 packages with express as the first", async () => {

diff --git a/backend/src/handlers/__tests__/end_to_end/packageRetrieve.test.ts b/backend/src/handlers/__tests__/end_to_end/packageRetrieve.test.ts
@@ -10,7 +10,7 @@ describe("E2E Test for Package Retrieve Endpoint", () => {
     let url_id: string;
     beforeAll(async () => {
         // Reset the registry before running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
 
         // Upload a package with Content to the registry
         const requestBody: PackageData = {
@@ -39,7 +39,7 @@ describe("E2E Test for Package Retrieve Endpoint", () => {
     }, 90000);
     afterAll(async () => {
         // Reset the registry after running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, timeout);
 
     it("should return a 200 status for a package uploaded via Content", async () => {

diff --git a/backend/src/handlers/__tests__/end_to_end/packageUpdate.test.ts b/backend/src/handlers/__tests__/end_to_end/packageUpdate.test.ts
@@ -9,7 +9,7 @@ describe("E2E Test for Package Update Endpoint", () => {
     let url_id: string;
     beforeAll(async () => {
         // Reset the registry before running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
 
         // Upload a package with Content to the registry
         const requestBody: PackageData = {
@@ -38,7 +38,7 @@ describe("E2E Test for Package Update Endpoint", () => {
     }, 60000);
     afterAll(async () => {
         // Reset the registry after running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, timeout);
     it("should return a 200 status for a package with Content set", async () => {
         const requestBody: Package = {

diff --git a/backend/src/handlers/__tests__/end_to_end/packagesList.test.ts b/backend/src/handlers/__tests__/end_to_end/packagesList.test.ts
@@ -9,7 +9,7 @@ describe("E2E Test for Packages List Endpoint", () => {
     const ids: string[] = [];
     beforeAll(async () => {
         // Reset the registry before running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
 
         // Upload a package with Content to the registry
         const requestBody: PackageData = {
@@ -47,7 +47,7 @@ describe("E2E Test for Packages List Endpoint", () => {
     }, 240000);
     afterAll(async () => {
         // Reset the registry after running the tests
-        await axios.delete(`${baseUrl}/reset`);
+        await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
     }, timeout);
 
     it("should fetch all packages and return 200 status", async () => {

diff --git a/backend/src/handlers/__tests__/end_to_end/registryReset.test.ts b/backend/src/handlers/__tests__/end_to_end/registryReset.test.ts
@@ -4,7 +4,7 @@ import { baseUrl } from "./config";
 const timeout = 30000;
 describe("E2E Test for Registry Reset Endpoint", () => {
     it("should return a 200 status", async () => {
-        const response = await axios.delete(`${baseUrl}/reset`);
+        const response = await axios.delete(`${baseUrl}/reset`, { timeout: 60000 });
         expect(response.status).toBe(200);
     }, timeout);
 });