Move a few tests to integration from functional. (#3919)

* Make testInsecurePackage an integration test. * Add SAConvertTest
drush-ops · Feb 7, 2019 · 9203f68 · 9203f68
1 parent 519110f
commit 9203f68
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 8 deletions.
diff --git a/tests/functional/SecurityUpdatesTest.php → tests/integration/SecurityUpdatesTest.php b/tests/functional/SecurityUpdatesTest.php → tests/integration/SecurityUpdatesTest.php
@@ -2,23 +2,20 @@
 
 namespace Unish;
 
-use Drush\Commands\pm\SecurityUpdateCommands;
-
 /**
- * Tests "pm:security" commands for D8+.
+ * Tests "pm:security" command.
  * @group commands
- * @group slow
  * @group pm
  */
-class SecurityUpdatesTest extends CommandUnishTestCase
+class SecurityUpdatesTest extends UnishIntegrationTestCase
 {
 
   /**
    * Test that insecure packages are correctly identified.
    */
     public function testInsecurePackage()
     {
-        $this->drush('pm:security', [], ['format' => 'json'], null, null, self::EXIT_ERROR);
+        $this->drush('pm:security', [], ['format' => 'json'], self::EXIT_ERROR);
         $this->assertContains('One or more of your dependencies has an outstanding security update.', $this->getErrorOutput());
         $this->assertContains('Try running: composer require drupal/alinks --update-with-dependencies', $this->getErrorOutput());
         $security_advisories = $this->getOutputFromJSON();

diff --git a/tests/functional/SiteAliasConvertTest.php → tests/integration/SiteAliasConvertTest.php b/tests/functional/SiteAliasConvertTest.php → tests/integration/SiteAliasConvertTest.php
@@ -8,7 +8,7 @@
 /**
  * @group base
  */
-class SiteAliasConvertTest extends CommandUnishTestCase
+class SiteAliasConvertTest extends UnishIntegrationTestCase
 {
 
   /**
@@ -18,7 +18,7 @@ public function testSiteAliasConvert()
     {
         // Use a custom destination so we don't have to install a Drupal.
         $destination = Path::join(self::getSandbox(), 'testSiteAliasConvert');
-        $this->drush('site:alias-convert', ['destination' => $destination], ['sources' =>  Path::join(__DIR__, '/resources/alias-fixtures')]);
+        $this->drush('site:alias-convert', ['destination' => $destination], ['sources' =>  Path::join(dirname(__DIR__), 'functional/resources/alias-fixtures')]);
 
         // Write config alias-path that specifies our destination.
         $config['drush']['paths']['alias-path'][] = $destination;