fix: use throw and segregate the test cases

Author: piyushk77

Conversions/RgbHslConversion.js

@@ -18,9 +18,9 @@
 const checkRgbFormat = (colorRgb) => colorRgb.every(c => c >= 0 && c <= 255);
 
 const rgbToHsl = (colorRgb) => {
-  if (checkRgbFormat(colorRgb) === false) {
-    return 'Input is not a valid RGB color.'
-  }
+  if (!checkRgbFormat(colorRgb)) {
+    throw new Error('Input is not a valid RGB color.');
+  }  
 
   let colorHsl = colorRgb
 

Conversions/test/RgbHslConversion.test.js

@@ -1,6 +1,6 @@
 import { rgbToHsl } from '../RgbHslConversion'
 describe('RgbHslConversion', () => {
-  const testCases = [
+  const validTestCases = [
     [
       [215, 19, 180],
       [311, 84, 46]
@@ -28,16 +28,27 @@ describe('RgbHslConversion', () => {
     [
       [255, 255, 255],
       [0, 0, 100]
-    ],
+    ]
+  ]
+
+  const errorTestCases = [
     [[256, 180, 9], 'Input is not a valid RGB color.'],
     [[-90, 46, 8], 'Input is not a valid RGB color.'],
     [[1, 39, 900], 'Input is not a valid RGB color.']
   ]
 
-  test.each(testCases)(
+
+  test.each(validTestCases)(
     'Should return the color in HSL format.',
     (colorRgb, expected) => {
       expect(rgbToHsl(colorRgb)).toEqual(expected)
     }
   )
+
+  test.each(errorTestCases)(
+    'Should return the error message.',
+    (colorRgb, expected) => {
+      expect(() => rgbToHsl(colorRgb)).toThrowError(expected)
+    }
+  )
 })