Browse Source

[CI] Can we get some faster phpunit execution? (#27113)

* Don't backup globals and static attributes

* Unset global variable to remove impact on followup tests

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
pull/3955/head
Thomas Müller 9 years ago
committed by Morris Jobke
parent
commit
5bfce597a9
No known key found for this signature in database GPG Key ID: 9CE5ED29E7FCD38A
  1. 3
      apps/dav/tests/unit/Connector/Sabre/FileTest.php
  2. 2
      apps/dav/tests/unit/Connector/Sabre/RequestTest/RequestTest.php
  3. 3
      apps/files_sharing/tests/Controller/ShareAPIControllerTest.php
  4. 13
      tests/lib/Repair/RepairCollationTest.php
  5. 1
      tests/phpunit-autotest.xml

3
apps/dav/tests/unit/Connector/Sabre/FileTest.php

@ -48,6 +48,9 @@ class FileTest extends \Test\TestCase {
public function setUp() {
parent::setUp();
unset($_SERVER['HTTP_OC_CHUNKED']);
unset($_SERVER['CONTENT_LENGTH']);
unset($_SERVER['REQUEST_METHOD']);
\OC_Hook::clear();

2
apps/dav/tests/unit/Connector/Sabre/RequestTest/RequestTest.php

@ -57,6 +57,8 @@ abstract class RequestTest extends TestCase {
protected function setUp() {
parent::setUp();
unset($_SERVER['HTTP_OC_CHUNKED']);
$this->serverFactory = new ServerFactory(
\OC::$server->getConfig(),
\OC::$server->getLogger(),

3
apps/files_sharing/tests/Controller/ShareAPIControllerTest.php

@ -38,6 +38,7 @@ use OCP\Files\IRootFolder;
use OCP\Lock\LockedException;
use OCP\Share\IManager;
use OCP\Share;
use Test\TestCase;
/**
* Class ShareAPIControllerTest
@ -45,7 +46,7 @@ use OCP\Share;
* @package OCA\Files_Sharing\Tests\Controller
* @group DB
*/
class ShareAPIControllerTest extends \Test\TestCase {
class ShareAPIControllerTest extends TestCase {
/** @var string */
private $appName = 'files_sharing';

13
tests/lib/Repair/RepairCollationTest.php

@ -8,10 +8,15 @@
namespace Test\Repair;
use Doctrine\DBAL\Connection;
use Doctrine\DBAL\Platforms\MySqlPlatform;
use OC\Repair\Collation;
use OCP\IDBConnection;
use OCP\ILogger;
use OCP\Migration\IOutput;
use Test\TestCase;
class TestCollationRepair extends \OC\Repair\Collation {
class TestCollationRepair extends Collation {
/**
* @param \Doctrine\DBAL\Connection $connection
* @return string[]
@ -28,7 +33,7 @@ class TestCollationRepair extends \OC\Repair\Collation {
*
* @see \OC\Repair\RepairMimeTypes
*/
class RepairCollationTest extends \Test\TestCase {
class RepairCollationTest extends TestCase {
/**
* @var TestCollationRepair
@ -36,7 +41,7 @@ class RepairCollationTest extends \Test\TestCase {
private $repair;
/**
* @var \Doctrine\DBAL\Connection
* @var Connection|IDBConnection
*/
private $connection;
@ -59,7 +64,7 @@ class RepairCollationTest extends \Test\TestCase {
$this->connection = \OC::$server->getDatabaseConnection();
$this->logger = $this->createMock(ILogger::class);
$this->config = \OC::$server->getConfig();
if (!$this->connection->getDatabasePlatform() instanceof \Doctrine\DBAL\Platforms\MySqlPlatform) {
if (!$this->connection->getDatabasePlatform() instanceof MySqlPlatform) {
$this->markTestSkipped("Test only relevant on MySql");
}

1
tests/phpunit-autotest.xml

@ -2,6 +2,7 @@
<phpunit bootstrap="bootstrap.php"
strict="true"
verbose="true"
backupGlobals="false"
timeoutForSmallTests="900"
timeoutForMediumTests="900"
timeoutForLargeTests="900"

Loading…
Cancel
Save