Merge branch 'winh-idempotent-parseboolean' into 'master'

Make parseBoolean idempotent

See merge request gitlab-org/gitlab-ce!24365

1 files changed, 15 insertions, 4 deletions

diff --git a/spec/javascripts/lib/utils/common_utils_spec.js b/spec/javascripts/lib/utils/common_utils_spec.js
index f320f232687..0dc7e93539a 100644
--- a/spec/javascripts/lib/utils/common_utils_spec.js
+++ b/spec/javascripts/lib/utils/common_utils_spec.js
@@ -347,20 +347,31 @@ describe('common_utils', () => {
   });
 
   describe('parseBoolean', () => {
+    const { parseBoolean } = commonUtils;
+
     it('returns true for "true"', () => {
-      expect(commonUtils.parseBoolean('true')).toEqual(true);
+      expect(parseBoolean('true')).toEqual(true);
     });
 
     it('returns false for "false"', () => {
-      expect(commonUtils.parseBoolean('false')).toEqual(false);
+      expect(parseBoolean('false')).toEqual(false);
     });
 
     it('returns false for "something"', () => {
-      expect(commonUtils.parseBoolean('something')).toEqual(false);
+      expect(parseBoolean('something')).toEqual(false);
     });
 
     it('returns false for null', () => {
-      expect(commonUtils.parseBoolean(null)).toEqual(false);
+      expect(parseBoolean(null)).toEqual(false);
+    });
+
+    it('is idempotent', () => {
+      const input = ['true', 'false', 'something', null];
+      input.forEach(value => {
+        const result = parseBoolean(value);
+
+        expect(parseBoolean(result)).toBe(result);
+      });
     });
   });