cover development strings

Author: jawnsy

src/node/constants.ts

@@ -23,7 +23,7 @@ export const version = pkg.version || "development"
 export const commit = pkg.commit || "development"
 export const rootPath = path.resolve(__dirname, "../..")
 export const vsRootPath = path.join(rootPath, "vendor/modules/code-oss-dev")
-export const codeVersion = require(path.join(vsRootPath, "package.json")).version
+export const codeVersion = require(path.join(vsRootPath, "package.json")).version || "development"
 export const tmpdir = path.join(os.tmpdir(), "code-server")
 export const isDevMode = commit === "development"
 export const httpProxyUri =

test/unit/node/constants.test.ts

@@ -119,5 +119,18 @@ describe("constants", () => {
     it("commit should return 'development'", () => {
       expect(constants.commit).toBe("development")
     })
+
+    it("should return a human-readable version string", () => {
+      // this string is not super useful
+      expect(constants.getVersionString()).toStrictEqual("development development")
+    })
+
+    it("should return a machine-readable version string", () => {
+      expect(constants.getVersionJsonString()).toStrictEqual(JSON.stringify({
+        codeServer: "development",
+        commit: "development",
+        vscode: "development",
+      }))
+    })
   })
 })