Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Lines changed: 1 addition & 1 deletion
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

Lines changed: 4 additions & 0 deletions
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)

AltStyle によって変換されたページ (->オリジナル) /