Commit 83b4dd8

authored

fix: cleanup CheckKishnamurthyNumber (TheAlgorithms#1626)

1 parent 894a46c commit 83b4dd8Copy full SHA for 83b4dd8

File tree

+22

-1

lines changed

+22

-1

lines changed

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,10 @@ const factorial = (n) => {`
`24`	`24`	`const CheckKishnamurthyNumber = (number) => {`
`25`	`25`	`// firstly, check that input is a number or not.`
`26`	`26`	`if (typeof number !== 'number') {`
`27`		`- return new TypeError('Argument is not a number.')`
	`27`	`+ throw new TypeError('Argument is not a number.')`
	`28`	`+ }`
	`29`	`+ if (number === 0) {`
	`30`	`+ return false`
`28`	`31`	`}`
`29`	`32`	`// create a variable to store the sum of all digits factorial.`
`30`	`33`	`let sumOfAllDigitFactorial = 0`

Lines changed: 18 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,18 @@`
	`1`	`+import { CheckKishnamurthyNumber } from '../CheckKishnamurthyNumber'`
	`2`	`+`
	`3`	`+describe('CheckKishnamurthyNumber', () => {`
	`4`	`+ it.each([1, 2, 145, 40585])('returns true for %i', (num) => {`
	`5`	`+ expect(CheckKishnamurthyNumber(num)).toBe(true)`
	`6`	`+ })`
	`7`	`+`
	`8`	`+ it.each([0, 3, 4, 5, 100, 146, 1019823, -1])(`
	`9`	`+ 'returns false for %i',`
	`10`	`+ (num) => {`
	`11`	`+ expect(CheckKishnamurthyNumber(num)).toBe(false)`
	`12`	`+ }`
	`13`	`+ )`
	`14`	`+`
	`15`	`+ it('should throw when input is not a number', () => {`
	`16`	`+ expect(() => CheckKishnamurthyNumber('2')).toThrowError()`
	`17`	`+ })`
	`18`	`+})`

Comments

(0)