From f181dc84e2348315389890b1cf5280a2a40679bc Mon Sep 17 00:00:00 2001
From: Christophe Coevoet <stof@notk.org>
Date: Wed, 5 Sep 2012 20:39:45 +0200
Subject: [PATCH] Added tests for the suggestions

The test about replaced packages is failing because of #752.
---
 .../Fixtures/installer/suggest-installed.test | 29 +++++++++++++++++++
 .../Fixtures/installer/suggest-replaced.test  | 29 +++++++++++++++++++
 .../installer/suggest-uninstalled.test        | 27 +++++++++++++++++
 tests/Composer/Test/InstallerTest.php         | 10 +++++--
 4 files changed, 93 insertions(+), 2 deletions(-)
 create mode 100644 tests/Composer/Test/Fixtures/installer/suggest-installed.test
 create mode 100644 tests/Composer/Test/Fixtures/installer/suggest-replaced.test
 create mode 100644 tests/Composer/Test/Fixtures/installer/suggest-uninstalled.test

diff --git a/tests/Composer/Test/Fixtures/installer/suggest-installed.test b/tests/Composer/Test/Fixtures/installer/suggest-installed.test
new file mode 100644
index 000000000..b84f0a252
--- /dev/null
+++ b/tests/Composer/Test/Fixtures/installer/suggest-installed.test
@@ -0,0 +1,29 @@
+--TEST--
+Suggestions are not displayed for installed packages if they are also installed
+--COMPOSER--
+{
+    "repositories": [
+        {
+            "type": "package",
+            "package": [
+                { "name": "a/a", "version": "1.0.0", "suggest": { "b/b": "an obscure reason" } },
+                { "name": "b/b", "version": "1.0.0" }
+            ]
+        }
+    ],
+    "require": {
+        "a/a": "1.0.0",
+        "b/b": "1.0.0"
+    }
+}
+--RUN--
+install
+--EXPECT-OUTPUT--
+<info>Loading composer repositories with package information</info>
+<info>Installing dependencies</info>
+<info>Writing lock file</info>
+<info>Generating autoload files</info>
+
+--EXPECT--
+Installing a/a (1.0.0)
+Installing b/b (1.0.0)
\ No newline at end of file
diff --git a/tests/Composer/Test/Fixtures/installer/suggest-replaced.test b/tests/Composer/Test/Fixtures/installer/suggest-replaced.test
new file mode 100644
index 000000000..3691e6070
--- /dev/null
+++ b/tests/Composer/Test/Fixtures/installer/suggest-replaced.test
@@ -0,0 +1,29 @@
+--TEST--
+Suggestions are not displayed for installed packages if they are replaced
+--COMPOSER--
+{
+    "repositories": [
+        {
+            "type": "package",
+            "package": [
+                { "name": "a/a", "version": "1.0.0", "suggest": { "b/b": "an obscure reason" } },
+                { "name": "c/c", "version": "1.0.0", "replace": { "b/b": "1.0.0" } }
+            ]
+        }
+    ],
+    "require": {
+        "a/a": "1.0.0",
+        "b/b": "1.0.0"
+    }
+}
+--RUN--
+install
+--EXPECT-OUTPUT--
+<info>Loading composer repositories with package information</info>
+<info>Installing dependencies</info>
+<info>Writing lock file</info>
+<info>Generating autoload files</info>
+
+--EXPECT--
+Installing a/a (1.0.0)
+Installing c/c (1.0.0)
\ No newline at end of file
diff --git a/tests/Composer/Test/Fixtures/installer/suggest-uninstalled.test b/tests/Composer/Test/Fixtures/installer/suggest-uninstalled.test
new file mode 100644
index 000000000..a4ad14fd7
--- /dev/null
+++ b/tests/Composer/Test/Fixtures/installer/suggest-uninstalled.test
@@ -0,0 +1,27 @@
+--TEST--
+Suggestions are displayed for installed packages
+--COMPOSER--
+{
+    "repositories": [
+        {
+            "type": "package",
+            "package": [
+                { "name": "a/a", "version": "1.0.0", "suggest": { "b/b": "an obscure reason" } }
+            ]
+        }
+    ],
+    "require": {
+        "a/a": "1.0.0"
+    }
+}
+--RUN--
+install
+--EXPECT-OUTPUT--
+<info>Loading composer repositories with package information</info>
+<info>Installing dependencies</info>
+a/a suggests installing b/b (an obscure reason)
+<info>Writing lock file</info>
+<info>Generating autoload files</info>
+
+--EXPECT--
+Installing a/a (1.0.0)
\ No newline at end of file
diff --git a/tests/Composer/Test/InstallerTest.php b/tests/Composer/Test/InstallerTest.php
index d4635f246..cca2fd95f 100644
--- a/tests/Composer/Test/InstallerTest.php
+++ b/tests/Composer/Test/InstallerTest.php
@@ -123,7 +123,7 @@ class InstallerTest extends TestCase
     /**
      * @dataProvider getIntegrationTests
      */
-    public function testIntegration($file, $message, $condition, $composerConfig, $lock, $installed, $installedDev, $run, $expectLock, $expect)
+    public function testIntegration($file, $message, $condition, $composerConfig, $lock, $installed, $installedDev, $run, $expectLock, $expectOutput, $expect)
     {
         if ($condition) {
             eval('$res = '.$condition.';');
@@ -226,6 +226,10 @@ class InstallerTest extends TestCase
 
         $installationManager = $composer->getInstallationManager();
         $this->assertSame($expect, implode("\n", $installationManager->getTrace()));
+
+        if ($expectOutput) {
+            $this->assertEquals($expectOutput, $output);
+        }
     }
 
     public function getIntegrationTests()
@@ -250,6 +254,7 @@ class InstallerTest extends TestCase
                 (?:--INSTALLED-DEV--\s*(?P<installedDev>'.$content.'))?\s*
                 --RUN--\s*(?P<run>.*?)\s*
                 (?:--EXPECT-LOCK--\s*(?P<expectLock>'.$content.'))?\s*
+                (?:--EXPECT-OUTPUT--\s*(?P<expectOutput>'.$content.'))?\s*
                 --EXPECT--\s*(?P<expect>.*?)\s*
             $}xs';
 
@@ -279,6 +284,7 @@ class InstallerTest extends TestCase
                     if (!empty($match['expectLock'])) {
                         $expectLock = JsonFile::parseJson($match['expectLock']);
                     }
+                    $expectOutput = $match['expectOutput'];
                     $expect = $match['expect'];
                 } catch (\Exception $e) {
                     die(sprintf('Test "%s" is not valid: '.$e->getMessage(), str_replace($fixturesDir.'/', '', $file)));
@@ -287,7 +293,7 @@ class InstallerTest extends TestCase
                 die(sprintf('Test "%s" is not valid, did not match the expected format.', str_replace($fixturesDir.'/', '', $file)));
             }
 
-            $tests[] = array(str_replace($fixturesDir.'/', '', $file), $message, $condition, $composer, $lock, $installed, $installedDev, $run, $expectLock, $expect);
+            $tests[] = array(str_replace($fixturesDir.'/', '', $file), $message, $condition, $composer, $lock, $installed, $installedDev, $run, $expectLock, $expectOutput, $expect);
         }
 
         return $tests;