Merge pull request #345 from shapehq/develop

Deploy to production

Author: simonbs

.github/dependabot.yml

@@ -14,3 +14,6 @@ updates:
       mui:
         patterns:
           - "@mui/*"
+      typescript-eslint:
+        patterns:
+          - "@typescript-eslint/*"

.github/workflows/test-sql-queries.yml

@@ -2,6 +2,9 @@ name: Test SQL Queries
 on:
   workflow_dispatch:
   pull_request:
+env:
+  PGPASSWORD: postgres
+  DATABASE_NAME: testdb
 jobs:
   test:
     name: Test SQL Queries
@@ -18,8 +21,8 @@ jobs:
           --health-retries=5
         env:
           POSTGRES_USER: postgres
-          POSTGRES_PASSWORD: postgres
-          POSTGRES_DB: testdb
+          POSTGRES_PASSWORD: ${{ env.PGPASSWORD }}
+          POSTGRES_DB: ${{ env.DATABASE_NAME }}
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
@@ -29,10 +32,20 @@ jobs:
             sleep 1
           done
       - name: Run create-tables.sql
-        env:
-          PGPASSWORD: postgres
-        run: psql -h localhost -U postgres -d testdb -f create-tables.sql
+        run: psql -v ON_ERROR_STOP=1 -h localhost -U postgres -d ${{ env.DATABASE_NAME }} -f create-tables.sql
+      - name: Ensure tables were created
+        run: |
+          COUNT=$(psql -v ON_ERROR_STOP=1 -h localhost -U postgres -d ${{ env.DATABASE_NAME }} -t -A -c "SELECT count(*) FROM information_schema.tables WHERE table_schema = 'public';")
+          if [ "$COUNT" -eq 0 ]; then
+            echo "No tables found. Exiting with error."
+            exit 1
+          fi
       - name: Run drop-tables.sql
-        env:
-          PGPASSWORD: postgres
-        run: psql -h localhost -U postgres -d testdb -f drop-tables.sql
+        run: psql -v ON_ERROR_STOP=1 -h localhost -U postgres -d ${{ env.DATABASE_NAME }} -f drop-tables.sql
+      - name: Ensure all tables were dropped
+        run: |
+          COUNT=$(psql -v ON_ERROR_STOP=1 -h localhost -U postgres -d ${{ env.DATABASE_NAME }} -t -A -c "SELECT count(*) FROM information_schema.tables WHERE table_schema = 'public';")
+          if [ "$COUNT" -gt 0 ]; then
+            echo "Tables found: $count. Exiting with error."
+            exit 1
+          fi

.nvmrc

@@ -0,0 +1 @@
+v20.13.1

__test__/common/utils/makeFullRepositoryName.test.ts

@@ -0,0 +1,51 @@
+import { makeFullRepositoryName } from "@/common/utils/makeFullRepositoryName";
+
+test("It returns a sanitized repository name with a suffix", async () => {
+  const result = makeFullRepositoryName({ name: "My Project", suffix: "-repo" });
+  expect(result).toEqual("myproject-repo");
+});
+
+test("It returns a lowercase repository name", async () => {
+  const result = makeFullRepositoryName({ name: "MyProject", suffix: "-repo" });
+  expect(result).toEqual("myproject-repo");
+});
+
+test("It trims spaces from the name", async () => {
+  const result = makeFullRepositoryName({ name: "  My Project  ", suffix: "-repo" });
+  expect(result).toEqual("myproject-repo");
+});
+
+test("It removes non-alphanumeric characters except dashes", async () => {
+  const result = makeFullRepositoryName({ name: "My!@#Pro$%^ject", suffix: "-repo" });
+  expect(result).toEqual("myproject-repo");
+});
+
+test("It replaces spaces with dashes", async () => {
+  const result = makeFullRepositoryName({ name: "My Project", suffix: "-repo" });
+  expect(result).toEqual("myproject-repo");
+});
+
+test("It handles names with multiple spaces correctly", async () => {
+  const result = makeFullRepositoryName({ name: "My    Project", suffix: "-repo" });
+  expect(result).toEqual("myproject-repo");
+});
+
+test("It handles names with no characters left after sanitization", async () => {
+  const result = makeFullRepositoryName({ name: "!@#$%^", suffix: "-repo" });
+  expect(result).toEqual("-repo");
+});
+
+test("It handles names that are already safe", async () => {
+  const result = makeFullRepositoryName({ name: "safe-project-name", suffix: "-repo" });
+  expect(result).toEqual("safe-project-name-repo");
+});
+
+test("It returns just the suffix if the name is an empty string", async () => {
+  const result = makeFullRepositoryName({ name: "", suffix: "-repo" });
+  expect(result).toEqual("-repo");
+});
+
+test("It handles a suffix with special characters correctly", async () => {
+  const result = makeFullRepositoryName({ name: "My Project", suffix: "!-repo" });
+  expect(result).toEqual("myproject!-repo");
+});

__test__/common/utils/makeNewGitHubRepositoryLink.test.ts

@@ -0,0 +1,64 @@
+import { makeNewGitHubRepositoryLink } from "@/common/utils/makeNewGitHubRepositoryLink";
+
+test("It generates a URL with repository name and description", async () => {
+  const result = makeNewGitHubRepositoryLink({
+    repositoryName: "test-repo",
+    description: "A test repository"
+  });
+  expect(result).toEqual(
+    "https://github.com/new?name=test-repo&description=A%20test%20repository&visibility=private"
+  );
+});
+
+test("It encodes special characters in the repository name and description", async () => {
+  const result = makeNewGitHubRepositoryLink({
+    repositoryName: "test repo",
+    description: "A test & description"
+  });
+  expect(result).toEqual(
+    "https://github.com/new?name=test%20repo&description=A%20test%20%26%20description&visibility=private"
+  );
+});
+
+test("It generates a URL with a template repository", async () => {
+  const result = makeNewGitHubRepositoryLink({
+    templateName: "owner/template-repo",
+    repositoryName: "test-repo",
+    description: "A test repository"
+  });
+  expect(result).toEqual(
+    "https://github.com/new?name=test-repo&description=A%20test%20repository&visibility=private&template_owner=owner&template_name=template-repo"
+  );
+});
+
+test("It handles template names with special characters", async () => {
+  const result = makeNewGitHubRepositoryLink({
+    templateName: "owner/template repo",
+    repositoryName: "test-repo",
+    description: "A test repository"
+  });
+  expect(result).toEqual(
+    "https://github.com/new?name=test-repo&description=A%20test%20repository&visibility=private&template_owner=owner&template_name=template%20repo"
+  );
+});
+
+test("It returns the URL without template parameters if templateName is not provided", async () => {
+  const result = makeNewGitHubRepositoryLink({
+    repositoryName: "test-repo",
+    description: "A test repository"
+  });
+  expect(result).toEqual(
+    "https://github.com/new?name=test-repo&description=A%20test%20repository&visibility=private"
+  );
+});
+
+test("It returns a URL even if the template name cannot be split into owner and repository", async () => {
+  const result = makeNewGitHubRepositoryLink({
+    templateName: "invalidTemplateName",
+    repositoryName: "test-repo",
+    description: "A test repository"
+  });
+  expect(result).toEqual(
+    "https://github.com/new?name=test-repo&description=A%20test%20repository&visibility=private"
+  );
+});

drop-tables.sql

@@ -1,5 +1,4 @@
 DROP TABLE oauth_tokens;
-DROP TABLE verification_token;
 DROP TABLE accounts;
 DROP TABLE sessions;
 DROP TABLE users;