WidgetTest

Complexity

Total Complexity

5

Size/Duplication

Total Lines	58
Duplicated Lines	70.69 %

Coupling/Cohesion

Components	0
Dependencies	6

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	test_presenter_method_is_called_with_data_from_controller()	0	10	1
A	test_default_view_name_is_figured_out_correctly()	11	11	1
A	test_context_as_is_passes_to_view_correctly()	11	11	1
A	test_controller_method_is_called_on_some_other_class()	10	10	1
A	test_data_is_passed_to_data_method_from_view()	9	9	1

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 5 and the first side effect is on line 3.

require_once 'test_stubs.php';

class WidgetTest extends TestCase

There is at least one abstract method in this class. Maybe declare it as abstract, or implement the remaining methods: artisan, be, call, seed

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public function test_presenter_method_is_called_with_data_from_controller()
    {
        View::shouldReceive('exists')->once()->andReturn(true);
        View::shouldReceive('make')->once()->with('hello', ['data' => 'barfoo'], [])->andReturn(app('view'));
        View::shouldReceive('render')->once()->andReturn('<br>');

        //act
        $widget = new Widget5();
        render_widget($widget);
    }

    public function test_default_view_name_is_figured_out_correctly()

This method seems to be duplicated in your project.

    {
        View::shouldReceive('exists')->once()->andReturn(true);
        View::shouldReceive('make')->once()->with('Widgets::Foo.Widget1View', ['data' => 'foo'], [])->andReturn(app('view'));
        View::shouldReceive('render')->once()->andReturn('<br>');
        \App::shouldReceive('call')->once()->andReturn('foo');

        //act
        $widget = new \App\Widgets\Foo\Widget1();
        render_widget($widget);
    }

    public function test_context_as_is_passes_to_view_correctly()

This method seems to be duplicated in your project.

    {
        View::shouldReceive('exists')->once()->andReturn(true);
        View::shouldReceive('make')->once()->with('hello', ['myData' => 'foo'], [])->andReturn(app('view'));
        View::shouldReceive('render')->once()->andReturn('<br>');
        \App::shouldReceive('call')->once()->andReturn('foo');

        //act
        $widget = new Widget3();
        render_widget($widget);
    }

    public function test_controller_method_is_called_on_some_other_class()

This method seems to be duplicated in your project.

    {
        View::shouldReceive('exists')->once()->andReturn(true);
        View::shouldReceive('make')->once()->with('hello', ['data' => 'aaabbb'], [])->andReturn(app('view'));
        View::shouldReceive('render')->once()->andReturn('<br>');

        //act
        $widget = new Widget4();
        render_widget($widget, ['arg1' => 'aaa', 'arg2' => 'bbb']);
    }

    public function test_data_is_passed_to_data_method_from_view()

This method seems to be duplicated in your project.

    {
        View::shouldReceive('exists')->once()->andReturn(true);
        View::shouldReceive('make')->once()->with('hello', ['data' => '222111'], [])->andReturn(app('view'));
        View::shouldReceive('render')->once()->andReturn('<br>');
        //act
        $widget = new Widget6();
        render_widget($widget, ['foo' => '111', 'bar' => '222']);
    }
}