Removed hard dependencies on doctrine odm to run the tests

Toflar · Toflar · commit e7f7813b38d3 · 2019-04-08T11:09:58.000+02:00
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -87,6 +87,10 @@ reusable-steps:
     run:
       name: Update Composer
       command: sudo composer self-update
+  - &require-doctrine-odm
+    run:
+      name: Require doctrine/mongodb-odm an doctrine/mongodb-odm-bundle
+      command: composer require --no-update doctrine/mongodb-odm:^2.0.0@beta doctrine/mongodb-odm-bundle:^4.0.0@beta
   - &update-project-dependencies
     run:
       name: Update project dependencies
@@ -134,6 +138,7 @@ jobs:
       - *disable-php-memory-limit
       - *install-php-extensions
       - *update-composer
+      - *require-doctrine-odm
       - *update-project-dependencies
       - *save-composer-cache-by-revision
       - *save-composer-cache-by-branch
diff --git a/.travis.yml b/.travis.yml
@@ -8,10 +8,6 @@ cache:
 jobs:
   include:
     - php: '7.1'
-      env: NO_UNIT_TESTS=true
-      before_install:
-        - composer remove --dev ext-mongodb doctrine/mongodb-odm doctrine/mongodb-odm-bundle
-        - sed -i '33,39d' tests/Fixtures/app/config/config_common.yml
     - php: '7.2'
     - php: '7.3'
     - php: '7.3'
@@ -46,12 +42,15 @@ before_install:
   - if [[ $APP_ENV = 'elasticsearch' ]]; then
       curl -O https://artifacts.elastic.co/downloads/elasticsearch/elasticsearch-6.5.0.deb && sudo dpkg -i --force-confnew elasticsearch-6.5.0.deb && sudo service elasticsearch restart;
     fi
-  - phpenv config-rm xdebug.ini || echo "xdebug not available"
   - echo "memory_limit=-1" >> ~/.phpenv/versions/$(phpenv version-name)/etc/conf.d/travis.ini
-  - echo "extension=mongodb.so" >> ~/.phpenv/versions/$(phpenv version-name)/etc/php.ini
+  - if [[ $APP_ENV = 'mongodb' ]]; then
+      echo "extension=mongodb.so" >> ~/.phpenv/versions/$(phpenv version-name)/etc/php.ini;
+    fi
+  - phpenv config-rm xdebug.ini || echo "xdebug not available"
   - export PATH="$PATH:$HOME/.composer/vendor/bin"
 
 install:
+  - if [[ $APP_ENV = 'mongodb' ]]; then composer require --no-update doctrine/mongodb-odm:^2.0.0@beta doctrine/mongodb-odm-bundle:^4.0.0@beta; fi
   - if [[ $deps = 'low' ]]; then
       composer update --prefer-dist --no-progress --no-suggest --prefer-stable --prefer-lowest --ansi;
     else
@@ -60,11 +59,10 @@ install:
 
 script:
   - tests/Fixtures/app/console cache:clear
-  - if [[ $NO_UNIT_TESTS != true ]]; then
-      vendor/bin/phpunit;
-    fi
   - if [[ $APP_ENV = 'mongodb' ]]; then
       vendor/bin/phpunit -c phpunit.mongo.xml;
+    else
+      vendor/bin/phpunit;
     fi
   - tests/Fixtures/app/console cache:clear
   - if [[ $APP_ENV = 'postgres' ]]; then
diff --git a/composer.json b/composer.json
@@ -35,8 +35,6 @@
         "doctrine/annotations": "^1.2",
         "doctrine/doctrine-bundle": "^1.8",
         "doctrine/doctrine-cache-bundle": "^1.3.5",
-        "doctrine/mongodb-odm": "^2.0.0@beta",
-        "doctrine/mongodb-odm-bundle": "^4.0.0@beta",
         "doctrine/orm": "^2.6.3",
         "elasticsearch/elasticsearch": "^6.0",
         "friendsofsymfony/user-bundle": "^2.2@dev",
diff --git a/phpunit.mongo.xml b/phpunit.mongo.xml
@@ -13,7 +13,7 @@
         <env name="SYMFONY_DEPRECATIONS_HELPER" value="weak_vendors" />
         <server name="KERNEL_DIR" value="tests/Fixtures/app/" />
         <server name="KERNEL_CLASS" value="AppKernel" />
-        <server name="APP_ENV" value="test_mongodb" />
+        <server name="APP_ENV" value="mongodb" />
         <server name="LEGACY" value="0" />
     </php>
 
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -37,4 +37,10 @@
     <listeners>
         <listener class="Symfony\Bridge\PhpUnit\SymfonyTestsListener" />
     </listeners>
+
+    <groups>
+        <exclude>
+            <group>mongodb</group>
+        </exclude>
+    </groups>
 </phpunit>
