Refresh (Combined)Status entities

php-api-clients · WyriHaximus · Oct 11, 2019 · Oct 11, 2019 · Oct 11, 2019 · Oct 11, 2019
commit a23c885830a49af8321157774cae895e7b8daaf5
diff --git a/examples/advanced-add-github-sponsors-file.php b/examples/advanced-add-github-sponsors-file.php
@@ -8,11 +8,10 @@
 use ApiClients\Client\Github\Resource\Git\RefInterface;
 use ApiClients\Client\Github\Resource\TreeInterface;
 use ApiClients\Client\Github\VO\NamedBlob;
+use function ApiClients\Tools\Rx\unwrapObservableFromPromise;
 use React\EventLoop\Factory;
 use React\Stream\ThroughStream;
 use Rx\React\Promise;
-use function ApiClients\Tools\Rx\unwrapObservableFromPromise;
-use function React\Promise\reject;
 
 require \dirname(__DIR__) . \DIRECTORY_SEPARATOR . 'vendor/autoload.php';
 
@@ -28,7 +27,7 @@
 })->filter(function (Repository $repository) {
     return $repository->name() !== 'reactphp-simple-orm';
 })->filter(function (Repository $repository) {
-    return strpos($repository->name(), 'reactphp') !== false;
+    return \strpos($repository->name(), 'reactphp') !== false;
 })->subscribe(function (Repository $repository) use ($argv, $loop) {
     $stream = new ThroughStream();
 
@@ -66,7 +65,7 @@
             return $repository->ref($ref, $commit);
         });
     })->done(function () use ($repository) {
-        echo $repository->htmlUrl(), PHP_EOL;
+        echo $repository->htmlUrl(), \PHP_EOL;
     }, 'display_throwable');
 }, 'display_throwable');
 

diff --git a/examples/user-repositories-commit-combined-status-async.php b/examples/user-repositories-commit-combined-status-async.php
@@ -18,11 +18,16 @@
     return $user->repository($argv[2] ?? 'github');
 })->then(function (Repository $repository) {
     resource_pretty_print($repository, 1, true);
-    $repository->commits()->take(1)->subscribe(function (Repository\Commit $commit) {
-        $commit->status()->done(function (CombinedStatusInterface $combinedStatus) {
-            resource_pretty_print($combinedStatus, 3, true);
-        }, 'display_throwable');
-    }, 'display_throwable');
+
+    return $repository->commits()->take(1)->toPromise();
+})->then(function (Repository\Commit $commit) {
+    return $commit->status();
+})->then(function (CombinedStatusInterface $combinedStatus) {
+    resource_pretty_print($combinedStatus, 3, true);
+
+    return $combinedStatus->refresh();
+})->then(function (CombinedStatusInterface $combinedStatus) {
+    resource_pretty_print($combinedStatus, 3, true);
 })->done(null, 'display_throwable');
 
 $loop->run();

diff --git a/src/Resource/Async/Repository/Commit/CombinedStatus.php b/src/Resource/Async/Repository/Commit/CombinedStatus.php
@@ -2,12 +2,16 @@
 
 namespace ApiClients\Client\Github\Resource\Async\Repository\Commit;
 
+use ApiClients\Client\Github\CommandBus\Command\RefreshCommand;
 use ApiClients\Client\Github\Resource\Repository\Commit\CombinedStatus as BaseCombinedStatus;
+use React\Promise\PromiseInterface;
 
 class CombinedStatus extends BaseCombinedStatus
 {
-    public function refresh(): CombinedStatus
+    public function refresh(): PromiseInterface
     {
-        throw new \Exception('TODO: create refresh method!');
+        return $this->handleCommand(
+            new RefreshCommand($this)
+        );
     }
 }
diff --git a/src/Resource/Async/Repository/Commit/Status.php b/src/Resource/Async/Repository/Commit/Status.php
@@ -2,12 +2,16 @@
 
 namespace ApiClients\Client\Github\Resource\Async\Repository\Commit;
 
+use ApiClients\Client\Github\CommandBus\Command\RefreshCommand;
 use ApiClients\Client\Github\Resource\Repository\Commit\Status as BaseStatus;
+use React\Promise\PromiseInterface;
 
 class Status extends BaseStatus
 {
-    public function refresh(): Status
+    public function refresh(): PromiseInterface
     {
-        throw new \Exception('TODO: create refresh method!');
+        return $this->handleCommand(
+            new RefreshCommand($this)
+        );
     }
 }
diff --git a/src/Resource/Repository/Commit/CombinedStatus.php b/src/Resource/Repository/Commit/CombinedStatus.php
@@ -28,6 +28,11 @@ abstract class CombinedStatus extends AbstractResource implements CombinedStatus
      */
     protected $sha;
 
+    /**
+     * @var string
+     */
+    protected $url;
+
     /**
      * @var int
      */
@@ -59,6 +64,14 @@ public function sha(): string
         return $this->sha;
     }
 
+    /**
+     * @return string
+     */
+    public function url(): string
+    {
+        return $this->url;
+    }
+
     /**
      * @return int
      */

diff --git a/src/Resource/Repository/Commit/CombinedStatusInterface.php b/src/Resource/Repository/Commit/CombinedStatusInterface.php
@@ -18,6 +18,11 @@ public function state(): string;
      */
     public function sha(): string;
 
+    /**
+     * @return string
+     */
+    public function url(): string;
+
     /**
      * @return int
      */

diff --git a/src/Resource/Repository/Commit/EmptyCombinedStatus.php b/src/Resource/Repository/Commit/EmptyCombinedStatus.php
@@ -22,6 +22,14 @@ public function sha(): string
         return null;
     }
 
+    /**
+     * @return string
+     */
+    public function url(): string
+    {
+        return null;
+    }
+
     /**
      * @return int
      */

diff --git a/tests/CommandBus/Handler/EmojisHandlerTest.php b/tests/CommandBus/Handler/EmojisHandlerTest.php
@@ -29,14 +29,14 @@ public function testCommand()
         $hydrator = $this->prophesize(Hydrator::class);
         $hydrator->hydrate(
             EmojiInterface::HYDRATE_CLASS,
-                [
+            [
                     'name' => 'foo',
                     'image' => 'bar',
                 ]
         )->shouldBeCalled()->willReturn($this->prophesize(EmojiInterface::class)->reveal());
         $hydrator->hydrate(
             EmojiInterface::HYDRATE_CLASS,
-                [
+            [
                     'name' => 'bar',
                     'image' => 'foo',
                 ]