tests to .ts, formatting, test infra

.eslintrc.json

@@ -10,7 +10,8 @@
         "plugin:@typescript-eslint/recommended"
     ],
     "rules": {
-        "indent": ["error", 2, { "SwitchCase": 1 }]
+        "indent": ["error", 2, { "SwitchCase": 1 }],
+        "semi": [2, "always"]
     },
     "ignorePatterns": ["dist"]
 }

.gitignore

@@ -4,4 +4,5 @@ node_modules
 dist
 /data
 .nyc_output
+.test_output
 coverage

.mocharc.json

@@ -1,5 +1,5 @@
 {
-    "spec": "./test/unit/**/*.test.js",
+    "spec": "./test/unit/**/*.test.ts",
     "recursive": true,
     "require": ["ts-node/register"]
 }

.vscode/extensions.json

@@ -1,7 +1,4 @@
 {
-	// See https://go.microsoft.com/fwlink/?LinkId=733558
-	// for the documentation about the extensions.json format
-	"recommendations": [
-		"dbaeumer.vscode-eslint"
-	]
+    "recommendations": [
+    ]
 }

.vscode/launch.json

@@ -1,39 +1,41 @@
-// A launch configuration that launches the extension inside a new window
-// 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": [
-		{
-			"name": "Run Extension",
-			"type": "extensionHost",
-			"request": "launch",
-			"args": [
-				"${workspaceFolder}/data",
-				"--extensionDevelopmentPath=${workspaceFolder}"
-			]
-		},
-		{
-			"name": "Extension Tests",
-			"type": "extensionHost",
-			"request": "launch",
-			"args": [
-				"--extensionDevelopmentPath=${workspaceFolder}",
-				"--extensionTestsPath=${workspaceFolder}/test/suite/index"
-			]
-		},
-		{
-			"name": "Unit Tests",
-			"type": "pwa-node",
-			"request": "launch",
-			"program": "${workspaceFolder}/node_modules/mocha/bin/_mocha",
-			"stopOnEntry": false,
-			"sourceMaps": true,
-			"resolveSourceMapLocations": [
-				"${workspaceFolder}/**",
-				"!**/node_modules/**"
-			],
-		},
-	]
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "name": "Run Extension",
+            "type": "extensionHost",
+            "request": "launch",
+            "args": [
+                "${workspaceFolder}/data",
+                "--extensionDevelopmentPath=${workspaceFolder}",
+                "--disable-extensions"
+            ]
+        },
+        {
+            "name": "Run Extension Tests",
+            "type": "pwa-node",
+            "request": "launch",
+            "runtimeExecutable": "npm",
+            "runtimeArgs": [
+                "run",
+                "test:vscode",
+            ],
+            "skipFiles": [
+                "<node_internals>/**"
+            ]
+        },
+        {
+            "name": "Unit Tests",
+            "type": "pwa-node",
+            "request": "launch",
+            "runtimeExecutable": "npm",
+            "runtimeArgs": [
+                "run",
+                "test:unittests",
+            ],
+            "skipFiles": [
+                "<node_internals>/**"
+            ]
+        },
+    ]
 }

.vscodeignore

@@ -3,7 +3,6 @@
 test/**
 .gitignore
 .yarnrc
-vsc-extension-quickstart.md
 **/jsconfig.json
 **/*.map
 **/.eslintrc.json
@@ -13,4 +12,6 @@ tsconfig.json
 data
 extension
 webview
-*.ipynb
+*.ipynb
+.nyc_output
+.test_output

extension/formats/nifti/base.ts

@@ -227,7 +227,7 @@ export abstract class Nifti {
             intercept: scalingIntercept,
           },
         }
-      }
+      };
 
       if (min === max && min === 0) {
         const values = await this.values();

extension/utils.ts

@@ -23,7 +23,7 @@ export function isGzipped(fd: number): boolean {
   const view = new DataView(buffer);
   const bytes = fs.readSync(fd, view, 0, 2, 0);
   if (bytes < 2){
-    throw new Error("Invalid file: empty")
+    throw new Error("Invalid file: empty");
   }
   return (
     view.getUint8(0) === GZIP_MAGIC_COOKIE[0] &&
@@ -88,7 +88,7 @@ export class Slicer extends Transform {
     }
     if (this._offset_end !== null && this._passedThrough >= this._offset_end) {
       callback();
-      return
+      return;
     }
     const begin = this._passedThrough > this._offset ? 0 : this._offset - this._passedThrough;
     let end = undefined;
@@ -181,8 +181,8 @@ export class Bufferizer extends Transform {
   }
 
   _transform(chunk: number[], encoding: string, callback: TransformCallback) {
-    // const rounded: number[] = chunk.map((v) => Math.round(v));
-    this.push(Buffer.from(Int16Array.from(chunk).buffer));
+    const int16Array = Int16Array.from(chunk);
+    this.push(Buffer.from(int16Array.buffer));
     callback();
   }
 }

jsconfig.json

@@ -1,13 +1,13 @@
 {
-	"compilerOptions": {
-		"module": "commonjs",
-		"target": "es6",
-		"checkJs": true,  /* Typecheck .js files. */
-		"lib": [
-			"es6"
-		]
-	},
-	"exclude": [
-		"node_modules"
-	]
+    "compilerOptions": {
+        "module": "commonjs",
+        "target": "es6",
+        "checkJs": true,
+        "lib": [
+            "es6"
+        ]
+    },
+    "exclude": [
+        "node_modules"
+    ]
 }