Skip to content

Commit 894a46c

Browse files
authored
fix: throw error instead of returning it RailwayTimeConversion (TheAlgorithms#1625)
1 parent 8734dfc commit 894a46c

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

Conversions/RailwayTimeConversion.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
const RailwayTimeConversion = (timeString) => {
1919
// firstly, check that input is a string or not.
2020
if (typeof timeString !== 'string') {
21-
return new TypeError('Argument is not a string.')
21+
throw new TypeError('Argument is not a string.')
2222
}
2323
// split the string by ':' character.
2424
const [hour, minute, secondWithShift] = timeString.split(':')

Conversions/test/RailwayTimeConversion.test.js

+4
Original file line numberDiff line numberDiff line change
@@ -19,3 +19,7 @@ test('The RailwayTimeConversion of 11:20:00PM is 23:20:00', () => {
1919
const res = RailwayTimeConversion('11:20:00PM')
2020
expect(res).toEqual('23:20:00')
2121
})
22+
23+
test('The RailwayTimeConversion throws when input is not a string', () => {
24+
expect(() => RailwayTimeConversion(1120)).toThrowError()
25+
})

0 commit comments

Comments
 (0)