From e33158fd1b9336f939513778deea25b536beac96 Mon Sep 17 00:00:00 2001
From: peters <p1100i@gmx.com>
Date: Sun, 4 Feb 2024 20:51:31 +0100
Subject: [PATCH] Rework development mode related inferno warnings #1573 #1582
 (#1666)

* adds checking of variable `SKIP_INFERNO_WARNINGS` to skip warnings
  in non-production mode

* `SKIP_INFERNO_WARNINGS` can be also declared globally for browser
  functionality, or just defined on the process.env in case of node

* FYI: JEST related warning skip about development mode was removed
  with 2ed75dc9e8d8dfa when fixing #1582, only the skipping of
  minified code warning was in place
---
 packages/inferno/src/core/types.ts |  2 ++
 packages/inferno/src/index.ts      | 39 ++++++++++++++++--------------
 2 files changed, 23 insertions(+), 18 deletions(-)

diff --git a/packages/inferno/src/core/types.ts b/packages/inferno/src/core/types.ts
index 9a098868d..1d9bb9012 100644
--- a/packages/inferno/src/core/types.ts
+++ b/packages/inferno/src/core/types.ts
@@ -2326,4 +2326,6 @@ declare global {
       semantics: Inferno.DetailedHTMLProps<Inferno.HTMLAttributes<MathMLElement>, MathMLElement>;
     }
   }
+
+  var SKIP_INFERNO_WARNINGS: string;
 }
diff --git a/packages/inferno/src/index.ts b/packages/inferno/src/index.ts
index c0bc4fea0..ca325c041 100644
--- a/packages/inferno/src/index.ts
+++ b/packages/inferno/src/index.ts
@@ -35,27 +35,30 @@ import { createRef, forwardRef, mountRef } from './core/refs';
 export * from './core/types';
 
 if (process.env.NODE_ENV !== 'production') {
-  // Checks if Inferno is running in jest testing environment.
-  const testingEnv =
-    typeof process === 'object' && process.env?.JEST_WORKER_ID !== undefined;
+  const skipWarnings =
+    typeof SKIP_INFERNO_WARNINGS !== 'undefined' ||
+    (typeof process === 'object' &&
+      (process.env?.SKIP_INFERNO_WARNINGS !== undefined ||
+        process.env?.JEST_WORKER_ID !== undefined));
 
-  // This message informs developers that they are using development mode (can happen
-  // in production because of bundling mistakes) and, therefore, Inferno is slower
-  // than in production mode. Skipping the notification for testing mode to keep testing
-  // console clear.
+  if (!skipWarnings) {
+    // This message informs developers that they are using development mode
+    // (can happen in production because of bundling mistakes) and, therefore,
+    // Inferno is slower than in production mode.
+    console.log('Inferno is in development mode.');
 
-  // eslint-disable-next-line @typescript-eslint/explicit-function-return-type
-  const testFunc = function testFn() {};
-  console.log('Inferno is in development mode.');
+    // eslint-disable-next-line @typescript-eslint/explicit-function-return-type
+    const testFunc = function testFn() {};
 
-  // eslint-disable-next-line
-  if (!testingEnv && !((testFunc as Function).name || testFunc.toString()).includes('testFn')) {
-    warning(
-      "It looks like you're using a minified copy of the development build " +
-        'of Inferno. When deploying Inferno apps to production, make sure to use ' +
-        'the production build which skips development warnings and is faster. ' +
-        'See http://infernojs.org for more details.',
-    );
+    // eslint-disable-next-line
+    if (!((testFunc as Function).name || testFunc.toString()).includes('testFn')) {
+      warning(
+        "It looks like you're using a minified copy of the development build " +
+          'of Inferno. When deploying Inferno apps to production, make sure to use ' +
+          'the production build which skips development warnings and is faster. ' +
+          'See http://infernojs.org for more details.',
+      );
+    }
   }
 }