Merge branch 'Aiven-Open:main' into 2001-use-id-instead-of-name-for-q…

…uerying-teams
Aiven-Open · Jan 11, 2024 · dcca327 · dcca327
2 parents c696919 + deb1c31
commit dcca327
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 4 deletions.
diff --git a/coral/src/app/components/AsyncNativeSelectWrapper.tsx b/coral/src/app/components/AsyncNativeSelectWrapper.tsx
@@ -1,6 +1,7 @@
 import { NativeSelect, NativeSelectProps, useToast } from "@aivenio/aquarium";
 import { isValidElement, ReactElement, ReactNode, useEffect } from "react";
 import { parseErrorMsg } from "src/services/mutation-utils";
+import { isDevMode } from "src/services/is-dev-mode";
 
 function isNativeSelectComponent(
   child: ReactNode
@@ -39,9 +40,14 @@ function AsyncNativeSelectWrapper(props: AsyncNativeSelectWrapperProps) {
   // <NativeSelect> as a child component
   useEffect(() => {
     if (!isNativeSelectComponent(children)) {
-      throw new Error(
-        "Invalid child component. `AsyncNativeSelectWrapper` only accepts `NativeSelect` as a child."
-      );
+      const errorMessage =
+        "Invalid child component. `AsyncNativeSelectWrapper` only accepts `NativeSelect` as a child.";
+
+      if (isDevMode()) {
+        throw new Error(errorMessage);
+      } else {
+        console.error(errorMessage);
+      }
     }
   }, [children]);
 

diff --git a/coral/src/app/main.tsx b/coral/src/app/main.tsx
@@ -18,8 +18,9 @@ import { AuthProvider } from "src/app/context-provider/AuthProvider";
 import { BasePage } from "src/app/layout/page/BasePage";
 // https://github.com/microsoft/monaco-editor/tree/main/samples/browser-esm-vite-react
 import "/src/services/configure-monaco-editor";
+import { isDevMode } from "src/services/is-dev-mode";
 
-const DEV_MODE = import.meta.env.DEV;
+const DEV_MODE = isDevMode();
 
 const root = createRoot(document.getElementById("root") as HTMLElement);
 

diff --git a/coral/src/services/is-dev-mode.ts b/coral/src/services/is-dev-mode.ts
@@ -0,0 +1,5 @@
+function isDevMode() {
+  return import.meta.env.DEV;
+}
+
+export { isDevMode };
diff --git a/coral/test-setup/setup-files-after-env.ts b/coral/test-setup/setup-files-after-env.ts
@@ -13,3 +13,7 @@ Object.defineProperty(global, "crypto", {
 
 process.env.API_BASE_URL = "http://localhost:8080";
 process.env.FEATURE_FLAG_TOPIC_REQUEST = "true";
+
+jest.mock("src/services/is-dev-mode", () => ({
+  isDevMode: () => true,
+}));