Skip to content

Commit 5c2e8a0

Browse files
committed
user: add test-cases for maxID (math.MaxInt32)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
1 parent e001aea commit 5c2e8a0

1 file changed

Lines changed: 38 additions & 0 deletions

File tree

user/user_test.go

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,8 @@ root:x:0:0:root user:/root:/bin/bash
104104
adm:x:42:43:adm:/var/adm:/bin/false
105105
111:x:222:333::/var/garbage
106106
odd:x:111:112::/home/odd:::::
107+
2147483647:x:0:0:maxint32:/root:/bin/bash
108+
2147483648:x:0:0:toolarge:/root:/bin/bash
107109
user7456:x:7456:100:Vasya:/home/user7456
108110
this is just some garbage data
109111
`
@@ -113,6 +115,8 @@ adm:x:43:
113115
grp:x:1234:root,adm,user7456
114116
444:x:555:111
115117
odd:x:444:
118+
2147483647:x:1235:
119+
2147483648:x:1236:
116120
this is just some garbage data
117121
` + largeGroup()
118122

@@ -229,6 +233,33 @@ this is just some garbage data
229233
Home: "/home/user7456",
230234
},
231235
},
236+
{
237+
ref: "7456:2147483647",
238+
expected: ExecUser{
239+
Uid: 7456,
240+
Gid: 2147483647, // maxID
241+
Sgids: defaultExecUser.Sgids,
242+
Home: "/home/user7456",
243+
},
244+
},
245+
{
246+
ref: "2147483647:43",
247+
expected: ExecUser{
248+
Uid: 2147483647, // maxID
249+
Gid: 43,
250+
Sgids: defaultExecUser.Sgids,
251+
Home: defaultExecUser.Home,
252+
},
253+
},
254+
{
255+
ref: "2147483647",
256+
expected: ExecUser{
257+
Uid: 2147483647, // maxID
258+
Gid: defaultExecUser.Gid,
259+
Sgids: defaultExecUser.Sgids,
260+
Home: defaultExecUser.Home,
261+
},
262+
},
232263
}
233264

234265
for _, tc := range tests {
@@ -260,12 +291,16 @@ func TestGetExecUserInvalid(t *testing.T) {
260291
root:x:0:0:root user:/root:/bin/bash
261292
adm:x:42:43:adm:/var/adm:/bin/false
262293
-42:x:12:13:broken:/very/broken
294+
2147483647:x:0:0:maxint32:/root:/bin/bash
295+
2147483648:x:0:0:toolarge:/root:/bin/bash
263296
this is just some garbage data
264297
`
265298
const groupContent = `
266299
root:x:0:root
267300
adm:x:43:
268301
grp:x:1234:root,adm
302+
2147483647:x:1235:
303+
2147483648:x:1236:
269304
this is just some garbage data
270305
`
271306

@@ -284,6 +319,9 @@ this is just some garbage data
284319
"-5:-2",
285320
"-42",
286321
"-43",
322+
"42:2147483648", // maxID + 1
323+
"2147483648:43", // maxID + 1
324+
"2147483648", // maxID + 1
287325
}
288326

289327
for _, tc := range tests {

0 commit comments

Comments
 (0)