Skip to content

Commit 972fa16

Browse files
committed
test(model): fix testifylint expected/actual order for Duration constants
Signed-off-by: Dean Chen <862469039@qq.com>
1 parent a14d955 commit 972fa16

1 file changed

Lines changed: 11 additions & 10 deletions

File tree

model/time_test.go

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -68,18 +68,19 @@ func TestDuration(t *testing.T) {
6868
}
6969

7070
func TestDurationConstants(t *testing.T) {
71-
require.Equal(t, Duration(time.Nanosecond), Nanosecond)
72-
require.Equal(t, Duration(time.Microsecond), Microsecond)
73-
require.Equal(t, Duration(time.Millisecond), Millisecond)
74-
require.Equal(t, Duration(time.Second), Second)
75-
require.Equal(t, Duration(time.Minute), Minute)
76-
require.Equal(t, Duration(time.Hour), Hour)
77-
require.Equal(t, Duration(24*time.Hour), Day)
78-
require.Equal(t, Duration(7*24*time.Hour), Week)
71+
// require.Equal(expected, actual) — expected is the reference time.Duration cast.
72+
require.Equal(t, Nanosecond, Duration(time.Nanosecond))
73+
require.Equal(t, Microsecond, Duration(time.Microsecond))
74+
require.Equal(t, Millisecond, Duration(time.Millisecond))
75+
require.Equal(t, Second, Duration(time.Second))
76+
require.Equal(t, Minute, Duration(time.Minute))
77+
require.Equal(t, Hour, Duration(time.Hour))
78+
require.Equal(t, Day, Duration(24*time.Hour))
79+
require.Equal(t, Week, Duration(7*24*time.Hour))
7980

8081
// Typical usage from the issue: multiply integers by model constants.
81-
require.Equal(t, Duration(15*24*time.Hour), 15*Day)
82-
require.Equal(t, Duration(5000*time.Millisecond), 5000*Millisecond)
82+
require.Equal(t, 15*Day, Duration(15*24*time.Hour))
83+
require.Equal(t, 5000*Millisecond, Duration(5000*time.Millisecond))
8384
}
8485

8586
func TestDurationMethods(t *testing.T) {

0 commit comments

Comments
 (0)