search

elliotchance / concise

✅ Concise is test framework for using plain English and minimal code, built on PHPUnit.
MIT License
45 stars 3 forks source link

Multiple with() can cause bad index #225

Closed elliotchance closed 9 years ago

elliotchance commented 9 years ago 
    public function testRedirectToWillOnlyHappenIfOnProduction()
    {
        /** @var \LoginController $controller */
        $controller = $this->niceMock('LoginController', ["login"])
            ->expect('redirect')->with('https://mydomain')->never()
                                ->with('foo')
            ->stub(['getDomain' => 'mydomain.local', 'getAccessCodeUrl' => 'foo'])
            ->get();

        $controller->actionGo();
    }
Undefined index: 94fca0fba018df9661b4b457a2a54cfa
coveralls commented 9 years ago

Coverage Status

Coverage increased (+0.06%) when pulling 521dd631479e4cf783e98b25861cdbd4d33c5edd on 1.4.4/225-multiple-with-can-cause-bad-index into 79c157ea0d3a1af699dc1477d78b743fd354a715 on release/1.4.4.

coveralls commented 9 years ago

Coverage Status

Coverage increased (+0.06%) when pulling 3a69f000922bb18e327efd95df28c5257aa089ed on 1.4.4/225-multiple-with-can-cause-bad-index into 79c157ea0d3a1af699dc1477d78b743fd354a715 on release/1.4.4.

coveralls commented 9 years ago

Coverage Status

Coverage increased (+0.06%) when pulling 3a69f000922bb18e327efd95df28c5257aa089ed on 1.4.4/225-multiple-with-can-cause-bad-index into 79c157ea0d3a1af699dc1477d78b743fd354a715 on release/1.4.4.

scrutinizer-notifier commented 9 years ago

The inspection completed: No new issues