johnnymast / redbox-testsuite

spec/ContainerSpec.php

<?php

/**
 * ContainerSpec.php
 *
 * This file tests the behavior of the Container class.
 *
 * PHP version 7.4
 *
 * @category Core
 * @package  RedboxTestSuite
 * @author   Johnny Mast <mastjohnny@gmail.com>
 * @license  https://opensource.org/licenses/MIT MIT
 * @link     https://github.com/johnnymast/redbox-testsuite
 * @since    1.0
 */

namespace spec\Redbox\Testsuite;

use PhpSpec\ObjectBehavior;
use Redbox\Testsuite\Container;

/**
 * Class ContainerSpec
 *
 * @package spec\Redbox\Testsuite
 */
class ContainerSpec extends ObjectBehavior
{
    /**
     * Check to see if the class can be initialized.
     *
     * @return void
     */
    function it_is_initializable()

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->shouldHaveType(Container::class);
    }
    
    /**
     * Check if there is a function called has.
     *
     * @return void
     */
    function it_should_have_a_workable_has_function()
    {
        $this->set('__HELLO__', 'world');

The method set() does not exist on spec\Redbox\Testsuite\ContainerSpec. Since you implemented __call, consider adding a @method annotation.

        $this->has('__HELLO__')->shouldReturn(true);

The method has() does not exist on spec\Redbox\Testsuite\ContainerSpec. Since you implemented __call, consider adding a @method annotation.

    }
    
    /**
     * Check if there is a function called get.
     *
     * @return void
     */
    function it_should_workable_get_and_set_function()
    {
        $this->set('__GOODBYE__', 'world');
        $this->get('__GOODBYE__');

The method get() does not exist on spec\Redbox\Testsuite\ContainerSpec. Since you implemented __call, consider adding a @method annotation.

    }
    
    /**
     * Test that get should return false if a key does not exist.
     *
     * @return void
     */
    function it_should_return_false_on_get_if_a_given_key_does_not_exist()
    {
        $this->get('__NON_EXISTING_KEY')->shouldReturn(false);
    }
    
    /**
     * Check if there is a workable forget function.
     *
     * @return void
     */
    function it_should_have_a_workable_forget_function()
    {
        $this->set('__BLEEP__', 'world');
        $this->has('__BLEEP__')->shouldReturn(true);
        
        $this->forget('__BLEEP__');

The method forget() does not exist on spec\Redbox\Testsuite\ContainerSpec. Since you implemented __call, consider adding a @method annotation.

        
        $this->has('__BLEEP__')->shouldReturn(false);
    }
}