[Asterisk-code-review] Test/ARI: test for getting all module information (testsuite[master])

[Kevin Harwell]

Kevin Harwell has posted comments on this change.

Change subject: Test/ARI: test for getting all module information
......................................................................


Patch Set 2:

(2 comments)

https://gerrit.asterisk.org/#/c/838/2/tests/rest_api/asterisk/modules/get_modules/test-config.yaml
File tests/rest_api/asterisk/modules/get_modules/test-config.yaml:

Line 4:        * Receives list of all modules
The description needs to be expanded some. I'd take what you wrote for the commit message and use it here as the description, or something similar.


Line 211:     minversion: '13.1.0'
Not sure what it will be, but the minversion won't be 13.1.0. Are the Asterisk changes master only? Then it will be 14.0.0.


-- 
To view, visit https://gerrit.asterisk.org/838
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic16508cf03548a55ffa0339b94d432c7d8f660c1
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Scott Emidy <jemidy at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-HasComments: Yes