Merged in wbinnssmith/add-back-packaging-invariant (pull request #1)

Restore invariant asserting bundles don't have external assets Approved-by: Trey Shugart <tshugart@atlassian.com>
atlassian · Sep 23, 2019 · d6cebc2 · d6cebc2
1 parent 642f64f
commit d6cebc2
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/packages/packagers/js/src/JSPackager.js b/packages/packagers/js/src/JSPackager.js
@@ -54,12 +54,12 @@ export default new Packager({
         ) {
           // if this is a reference to another javascript asset, we should not include
           // its output, as its contents should already be loaded.
-          // invariant(
-          //   !bundle.hasAsset(resolved),
-          //   `Expected JS bundle ${
-          //     bundle.name
-          //   } to only contain JS, but contained ${resolved.filePath}`
-          // );
+          invariant(
+            !bundle.hasAsset(resolved),
+            `Expected JS bundle ${
+              bundle.name
+            } to only contain JS, but contained ${resolved.filePath}`
+          );
           wrapped += JSON.stringify(resolved.id) + ':[function() {},{}]';
         } else {
           return;