test: convert the tests to typescript

Author: aminya

package.json

@@ -11,16 +11,16 @@
     "lint": "eslint . --fix",
     "test.lint": "eslint .",
     "test": "npm run build.unit && npm run test.only",
-    "test.only": "atom --test spec",
+    "test.only": "atom --test ./dist/spec",
     "clean": "shx rm -rf dist",
-    "build.unit": "tsc -p ./src/tsconfig.json || echo done",
+    "build.unit": "tsc -p ./tsconfig.json || echo done",
     "dev": "npm run clean && cross-env NODE_ENV=development cross-env BABEL_KEEP_MODULES=true rollup -c -w",
     "build": "npm run clean && cross-env NODE_ENV=production cross-env BABEL_KEEP_MODULES=true rollup -c ",
     "build-commit": "build-commit -o dist",
     "bump": "ncu -u -x coffeescript",
     "prepare": "npm run build"
   },
-  "atomTestRunner": "./spec/runner",
+  "atomTestRunner": "./dist/spec/runner",
   "activationHooks": [
     "core:loaded-shell-environment"
   ],

rollup.config.js

@@ -1,6 +1,6 @@
 import { createPlugins } from "rollup-plugin-atomic"
 
-const plugins = createPlugins([["ts", { tsconfig: "./src/tsconfig.json" }, true], "js", "json"])
+const plugins = createPlugins([["ts", { tsconfig: "./tsconfig.json" }, true], "js", "json"])
 
 const RollupConfig = [
   {

spec/CodeFormatManager-spec.js

@@ -0,0 +1,125 @@
+import { Range } from "atom"
+import { observeTextEditors } from "@atom-ide-community/nuclide-commons-atom/FileEventHandlers"
+import CodeFormatManager, { SAVE_TIMEOUT } from "../src/CodeFormatManager"
+import UniversalDisposable from "@atom-ide-community/nuclide-commons/UniversalDisposable"
+import temp from "temp"
+import * as config from "../src/config"
+import { waitsFor } from "waitsfor"
+
+const sleep = (n) => new Promise((r) => setTimeout(r, n))
+
+jasmine.DEFAULT_TIMEOUT_INTERVAL = SAVE_TIMEOUT + 100
+describe("CodeFormatManager", () => {
+  let textEditor
+  let manager
+  let disposables
+  beforeEach(async () => {
+    manager = new CodeFormatManager()
+    disposables = new UniversalDisposable(observeTextEditors())
+    temp.track()
+    const file = temp.openSync()
+    textEditor = await atom.workspace.open(file.path)
+  })
+  afterEach(async () => {
+    manager.dispose()
+    disposables.dispose()
+  })
+  it("formats an editor on request", async () => {
+    manager.addRangeProvider({
+      grammarScopes: ["text.plain.null-grammar"],
+      priority: 1,
+      formatCode: () =>
+        Promise.resolve([
+          {
+            oldRange: new Range([0, 0], [0, 3]),
+            oldText: "abc",
+            newText: "def",
+          },
+        ]),
+    })
+    textEditor.setText("abc")
+    atom.commands.dispatch(atom.views.getView(textEditor), "code-format:format-code")
+    await waitsFor(() => textEditor.getText() === "def", {
+      timeout: SAVE_TIMEOUT,
+    })
+  })
+  it("format an editor using formatEntireFile", async () => {
+    manager.addFileProvider({
+      grammarScopes: ["text.plain.null-grammar"],
+      priority: 1,
+      formatEntireFile: () =>
+        Promise.resolve({
+          formatted: "ghi",
+        }),
+    })
+    textEditor.setText("abc")
+    atom.commands.dispatch(atom.views.getView(textEditor), "code-format:format-code")
+    await waitsFor(() => textEditor.getText() === "ghi", {
+      timeout: SAVE_TIMEOUT,
+    })
+  })
+  it("formats an editor on type", async () => {
+    spyOn(config, "getFormatOnType").and.returnValue(true)
+    const provider = {
+      grammarScopes: ["text.plain.null-grammar"],
+      priority: 1,
+      formatAtPosition: () =>
+        Promise.resolve([
+          {
+            oldRange: new Range([0, 0], [0, 3]),
+            oldText: "abc",
+            newText: "def",
+          },
+        ]),
+      keepCursorPosition: false,
+    }
+    const spy = spyOn(provider, "formatAtPosition")
+    manager.addOnTypeProvider(provider)
+    textEditor.setText("a")
+    textEditor.setCursorBufferPosition([0, 1])
+    textEditor.insertText("b")
+    textEditor.insertText("c")
+    await waitsFor(() => textEditor.getText() === "def", {
+      timeout: SAVE_TIMEOUT,
+    })
+    // Debouncing should ensure only one format call.
+    expect(spy.mock.calls.length).toBe(1)
+  })
+  it("formats an editor on save", async () => {
+    spyOn(config, "getFormatOnSave").and.returnValue(true)
+    manager.addOnSaveProvider({
+      grammarScopes: ["text.plain.null-grammar"],
+      priority: 1,
+      formatOnSave: () =>
+        Promise.resolve([
+          {
+            oldRange: new Range([0, 0], [0, 3]),
+            oldText: "abc",
+            newText: "def",
+          },
+        ]),
+    })
+    textEditor.setText("abc")
+    await textEditor.save()
+    expect(textEditor.getText()).toBe("def")
+  })
+  it("should still save on timeout", async () => {
+    spyOn(config, "getFormatOnSave").and.returnValue(true)
+    manager.addRangeProvider({
+      grammarScopes: ["text.plain.null-grammar"],
+      priority: 1,
+      formatCode: async () => {
+        await sleep(SAVE_TIMEOUT + 1000)
+        return []
+      },
+    })
+    const spy = spyOn(textEditor.getBuffer(), "save")
+    textEditor.save()
+    // Wait until the buffer has been saved and verify it has been saved exactly
+    // once.
+    await waitsFor(() => spy.mock.calls.length > 0, {
+      timeout: SAVE_TIMEOUT,
+    })
+    expect(spy.mock.calls.length).toBe(1)
+  })
+})

spec/CodeFormatManager-spec.ts

@@ -1,4 +1,3 @@
-"use babel"
 import { createRunner } from "atom-jasmine3-test-runner"
 
 // https://github.com/UziTech/atom-jasmine3-test-runner#api

spec/runner.js renamed to spec/runner.ts

@@ -28,7 +28,8 @@
     "module": "commonjs",
     "moduleResolution": "node",
     "importHelpers": false,
-    "outDir": "../dist"
+    "outDir": "./dist"
   },
-  "compileOnSave": true
+  "compileOnSave": true,
+  "include": ["src/**/*", "spec/**/*"]
 }