Throw error on empty options when creating server

Author: nrayburn-tech

lib/proxyServer.ts

@@ -42,6 +42,7 @@ export interface ProxyTargetDetailed {
   ciphers?: string | undefined;
   secureProtocol?: string | undefined;
 }
+
 export type ProxyTargetUrl = string | Partial<url.Url>;
 
 export type ProxyTarget = ProxyTargetUrl | ProxyTargetDetailed;
@@ -126,6 +127,9 @@ export class ProxyServer extends EE3 {
 
   constructor(options: ProxyServerOptions) {
     super();
+    if (!options || Object.keys(options).length === 0) {
+      throw new Error('Options must not be empty.');
+    }
     options = options || {};
     options.prependPath = options.prependPath !== false;
 

test/lib-http-proxy-passes-web-incoming-test.js

@@ -495,7 +495,7 @@ describe('#createProxy.web() using own http server', () => {
   });
 
   it('should proxy requests to multiple servers with different options', async () => {
-    const proxy = createProxy();
+    const proxy = createProxy({target: 'http://ignored:8080'});
 
     // proxies to two servers depending on url, rewriting the url as well
     // http://127.0.0.1:8080/s1/ -> http://127.0.0.1:8081/

test/lib-http-proxy-test.js

@@ -23,7 +23,7 @@ Object.defineProperty(gen, 'port', {
 
 describe('lib/http-proxy.js', () => {
   describe('#createProxy', () => {
-    it.skip('should throw without options', function () {
+    it('should throw without options', function () {
       let error;
       try {
         createProxy();