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

[Mark Michelson]

Mark Michelson has posted comments on this change.

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


Patch Set 2: Code-Review-1

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.

-- 
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