Skip to content

Commit 9d7a794

Browse files
committed
Fix some lint warnings in tests
Signed-off-by: Bryan Boreham <[email protected]>
1 parent cfaa978 commit 9d7a794

File tree

2 files changed

+9
-10
lines changed

2 files changed

+9
-10
lines changed

pkg/ring/lifecycler_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -340,7 +340,6 @@ func TestCheckReady(t *testing.T) {
340340
}
341341

342342
type noopFlushTransferer struct {
343-
lifecycler *Lifecycler
344343
}
345344

346345
func (f *noopFlushTransferer) StopIncomingRequests() {}

pkg/ruler/rules/store_test.go

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ func Test_ConfigRuleStoreError(t *testing.T) {
3333
}
3434

3535
store := NewConfigRuleStore(mock)
36-
_, err := store.ListAllRuleGroups(nil)
36+
_, err := store.ListAllRuleGroups(context.Background())
3737

3838
assert.Equal(t, mock.err, err, "Unexpected error returned")
3939
}
@@ -52,7 +52,7 @@ func Test_ConfigRuleStoreReturn(t *testing.T) {
5252
}
5353

5454
store := NewConfigRuleStore(mock)
55-
rules, _ := store.ListAllRuleGroups(nil)
55+
rules, _ := store.ListAllRuleGroups(context.Background())
5656

5757
assert.Equal(t, 1, len(rules["user"]))
5858
assert.Equal(t, id, store.since)
@@ -71,15 +71,15 @@ func Test_ConfigRuleStoreDelete(t *testing.T) {
7171
}
7272

7373
store := NewConfigRuleStore(mock)
74-
store.ListAllRuleGroups(nil)
74+
_, _ = store.ListAllRuleGroups(context.Background())
7575

7676
mock.cfgs["user"] = configs.VersionedRulesConfig{
7777
ID: 1,
7878
Config: configs.RulesConfig{},
7979
DeletedAt: time.Unix(0, 1),
8080
}
8181

82-
rules, _ := store.ListAllRuleGroups(nil)
82+
rules, _ := store.ListAllRuleGroups(context.Background())
8383

8484
assert.Equal(t, 0, len(rules["user"]))
8585
}
@@ -97,7 +97,7 @@ func Test_ConfigRuleStoreAppend(t *testing.T) {
9797
}
9898

9999
store := NewConfigRuleStore(mock)
100-
store.ListAllRuleGroups(nil)
100+
_, _ = store.ListAllRuleGroups(context.Background())
101101

102102
delete(mock.cfgs, "user")
103103
mock.cfgs["user2"] = configs.VersionedRulesConfig{
@@ -106,7 +106,7 @@ func Test_ConfigRuleStoreAppend(t *testing.T) {
106106
DeletedAt: zeroTime,
107107
}
108108

109-
rules, _ := store.ListAllRuleGroups(nil)
109+
rules, _ := store.ListAllRuleGroups(context.Background())
110110

111111
assert.Equal(t, 2, len(rules))
112112
}
@@ -134,7 +134,7 @@ func Test_ConfigRuleStoreSinceSet(t *testing.T) {
134134
}
135135

136136
store := NewConfigRuleStore(mock)
137-
store.ListAllRuleGroups(nil)
137+
_, _ = store.ListAllRuleGroups(context.Background())
138138
assert.Equal(t, configs.ID(100), store.since)
139139

140140
delete(mock.cfgs, "user")
@@ -145,7 +145,7 @@ func Test_ConfigRuleStoreSinceSet(t *testing.T) {
145145
DeletedAt: zeroTime,
146146
}
147147

148-
store.ListAllRuleGroups(nil)
148+
_, _ = store.ListAllRuleGroups(context.Background())
149149
assert.Equal(t, configs.ID(100), store.since)
150150

151151
mock.cfgs["user2"] = configs.VersionedRulesConfig{
@@ -154,7 +154,7 @@ func Test_ConfigRuleStoreSinceSet(t *testing.T) {
154154
DeletedAt: zeroTime,
155155
}
156156

157-
store.ListAllRuleGroups(nil)
157+
_, _ = store.ListAllRuleGroups(context.Background())
158158
assert.Equal(t, configs.ID(101), store.since)
159159
}
160160

0 commit comments

Comments
 (0)