Always replace svelte flag with boolean value in non-debug builds.

Author: Robert Jackson

fixtures/development-svelte-builds/expectation6.js

@@ -11,6 +11,10 @@ if (DEPRECATED_PARTIALS) {
   };
 }
 
+if (DEPRECATED_PARTIALS && someOtherThing()) {
+  doStuff();
+}
+
 export let ObjectController;
 if (DEPRECATED_CONTROLLERS) {
   ObjectController = class {

fixtures/development-svelte-builds/expectation7.js

@@ -11,6 +11,10 @@ if (DEPRECATED_PARTIALS) {
   };
 }
 
+if (DEPRECATED_PARTIALS && someOtherThing()) {
+  doStuff();
+}
+
 export let ObjectController;
 
 if (DEPRECATED_CONTROLLERS) {

fixtures/development-svelte-builds/sample.js

@@ -9,6 +9,10 @@ if (DEPRECATED_PARTIALS) {
   };
 }
 
+if (DEPRECATED_PARTIALS && someOtherThing()) {
+  doStuff();
+}
+
 export let ObjectController;
 if (DEPRECATED_CONTROLLERS) {
   ObjectController = class {

fixtures/production-svelte-builds/expectation6.js

@@ -9,6 +9,10 @@ if (false) {
   };
 }
 
+if (false && someOtherThing()) {
+  doStuff();
+}
+
 export let ObjectController;
 if (true) {
   ObjectController = class {

fixtures/production-svelte-builds/expectation7.js

@@ -9,6 +9,10 @@ if (false) {
   };
 }
 
+if (false && someOtherThing()) {
+  doStuff();
+}
+
 export let ObjectController;
 
 if (true) {

fixtures/production-svelte-builds/sample.js

@@ -9,6 +9,10 @@ if (DEPRECATED_PARTIALS) {
   };
 }
 
+if (DEPRECATED_PARTIALS && someOtherThing()) {
+  doStuff();
+}
+
 export let ObjectController;
 if (DEPRECATED_CONTROLLERS) {
   ObjectController = class {
@@ -22,4 +26,4 @@ export default class TheThingToReplaceControllers {
   constructor() {
     this.isNew = true;
   }
-}
+}

src/utils/macros.js

@@ -98,11 +98,10 @@ module.exports = class Macros {
         if (binding !== undefined) {
           binding.referencePaths.forEach(p => {
             let t = builder.t;
-            if (envFlags.DEBUG) {
-              if (svelteMap[source][flag] === false) {
-                if (!p.parentPath.isIfStatement()) {
-                  return;
-                }
+            // in debug builds add an error after a conditional (to ensure if the
+            // specific branch is taken, an error is thrown)
+            if (envFlags.DEBUG && svelteMap[source][flag] === false) {
+              if (p.parentPath.isIfStatement()) {
                 let consequent = p.parentPath.get('consequent');
                 consequent.unshiftContainer(
                   'body',
@@ -115,10 +114,8 @@ module.exports = class Macros {
                   )
                 );
               }
-            } else {
-              if (p.parentPath.isIfStatement()) {
-                p.replaceWith(t.booleanLiteral(svelteMap[source][flag]));
-              }
+            } else if (envFlags.DEBUG === false) {
+              p.replaceWith(t.booleanLiteral(svelteMap[source][flag]));
             }
           });