@@ -1150,7 +1150,7 @@ func TestProcessPolicyDetails(t *testing.T) {
11501150 req * CheckinRequest
11511151 getPolicyMonitor func () * mockPolicyMonitor
11521152 revIDX int64
1153- returnsOps bool
1153+ returnsOpts bool
11541154 err error
11551155 }{{
11561156 name : "request has no policy details" ,
@@ -1161,9 +1161,9 @@ func TestProcessPolicyDetails(t *testing.T) {
11611161 getPolicyMonitor : func () * mockPolicyMonitor {
11621162 return & mockPolicyMonitor {}
11631163 },
1164- revIDX : 1 ,
1165- returnsOps : false ,
1166- err : nil ,
1164+ revIDX : 1 ,
1165+ returnsOpts : false ,
1166+ err : nil ,
11671167 }, {
11681168 name : "policy reassign detected" ,
11691169 agent : & model.Agent {
@@ -1181,9 +1181,9 @@ func TestProcessPolicyDetails(t *testing.T) {
11811181 getPolicyMonitor : func () * mockPolicyMonitor {
11821182 return & mockPolicyMonitor {}
11831183 },
1184- revIDX : 0 ,
1185- returnsOps : false ,
1186- err : nil ,
1184+ revIDX : 0 ,
1185+ returnsOpts : false ,
1186+ err : nil ,
11871187 }, {
11881188 name : "revision updated" ,
11891189 agent : & model.Agent {
@@ -1203,9 +1203,9 @@ func TestProcessPolicyDetails(t *testing.T) {
12031203 pm .On ("LatestRev" , mock .Anything , policyID ).Return (int64 (2 )).Once ()
12041204 return pm
12051205 },
1206- revIDX : 2 ,
1207- returnsOps : true ,
1208- err : nil ,
1206+ revIDX : 2 ,
1207+ returnsOpts : true ,
1208+ err : nil ,
12091209 }, {
12101210 name : "checkin revision is greater than the policy's latest revision" ,
12111211 agent : & model.Agent {
@@ -1225,9 +1225,9 @@ func TestProcessPolicyDetails(t *testing.T) {
12251225 pm .On ("LatestRev" , mock .Anything , policyID ).Return (int64 (1 )).Once ()
12261226 return pm
12271227 },
1228- revIDX : 0 ,
1229- returnsOps : true ,
1230- err : nil ,
1228+ revIDX : 0 ,
1229+ returnsOpts : true ,
1230+ err : nil ,
12311231 }, {
12321232 name : "agent_policy_id has changed" ,
12331233 agent : & model.Agent {
@@ -1247,9 +1247,9 @@ func TestProcessPolicyDetails(t *testing.T) {
12471247 pm .On ("LatestRev" , mock .Anything , policyID ).Return (int64 (2 )).Once ()
12481248 return pm
12491249 },
1250- revIDX : 2 ,
1251- returnsOps : true ,
1252- err : nil ,
1250+ revIDX : 2 ,
1251+ returnsOpts : true ,
1252+ err : nil ,
12531253 }, {
12541254 name : "agent does not have agent_policy_id present" ,
12551255 agent : & model.Agent {
@@ -1268,9 +1268,9 @@ func TestProcessPolicyDetails(t *testing.T) {
12681268 pm .On ("LatestRev" , mock .Anything , policyID ).Return (int64 (2 )).Once ()
12691269 return pm
12701270 },
1271- revIDX : 2 ,
1272- returnsOps : true ,
1273- err : nil ,
1271+ revIDX : 2 ,
1272+ returnsOpts : true ,
1273+ err : nil ,
12741274 }, {
12751275 name : "details present with no changes from agent doc" ,
12761276 agent : & model.Agent {
@@ -1290,9 +1290,9 @@ func TestProcessPolicyDetails(t *testing.T) {
12901290 pm .On ("LatestRev" , mock .Anything , policyID ).Return (int64 (2 )).Once ()
12911291 return pm
12921292 },
1293- revIDX : 2 ,
1294- returnsOps : false ,
1295- err : nil ,
1293+ revIDX : 2 ,
1294+ returnsOpts : false ,
1295+ err : nil ,
12961296 }}
12971297
12981298 for _ , tc := range tests {
@@ -1307,7 +1307,7 @@ func TestProcessPolicyDetails(t *testing.T) {
13071307
13081308 revIDX , opts , err := checkin .processPolicyDetails (t .Context (), logger , tc .agent , tc .req )
13091309 assert .Equal (t , tc .revIDX , revIDX )
1310- if tc .returnsOps {
1310+ if tc .returnsOpts {
13111311 assert .NotEmpty (t , opts )
13121312 } else {
13131313 assert .Empty (t , opts )
0 commit comments