Skip to content

Commit a5d156d

Browse files
gtjosephgithub-actions[bot]
authored andcommittedMar 6, 2025
manager: Add scenarios to the config/restricted test and skip Lua tests.
* Added scenarios to the manager/config/restricted test to cover CreateConfig. * Fixed the spelling of "privileges" in manager/config/restricted. * Marked manager tests that require Lua as skipped until they can be rewritten. * action_events_response * authlimit * authtimeout
1 parent 12eab75 commit a5d156d

File tree

4 files changed

+29
-5
lines changed

4 files changed

+29
-5
lines changed
 

‎tests/manager/action-events-response/test-config.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
testinfo:
22
summary: 'Test the presence and absence of a response from the Events manager action'
3+
skip: 'Lua test that needs to be updated'
34
description: |
45
"This test ensure asterisk either responds or does not respond to
56
various values of the EventMaks parameter of the Events manager

‎tests/manager/authlimit/test-config.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
testinfo:
22
summary: 'Test the authlimit manager config option.'
3+
skip: 'Lua test that needs to be updated'
34
description: |
45
"This test ensures that the manager respects the authlimit config
56
option."

‎tests/manager/authtimeout/test-config.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
testinfo:
22
summary: 'Test the authtimeout manager config option.'
3+
skip: 'Lua test that needs to be updated'
34
description: |
45
"This test ensures that the manager respects the authtimeout config
56
option."

‎tests/manager/config/restricted/test-config.yaml

+26-5
Original file line numberDiff line numberDiff line change
@@ -24,13 +24,34 @@ object-config:
2424
reactor-timeout: 15
2525
testfile: '/tmp/test.conf'
2626
ami-config:
27-
- ### Create a new config file outside the asterisk config directory
27+
- ### Create a config file outside the asterisk config directory
28+
message:
29+
Action: 'CreateConfig'
30+
Filename: '/tmp/test.conf'
31+
expected:
32+
Response: 'Error'
33+
Message: 'File requires escalated privileges'
34+
- ### Create a config file outside the asterisk config directory
35+
message:
36+
Action: 'CreateConfig'
37+
Filename: '../test.conf'
38+
expected:
39+
Response: 'Error'
40+
Message: 'File requires escalated privileges'
41+
- ### Create a config file outside the asterisk config directory
42+
message:
43+
Action: 'CreateConfig'
44+
Filename: '/etc/asterisk/../test.conf'
45+
expected:
46+
Response: 'Error'
47+
Message: 'File requires escalated privileges'
48+
- ### Retrieve a config file outside the asterisk config directory
2849
message:
2950
Action: 'GetConfig'
3051
Filename: '/tmp/test.conf'
3152
expected:
3253
Response: 'Error'
33-
Message: 'File requires escalated priveledges'
54+
Message: 'File requires escalated privileges'
3455
- ### Update a config file outside the asterisk config directory
3556
message:
3657
Action: 'UpdateConfig'
@@ -41,7 +62,7 @@ object-config:
4162
Cat-000000: 'Cat1'
4263
expected:
4364
Response: 'Error'
44-
Message: 'File requires escalated priveledges'
65+
Message: 'File requires escalated privileges'
4566
- ### Update from a config file outside the asterisk config directory
4667
message:
4768
Action: 'UpdateConfig'
@@ -52,7 +73,7 @@ object-config:
5273
Cat-000000: 'Cat1'
5374
expected:
5475
Response: 'Error'
55-
Message: 'File requires escalated priveledges'
76+
Message: 'File requires escalated privileges'
5677
- ### Update to a config file outside the asterisk config directory
5778
message:
5879
Action: 'UpdateConfig'
@@ -63,6 +84,6 @@ object-config:
6384
Cat-000000: 'Cat1'
6485
expected:
6586
Response: 'Error'
66-
Message: 'File requires escalated priveledges'
87+
Message: 'File requires escalated privileges'
6788

6889

0 commit comments

Comments
 (0)