[Asterisk-code-review] tests/remote-test: Add a sample remote test (testsuite[master])

[Matt Jordan]

Matt Jordan has posted comments on this change.

Change subject: tests/remote-test: Add a sample remote test
......................................................................


Patch Set 2:

> I think that somewhere in the test-config.yaml file, there needs to
 > be some sort of indication of what is expected to exist on the
 > remote Asterisk instance to enable the test to execute.
 > 
 > For instance, in this test, it is expected that there is a default
 > context with an s extension in it, and app_echo is expected to be
 > loaded.
 > 
 > For a small example like this, one can deduce what's expected to be
 > there simply from the description, but I think setting a precedent
 > of spelling out the expected content on the remote Asterisk will
 > make it easier for people to know what to expect.
 > 
 > I would hope that eventually it would be expected to push expected
 > configuration for individual tests to the remote Asterisk so there
 > is no assumption of what is on the remote Asterisk when the test
 > runs. This would also allow for tests to be able to push their own
 > configuration that would otherwise conflict with each other.
 > 
 > For the time being, push configuration to remote Asterisks is a
 > ways off, but I think that noting the expectations for the remote
 > Asterisk can be done for the time being.

Do you think the description should just be updated, or are you envisioning some other fields in the YAML that describe the expected configuration of the remote system?

And yes, push configuration to remote systems is a different task than this one. For example, one could write a pluggable module that used AMI to dynamically construct a dialplan - but that'd make these reviews a big larger :-)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I857c5f20d4c260be73697062506f63cb3533c3a8
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: No