Fix dealer role missing issue add test

voryx · Jul 15, 2017 · 2927b39 · 2927b39
1 parent 37b7c64
commit 2927b39
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/Thruway/Role/Dealer.php b/src/Thruway/Role/Dealer.php
@@ -77,7 +77,7 @@ public function getSubscribedRealmEvents()
           "YieldMessageEvent"      => ["handleYieldMessage", 10],
           "ErrorMessageEvent"      => ["handleErrorMessage", 10],
           "LeaveRealm"             => ["handleLeaveRealm", 10],
-          "SendWelcomeMessageEvent" => ["handleSendWelcomeMessage", 10]
+          "SendWelcomeMessageEvent" => ["handleSendWelcomeMessage", 20]
         ];
     }
 

diff --git a/tests/Unit/Peer/RouterTest.php b/tests/Unit/Peer/RouterTest.php
@@ -1,4 +1,5 @@
 <?php
+use Thruway\Message\WelcomeMessage;
 
 /**
  * Class RouterTest
@@ -110,8 +111,12 @@ public function testHelloMessage()
             ->method('sendMessage')
             ->with(
                 $this->callback(
-                    function ($msg) {
+                    function (WelcomeMessage $msg) {
                         $this->assertInstanceOf('\Thruway\Message\WelcomeMessage', $msg);
+                        $this->assertNotEmpty($msg->getDetails());
+                        $this->assertObjectHasAttribute('roles', $msg->getDetails());
+                        $this->assertObjectHasAttribute('dealer', $msg->getDetails()->roles);
+                        $this->assertObjectHasAttribute('broker', $msg->getDetails()->roles);
 
                         return $msg instanceof Thruway\Message\WelcomeMessage;
                     }