From bb50f586a37bbecf72b841e900a4ae3c2d0bbe11 Mon Sep 17 00:00:00 2001
From: Jose Celano <josecelano@gmail.com>
Date: Fri, 11 Aug 2023 13:16:48 +0100
Subject: [PATCH 1/4] test: [#226] E2E test: delete category

The category list is reloaded from the API before mounting the admin
settings tab for categories. It's needed becuase we insert the category
directly in the database no via the UI.
---
 cypress.config.ts                             | 18 ++++--
 cypress/e2e/contexts/category/commands.ts     |  4 ++
 cypress/e2e/contexts/category/fixtures.ts     |  7 ++
 cypress/e2e/contexts/category/specs/add.cy.ts |  2 -
 .../e2e/contexts/category/specs/delete.cy.ts  | 64 +++++++++++++++++++
 cypress/e2e/contexts/category/tasks.ts        | 17 +++++
 cypress/support/commands.ts                   |  1 +
 pages/admin/settings/categories.vue           | 10 ++-
 8 files changed, 114 insertions(+), 9 deletions(-)
 create mode 100644 cypress/e2e/contexts/category/fixtures.ts
 create mode 100644 cypress/e2e/contexts/category/specs/delete.cy.ts

diff --git a/cypress.config.ts b/cypress.config.ts
index 26d306c5..0297ae27 100644
--- a/cypress.config.ts
+++ b/cypress.config.ts
@@ -1,7 +1,7 @@
 import { defineConfig } from "cypress";
 import { grantAdminRole, deleteUser } from "./cypress/e2e/contexts/user/tasks";
 import { deleteTorrent } from "./cypress/e2e/contexts/torrent/tasks";
-import { deleteCategory } from "./cypress/e2e/contexts/category/tasks";
+import { deleteCategory, addCategory } from "./cypress/e2e/contexts/category/tasks";
 import { DatabaseConfig } from "./cypress/e2e/common/database";
 
 function databaseConfig (config: Cypress.PluginConfigOptions): DatabaseConfig {
@@ -15,17 +15,23 @@ export default defineConfig({
     baseUrl: "http://localhost:3000",
     setupNodeEvents (on, config) {
       on("task", {
-        grantAdminRole: ({ username }) => {
-          return grantAdminRole(username, databaseConfig(config));
+        // Category context
+        deleteCategory: ({ name }) => {
+          return deleteCategory(name, databaseConfig(config));
+        },
+        addCategory: ({ name }) => {
+          return addCategory(name, databaseConfig(config));
         },
+        // Torrent context
         deleteTorrent: ({ infohash }) => {
           return deleteTorrent(infohash, databaseConfig(config));
         },
+        // User context
+        grantAdminRole: ({ username }) => {
+          return grantAdminRole(username, databaseConfig(config));
+        },
         deleteUser: ({ username }) => {
           return deleteUser(username, databaseConfig(config));
-        },
-        deleteCategory: ({ name }) => {
-          return deleteCategory(name, databaseConfig(config));
         }
       });
     }
diff --git a/cypress/e2e/contexts/category/commands.ts b/cypress/e2e/contexts/category/commands.ts
index 9bc06db0..f6429725 100644
--- a/cypress/e2e/contexts/category/commands.ts
+++ b/cypress/e2e/contexts/category/commands.ts
@@ -3,3 +3,7 @@
 Cypress.Commands.add("delete_category", (name) => {
   cy.task("deleteCategory", { name });
 });
+
+Cypress.Commands.add("add_category", (name) => {
+  cy.task("addCategory", { name });
+});
diff --git a/cypress/e2e/contexts/category/fixtures.ts b/cypress/e2e/contexts/category/fixtures.ts
new file mode 100644
index 00000000..cbde8076
--- /dev/null
+++ b/cypress/e2e/contexts/category/fixtures.ts
@@ -0,0 +1,7 @@
+export function random_category_name (): string {
+  return `category-${random_category_id()}`;
+}
+
+function random_category_id (): number {
+  return Math.floor(Math.random() * 1000000);
+}
diff --git a/cypress/e2e/contexts/category/specs/add.cy.ts b/cypress/e2e/contexts/category/specs/add.cy.ts
index dd244657..0c9d85a4 100644
--- a/cypress/e2e/contexts/category/specs/add.cy.ts
+++ b/cypress/e2e/contexts/category/specs/add.cy.ts
@@ -50,7 +50,6 @@ describe("A non admin authenticated user", () => {
 
   it("should not be able to add a new category", () => {
     cy.visit("/admin/settings/categories");
-
     cy.contains("Please login to manage admin settings.");
   });
 });
@@ -58,7 +57,6 @@ describe("A non admin authenticated user", () => {
 describe("A guest user", () => {
   it("should not be able to add a new category", () => {
     cy.visit("/admin/settings/categories");
-
     cy.contains("Please login to manage admin settings.");
   });
 });
diff --git a/cypress/e2e/contexts/category/specs/delete.cy.ts b/cypress/e2e/contexts/category/specs/delete.cy.ts
new file mode 100644
index 00000000..1e7c97b6
--- /dev/null
+++ b/cypress/e2e/contexts/category/specs/delete.cy.ts
@@ -0,0 +1,64 @@
+import { RegistrationForm, random_user_registration_data } from "../../user/registration";
+import { random_category_name } from "../fixtures";
+
+describe("The admin user", () => {
+  let registration_form: RegistrationForm;
+
+  before(() => {
+    registration_form = random_user_registration_data();
+    cy.register_as_admin_and_login(registration_form);
+  });
+
+  after(() => {
+    cy.delete_user(registration_form.username);
+  });
+
+  it("should be able to delete a category", () => {
+    const category_name = random_category_name();
+
+    cy.add_category(category_name);
+
+    // Go to admin settings
+    cy.get("div[data-cy=\"user-menu\"]").click();
+    cy.get("li[data-cy=\"admin-settings-link\"]").click();
+
+    // Click categories tab
+    cy.contains("a", "categories").click();
+
+    // Delete the category
+    cy.get(`button[data-cy="delete-category-${category_name}"]`).click();
+
+    cy.on("window:confirm", (str) => {
+      expect(str).to.equal(`Are you sure you want to delete ${category_name}?`);
+    });
+
+    cy.on("window:confirm", () => true);
+
+    cy.get(`[data-cy="delete-category-${category_name}"]`).should("not.exist");
+  });
+});
+
+describe("A non admin authenticated user", () => {
+  let registration_form: RegistrationForm;
+
+  before(() => {
+    registration_form = random_user_registration_data();
+    cy.register_and_login(registration_form);
+  });
+
+  after(() => {
+    cy.delete_user(registration_form.username);
+  });
+
+  it("should not be able to delete category", () => {
+    cy.visit("/admin/settings/categories");
+    cy.contains("Please login to manage admin settings.");
+  });
+});
+
+describe("A guest user", () => {
+  it("should not be able to delete a category", () => {
+    cy.visit("/admin/settings/categories");
+    cy.contains("Please login to manage admin settings.");
+  });
+});
diff --git a/cypress/e2e/contexts/category/tasks.ts b/cypress/e2e/contexts/category/tasks.ts
index 4273bdae..3ff4660f 100644
--- a/cypress/e2e/contexts/category/tasks.ts
+++ b/cypress/e2e/contexts/category/tasks.ts
@@ -12,6 +12,16 @@ export const deleteCategory = async (name: string, db_config: DatabaseConfig): P
   }
 };
 
+// Task to add a new category
+export const addCategory = async (name: string, db_config: DatabaseConfig): Promise<string> => {
+  try {
+    const result = await runDatabaseQuery(addCategoryQuery(name), db_config);
+    return name;
+  } catch (err) {
+    return await Promise.reject(err);
+  }
+};
+
 // Database query specifications
 
 function deleteCategoryQuery (name: string): DatabaseQuery {
@@ -20,3 +30,10 @@ function deleteCategoryQuery (name: string): DatabaseQuery {
     params: [name]
   };
 }
+
+function addCategoryQuery (name: string): DatabaseQuery {
+  return {
+    query: "INSERT INTO torrust_categories (name) VALUES (?)",
+    params: [name]
+  };
+}
diff --git a/cypress/support/commands.ts b/cypress/support/commands.ts
index 506f37fd..9a7f3099 100644
--- a/cypress/support/commands.ts
+++ b/cypress/support/commands.ts
@@ -22,6 +22,7 @@ declare global {
       delete_torrent(torrent_info: TestTorrentInfo, infohash: string): Chainable<void>
       // Category
       delete_category(name: string): Chainable<void>
+      add_category(name: string): Chainable<void>
     }
   }
 }
diff --git a/pages/admin/settings/categories.vue b/pages/admin/settings/categories.vue
index 6fde6b6b..38dcb918 100644
--- a/pages/admin/settings/categories.vue
+++ b/pages/admin/settings/categories.vue
@@ -4,7 +4,7 @@
       <template v-for="category in categories">
         <div class="flex justify-between p-2 rounded bg-base-100">
           <span class="text-base-content">{{ category.name }} ({{ category.num_torrents }})</span>
-          <button class="text-error-content hover:text-error" @click="deleteCategory(category.name)">
+          <button :data-cy="getDeleteButtonDataCy(category.name)" class="text-error-content hover:text-error" @click="deleteCategory(category.name)">
             Delete
           </button>
         </div>
@@ -30,6 +30,14 @@ const rest = useRestApi().value;
 const newCategory = ref("");
 const addingCategory = ref(false);
 
+onBeforeMount(() => {
+  getCategories();
+});
+
+function getDeleteButtonDataCy (name: string) {
+  return "delete-category-" + name.toLowerCase().replace(/ /g, "-");
+}
+
 function addCategory () {
   if (newCategory.value) {
     addingCategory.value = true;

From 90332ee551b7ebc9ced7754b4287953188b0a2ea Mon Sep 17 00:00:00 2001
From: Jose Celano <josecelano@gmail.com>
Date: Fri, 11 Aug 2023 13:32:14 +0100
Subject: [PATCH 2/4] refactor: [#226] rename cypress commands

to diferentiate from browser macros (login, logout, ... things done via the
browser) and tasks (like running database queries, ... things done
indirectly).
---
 cypress/e2e/contexts/category/commands.ts     |  4 ++--
 cypress/e2e/contexts/category/specs/add.cy.ts |  8 +++----
 .../e2e/contexts/category/specs/delete.cy.ts  |  6 ++---
 cypress/e2e/contexts/torrent/commands.ts      |  2 +-
 .../e2e/contexts/torrent/specs/download.cy.ts |  8 +++----
 .../e2e/contexts/torrent/specs/upload.cy.ts   |  2 +-
 cypress/e2e/contexts/user/commands.ts         |  2 +-
 .../contexts/user/specs/authentication.cy.ts  |  4 ++--
 cypress/support/commands.ts                   | 22 +++++++++++--------
 9 files changed, 31 insertions(+), 27 deletions(-)

diff --git a/cypress/e2e/contexts/category/commands.ts b/cypress/e2e/contexts/category/commands.ts
index f6429725..2f1c492b 100644
--- a/cypress/e2e/contexts/category/commands.ts
+++ b/cypress/e2e/contexts/category/commands.ts
@@ -1,9 +1,9 @@
 // Custom commands for category context
 
-Cypress.Commands.add("delete_category", (name) => {
+Cypress.Commands.add("delete_category_from_database", (name) => {
   cy.task("deleteCategory", { name });
 });
 
-Cypress.Commands.add("add_category", (name) => {
+Cypress.Commands.add("add_category_to_database", (name) => {
   cy.task("addCategory", { name });
 });
diff --git a/cypress/e2e/contexts/category/specs/add.cy.ts b/cypress/e2e/contexts/category/specs/add.cy.ts
index 0c9d85a4..792b6313 100644
--- a/cypress/e2e/contexts/category/specs/add.cy.ts
+++ b/cypress/e2e/contexts/category/specs/add.cy.ts
@@ -9,12 +9,12 @@ describe("The admin user", () => {
   });
 
   after(() => {
-    cy.delete_user(registration_form.username);
+    cy.delete_user_from_database(registration_form.username);
   });
 
   it("should be able to add a new category", () => {
     // Make sure the category does not exist
-    cy.delete_category("new category");
+    cy.delete_category_from_database("new category");
 
     // Go to admin settings
     cy.get("div[data-cy=\"user-menu\"]").click();
@@ -32,7 +32,7 @@ describe("The admin user", () => {
     // The new category should appear in the list
     cy.contains("new category (0)");
 
-    cy.delete_category("new category");
+    cy.delete_category_from_database("new category");
   });
 });
 
@@ -45,7 +45,7 @@ describe("A non admin authenticated user", () => {
   });
 
   after(() => {
-    cy.delete_user(registration_form.username);
+    cy.delete_user_from_database(registration_form.username);
   });
 
   it("should not be able to add a new category", () => {
diff --git a/cypress/e2e/contexts/category/specs/delete.cy.ts b/cypress/e2e/contexts/category/specs/delete.cy.ts
index 1e7c97b6..3feea0ac 100644
--- a/cypress/e2e/contexts/category/specs/delete.cy.ts
+++ b/cypress/e2e/contexts/category/specs/delete.cy.ts
@@ -10,13 +10,13 @@ describe("The admin user", () => {
   });
 
   after(() => {
-    cy.delete_user(registration_form.username);
+    cy.delete_user_from_database(registration_form.username);
   });
 
   it("should be able to delete a category", () => {
     const category_name = random_category_name();
 
-    cy.add_category(category_name);
+    cy.add_category_to_database(category_name);
 
     // Go to admin settings
     cy.get("div[data-cy=\"user-menu\"]").click();
@@ -47,7 +47,7 @@ describe("A non admin authenticated user", () => {
   });
 
   after(() => {
-    cy.delete_user(registration_form.username);
+    cy.delete_user_from_database(registration_form.username);
   });
 
   it("should not be able to delete category", () => {
diff --git a/cypress/e2e/contexts/torrent/commands.ts b/cypress/e2e/contexts/torrent/commands.ts
index 7691b1c7..47d76bca 100644
--- a/cypress/e2e/contexts/torrent/commands.ts
+++ b/cypress/e2e/contexts/torrent/commands.ts
@@ -33,7 +33,7 @@ Cypress.Commands.add("upload_torrent", (torrent_info) => {
   cy.get("button[data-cy=\"upload-form-submit\"]").click();
 });
 
-Cypress.Commands.add("delete_torrent", (torrent_info, infohash) => {
+Cypress.Commands.add("delete_torrent_from_database_and_fixture", (torrent_info, infohash) => {
   // Delete the torrent in the database
   cy.task("deleteTorrent", { infohash });
 
diff --git a/cypress/e2e/contexts/torrent/specs/download.cy.ts b/cypress/e2e/contexts/torrent/specs/download.cy.ts
index 50b97951..7ed7500f 100644
--- a/cypress/e2e/contexts/torrent/specs/download.cy.ts
+++ b/cypress/e2e/contexts/torrent/specs/download.cy.ts
@@ -11,7 +11,7 @@ describe("A registered user", () => {
   });
 
   after(() => {
-    cy.delete_user(registration_form.username);
+    cy.delete_user_from_database(registration_form.username);
   });
 
   it("should be able to download a preexisting torrent", () => {
@@ -32,7 +32,7 @@ describe("A registered user", () => {
 
       // Delete the test torrent generated for this test
       const torrentInfoHash = parseInfoHash(interception.response.headers["x-torrust-torrent-infohash"]);
-      cy.delete_torrent(torrent_info, torrentInfoHash);
+      cy.delete_torrent_from_database_and_fixture(torrent_info, torrentInfoHash);
     });
   });
 });
@@ -49,7 +49,7 @@ describe("A guest user", () => {
   });
 
   after(() => {
-    cy.delete_user(uploader_registration_form.username);
+    cy.delete_user_from_database(uploader_registration_form.username);
   });
 
   it("should be able to download a preexisting torrent", () => {
@@ -79,7 +79,7 @@ describe("A guest user", () => {
 
       // Delete the test torrent generated for this test
       const torrentInfoHash = parseInfoHash(interception.response.headers["x-torrust-torrent-infohash"]);
-      cy.delete_torrent(torrent_info, torrentInfoHash);
+      cy.delete_torrent_from_database_and_fixture(torrent_info, torrentInfoHash);
     });
   });
 });
diff --git a/cypress/e2e/contexts/torrent/specs/upload.cy.ts b/cypress/e2e/contexts/torrent/specs/upload.cy.ts
index b28eb201..d9cda849 100644
--- a/cypress/e2e/contexts/torrent/specs/upload.cy.ts
+++ b/cypress/e2e/contexts/torrent/specs/upload.cy.ts
@@ -11,7 +11,7 @@ describe("A registered user", () => {
   });
 
   after(() => {
-    cy.delete_user(registration_form.username);
+    cy.delete_user_from_database(registration_form.username);
   });
 
   it("should be able to upload a torrent", () => {
diff --git a/cypress/e2e/contexts/user/commands.ts b/cypress/e2e/contexts/user/commands.ts
index 8ee08694..7489b01d 100644
--- a/cypress/e2e/contexts/user/commands.ts
+++ b/cypress/e2e/contexts/user/commands.ts
@@ -21,7 +21,7 @@ Cypress.Commands.add("register_as_admin", (registration_form) => {
   cy.task("grantAdminRole", { username: registration_form.username });
 });
 
-Cypress.Commands.add("delete_user", (username) => {
+Cypress.Commands.add("delete_user_from_database", (username) => {
   cy.task("deleteUser", { username });
 });
 
diff --git a/cypress/e2e/contexts/user/specs/authentication.cy.ts b/cypress/e2e/contexts/user/specs/authentication.cy.ts
index 8a039152..1bb5e9c2 100644
--- a/cypress/e2e/contexts/user/specs/authentication.cy.ts
+++ b/cypress/e2e/contexts/user/specs/authentication.cy.ts
@@ -21,7 +21,7 @@ describe("A registered user", () => {
 
     cy.url().should("include", "/torrents");
 
-    cy.delete_user(registration_form.username);
+    cy.delete_user_from_database(registration_form.username);
   });
 });
 
@@ -40,6 +40,6 @@ describe("The website admin", () => {
     // If the user is an admin, the link to admin settings should be available
     cy.get("li[data-cy=\"admin-settings-link\"]");
 
-    cy.delete_user(registration_form.username);
+    cy.delete_user_from_database(registration_form.username);
   });
 });
diff --git a/cypress/support/commands.ts b/cypress/support/commands.ts
index 9a7f3099..f3cc1d09 100644
--- a/cypress/support/commands.ts
+++ b/cypress/support/commands.ts
@@ -7,22 +7,26 @@ import { TestTorrentInfo } from "cypress/e2e/contexts/torrent/test_torrent_info"
 declare global {
   namespace Cypress {
     interface Chainable {
-      // User: Registration
+      // User context: Registration
       register(registration_form: RegistrationForm): Chainable<void>
       register_as_admin(registration_form: RegistrationForm): Chainable<void>
-      delete_user(username: string): Chainable<void>
-      // User: Authentication
+
+      // User context: Authentication
       login(username: string, password: string): Chainable<void>
       logout(): Chainable<void>
-      // User: others
+
+      // User context: Others
       register_and_login(registration_form: RegistrationForm): Chainable<void>
       register_as_admin_and_login(registration_form: RegistrationForm): Chainable<void>
-      // Torrent
+      delete_user_from_database(username: string): Chainable<void>
+
+      // Torrent context
       upload_torrent(torrent_info: TestTorrentInfo): Chainable<void>
-      delete_torrent(torrent_info: TestTorrentInfo, infohash: string): Chainable<void>
-      // Category
-      delete_category(name: string): Chainable<void>
-      add_category(name: string): Chainable<void>
+      delete_torrent_from_database_and_fixture(torrent_info: TestTorrentInfo, infohash: string): Chainable<void>
+
+      // Category context
+      delete_category_from_database(name: string): Chainable<void>
+      add_category_to_database(name: string): Chainable<void>
     }
   }
 }

From 8e84778da1167be1b3b969a65faf06b1ec9bb96e Mon Sep 17 00:00:00 2001
From: Jose Celano <josecelano@gmail.com>
Date: Fri, 11 Aug 2023 13:37:44 +0100
Subject: [PATCH 3/4] refactor: [#226] use random category names

If the test fail for some reason you do not need to clean the database
to run it again.
---
 cypress/e2e/contexts/category/specs/add.cy.ts | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/cypress/e2e/contexts/category/specs/add.cy.ts b/cypress/e2e/contexts/category/specs/add.cy.ts
index 792b6313..63436227 100644
--- a/cypress/e2e/contexts/category/specs/add.cy.ts
+++ b/cypress/e2e/contexts/category/specs/add.cy.ts
@@ -1,4 +1,5 @@
 import { RegistrationForm, random_user_registration_data } from "../../user/registration";
+import { random_category_name } from "../fixtures";
 
 describe("The admin user", () => {
   let registration_form: RegistrationForm;
@@ -13,8 +14,10 @@ describe("The admin user", () => {
   });
 
   it("should be able to add a new category", () => {
+    const category_name = random_category_name();
+
     // Make sure the category does not exist
-    cy.delete_category_from_database("new category");
+    cy.delete_category_from_database(category_name);
 
     // Go to admin settings
     cy.get("div[data-cy=\"user-menu\"]").click();
@@ -24,15 +27,15 @@ describe("The admin user", () => {
     cy.contains("a", "categories").click();
 
     // Fill new category name
-    cy.get("input[data-cy=\"add-category-input\"]").type("new category");
+    cy.get("input[data-cy=\"add-category-input\"]").type(category_name);
 
     // Add category
     cy.get("button[data-cy=\"add-category-button\"]").click();
 
     // The new category should appear in the list
-    cy.contains("new category (0)");
+    cy.contains(`${category_name} (0)`);
 
-    cy.delete_category_from_database("new category");
+    cy.delete_category_from_database(category_name);
   });
 });
 

From 338d2f93f5698940fc03109483d584a16eb43d0e Mon Sep 17 00:00:00 2001
From: Jose Celano <josecelano@gmail.com>
Date: Fri, 11 Aug 2023 15:01:23 +0100
Subject: [PATCH 4/4] refactor: [#226] extract cypress command to go to admin
 settings

---
 cypress/e2e/common/commands.ts                   | 6 ++++++
 cypress/e2e/contexts/category/specs/add.cy.ts    | 4 +---
 cypress/e2e/contexts/category/specs/delete.cy.ts | 6 +++---
 cypress/support/commands.ts                      | 4 ++++
 4 files changed, 14 insertions(+), 6 deletions(-)
 create mode 100644 cypress/e2e/common/commands.ts

diff --git a/cypress/e2e/common/commands.ts b/cypress/e2e/common/commands.ts
new file mode 100644
index 00000000..1f01f832
--- /dev/null
+++ b/cypress/e2e/common/commands.ts
@@ -0,0 +1,6 @@
+// Common commands
+
+Cypress.Commands.add("go_to_settings", () => {
+  cy.get("div[data-cy=\"user-menu\"]").click();
+  cy.get("li[data-cy=\"admin-settings-link\"]").click();
+});
diff --git a/cypress/e2e/contexts/category/specs/add.cy.ts b/cypress/e2e/contexts/category/specs/add.cy.ts
index 63436227..49ffbe07 100644
--- a/cypress/e2e/contexts/category/specs/add.cy.ts
+++ b/cypress/e2e/contexts/category/specs/add.cy.ts
@@ -19,9 +19,7 @@ describe("The admin user", () => {
     // Make sure the category does not exist
     cy.delete_category_from_database(category_name);
 
-    // Go to admin settings
-    cy.get("div[data-cy=\"user-menu\"]").click();
-    cy.get("li[data-cy=\"admin-settings-link\"]").click();
+    cy.go_to_settings();
 
     // Click categories tab
     cy.contains("a", "categories").click();
diff --git a/cypress/e2e/contexts/category/specs/delete.cy.ts b/cypress/e2e/contexts/category/specs/delete.cy.ts
index 3feea0ac..6a38f77c 100644
--- a/cypress/e2e/contexts/category/specs/delete.cy.ts
+++ b/cypress/e2e/contexts/category/specs/delete.cy.ts
@@ -18,9 +18,7 @@ describe("The admin user", () => {
 
     cy.add_category_to_database(category_name);
 
-    // Go to admin settings
-    cy.get("div[data-cy=\"user-menu\"]").click();
-    cy.get("li[data-cy=\"admin-settings-link\"]").click();
+    cy.go_to_settings();
 
     // Click categories tab
     cy.contains("a", "categories").click();
@@ -28,10 +26,12 @@ describe("The admin user", () => {
     // Delete the category
     cy.get(`button[data-cy="delete-category-${category_name}"]`).click();
 
+    // Confirm alert should pop up
     cy.on("window:confirm", (str) => {
       expect(str).to.equal(`Are you sure you want to delete ${category_name}?`);
     });
 
+    // Confirm delete
     cy.on("window:confirm", () => true);
 
     cy.get(`[data-cy="delete-category-${category_name}"]`).should("not.exist");
diff --git a/cypress/support/commands.ts b/cypress/support/commands.ts
index f3cc1d09..9296f10c 100644
--- a/cypress/support/commands.ts
+++ b/cypress/support/commands.ts
@@ -1,12 +1,16 @@
 import "../e2e/contexts/user/commands";
 import "../e2e/contexts/torrent/commands";
 import "../e2e/contexts/category/commands";
+import "../e2e/common/commands";
 import { RegistrationForm } from "../e2e/contexts/user/registration";
 import { TestTorrentInfo } from "cypress/e2e/contexts/torrent/test_torrent_info";
 
 declare global {
   namespace Cypress {
     interface Chainable {
+      // Common command
+      go_to_settings(): Chainable<void>
+
       // User context: Registration
       register(registration_form: RegistrationForm): Chainable<void>
       register_as_admin(registration_form: RegistrationForm): Chainable<void>