Butler\Graphql\Concerns\AssertsPromises::assertPromiseFulfills()

Complexity

Conditions	5
Paths	12

Size

Total Lines	29
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	5

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function assertPromiseFulfills($promise, $expectedValue = null): void
    {
        $this->addToAssertionCount(1);

It seems like addToAssertionCount() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if (is_array($promise)) {
            $promise = all($promise);
        }

        try {
            $result = null;
            Loop::run(function () use (&$result, $promise) {
                $result = yield call(function () use ($promise) {
                    return $promise;
                });
            });
        } catch (Exception $e) {
            $this->fail('Failed asserting that promise fulfills. Promise was rejected: ' . $e->getMessage());

It seems like fail() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        if ($expectedValue instanceof Closure) {
            $this->assertTrue($expectedValue($result));

It seems like assertTrue() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return;
        }

        if (! is_null($expectedValue)) {
            $this->assertSame(

It seems like assertSame() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                $expectedValue,
                $result,
                'Failed asserting that promise fulfills with a specified value.'
            );