Improve coverage

src/ast/nodes/IfStatement.ts

@@ -70,12 +70,18 @@ export default class IfStatement extends StatementBase implements DeoptimizableE
 
 	parseNode(esTreeNode: GenericEsTreeNode) {
 		this.consequentScope = new TrackingScope(this.scope);
-		this.consequent = new (this.context.nodeConstructors[esTreeNode.consequent.type] ||
-			this.context.nodeConstructors.UnknownNode)(esTreeNode.consequent, this, this.consequentScope);
+		this.consequent = new this.context.nodeConstructors[esTreeNode.consequent.type](
+			esTreeNode.consequent,
+			this,
+			this.consequentScope
+		);
 		if (esTreeNode.alternate) {
 			this.alternateScope = new TrackingScope(this.scope);
-			this.alternate = new (this.context.nodeConstructors[esTreeNode.alternate.type] ||
-				this.context.nodeConstructors.UnknownNode)(esTreeNode.alternate, this, this.alternateScope);
+			this.alternate = new this.context.nodeConstructors[esTreeNode.alternate.type](
+				esTreeNode.alternate,
+				this,
+				this.alternateScope
+			);
 		}
 		super.parseNode(esTreeNode);
 	}

src/ast/nodes/VariableDeclarator.ts

@@ -1,5 +1,3 @@
-import MagicString from 'magic-string';
-import { RenderOptions } from '../../utils/renderHelpers';
 import { ObjectPath } from '../utils/PathTracker';
 import { UNDEFINED_EXPRESSION } from '../values';
 import * as NodeType from './NodeType';
@@ -18,14 +16,4 @@ export default class VariableDeclarator extends NodeBase {
 	deoptimizePath(path: ObjectPath) {
 		this.id.deoptimizePath(path);
 	}
-
-	render(code: MagicString, options: RenderOptions) {
-		// This can happen for hoisted variables in dead branches
-		if (this.init !== null && !this.init.included) {
-			code.remove(this.id.end, this.end);
-			this.id.render(code, options);
-		} else {
-			super.render(code, options);
-		}
-	}
 }

src/ast/scopes/BlockScope.ts

@@ -9,7 +9,7 @@ export default class BlockScope extends ChildScope {
 	addDeclaration(
 		identifier: Identifier,
 		context: AstContext,
-		init: ExpressionEntity | null = null,
+		init: ExpressionEntity | null,
 		isHoisted: boolean
 	): LocalVariable {
 		if (isHoisted) {

src/ast/scopes/Scope.ts

@@ -13,7 +13,7 @@ export default class Scope {
 	addDeclaration(
 		identifier: Identifier,
 		context: AstContext,
-		init: ExpressionEntity | null = null,
+		init: ExpressionEntity | null,
 		_isHoisted: boolean
 	) {
 		const name = identifier.name;

src/ast/scopes/TrackingScope.ts

@@ -10,12 +10,10 @@ export default class TrackingScope extends BlockScope {
 	addDeclaration(
 		identifier: Identifier,
 		context: AstContext,
-		init: ExpressionEntity | null = null,
+		init: ExpressionEntity | null,
 		isHoisted: boolean
 	): LocalVariable {
-		if (isHoisted) {
-			this.hoistedDeclarations.push(identifier);
-		}
+		this.hoistedDeclarations.push(identifier);
 		return this.parent.addDeclaration(identifier, context, init, isHoisted);
 	}
 }