diff --git a/tests/Bridge/Doctrine/Common/Util/IdentifierManagerTraitTest.php b/tests/Bridge/Doctrine/Common/Util/IdentifierManagerTraitTest.php
@@ -70,6 +70,7 @@ public function testSingleIdentifier()
 
     /**
      * @group legacy
+     * @group mongodb
      */
     public function testSingleDocumentIdentifier()
     {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/CollectionDataProviderTest.php b/tests/Bridge/Doctrine/MongoDbOdm/CollectionDataProviderTest.php
@@ -28,6 +28,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class CollectionDataProviderTest extends TestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Extension/FilterExtensionTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Extension/FilterExtensionTest.php
@@ -25,6 +25,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class FilterExtensionTest extends TestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Extension/OrderExtensionTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Extension/OrderExtensionTest.php
@@ -26,6 +26,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class OrderExtensionTest extends TestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Extension/PaginationExtensionTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Extension/PaginationExtensionTest.php
@@ -35,6 +35,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class PaginationExtensionTest extends TestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Filter/BooleanFilterTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Filter/BooleanFilterTest.php
@@ -19,6 +19,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class BooleanFilterTest extends DoctrineMongoDbOdmFilterTestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Filter/DateFilterTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Filter/DateFilterTest.php
@@ -20,6 +20,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class DateFilterTest extends DoctrineMongoDbOdmFilterTestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Filter/ExistsFilterTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Filter/ExistsFilterTest.php
@@ -19,6 +19,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class ExistsFilterTest extends DoctrineMongoDbOdmFilterTestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Filter/NumericFilterTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Filter/NumericFilterTest.php
@@ -19,6 +19,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class NumericFilterTest extends DoctrineMongoDbOdmFilterTestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Filter/OrderFilterTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Filter/OrderFilterTest.php
@@ -20,6 +20,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class OrderFilterTest extends DoctrineMongoDbOdmFilterTestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Filter/RangeFilterTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Filter/RangeFilterTest.php
@@ -19,6 +19,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class RangeFilterTest extends DoctrineMongoDbOdmFilterTestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Filter/SearchFilterTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Filter/SearchFilterTest.php
@@ -25,6 +25,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class SearchFilterTest extends DoctrineMongoDbOdmFilterTestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/ItemDataProviderTest.php b/tests/Bridge/Doctrine/MongoDbOdm/ItemDataProviderTest.php
@@ -37,6 +37,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class ItemDataProviderTest extends TestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/Metadata/Property/DoctrineMongoDbOdmPropertyMetadataFactoryTest.php b/tests/Bridge/Doctrine/MongoDbOdm/Metadata/Property/DoctrineMongoDbOdmPropertyMetadataFactoryTest.php
@@ -24,6 +24,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class DoctrineMongoDbOdmPropertyMetadataFactoryTest extends TestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/PaginatorTest.php b/tests/Bridge/Doctrine/MongoDbOdm/PaginatorTest.php
@@ -20,6 +20,9 @@
 use Doctrine\ODM\MongoDB\UnitOfWork;
 use PHPUnit\Framework\TestCase;
 
+/**
+ * @group mongodb
+ */
 class PaginatorTest extends TestCase
 {
     /**
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/PropertyInfo/DoctrineExtractorTest.php b/tests/Bridge/Doctrine/MongoDbOdm/PropertyInfo/DoctrineExtractorTest.php
@@ -29,6 +29,7 @@
 /**
  * @author Kévin Dunglas <dunglas@gmail.com>
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class DoctrineExtractorTest extends TestCase
 {
diff --git a/tests/Bridge/Doctrine/MongoDbOdm/SubresourceDataProviderTest.php b/tests/Bridge/Doctrine/MongoDbOdm/SubresourceDataProviderTest.php
@@ -40,6 +40,7 @@
 
 /**
  * @author Alan Poulain <contact@alanpoulain.eu>
+ * @group mongodb
  */
 class SubresourceDataProviderTest extends TestCase
 {
diff --git a/tests/Bridge/Symfony/Bundle/DependencyInjection/ApiPlatformExtensionTest.php b/tests/Bridge/Symfony/Bundle/DependencyInjection/ApiPlatformExtensionTest.php
diff --git a/tests/Bridge/Symfony/Bundle/DependencyInjection/ConfigurationTest.php b/tests/Bridge/Symfony/Bundle/DependencyInjection/ConfigurationTest.php
diff --git a/tests/Bridge/Symfony/Bundle/Twig/ApiPlatformProfilerPanelTest.php b/tests/Bridge/Symfony/Bundle/Twig/ApiPlatformProfilerPanelTest.php
diff --git a/tests/Fixtures/app/config/config_common.yml b/tests/Fixtures/app/config/config_common.yml
diff --git a/tests/Fixtures/app/config/config_mongodb.yml b/tests/Fixtures/app/config/config_mongodb.yml

Original file line number	Diff line number	Diff line change
`@@ -70,6 +70,7 @@ public function testSingleIdentifier()`
`70`	`70`
`71`	`71`	`/**`
`72`	`72`	`* @group legacy`
	`73`	`+ * @group mongodb`
`73`	`74`	`*/`
`74`	`75`	`public function testSingleDocumentIdentifier()`
`75`	`76`	`{`
Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@`
`28`	`28`
`29`	`29`	`/**`
`30`	`30`	`* @author Alan Poulain <[email protected]>`
	`31`	`+ * @group mongodb`
`31`	`32`	`*/`
`32`	`33`	`class CollectionDataProviderTest extends TestCase`
`33`	`34`	`{`
Original file line number	Diff line number	Diff line change
`@@ -25,6 +25,7 @@`
`25`	`25`
`26`	`26`	`/**`
`27`	`27`	`* @author Alan Poulain <[email protected]>`
	`28`	`+ * @group mongodb`
`28`	`29`	`*/`
`29`	`30`	`class FilterExtensionTest extends TestCase`
`30`	`31`	`{`
Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,7 @@`
`26`	`26`
`27`	`27`	`/**`
`28`	`28`	`* @author Alan Poulain <[email protected]>`
	`29`	`+ * @group mongodb`
`29`	`30`	`*/`
`30`	`31`	`class OrderExtensionTest extends TestCase`
`31`	`32`	`{`
Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,7 @@`
`35`	`35`
`36`	`36`	`/**`
`37`	`37`	`* @author Alan Poulain <[email protected]>`
	`38`	`+ * @group mongodb`
`38`	`39`	`*/`
`39`	`40`	`class PaginationExtensionTest extends TestCase`
`40`	`41`	`{`