Add new test case and refactor ScopeTest

Added test case 'testEqual' to enhance the coverage of code in ScopeTest. Simplified the occurrences of UnknownScopeCodeException in all test cases, and corrected the order of scope codes inside the 'testInitFromString' test case.

Signed-off-by: mesilov <[email protected]>

tests/Unit/Core/Credentials/ScopeTest.php

@@ -8,15 +8,10 @@
 use Bitrix24\SDK\Core\Exceptions\UnknownScopeCodeException;
 use PHPUnit\Framework\TestCase;
 
-/**
- * Class ScopeTest
- *
- * @package Bitrix24\SDK\Tests\Unit\Core
- */
 class ScopeTest extends TestCase
 {
     /**
-     * @throws \Bitrix24\SDK\Core\Exceptions\UnknownScopeCodeException
+     * @throws UnknownScopeCodeException
      */
     public function testBuildScopeFromArray(): void
     {
@@ -73,6 +68,16 @@ public function testUnknownScope(): void
         $scope = new Scope(['fooo']);
     }
 
+    /**
+     * @throws UnknownScopeCodeException
+     */
+    public function testEqual(): void
+    {
+        $scope = Scope::initFromString('crm,telephony');
+        $this->assertTrue($scope->equal(Scope::initFromString('telephony,crm')));
+        $this->assertFalse($scope->equal(Scope::initFromString('telephony')));
+    }
+
     /**
      * @throws UnknownScopeCodeException
      */
@@ -89,18 +94,20 @@ public function testWrongScopeCode(): void
     {
         $scope = new Scope(['CRM', 'Call', 'im']);
 
-        $this->assertEquals(['crm', 'call', 'im'], $scope->getScopeCodes());
+        $this->assertEquals(['call', 'crm', 'im'], $scope->getScopeCodes());
     }
 
     /**
      * @return void
-     * @throws \Bitrix24\SDK\Core\Exceptions\UnknownScopeCodeException
+     * @throws UnknownScopeCodeException
      * @covers  \Bitrix24\SDK\Core\Credentials\Scope::initFromString
      * @testdox Test init Scope from string
      */
     public function testInitFromString(): void
     {
+        $scopeList = ['crm', 'telephony', 'call', 'user_basic', 'placement', 'im', 'imopenlines'];
+        sort($scopeList);
         $scope = Scope::initFromString('crm,telephony,call,user_basic,placement,im,imopenlines');
-        $this->assertEquals(['crm', 'telephony', 'call', 'user_basic', 'placement', 'im', 'imopenlines'], $scope->getScopeCodes());
+        $this->assertEquals($scopeList, $scope->getScopeCodes());
     }
 }