Cerbero\OctaneTestbench\Concerns\InteractsWithCache

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	assertOctaneCacheMissing()	0	7	1
A	assertOctaneCacheHas()	0	12	2

<?php

namespace Cerbero\OctaneTestbench\Concerns;

/**
 * The trait to interact with the Octane cache.
 *
 */
trait InteractsWithCache
{
    /**
     * Assert that the Laravel Octane cache has the given key set
     *
     * @param string $key
     * @param mixed|null $value
     * @return static
     */
    public function assertOctaneCacheHas(string $key, mixed $value = null): static
    {
        $cache = $this->app->cache->store('octane');

        if ($value === null) {
            $this->assertTrue($cache->has($key), "The Octane cache doesn't have the [$key] key set");

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

        } else {
            $actual = $cache->get($key);
            $this->assertSame($value, $actual, "Expected cached value [$value], got [$actual] instead");

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

        }

        return $this;
    }

    /**
     * Assert that the Laravel Octane cache doesn't have the given key set
     *
     * @param string $key
     * @return static
     */
    public function assertOctaneCacheMissing(string $key): static
    {
        $cache = $this->app->cache->store('octane');

        $this->assertFalse($cache->has($key), "The Octane cache has the [$key] key set");

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

        return $this;
    }
}