Commit defaf5d4 authored by Aaron Wells's avatar Aaron Wells Committed by Robert Lyon
Browse files

Move WebService "markTestSkipped" to setup (Bug 1515473)

... so that we don't bother doing setup just to get
to testRun() and realize we're skipping this test

behatnotneeded: phpunit, not behat

Change-Id: I0c96e364129435f9eca3f0cb3bb44beaf9db4701
parent 48ea9ab8
......@@ -79,6 +79,8 @@ class WebServiceTestBase extends MaharaUnitTest {
protected function setUp() {
parent::setUp();
$this->markTestSkipped('cURL requests need to be mocked properly. Skipping for now.');
// default current user to admin
global $USER;
$USER->id = 1;
......@@ -279,7 +281,6 @@ class WebServiceTestBase extends MaharaUnitTest {
* of iterations, auth types, and protocols
*/
public function testRun() {
$this->markTestSkipped('cURL requests need to be mocked properly. Skipping for now.');
// do we have any tests
if (!$this->testrest and !$this->testxmlrpc and !$this->testsoap) {
log_debug("Web service unit tests are not run as not setup (see /webservice/simpletest/testwebservice.php)");
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment