diff options
author | Stefan Giehl <stefan@piwik.org> | 2017-02-19 13:28:59 +0300 |
---|---|---|
committer | Matthieu Aubry <mattab@users.noreply.github.com> | 2017-02-19 13:28:59 +0300 |
commit | 471aaea1166bc71c8bd28c953e4b192f72646070 (patch) | |
tree | db3f3dcbb6e6e21e312b0babd11ac4c2dd16fb4d /tests/UI | |
parent | 918ef6f92c9a44be67fc8d11442f6eefd925e781 (diff) |
Implements new SMS Provider ASPSMS.com (#11263)
* makes it possible to define the required credential fields in sms provider
* show error message if configured sms provider doesn't work, instead of throwing uncatched exception
* Adds new SMS provider ASPSMS.com
* Development SMS notifications didn't work when clicking 'send now'
Caused by the session already closed before, as it's done using a ajax request
calling a Session:start() reopens the session. Might be a bit hackish, but should be fine for development
* adds UI test for switching sms provider
* Makes API-Key default credential field to prevent breaking third party plugins
* fixes issue where save button was disabled falsely
Diffstat (limited to 'tests/UI')
4 files changed, 15 insertions, 4 deletions
diff --git a/tests/UI/expected-screenshots/UIIntegrationTest_admin_settings_mobilemessaging.png b/tests/UI/expected-screenshots/UIIntegrationTest_admin_settings_mobilemessaging.png index 0aa2d9e038..a45b6971c4 100644 --- a/tests/UI/expected-screenshots/UIIntegrationTest_admin_settings_mobilemessaging.png +++ b/tests/UI/expected-screenshots/UIIntegrationTest_admin_settings_mobilemessaging.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:3bfb217b2559a0903376b184d06acaaac3818b16e265c0a45bfab9b7aa17cdf9 -size 121285 +oid sha256:7965bbcbad4cea0d8aa9431c7317e2923d54073fe8f212353ee2b8ff0d25d28e +size 152907 diff --git a/tests/UI/expected-screenshots/UIIntegrationTest_admin_settings_mobilemessaging_provider.png b/tests/UI/expected-screenshots/UIIntegrationTest_admin_settings_mobilemessaging_provider.png new file mode 100644 index 0000000000..0aa2d9e038 --- /dev/null +++ b/tests/UI/expected-screenshots/UIIntegrationTest_admin_settings_mobilemessaging_provider.png @@ -0,0 +1,3 @@ +version https://git-lfs.github.com/spec/v1 +oid sha256:3bfb217b2559a0903376b184d06acaaac3818b16e265c0a45bfab9b7aa17cdf9 +size 121285 diff --git a/tests/UI/expected-screenshots/UIIntegrationTest_api_listing.png b/tests/UI/expected-screenshots/UIIntegrationTest_api_listing.png index 2890c755f5..a53594da01 100644 --- a/tests/UI/expected-screenshots/UIIntegrationTest_api_listing.png +++ b/tests/UI/expected-screenshots/UIIntegrationTest_api_listing.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:7bd1af76f0fd5e578d67dc6191542344dacb6d1d4be40a6a4f2ed0d22391e87a -size 4187838 +oid sha256:3329e42371a9eaeffa71a082d03e99bf3128161ecb2cec73413957bb7f483cd5 +size 4188700 diff --git a/tests/UI/specs/UIIntegration_spec.js b/tests/UI/specs/UIIntegration_spec.js index 6923b75377..a1e43ead5c 100644 --- a/tests/UI/specs/UIIntegration_spec.js +++ b/tests/UI/specs/UIIntegration_spec.js @@ -505,6 +505,14 @@ describe("UIIntegrationTest", function () { // TODO: Rename to Piwik? }, done); }); + it('should switch the SMS provider correctly', function (done) { + expect.screenshot('admin_settings_mobilemessaging_provider').to.be.captureSelector('.pageWrap', function (page) { + page.evaluate(function() { + $('[name=smsProviders] ul li:nth-child(2)').click(); + }, 150); + }, done); + }); + it('should load the themes admin page correctly', function (done) { expect.screenshot('admin_themes').to.be.captureSelector('.pageWrap', function (page) { page.load("?" + generalParams + "&module=CorePluginsAdmin&action=themes"); |