Fix switch case not being included correctly (#3178)

* Fix switch case not being included correctly * Update dependencies
rollup · Oct 20, 2019 · 06047fc · 06047fc
1 parent 73faf06
commit 06047fc
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 1 deletion.
diff --git a/src/ast/nodes/SwitchStatement.ts b/src/ast/nodes/SwitchStatement.ts
@@ -67,7 +67,11 @@ export default class SwitchStatement extends StatementBase {
 				context.brokenFlow = brokenFlow;
 			}
 		}
-		if (this.defaultCase !== null && !(minBrokenFlow === BROKEN_FLOW_BREAK_CONTINUE)) {
+		if (
+			isCaseIncluded &&
+			this.defaultCase !== null &&
+			!(minBrokenFlow === BROKEN_FLOW_BREAK_CONTINUE)
+		) {
 			context.brokenFlow = minBrokenFlow;
 		}
 	}

diff --git a/test/form/samples/side-effects-switch-statements/_expected.js b/test/form/samples/side-effects-switch-statements/_expected.js
@@ -53,3 +53,10 @@ for (var i = 0; i < 4; i++) {
 	}
 	effect();
 }
+
+var included;
+switch (effect()) {
+	default:
+		included = 1;
+}
+console.log(included);
diff --git a/test/form/samples/side-effects-switch-statements/main.js b/test/form/samples/side-effects-switch-statements/main.js
@@ -74,3 +74,10 @@ for (var i = 0; i < 4; i++) {
 	}
 	effect();
 }
+
+var included;
+switch (effect()) {
+	default:
+		included = 1;
+}
+console.log(included);