Merge pull request #53 from NativeScript/tachev/improve-demos

chore: fix the unit tests and the linting + start executing them on prepare

Author: Dimitar Tachev

.vscode/launch.json

@@ -0,0 +1,23 @@
+{
+    // Use IntelliSense to learn about possible attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "type": "node",
+            "request": "launch",
+            "name": "Launch Unit Test",
+            "skipFiles": [
+                "<node_internals>/**"
+            ],
+            "program": "${workspaceRoot}/src/node_modules/mocha/bin/_mocha",
+            "sourceMaps": true,
+            "autoAttachChildProcesses": true,
+            "cwd": "${workspaceRoot}/src",
+            "args": [
+                "test/test.es5.js"
+            ]
+        }
+    ]
+}

README.md

@@ -222,11 +222,6 @@ const worker = new Worker("./worker-script");
 
 ```
 
-## Demo apps
-For usage with NativeScript Angular, check out [`demo-angular`](./demo-angular) in this repo.
-
-For usage with NativeScript apps written in plain JavaScript, check out this repo: https://github.com/NativeScript/demo-workers.
-
 ## Related docs
 
 1. [Workers in NativeScript](https://docs.nativescript.org/core-concepts/multithreading-model)

src/index.js

@@ -71,7 +71,7 @@ module.exports.pitch = function pitch(request) {
     };
 
     const plugins = (pluginOptions.plugins || []).map(plugin => {
-        if (typeof plugin !== 'string') {
+        if (typeof plugin !== "string") {
             return plugin;
         }
         const found = compilerOptions.plugins.find(p => p.constructor.name === plugin);
@@ -109,7 +109,7 @@ module.exports.pitch = function pitch(request) {
         if (entries[0]) {
             const fileDeps = Array.from(childCompilation.fileDependencies);
             this.clearDependencies();
-            fileDeps.map(fileName => {
+            fileDeps.forEach(fileName => {
                 this.addDependency(fileName);
             });
             /**

src/package.json

@@ -6,7 +6,8 @@
   "scripts": {
     "pretest": "babel ./test/test.es6.js --out-file ./test/test.es5.js",
     "test": "mocha test/test.es5.js",
-    "posttest": "eslint ."
+    "posttest": "eslint .",
+    "prepare": "npm run test"
   },
   "eslintConfig": {
     "extends": "webpack",
@@ -29,6 +30,7 @@
           "functions": "never"
         }
       ],
+      "no-console": 0,
       "arrow-parens": 0,
       "func-names": 0,
       "import/no-extraneous-dependencies": 0,
@@ -48,7 +50,8 @@
     "babel-cli": "^6.26.0",
     "babel-plugin-transform-async-to-generator": "^6.24.1",
     "del": "^2.2.2",
-    "eslint": "^3.16.0",
+    "eslint": "^6.6.0",
+    "webpack": "~4.27.0",
     "eslint-config-webpack": "^1.0.0",
     "eslint-plugin-import": "^2.2.0",
     "mocha": "^5.2.0"

src/symbol.js

@@ -1,3 +1,3 @@
 
-const NATIVESCRIPT_WORKER_PLUGIN_SYMBOL = Symbol('NATIVESCRIPT_WORKER_PLUGIN_SYMBOL');
+const NATIVESCRIPT_WORKER_PLUGIN_SYMBOL = Symbol("NATIVESCRIPT_WORKER_PLUGIN_SYMBOL");
 module.exports = NATIVESCRIPT_WORKER_PLUGIN_SYMBOL;

src/test/fixtures/inline-fallbacks/w1.js

@@ -1 +1 @@
-// w1 inlined with fallback
+console.log("w1 inlined with fallback");

src/test/fixtures/inline-fallbacks/w2.js

@@ -1 +1 @@
-// w2 inlined with fallback
+console.log("w2 inlined with fallback");

src/test/fixtures/inline-options/w1.js

@@ -1 +1 @@
-// w1 inlined via options
+console.log("w1 inlined via options");

src/test/fixtures/inline-options/w2.js

@@ -1 +1 @@
-// w2 inlined via options
+console.log("w2 inlined via options");

src/test/fixtures/inline-query/worker.js

@@ -1 +1 @@
-// inlined worker test mark
+console.log("inlined worker test mark");