Conditions | 4 |
Total Lines | 54 |
Code Lines | 50 |
Lines | 0 |
Ratio | 0 % |
Changes | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
1 | package tests |
||
10 | func TestTakeEn(t *testing.T) { |
||
11 | cases := []struct { |
||
12 | date string |
||
13 | result string |
||
14 | lang string |
||
15 | }{ |
||
16 | {smallSubTime(-60 * time.Second), "1 minute ago", "en"}, |
||
17 | {smallSubTime(-1 * time.Minute), "1 minute ago", "en"}, |
||
18 | {smallSubTime(-2 * time.Minute), "2 minutes ago", "en"}, |
||
19 | {smallSubTime(-5 * time.Minute), "5 minutes ago", "en"}, |
||
20 | {smallSubTime(-9 * time.Minute), "9 minutes ago", "en"}, |
||
21 | {smallSubTime(-10 * time.Minute), "10 minutes ago", "en"}, |
||
22 | {smallSubTime(-11 * time.Minute), "11 minutes ago", "en"}, |
||
23 | {smallSubTime(-20 * time.Minute), "20 minutes ago", "en"}, |
||
24 | {smallSubTime(-21 * time.Minute), "21 minutes ago", "en"}, |
||
25 | {smallSubTime(-22 * time.Minute), "22 minutes ago", "en"}, |
||
26 | {smallSubTime(-30 * time.Minute), "30 minutes ago", "en"}, |
||
27 | {smallSubTime(-31 * time.Minute), "31 minutes ago", "en"}, |
||
28 | {smallSubTime(-59 * time.Minute), "59 minutes ago", "en"}, |
||
29 | {smallSubTime(-60 * time.Minute), "1 hour ago", "en"}, |
||
30 | {smallSubTime(-1 * time.Hour), "1 hour ago", "en"}, |
||
31 | {smallSubTime(-2 * time.Hour), "2 hours ago", "en"}, |
||
32 | {smallSubTime(-9 * time.Hour), "9 hours ago", "en"}, |
||
33 | {smallSubTime(-10 * time.Hour), "10 hours ago", "en"}, |
||
34 | {smallSubTime(-11 * time.Hour), "11 hours ago", "en"}, |
||
35 | {smallSubTime(-20 * time.Hour), "20 hours ago", "en"}, |
||
36 | {smallSubTime(-21 * time.Hour), "21 hours ago", "en"}, |
||
37 | {smallSubTime(-23 * time.Hour), "23 hours ago", "en"}, |
||
38 | {smallSubTime(-24 * time.Hour), "1 day ago", "en"}, |
||
39 | {smallSubTime(-30 * time.Hour), "1 day ago", "en"}, |
||
40 | {smallSubTime((-24 * 2) * time.Hour), "2 days ago", "en"}, |
||
41 | {smallSubTime((-24 * 6) * time.Hour), "6 days ago", "en"}, |
||
42 | {smallSubTime((-24 * 7) * time.Hour), "1 week ago", "en"}, |
||
43 | {smallSubTime((-24 * 14) * time.Hour), "2 weeks ago", "en"}, |
||
44 | {smallSubTime((-24 * 21) * time.Hour), "3 weeks ago", "en"}, |
||
45 | {bigSubTime(0, 1, 1), "1 month ago", "en"}, |
||
46 | {bigSubTime(0, 2, 1), "2 months ago", "en"}, |
||
47 | {bigSubTime(0, 9, 1), "9 months ago", "en"}, |
||
48 | {bigSubTime(0, 11, 1), "11 months ago", "en"}, |
||
49 | {bigSubTime(0, 12, 1), "1 year ago", "en"}, |
||
50 | {bigSubTime(1, 0, 1), "1 year ago", "en"}, |
||
51 | {bigSubTime(2, 0, 1), "2 years ago", "en"}, |
||
52 | {bigSubTime(21, 0, 1), "21 years ago", "en"}, |
||
53 | {bigSubTime(31, 0, 1), "31 years ago", "en"}, |
||
54 | {bigSubTime(100, 0, 1), "100 years ago", "en"}, |
||
55 | } |
||
56 | |||
57 | for _, tc := range cases { |
||
58 | t.Run("result for "+tc.date, func(test *testing.T) { |
||
59 | timeago.Set("language", tc.lang) |
||
60 | timeago.Set("location", "Europe/Kiev") |
||
61 | |||
62 | if res := timeago.Take(tc.date); res != tc.result { |
||
63 | test.Errorf("Result must be %s, but got %s instead", tc.result, res) |
||
64 | } |
||
168 |