From 63f4ae6ec623ea3cd94ad4076607693afdab03b4 Mon Sep 17 00:00:00 2001
From: evilebottnawi <sheo13666q@gmail.com>
Date: Tue, 17 Mar 2020 14:02:08 +0300
Subject: [PATCH] refactor: the `getOptions` method returns empty object on
 empty query

BREAKING CHANGE: the `getOptions` method returns empty object on empty query
---
 lib/getOptions.js       |   2 +-
 test/getOptions.test.js | 132 +++++++++++++++++++++++-----------------
 2 files changed, 76 insertions(+), 58 deletions(-)

diff --git a/lib/getOptions.js b/lib/getOptions.js
index 9cbbed4..d849d03 100644
--- a/lib/getOptions.js
+++ b/lib/getOptions.js
@@ -11,7 +11,7 @@ function getOptions(loaderContext) {
 
   if (!query || typeof query !== 'object') {
     // Not object-like queries are not supported.
-    return null;
+    return {};
   }
 
   return query;
diff --git a/test/getOptions.test.js b/test/getOptions.test.js
index 067b898..5b0c030 100644
--- a/test/getOptions.test.js
+++ b/test/getOptions.test.js
@@ -3,62 +3,80 @@
 const loaderUtils = require('../lib');
 
 describe('getOptions()', () => {
-  describe('when loaderContext.query is a string with length > 0', () => {
-    it('should call parseQuery() and return its result', () => {
-      expect(
-        loaderUtils.getOptions({
-          query: '?something=getOptions_cannot_parse',
-        })
-      ).toEqual({ something: 'getOptions_cannot_parse' });
-    });
-  });
-  describe('when loaderContext.query is an empty string', () => {
-    it('should return null', () => {
-      expect(
-        loaderUtils.getOptions({
-          query: '',
-        })
-      ).toEqual(null);
-    });
-  });
-  describe('when loaderContext.query is an object', () => {
-    it('should just return it', () => {
-      const query = {};
-      expect(
-        loaderUtils.getOptions({
-          query,
-        })
-      ).toEqual(query);
-    });
-  });
-  describe('when loaderContext.query is an array', () => {
-    it('should just return it', () => {
-      const query = [];
-      expect(loaderUtils.getOptions({ query })).toEqual(query);
-    });
-  });
-  describe('when loaderContext.query is anything else', () => {
-    it('should return null', () => {
-      expect(
-        loaderUtils.getOptions({
-          query: undefined,
-        })
-      ).toEqual(null);
-      expect(
-        loaderUtils.getOptions({
-          query: null,
-        })
-      ).toEqual(null);
-      expect(
-        loaderUtils.getOptions({
-          query: 1,
-        })
-      ).toEqual(null);
-      expect(
-        loaderUtils.getOptions({
-          query: 0,
-        })
-      ).toEqual(null);
-    });
+  it('should work', () => {
+    expect(
+      loaderUtils.getOptions({
+        query: true,
+      })
+    ).toEqual({});
+    expect(
+      loaderUtils.getOptions({
+        query: false,
+      })
+    ).toEqual({});
+    expect(
+      loaderUtils.getOptions({
+        query: null,
+      })
+    ).toEqual({});
+    expect(
+      loaderUtils.getOptions({
+        query: undefined,
+      })
+    ).toEqual({});
+    expect(
+      loaderUtils.getOptions({
+        query: 1,
+      })
+    ).toEqual({});
+    expect(
+      loaderUtils.getOptions({
+        query: 0,
+      })
+    ).toEqual({});
+    expect(
+      loaderUtils.getOptions({
+        query: -0,
+      })
+    ).toEqual({});
+    expect(
+      loaderUtils.getOptions({
+        query: -1,
+      })
+    ).toEqual({});
+    expect(
+      loaderUtils.getOptions({
+        query: '',
+      })
+    ).toEqual({});
+    expect(
+      loaderUtils.getOptions({
+        query: '?something=getOptions_cannot_parse',
+      })
+    ).toEqual({ something: 'getOptions_cannot_parse' });
+
+    const query1 = {};
+
+    expect(
+      loaderUtils.getOptions({
+        query: query1,
+      })
+    ).toEqual(query1);
+
+    const query2 = { foo: { bar: 'baz' } };
+
+    expect(
+      loaderUtils.getOptions({
+        query: query2,
+      })
+    ).toEqual(query2);
+
+    const query3 = [];
+
+    expect(loaderUtils.getOptions({ query: query3 })).toEqual(query3);
+
+    const query4 = [1, true, 'foobar'];
+
+    expect(loaderUtils.getOptions({ query: query4 })).toEqual(query4);
   });
 });