Commit5364a1c

authored

chore: check for invalid input to factorial (#1229)

1 parentd9d085f commit5364a1cCopy full SHA for 5364a1c

File tree

+16

-0

lines changed

+16

-0

lines changed

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -9,6 +9,14 @@`
`9`	`9`	`*/`
`10`	`10`
`11`	`11`	`constfactorial=(n)=>{`
	`12`	`+if(!Number.isInteger(n)){`
	`13`	`+thrownewRangeError('Not a Whole Number')`
	`14`	`+}`
	`15`	`+`
	`16`	`+if(n<0){`
	`17`	`+thrownewRangeError('Not a Positive Number')`
	`18`	`+}`
	`19`	`+`
`12`	`20`	`if(n===0){`
`13`	`21`	`return1`
`14`	`22`	`}`

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -8,4 +8,12 @@ describe('Factorial', () => {`
`8`	`8`	`it('should return factorial 120 for value "5"',()=>{`
`9`	`9`	`expect(factorial(5)).toBe(120)`
`10`	`10`	`})`
	`11`	`+`
	`12`	`+it('Throw Error for Invalid Input',()=>{`
	`13`	`+expect(()=>factorial('-')).toThrow('Not a Whole Number')`
	`14`	`+expect(()=>factorial(null)).toThrow('Not a Whole Number')`
	`15`	`+expect(()=>factorial(undefined)).toThrow('Not a Whole Number')`
	`16`	`+expect(()=>factorial(3.142)).toThrow('Not a Whole Number')`
	`17`	`+expect(()=>factorial(-1)).toThrow('Not a Positive Number')`
	`18`	`+})`
`11`	`19`	`})`

Comments

(0)