From de005fece18e668bc0089558420602a043f5fd0e Mon Sep 17 00:00:00 2001
From: Joe Previte <jjprevite@gmail.com>
Date: Mon, 14 Feb 2022 15:37:16 -0700
Subject: [PATCH 1/2] chore: fmt cleanup

---
 test/unit/node/update.test.ts | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/test/unit/node/update.test.ts b/test/unit/node/update.test.ts
index b900d8ae7c0d..d2f62531a73c 100644
--- a/test/unit/node/update.test.ts
+++ b/test/unit/node/update.test.ts
@@ -1,5 +1,5 @@
-import * as http from "http"
 import { logger } from "@coder/logger"
+import * as http from "http"
 import { AddressInfo } from "net"
 import * as path from "path"
 import { SettingsProvider, UpdateSettings } from "../../../src/node/settings"
@@ -218,8 +218,8 @@ describe("update", () => {
   it("should reject if response has status code 500", async () => {
     if (isAddressInfo(_address)) {
       const mockURL = `http://${_address.address}:${_address.port}/reject-status-code`
-      let provider = new UpdateProvider(mockURL, settings())
-      let update = await provider.getUpdate(true)
+      const provider = new UpdateProvider(mockURL, settings())
+      const update = await provider.getUpdate(true)
 
       expect(update.version).toBe("unknown")
       expect(logger.error).toHaveBeenCalled()
@@ -233,8 +233,8 @@ describe("update", () => {
   it("should reject if no location header provided", async () => {
     if (isAddressInfo(_address)) {
       const mockURL = `http://${_address.address}:${_address.port}/no-location-header`
-      let provider = new UpdateProvider(mockURL, settings())
-      let update = await provider.getUpdate(true)
+      const provider = new UpdateProvider(mockURL, settings())
+      const update = await provider.getUpdate(true)
 
       expect(update.version).toBe("unknown")
       expect(logger.error).toHaveBeenCalled()
@@ -249,8 +249,8 @@ describe("update", () => {
     version = "4.1.1"
     if (isAddressInfo(_address)) {
       const mockURL = `http://${_address.address}:${_address.port}/with-location-header`
-      let provider = new UpdateProvider(mockURL, settings())
-      let update = await provider.getUpdate(true)
+      const provider = new UpdateProvider(mockURL, settings())
+      const update = await provider.getUpdate(true)
 
       expect(logger.error).not.toHaveBeenCalled()
       expect(update.version).toBe("4.1.1")
@@ -260,8 +260,8 @@ describe("update", () => {
   it("should reject if more than 10 redirects", async () => {
     if (isAddressInfo(_address)) {
       const mockURL = `http://${_address.address}:${_address.port}/redirect/11`
-      let provider = new UpdateProvider(mockURL, settings())
-      let update = await provider.getUpdate(true)
+      const provider = new UpdateProvider(mockURL, settings())
+      const update = await provider.getUpdate(true)
 
       expect(update.version).toBe("unknown")
       expect(logger.error).toHaveBeenCalled()

From d8deb781cf68ecb0aee088ddde6b33378267ba94 Mon Sep 17 00:00:00 2001
From: Joe Previte <jjprevite@gmail.com>
Date: Mon, 14 Feb 2022 15:37:36 -0700
Subject: [PATCH 2/2] feat(parse): add test error w/config

---
 test/unit/node/cli.test.ts | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/test/unit/node/cli.test.ts b/test/unit/node/cli.test.ts
index a4cffb9a6bca..57d19317ded4 100644
--- a/test/unit/node/cli.test.ts
+++ b/test/unit/node/cli.test.ts
@@ -361,6 +361,16 @@ describe("parser", () => {
         "$argon2i$v=19$m=4096,t=3,p=1$0qr/o+0t00hsbjfqcksfdq$ofcm4rl6o+b7oxpua4qlxubypbbpsf+8l531u7p9hyy",
     })
   })
+  it("should throw an error for invalid config values", async () => {
+    const fakePath = "/fake-config-path"
+    const expectedErrMsg = `error reading ${fakePath}: `
+
+    expect(() =>
+      parse(["--foo"], {
+        configFile: fakePath,
+      }),
+    ).toThrowError(expectedErrMsg)
+  })
 })
 
 describe("cli", () => {