Write sourcemaps for the serve mode's default target again (#4569)

* Add test * Fix * Update packages/core/core/test/TargetResolver.test.js Co-authored-by: Jasper De Moor <[email protected]> Co-authored-by: Will Binns-Smith <[email protected]> Co-authored-by: Jasper De Moor <[email protected]>
parcel-bundler · May 5, 2020 · 3c1c7db · 3c1c7db
1 parent d29a741
commit 3c1c7db
Show file tree

Hide file tree

Showing 2 changed files with 39 additions and 0 deletions.
diff --git a/packages/core/core/src/TargetResolver.js b/packages/core/core/src/TargetResolver.js
@@ -175,6 +175,7 @@ export default class TargetResolver {
             name: 'default',
             distDir: this.options.distDir,
             publicUrl: this.options.publicUrl ?? '/',
+            sourceMap: this.options.sourceMaps ? {} : undefined,
             env: createEnvironment({
               context: 'browser',
               engines: {

diff --git a/packages/core/core/test/TargetResolver.test.js b/packages/core/core/test/TargetResolver.test.js
@@ -547,6 +547,44 @@ describe('TargetResolver', () => {
     );
   });
 
+  it('generates a default target in serve mode', async () => {
+    let targetResolver = new TargetResolver({
+      ...DEFAULT_OPTIONS,
+      serve: {port: 1234},
+    });
+
+    assert.deepEqual(
+      await targetResolver.resolve(COMMON_TARGETS_FIXTURE_PATH),
+      {
+        targets: [
+          {
+            name: 'default',
+            distDir: DEFAULT_OPTIONS.distDir,
+            publicUrl: '/',
+            env: {
+              context: 'browser',
+              engines: {
+                browsers: [
+                  'last 1 Chrome version',
+                  'last 1 Safari version',
+                  'last 1 Firefox version',
+                  'last 1 Edge version',
+                ],
+              },
+              includeNodeModules: true,
+              outputFormat: 'global',
+              isLibrary: false,
+              minify: false,
+              scopeHoist: false,
+            },
+            sourceMap: {},
+          },
+        ],
+        files: [],
+      },
+    );
+  });
+
   it('rejects invalid or unknown fields', async () => {
     let code =
       '{\n' +