[Asterisk-code-review] rest api/recording/stored: Add a test for downloading a stor... (testsuite[master])
Matt Jordan
asteriskteam at digium.com
Thu May 19 09:48:31 CDT 2016
Matt Jordan has uploaded a new change for review.
https://gerrit.asterisk.org/2877
Change subject: rest_api/recording/stored: Add a test for downloading a stored recording file
......................................................................
rest_api/recording/stored: Add a test for downloading a stored recording file
This patch adds a test that covers downloading the media associated with
a stored recording. The test:
- Spawn a Local channel and places both ends into the Stasis app
- One end of the channel has tt-monkeys played back on it
- The other end records the file
- When the playback is completed, the recording is finished
- A custom callback then attempts to download the media in the stored
recording. We verify that the file is downloaded, has some bytes in
it, and pass if that is the case.
ASTERISK-26042
Change-Id: I494d3c329cb00e8b20a5435da6a542a923b92ef6
---
A tests/rest_api/recording/stored/configs/extensions.conf
A tests/rest_api/recording/stored/file/recording.py
A tests/rest_api/recording/stored/file/test-config.yaml
M tests/rest_api/recording/stored/tests.yaml
4 files changed, 155 insertions(+), 0 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/77/2877/1
diff --git a/tests/rest_api/recording/stored/configs/extensions.conf b/tests/rest_api/recording/stored/configs/extensions.conf
new file mode 100644
index 0000000..aa88cf3
--- /dev/null
+++ b/tests/rest_api/recording/stored/configs/extensions.conf
@@ -0,0 +1,6 @@
+[default]
+
+exten => s,1,NoOp()
+ same => n,Answer()
+ same => n,Stasis(testsuite)
+ same => n,Hangup()
diff --git a/tests/rest_api/recording/stored/file/recording.py b/tests/rest_api/recording/stored/file/recording.py
new file mode 100644
index 0000000..bc0365b
--- /dev/null
+++ b/tests/rest_api/recording/stored/file/recording.py
@@ -0,0 +1,47 @@
+"""Check that we can retrieve the raw sound in a file
+
+Copyright (C) 2016, Digium, Inc.
+Matt Jordan <mjordan at digium.com>
+
+This program is free software, distributed under the terms of
+the GNU General Public License Version 2.
+"""
+
+import logging
+import requests
+import os
+
+from twisted.internet import reactor
+
+LOGGER = logging.getLogger(__name__)
+
+FILEPATH = '/tmp/superfly.wav'
+
+
+def on_recording_finished(ari, event, test_object):
+ LOGGER.info('Recording finished')
+
+ url = ari.build_url('recordings/stored', 'superfly', 'file')
+ resp = requests.get(url, stream=True, auth=ari.userpass)
+
+ if resp.status_code != 200:
+ LOGGER.error('Failed to download superfly: {0}'.format(
+ resp.status_code))
+ return False
+
+ with open(FILEPATH, 'wb') as f:
+ for chunk in resp:
+ f.write(chunk)
+
+ if (os.path.getsize(FILEPATH) == 0):
+ LOGGER.error('Sound file superfly is 0 bytes')
+ return False
+
+ LOGGER.info('Superfly downloaded successfully')
+ os.remove(FILEPATH)
+
+ ari.delete('channels', 'testsuite-default-id')
+
+ test_object.stop_reactor()
+
+ return True
diff --git a/tests/rest_api/recording/stored/file/test-config.yaml b/tests/rest_api/recording/stored/file/test-config.yaml
new file mode 100644
index 0000000..f498ee9
--- /dev/null
+++ b/tests/rest_api/recording/stored/file/test-config.yaml
@@ -0,0 +1,101 @@
+testinfo:
+ summary: Test downloading a stored recording
+ description: |
+ This test does the following:
+ - Makes a recording
+ - Retrieves the recording
+ - Verifies that the recording was downloaded successfully
+
+test-modules:
+ add-test-to-search-path: True
+ test-object:
+ config-section: test-object-config
+ typename: ari.AriOriginateTestObject
+ modules:
+ - config-section: ari-config
+ typename: ari.WebSocketEventModule
+ -
+ config-section: asterisk-config
+ typename: pluggable_modules.AsteriskConfigModule
+
+test-object-config:
+ stop-on-end: False
+ test-iterations:
+ -
+ endpoint: 'Local/s at default'
+ channelId: 'testsuite-default-id'
+ otherChannelId: 'testsuite-default-other-id'
+ extension: 's'
+ context: 'default'
+ priority: 1
+
+asterisk-config:
+ -
+ src: 'tests/rest_api/recording/stored/configs/extensions.conf'
+ dst: 'extensions.conf'
+
+ari-config:
+ apps: testsuite
+ events:
+ -
+ conditions:
+ match:
+ type: StasisStart
+ application: testsuite
+ channel:
+ id: 'testsuite-default-id$'
+ count: 1
+ requests:
+ method: 'post'
+ uri: 'channels/testsuite-default-id/record'
+ params:
+ name: 'superfly'
+ format: 'wav'
+ -
+ conditions:
+ match:
+ type: StasisStart
+ application: testsuite
+ channel:
+ id: 'testsuite-default-other-id'
+ count: 1
+ requests:
+ method: 'post'
+ uri: 'channels/testsuite-default-other-id/play/MyPlaybackId'
+ params:
+ media: 'sound:tt-monkeys'
+ -
+ conditions:
+ match:
+ type: PlaybackFinished
+ application: testsuite
+ playback:
+ id: MyPlaybackId
+ count: 1
+ requests:
+ method: 'post'
+ uri: 'recordings/live/superfly/stop'
+ -
+ conditions:
+ match:
+ type: RecordingFinished
+ recording:
+ name: superfly
+ count: 1
+ callback:
+ module: recording
+ method: on_recording_finished
+
+
+properties:
+ minversion: '14.0.0'
+ dependencies:
+ - python : autobahn.websocket
+ - python : requests
+ - python : twisted
+ - python : starpy
+ - asterisk : res_ari_channels
+ - asterisk : res_ari_recordings
+ - asterisk : app_echo
+ tags:
+ - ARI
diff --git a/tests/rest_api/recording/stored/tests.yaml b/tests/rest_api/recording/stored/tests.yaml
index b85272b..e3176f2 100644
--- a/tests/rest_api/recording/stored/tests.yaml
+++ b/tests/rest_api/recording/stored/tests.yaml
@@ -1,2 +1,3 @@
tests:
- dir: 'copy'
+ - test: 'file'
--
To view, visit https://gerrit.asterisk.org/2877
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I494d3c329cb00e8b20a5435da6a542a923b92ef6
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
More information about the asterisk-code-review
mailing list