Merge pull request #311 from creative-commoners/pulls/2.0/fix-dataprovider

FIX Update visibility of test dataprovider and update doc block
This commit is contained in:
Will Rossiter 2017-09-13 13:28:18 +12:00 committed by GitHub
commit ed8efc2144
1 changed files with 5 additions and 4 deletions

View File

@ -399,7 +399,7 @@ class SiteTreeSubsitesTest extends BaseSubsiteTest
);
}
protected function provideAlternateAbsoluteLink()
public function provideAlternateAbsoluteLink()
{
return [
['home', null, 'http://localhost/'],
@ -415,13 +415,14 @@ class SiteTreeSubsitesTest extends BaseSubsiteTest
/**
* @dataProvider provideAlternateAbsoluteLink
* @param $fixture
* @param $expected
* @param name $pageFixtureName
* @param string|null $action
* @param string $expectedAbsoluteLink
*/
public function testAlternateAbsoluteLink($pageFixtureName, $action, $expectedAbsoluteLink)
{
/** @var Page $page */
$page = $this->objFromFixture('Page', $pageFixtureName);
$page = $this->objFromFixture(Page::class, $pageFixtureName);
$result = $page->AbsoluteLink($action);