fix: throw error instead of returning it (TheAlgorithms#1624)

vil02 · web-flow · commit a5945e37c257 · 2024-02-29T10:25:11.000+05:30
diff --git a/String/CheckFlatCase.js b/String/CheckFlatCase.js
@@ -12,7 +12,7 @@
 const checkFlatCase = (varname) => {
   // firstly, check that input is a string or not.
   if (typeof varname !== 'string') {
-    return new TypeError('Argument is not a string.')
+    throw new TypeError('Argument is not a string.')
   }
 
   const pat = /^[a-z]*$/
diff --git a/String/CheckKebabCase.js b/String/CheckKebabCase.js
@@ -10,7 +10,7 @@
 const CheckKebabCase = (varName) => {
   // firstly, check that input is a string or not.
   if (typeof varName !== 'string') {
-    return new TypeError('Argument is not a string.')
+    throw new TypeError('Argument is not a string.')
   }
 
   const pat = /(\w+)-(\w)([\w-]*)/
diff --git a/String/CheckPascalCase.js b/String/CheckPascalCase.js
@@ -10,7 +10,7 @@
 const CheckPascalCase = (VarName) => {
   // firstly, check that input is a string or not.
   if (typeof VarName !== 'string') {
-    return new TypeError('Argument is not a string.')
+    throw new TypeError('Argument is not a string.')
   }
 
   const pat = /^[A-Z][A-Za-z]*$/
diff --git a/String/test/CheckCamelCase.test.js b/String/test/CheckCamelCase.test.js
@@ -15,4 +15,8 @@ describe('checkCamelCase', () => {
     const result = checkCamelCase(value)
     expect(result).toBe(false)
   })
+
+  it('should throw when input is not a string', () => {
+    expect(() => checkCamelCase(100)).toThrowError()
+  })
 })
diff --git a/String/test/CheckFlatCase.test.js b/String/test/CheckFlatCase.test.js
@@ -15,4 +15,8 @@ describe('checkFlatCase function', () => {
     const actual = checkFlatCase('abcdefghijklmnopqrstuvwxyz')
     expect(actual).toBe(true)
   })
+
+  it('should throw when input is not a string', () => {
+    expect(() => checkFlatCase(100)).toThrowError()
+  })
 })
diff --git a/String/test/CheckKebabCase.test.js b/String/test/CheckKebabCase.test.js
@@ -11,3 +11,7 @@ test('CheckKebabCase(The Algorithms) -> false', () => {
   const res = CheckKebabCase(word)
   expect(res).toBeFalsy()
 })
+
+test('CheckKebabCase throws when input is not a string', () => {
+  expect(() => CheckKebabCase(100)).toThrowError()
+})
diff --git a/String/test/CheckPascalCase.test.js b/String/test/CheckPascalCase.test.js
@@ -17,3 +17,7 @@ test('CheckPascalCase(The Algorithms) -> false', () => {
   const res = CheckPascalCase(word)
   expect(res).toBeFalsy()
 })
+
+test('CheckPascalCase throws when input is not a string', () => {
+  expect(() => CheckPascalCase(100)).toThrowError()
+})

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@`
`12`	`12`	`const checkFlatCase = (varname) => {`
`13`	`13`	`// firstly, check that input is a string or not.`
`14`	`14`	`if (typeof varname !== 'string') {`
`15`		`- return new TypeError('Argument is not a string.')`
	`15`	`+ throw new TypeError('Argument is not a string.')`
`16`	`16`	`}`
`17`	`17`
`18`	`18`	`const pat = /^[a-z]*$/`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`	`const CheckKebabCase = (varName) => {`
`11`	`11`	`// firstly, check that input is a string or not.`
`12`	`12`	`if (typeof varName !== 'string') {`
`13`		`- return new TypeError('Argument is not a string.')`
	`13`	`+ throw new TypeError('Argument is not a string.')`
`14`	`14`	`}`
`15`	`15`
`16`	`16`	`const pat = /(\w+)-(\w)([\w-]*)/`