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

This commit is contained in:
Piotr Idzik
2024-02-29 05:55:11 +01:00
committed by GitHub
parent c067a34fae
commit a5945e37c2
7 changed files with 19 additions and 3 deletions

View File

@ -12,7 +12,7 @@
const checkFlatCase = (varname) => { const checkFlatCase = (varname) => {
// firstly, check that input is a string or not. // firstly, check that input is a string or not.
if (typeof varname !== 'string') { if (typeof varname !== 'string') {
return new TypeError('Argument is not a string.') throw new TypeError('Argument is not a string.')
} }
const pat = /^[a-z]*$/ const pat = /^[a-z]*$/

View File

@ -10,7 +10,7 @@
const CheckKebabCase = (varName) => { const CheckKebabCase = (varName) => {
// firstly, check that input is a string or not. // firstly, check that input is a string or not.
if (typeof varName !== 'string') { 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-]*)/ const pat = /(\w+)-(\w)([\w-]*)/

View File

@ -10,7 +10,7 @@
const CheckPascalCase = (VarName) => { const CheckPascalCase = (VarName) => {
// firstly, check that input is a string or not. // firstly, check that input is a string or not.
if (typeof VarName !== 'string') { 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]*$/ const pat = /^[A-Z][A-Za-z]*$/

View File

@ -15,4 +15,8 @@ describe('checkCamelCase', () => {
const result = checkCamelCase(value) const result = checkCamelCase(value)
expect(result).toBe(false) expect(result).toBe(false)
}) })
it('should throw when input is not a string', () => {
expect(() => checkCamelCase(100)).toThrowError()
})
}) })

View File

@ -15,4 +15,8 @@ describe('checkFlatCase function', () => {
const actual = checkFlatCase('abcdefghijklmnopqrstuvwxyz') const actual = checkFlatCase('abcdefghijklmnopqrstuvwxyz')
expect(actual).toBe(true) expect(actual).toBe(true)
}) })
it('should throw when input is not a string', () => {
expect(() => checkFlatCase(100)).toThrowError()
})
}) })

View File

@ -11,3 +11,7 @@ test('CheckKebabCase(The Algorithms) -> false', () => {
const res = CheckKebabCase(word) const res = CheckKebabCase(word)
expect(res).toBeFalsy() expect(res).toBeFalsy()
}) })
test('CheckKebabCase throws when input is not a string', () => {
expect(() => CheckKebabCase(100)).toThrowError()
})

View File

@ -17,3 +17,7 @@ test('CheckPascalCase(The Algorithms) -> false', () => {
const res = CheckPascalCase(word) const res = CheckPascalCase(word)
expect(res).toBeFalsy() expect(res).toBeFalsy()
}) })
test('CheckPascalCase throws when input is not a string', () => {
expect(() => CheckPascalCase(100)).toThrowError()
})