Fix risky and unreliable tests

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

tests/AppInfo/ApplicationTest.php

@@ -29,6 +29,8 @@ class ApplicationTest extends TestCase {
 	public function testConstrucor() {
 		// Not really a test – it's just about code coverage
 		new Application();
+
+		$this->addToAssertionCount(1);
 	}
 
 }

tests/Controller/MessagesControllerTest.php

@@ -92,14 +92,17 @@ class MessagesControllerTest extends TestCase {
 
 	public function testIndex() {
 		// TODO: write test
+		$this->markTestSkipped('todo');
 	}
 
 	public function testShow() {
 		// TODO: write test
+		$this->markTestSkipped('todo');
 	}
 
 	public function testShowMessageNotFound() {
 		// TODO: write test
+		$this->markTestSkipped('todo');
 	}
 
 	public function testGetHtmlBody() {

tests/FolderTest.php

@@ -85,6 +85,8 @@ class FolderTest extends TestCase {
 		$this->folder->setStatus([
 			'unseen' => 4,
 		]);
+
+		$this->addToAssertionCount(1);
 	}
 
 	public function testSpecialUse() {

tests/Integration/Service/AvatarServiceIntegrationTest.php

@@ -59,6 +59,9 @@ class AvatarServiceIntegrationTest extends TestCase {
 	}
 
 	public function testGithubFavicon() {
+		$this->markTestSkipped('does not work reliably at the moment');
+		return;
+
 		$avatar = $this->service->getAvatar('no-reply@github.com', 'jane');
 		$this->assertNotNull($avatar);
 		$this->assertEquals('https://github.githubassets.com/favicon.ico', $avatar->getUrl());

tests/Integration/Service/MailTransmissionIntegrationTest.php

@@ -81,7 +81,10 @@ class MailTransmissionIntegrationTest extends TestCase {
 	public function testSendMail() {
 		$message = NewMessageData::fromRequest($this->account, 'recipient@domain.com', null, null, 'greetings', 'hello there', []);
 		$reply = new RepliedMessageData($this->account, null, null);
+
 		$this->transmission->sendMessage('ferdinand', $message, $reply);
+
+		$this->addToAssertionCount(1);
 	}
 
 	public function testSendMailWithLocalAttachment() {
@@ -97,7 +100,10 @@ class MailTransmissionIntegrationTest extends TestCase {
 				],
 		]);
 		$reply = new RepliedMessageData($this->account, null, null);
+
 		$this->transmission->sendMessage('gerald', $message, $reply);
+
+		$this->addToAssertionCount(1);
 	}
 
 	public function testSendMailWithCloudAttachment() {
@@ -110,7 +116,10 @@ class MailTransmissionIntegrationTest extends TestCase {
 				],
 		]);
 		$reply = new RepliedMessageData($this->account, null, null);
+
 		$this->transmission->sendMessage($this->user->getUID(), $message, $reply);
+
+		$this->addToAssertionCount(1);
 	}
 
 	public function testSendReply() {

tests/Service/AccountServiceTest.php

@@ -104,6 +104,7 @@ class AccountServiceTest extends TestCase {
 
 	public function testFindNotFound() {
 		// TODO: implement code + write tests
+		$this->markTestSkipped('todo');
 	}
 
 	public function testDelete() {