Merge pull request #370 from cwillisf/polyfill-fetch-in-node

fix: polyfill 'fetch' for Node target

Author: cwillisf

test/integration/download-known-assets.js

@@ -1,7 +1,5 @@
 const md5 = require('js-md5');
 const test = require('tap').test;
-// Polyfill the fetch API
-global.fetch = require('node-fetch');
 
 const ScratchStorage = require('../../dist/node/scratch-storage');
 
@@ -18,6 +16,7 @@ test('constructor', t => {
  * @typedef {object} AssetTestInfo
  * @property {AssetType} type - The type of the asset.
  * @property {string} id - The asset's unique ID.
+ * @property {string} md5 - The asset's MD5 hash.
  * @property {DataFormat} [ext] - Optional: the asset's data format / file extension.
  */
 const testAssets = [

webpack.config.js

@@ -1,4 +1,5 @@
 const path = require('path');
+const {ProvidePlugin} = require('webpack');
 const UglifyJsPlugin = require('uglifyjs-webpack-plugin');
 
 const base = {
@@ -64,6 +65,11 @@ module.exports = [
             'js-md5': true,
             'localforage': true,
             'text-encoding': true
-        }
+        },
+        plugins: [
+            new ProvidePlugin({
+                fetch: ['node-fetch', 'default']
+            })
+        ]
     })
 ];