Skip to content

Commit

Permalink
Removing PHPUnit::execute Coverage Tags (#1258)
Browse files Browse the repository at this point in the history
* Removing PHPUnit::execute Coverage Tags

* PSR-4 Change

Fixing Accidental Change To Code Standards
  • Loading branch information
Fenikkusu authored and Dan Cryer committed Dec 14, 2016
1 parent 98f8465 commit 99f4ef0
Show file tree
Hide file tree
Showing 7 changed files with 2 additions and 128 deletions.
6 changes: 0 additions & 6 deletions Tests/PHPCI/Helper/MailerFactoryTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,6 @@ public function setUp()
{
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestGetMailConfig()
{
$config = array(
Expand All @@ -46,9 +43,6 @@ public function testExecute_TestGetMailConfig()
$this->assertEquals($config['default_mailto_address'], $factory->getMailConfig('default_mailto_address'));
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestMailer()
{
$config = array(
Expand Down
12 changes: 0 additions & 12 deletions Tests/PHPCI/Model/BuildTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,6 @@ public function setUp()
{
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestIsAValidModel()
{
$build = new Build();
Expand All @@ -35,9 +32,6 @@ public function testExecute_TestIsAValidModel()
$this->assertTrue($build instanceof Model\Base\BuildBase);
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestBaseBuildDefaults()
{
$build = new Build();
Expand All @@ -46,9 +40,6 @@ public function testExecute_TestBaseBuildDefaults()
$this->assertEquals(null, $build->getFileLinkTemplate());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestIsSuccessful()
{
$build = new Build();
Expand All @@ -65,9 +56,6 @@ public function testExecute_TestIsSuccessful()
$this->assertTrue($build->isSuccessful());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestBuildExtra()
{
$info = array(
Expand Down
23 changes: 1 addition & 22 deletions Tests/PHPCI/Model/ProjectTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,6 @@ public function setUp()
{
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestIsAValidModel()
{
$project = new Project();
Expand All @@ -35,9 +32,6 @@ public function testExecute_TestIsAValidModel()
$this->assertTrue($project instanceof Model\Base\ProjectBase);
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestGitDefaultBranch()
{
$project = new Project();
Expand All @@ -46,9 +40,6 @@ public function testExecute_TestGitDefaultBranch()
$this->assertEquals('master', $project->getBranch());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestGithubDefaultBranch()
{
$project = new Project();
Expand All @@ -57,9 +48,6 @@ public function testExecute_TestGithubDefaultBranch()
$this->assertEquals('master', $project->getBranch());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestGitlabDefaultBranch()
{
$project = new Project();
Expand All @@ -68,20 +56,14 @@ public function testExecute_TestGitlabDefaultBranch()
$this->assertEquals('master', $project->getBranch());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestBitbucketDefaultBranch()
public function testExecute_TestBitbucketDefaultBranch()
{
$project = new Project();
$project->setType('bitbucket');

$this->assertEquals('master', $project->getBranch());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestMercurialDefaultBranch()
{
$project = new Project();
Expand All @@ -90,9 +72,6 @@ public function testExecute_TestMercurialDefaultBranch()
$this->assertEquals('default', $project->getBranch());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_TestProjectAccessInformation()
{
$info = array(
Expand Down
39 changes: 0 additions & 39 deletions Tests/PHPCI/Plugin/EmailTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -164,9 +164,6 @@ protected function loadEmailPluginWithOptions($arrOptions = array(), $buildStatu
}));
}

/**
* @covers PHPUnit::execute
*/
public function testReturnsFalseWithoutArgs()
{
$this->loadEmailPluginWithOptions();
Expand All @@ -179,9 +176,6 @@ public function testReturnsFalseWithoutArgs()
$this->assertEquals($expectedReturn, $returnValue);
}

/**
* @covers PHPUnit::execute
*/
public function testBuildsBasicEmails()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -196,9 +190,6 @@ public function testBuildsBasicEmails()
$this->assertContains('test-receiver@example.com', $this->message['to']);
}

/**
* @covers PHPUnit::execute
*/
public function testBuildsDefaultEmails()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -213,9 +204,6 @@ public function testBuildsDefaultEmails()
$this->assertContains('default-mailto-address@example.com', $this->message['to']);
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_UniqueRecipientsFromWithCommitter()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -233,9 +221,6 @@ public function testExecute_UniqueRecipientsFromWithCommitter()
$this->assertContains('test-receiver2@example.com', $this->message['to']);
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_UniqueRecipientsWithCommitter()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -252,9 +237,6 @@ public function testExecute_UniqueRecipientsWithCommitter()
$this->assertContains('committer@test.com', $this->message['to']);
}

/**
* @covers PHPUnit::execute
*/
public function testCcDefaultEmails()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -281,9 +263,6 @@ public function testCcDefaultEmails()
);
}

/**
* @covers PHPUnit::execute
*/
public function testBuildsCommitterEmails()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -298,9 +277,6 @@ public function testBuildsCommitterEmails()
$this->assertContains('committer-email@example.com', $this->message['to']);
}

/**
* @covers PHPUnit::execute
*/
public function testMailSuccessfulBuildHaveProjectName()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -316,9 +292,6 @@ public function testMailSuccessfulBuildHaveProjectName()
$this->assertContains('Test-Project', $this->message['body']);
}

/**
* @covers PHPUnit::execute
*/
public function testMailFailingBuildHaveProjectName()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -334,9 +307,6 @@ public function testMailFailingBuildHaveProjectName()
$this->assertContains('Test-Project', $this->message['body']);
}

/**
* @covers PHPUnit::execute
*/
public function testMailSuccessfulBuildHaveStatus()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -352,9 +322,6 @@ public function testMailSuccessfulBuildHaveStatus()
$this->assertContains('successful', $this->message['body']);
}

/**
* @covers PHPUnit::execute
*/
public function testMailFailingBuildHaveStatus()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -370,9 +337,6 @@ public function testMailFailingBuildHaveStatus()
$this->assertContains('failed', $this->message['body']);
}

/**
* @covers PHPUnit::execute
*/
public function testMailDeliverySuccess()
{
$this->loadEmailPluginWithOptions(
Expand All @@ -388,9 +352,6 @@ public function testMailDeliverySuccess()
$this->assertEquals(true, $returnValue);
}

/**
* @covers PHPUnit::execute
*/
public function testMailDeliveryFail()
{
$this->loadEmailPluginWithOptions(
Expand Down
15 changes: 0 additions & 15 deletions Tests/PHPCI/Service/BuildServiceTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,6 @@ public function setUp()
$this->testedService = new BuildService($this->mockBuildStore);
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_CreateBasicBuild()
{
$project = new Project();
Expand All @@ -65,9 +62,6 @@ public function testExecute_CreateBasicBuild()
$this->assertEquals('Manual', $returnValue->getCommitId());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_CreateBuildWithOptions()
{
$project = new Project();
Expand All @@ -82,9 +76,6 @@ public function testExecute_CreateBuildWithOptions()
$this->assertEquals('test@example.com', $returnValue->getCommitterEmail());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_CreateBuildWithExtra()
{
$project = new Project();
Expand All @@ -96,9 +87,6 @@ public function testExecute_CreateBuildWithExtra()
$this->assertEquals(1001, $returnValue->getExtra('item1'));
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_CreateDuplicateBuild()
{
$build = new Build();
Expand Down Expand Up @@ -131,9 +119,6 @@ public function testExecute_CreateDuplicateBuild()
$this->assertEquals($build->getExtra('item1'), $returnValue->getExtra('item1'));
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_DeleteBuild()
{
$store = $this->getMock('PHPCI\Store\BuildStore');
Expand Down
16 changes: 0 additions & 16 deletions Tests/PHPCI/Service/ProjectServiceTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,6 @@ public function setUp()
$this->testedService = new ProjectService($this->mockProjectStore);
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_CreateBasicProject()
{
$returnValue = $this->testedService->createProject('Test Project', 'github', 'block8/phpci');
Expand All @@ -54,9 +51,6 @@ public function testExecute_CreateBasicProject()
$this->assertEquals('master', $returnValue->getBranch());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_CreateProjectWithOptions()
{
$options = array(
Expand All @@ -78,7 +72,6 @@ public function testExecute_CreateProjectWithOptions()

/**
* @link https://github.com/Block8/PHPCI/issues/484
* @covers PHPUnit::execute
*/
public function testExecute_CreateGitlabProjectWithoutPort()
{
Expand All @@ -90,9 +83,6 @@ public function testExecute_CreateGitlabProjectWithoutPort()
$this->assertEquals('block8/phpci', $returnValue->getReference());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_UpdateExistingProject()
{
$project = new Project();
Expand All @@ -107,9 +97,6 @@ public function testExecute_UpdateExistingProject()
$this->assertEquals('bitbucket', $returnValue->getType());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_EmptyPublicStatus()
{
$project = new Project();
Expand All @@ -126,9 +113,6 @@ public function testExecute_EmptyPublicStatus()
$this->assertEquals(0, $returnValue->getAllowPublicStatus());
}

/**
* @covers PHPUnit::execute
*/
public function testExecute_DeleteProject()
{
$store = $this->getMock('PHPCI\Store\ProjectStore');
Expand Down

0 comments on commit 99f4ef0

Please sign in to comment.