[Asterisk-code-review] testsuite: Added testcase for channels/{channelId}/rtp_statistics (...testsuite[master])

[sungtae kim]

sungtae kim has posted comments on this change. ( https://gerrit.asterisk.org/c/testsuite/+/11178 )

Change subject: testsuite: Added testcase for channels/{channelId}/rtp_statistics
......................................................................


Patch Set 7:

> Patch Set 7: Code-Review-1
> 
> This doesn't seem to work for me, I receive the following:
> 
> <?xml version="1.0" encoding="utf-8"?>
> <testsuites>
>   <testsuite errors="0" failures="1" name="AsteriskTestSuite" tests="1" time="7.74" timestamp="2019-05-22T10:13:23 ADT">
>     <testcase classname="rest_api.channels.rtp_statistics" name="basic" time="7.74">
>       <failure>Running tests/rest_api/channels/rtp_statistics/basic ...
> [May 22 10:13:30] WARNING[22653]: twisted:154 publishToNewObserver: dropping connection to peer tcp4:127.0.0.1:8088 with abort=False: None
> [May 22 10:13:31] ERROR[22653]: asterisk.ari:670 on_stop: Expected 2 <= count <= 2; was 1 ({'match': {'application': 'testsuite', 'playback': {'id': 'MyPlaybackId', 'target_uri': 'channel:testsuite-default-id$'}, 'type': 'PlaybackStarted'}})
> </failure>
>     </testcase>
>   </testsuite>
> </testsuites>
> 
> Looking at the full.txt I show only a single PlaybackStarted:
> 
> [May 22 10:13:25] DEBUG[22653]: asterisk.ari:418 onMessage: rxed: {
>   "type": "PlaybackStarted",
>   "timestamp": "2019-05-22T10:13:25.476-0300",
>   "playback": {
>     "id": "MyPlaybackId",
>     "media_uri": "sound:silence/5",
>     "target_uri": "channel:testsuite-default-id",
>     "language": "fr",
>     "state": "playing"
>   },
>   "asterisk_id": "50:9a:4c:1c:45:9f",
>   "application": "testsuite"
> }

Hi Joshua,

Thank you for your kind reply.

There was a mistake when I push it.
I've fixed it. :)

Thank you.


-- 
To view, visit https://gerrit.asterisk.org/c/testsuite/+/11178
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Change-Id: Ie41127219a4fccc183f9374fbcb768835a7d2bb2
Gerrit-Change-Number: 11178
Gerrit-PatchSet: 7
Gerrit-Owner: sungtae kim <pchero21 at gmail.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: sungtae kim <pchero21 at gmail.com>
Gerrit-Comment-Date: Thu, 23 May 2019 21:56:58 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190523/bbd3cb01/attachment.html>