MDL-70725 oauth2: move expectException to separate method

diff --git a/lib/tests/oauth2_test.php b/lib/tests/oauth2_test.php
index 62ce761..26fd9e2 100644 (file)
--- a/lib/tests/oauth2_test.php
+++ b/lib/tests/oauth2_test.php
@@ -44,9 +44,6 @@ class core_oauth2_testcase extends advanced_testcase {
         \core\oauth2\api::create_standard_issuer('microsoft');
         \core\oauth2\api::create_standard_issuer('nextcloud', 'https://dummy.local/nextcloud/');
 
-        $this->expectException(\moodle_exception::class);
-        \core\oauth2\api::create_standard_issuer('nextcloud');
-
         $issuers = \core\oauth2\api::get_all_issuers();
 
         $this->assertEquals($issuers[0]->get('name'), 'Google');
@@ -72,6 +69,17 @@ class core_oauth2_testcase extends advanced_testcase {
         $this->assertEquals($issuers[2]->get('name'), 'Nextcloud');
     }
 
+    /**
+     * Tests the crud operations on oauth2 issuers.
+     */
+    public function test_create_nextcloud_without_url() {
+        $this->resetAfterTest();
+        $this->setAdminUser();
+
+        $this->expectException(\moodle_exception::class);
+        \core\oauth2\api::create_standard_issuer('nextcloud');
+    }
+
     /**
      * Tests we can list and delete each of the persistents related to an issuer.
      */