test: fix lint in tests

Author: jagregory

integration-tests/server.test.ts

@@ -14,7 +14,7 @@ describe("HTTP server", () => {
   describe("/", () => {
     it("errors with missing x-azm-target header", async () => {
       const router = jest.fn();
-      const server = createServer(router, MockLogger as any);
+      const server = createServer(router, MockLogger as any, {});
 
       const response = await supertest(server.application).post("/");
 
@@ -24,7 +24,7 @@ describe("HTTP server", () => {
 
     it("errors with an poorly formatted x-azm-target header", async () => {
       const router = jest.fn();
-      const server = createServer(router, MockLogger as any);
+      const server = createServer(router, MockLogger as any, {});
 
       const response = await supertest(server.application)
         .post("/")
@@ -43,7 +43,7 @@ describe("HTTP server", () => {
         });
         const router = (target: string) =>
           target === "valid" ? route : () => Promise.reject();
-        const server = createServer(router, MockLogger as any);
+        const server = createServer(router, MockLogger as any, {});
 
         const response = await supertest(server.application)
           .post("/")
@@ -61,7 +61,7 @@ describe("HTTP server", () => {
           .mockRejectedValue(new UnsupportedError("integration test"));
         const router = (target: string) =>
           target === "valid" ? route : () => Promise.reject();
-        const server = createServer(router, MockLogger as any);
+        const server = createServer(router, MockLogger as any, {});
 
         const response = await supertest(server.application)
           .post("/")
@@ -87,7 +87,7 @@ describe("HTTP server", () => {
           const route = jest.fn().mockRejectedValue(error);
           const router = (target: string) =>
             target === "valid" ? route : () => Promise.reject();
-          const server = createServer(router, MockLogger as any);
+          const server = createServer(router, MockLogger as any, {});
 
           const response = await supertest(server.application)
             .post("/")
@@ -105,7 +105,7 @@ describe("HTTP server", () => {
 
   describe("jwks endpoint", () => {
     it("responds with our public key", async () => {
-      const server = createServer(jest.fn(), MockLogger as any);
+      const server = createServer(jest.fn(), MockLogger as any, {});
 
       const response = await supertest(server.application).get(
         "/any-user-pool/.well-known/jwks.json"
@@ -129,7 +129,7 @@ describe("HTTP server", () => {
 
   describe("OpenId Configuration Endpoint", () => {
     it("responds with open id configuration", async () => {
-      const server = createServer(jest.fn(), MockLogger as any);
+      const server = createServer(jest.fn(), MockLogger as any, {});
 
       const response = await supertest(server.application).get(
         "/any-user-pool/.well-known/openid-configuration"

src/server/server.ts

@@ -12,12 +12,12 @@ import Pino from "pino-http";
 import { readFileSync } from "fs";
 
 export type ServerOptions = {
-  port: number;
-  hostname: string;
-  development: boolean;
+  port?: number;
+  hostname?: string;
+  development?: boolean;
 } & (
   | { https: true; key?: string; ca?: string; cert?: string }
-  | { https: false }
+  | { https?: false }
 );
 
 export interface Server {
@@ -143,8 +143,8 @@ export const createServer = (
   return {
     application: app,
     start() {
-      const hostname = options.hostname ?? "localhost";
-      const port = options.port ?? 9229;
+      const hostname = options.hostname;
+      const port = options.port;
 
       return new Promise<http.Server | https.Server>((resolve, reject) => {
         const server = options.https