Replace hard coded ids in E2E custom commands and helpers (#20729)

* Replace hard coded ids in `question.js`

* Replace hard coded ids in ad hoc question helpers

* Replace hard coded ids in `sandboxTable` custom command

* Replace hard coded ids in `updatePermissions` custom command

* Avoid circular dependencies

frontend/test/__support__/e2e/commands/api/question.js

+import { SAMPLE_DB_ID } from "__support__/e2e/cypress_data";
+
 Cypress.Commands.add("createQuestion", (questionDetails, customOptions) => {
   const { name, query } = questionDetails;
 
@@ -50,7 +52,7 @@ function question(
     dataset = false,
     native,
     query,
-    database = 1,
+    database = SAMPLE_DB_ID,
     display = "table",
     visualization_settings = {},
     collection_id,

frontend/test/__support__/e2e/commands/permissions/sandboxTable.js

+import { USER_GROUPS } from "__support__/e2e/cypress_data";
+
+const { COLLECTION_GROUP } = USER_GROUPS;
+
 Cypress.Commands.add(
   "sandboxTable",
   ({
     attribute_remappings = {},
     card_id = null,
-    group_id = 3,
+    group_id = COLLECTION_GROUP,
     table_id = 2,
   } = {}) => {
     // Extract the name of the table, as well as `schema` and `db_id` that we'll need later on for `cy.updatePermissionsSchemas()`

frontend/test/__support__/e2e/commands/permissions/updatePermissions.js

+import { SAMPLE_DB_ID, USER_GROUPS } from "__support__/e2e/cypress_data";
+
+const { COLLECTION_GROUP } = USER_GROUPS;
+
 /**
  * PERMISSIONS
  *
@@ -34,7 +38,11 @@ Cypress.Commands.add(
 
 Cypress.Commands.add(
   "updatePermissionsSchemas",
-  ({ schemas = {}, user_group = 3, database_id = 1 } = {}) => {
+  ({
+    schemas = {},
+    user_group = COLLECTION_GROUP,
+    database_id = SAMPLE_DB_ID,
+  } = {}) => {
     if (typeof schemas !== "object") {
       throw new Error("`schemas` must be an object!");
     }

frontend/test/__support__/e2e/helpers/e2e-ad-hoc-question-helpers.js

+import { SAMPLE_DB_ID } from "__support__/e2e/cypress_data";
+
 export function adhocQuestionHash(question) {
   if (question.display) {
     // without "locking" the display, the QB will run its picking logic and override the setting
@@ -32,7 +34,7 @@ export function visitQuestionAdhoc(question, { callback, mode } = {}) {
  * @param {{database:number, table: number, mode: (undefined|"notebook"), limit: number, callback: function}} config
  */
 export function openTable({
-  database = 1,
+  database = SAMPLE_DB_ID,
   table,
   mode = null,
   limit,
@@ -74,7 +76,7 @@ function getInterceptDetails(question, mode) {
   // Therefore, there is no `dataset` to wait for.
   // But we need to make sure the schema for our database is loaded before we can proceed.
   if (mode === "notebook") {
-    return ["/api/database/1/schema/PUBLIC", "publicSchema"];
+    return [`/api/database/${SAMPLE_DB_ID}/schema/PUBLIC`, "publicSchema"];
   }
 
   const {