Skip to content

Commit 0f5c54a

Browse files
committed
Fixed findlocalusers_unsafe test
Ticket: ENT-13533 Signed-off-by: Victor Moene <victor.moene@northern.tech>
1 parent ff09a71 commit 0f5c54a

File tree

2 files changed

+39
-92
lines changed

2 files changed

+39
-92
lines changed

tests/acceptance/01_vars/02_functions/unsafe/findlocalusers_unsafe.cf

Lines changed: 17 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -14,55 +14,50 @@ bundle agent init
1414
create => "true";
1515

1616
users:
17-
"test_user1"
17+
"testu1"
1818
policy => "present",
1919
home_dir => "/tmp/test_folder1",
2020
description => "TestUser 1",
21-
group_primary => "users",
21+
group_primary => "root", # we need a group which we know for sure the gid
2222
shell => "/bin/sh",
2323
uid => "12345";
2424

25-
"test_user2"
25+
"testu2"
2626
policy => "present",
2727
home_dir => "/tmp/test_folder2",
2828
description => "TestUser 2",
29-
group_primary => "users",
29+
group_primary => "root",
3030
shell => "/bin/sh",
3131
uid => "54321";
3232
vars:
3333
# simple filters
34-
"simple_filter" slist => { "name=root" };
35-
"number_filter" slist => { "uid=0" };
34+
"simple_filter" slist => { "name=testu1" };
35+
"number_filter" slist => { "uid=12345" };
3636

3737
# longer filters
38-
"slist_filter" slist => { "gid=0", "name=root" };
38+
"slist_filter" slist => { "gid=0", "name=testu1" };
3939

4040
# using data
41-
"data_filter" data => '[ "gid=0", "name=root" ]';
41+
"data_filter" data => '[ "gid=0", "name=testu1" ]';
4242

4343
# using regex
44-
"simple_regex" slist => { "name=test_user.*" };
44+
"simple_regex" slist => { "name=testu.*" };
4545
"number_regex" slist => { "uid=1234.*" };
46-
"longer_regex" slist => { "name=test_user.*", "uid=1.*" };
46+
"longer_regex" slist => { "name=testu.*", "uid=1.*" };
4747

4848
# non-existent user
4949
"unknown" slist => { "name=thisuserdoesntexist" };
5050
}
5151
bundle agent test
5252
{
53-
meta:
54-
"test_soft_fail" string => "windows|aix|solaris|hpux|redhat_10",
55-
comment => "RedHat 10 root user is named 'Super User' not 'root'",
56-
meta => { "CFE-2318" };
57-
5853
vars:
59-
"ulist1" data => findlocalusers("@(init.simple_filter)");
60-
"ulist2" data => findlocalusers("init.number_filter");
61-
"ulist4" data => findlocalusers("@(init.slist_filter)");
62-
"ulist3" data => findlocalusers("@(init.data_filter)");
63-
"ulist5" data => findlocalusers("@(init.simple_regex)");
64-
"ulist6" data => findlocalusers("@(init.number_regex)");
65-
"ulist7" data => findlocalusers("@(init.longer_regex)");
54+
"ulist1" data => mapdata(json, '"$(this.k)"', findlocalusers("@(init.simple_filter)"));
55+
"ulist2" data => mapdata(json, '"$(this.k)"', findlocalusers("init.number_filter"));
56+
"ulist4" data => mapdata(json, '"$(this.k)"', findlocalusers("@(init.slist_filter)"));
57+
"ulist3" data => mapdata(json, '"$(this.k)"', findlocalusers("@(init.data_filter)"));
58+
"ulist5" data => mapdata(json, '"$(this.k)"', findlocalusers("@(init.simple_regex)"));
59+
"ulist6" data => mapdata(json, '"$(this.k)"', findlocalusers("@(init.number_regex)"));
60+
"ulist7" data => mapdata(json, '"$(this.k)"', findlocalusers("@(init.longer_regex)"));
6661
"ulist8" data => findlocalusers("@(init.unknown)");
6762

6863
}
Lines changed: 22 additions & 70 deletions
Original file line numberDiff line numberDiff line change
@@ -1,74 +1,26 @@
11
{
2-
"ulist1": {
3-
"root": {
4-
"dir": "/root",
5-
"gecos": "root",
6-
"gid": 0,
7-
"shell": "/bin/bash",
8-
"uid": 0
9-
}
10-
},
11-
"ulist2": {
12-
"root": {
13-
"dir": "/root",
14-
"gecos": "root",
15-
"gid": 0,
16-
"shell": "/bin/bash",
17-
"uid": 0
18-
}
19-
},
20-
"ulist3": {
21-
"root": {
22-
"dir": "/root",
23-
"gecos": "root",
24-
"gid": 0,
25-
"shell": "/bin/bash",
26-
"uid": 0
27-
}
28-
},
29-
"ulist4": {
30-
"root": {
31-
"dir": "/root",
32-
"gecos": "root",
33-
"gid": 0,
34-
"shell": "/bin/bash",
35-
"uid": 0
36-
}
37-
},
38-
"ulist5": {
39-
"test_user1": {
40-
"dir": "/tmp/test_folder1",
41-
"gecos": "TestUser 1",
42-
"gid": 100,
43-
"shell": "/bin/sh",
44-
"uid": 12345
45-
},
46-
"test_user2": {
47-
"dir": "/tmp/test_folder2",
48-
"gecos": "TestUser 2",
49-
"gid": 100,
50-
"shell": "/bin/sh",
51-
"uid": 54321
52-
}
53-
},
54-
"ulist6": {
55-
"test_user1": {
56-
"dir": "/tmp/test_folder1",
57-
"gecos": "TestUser 1",
58-
"gid": 100,
59-
"shell": "/bin/sh",
60-
"uid": 12345
61-
}
62-
},
63-
"ulist7": {
64-
"test_user1": {
65-
"dir": "/tmp/test_folder1",
66-
"gecos": "TestUser 1",
67-
"gid": 100,
68-
"shell": "/bin/sh",
69-
"uid": 12345
70-
}
71-
},
2+
"ulist1": [
3+
"testu1"
4+
],
5+
"ulist2": [
6+
"testu1"
7+
],
8+
"ulist3": [
9+
"testu1"
10+
],
11+
"ulist4": [
12+
"testu1"
13+
],
14+
"ulist5": [
15+
"testu1",
16+
"testu2"
17+
],
18+
"ulist6": [
19+
"testu1"
20+
],
21+
"ulist7": [
22+
"testu1"
23+
],
7224
"ulist8": {
7325
}
7426
}

0 commit comments

Comments
 (0)