fix: simplify logGherkin configuration and pass when defined

Shelex · Sep 29, 2021 · 51a7be1 · 51a7be1
1 parent 6a5865c
commit 51a7be1
Showing 1 changed file with 6 additions and 12 deletions.
diff --git a/reporter/index.js b/reporter/index.js
@@ -31,18 +31,12 @@ const shouldEnableGherkinLogging = () => {
     const isLogCypressDefined = typeof logCypress !== 'undefined';
     const isLogGherkinDefined = typeof logGherkin !== 'undefined';
 
-    // enable by default
-    if (!isLogCypressDefined && !isLogGherkinDefined) {
-        return true;
-    }
-
-    // inherit logCypress in case directly set
-    if (isLogCypressDefined && !isLogGherkinDefined) {
-        return logCypress;
-    }
-
-    // use env var
-    return logGherkin !== false;
+    return isLogGherkinDefined
+        ? logGherkin
+        : // in case logGherkin is not defined use logCypress value or true by default
+        isLogCypressDefined
+        ? logCypress
+        : true;
 };
 
 const config = {