Browse Source

Update tests for replace conflicts

This reverts the test changes done in b4698568d2 to the original tests added
in 1425bb7fc3.
Christophe Coevoet 6 years ago
parent
commit
8c3898aa57

+ 5 - 7
tests/Composer/Test/Fixtures/installer/replaced-packages-wrong-version-install.test → tests/Composer/Test/Fixtures/installer/replaced-packages-should-not-be-installed-when-installing-from-lock.test

@@ -1,5 +1,5 @@
 --TEST--
-Requiring a replaced package in a version, that is not provided by the replacing package, should install correctly (although that is not a very smart idea)
+Requiring a replaced package in a version, that is not provided by the replacing package, should result in a conflict, when installing from lock
 --COMPOSER--
 {
     "repositories": [
@@ -17,9 +17,7 @@ Requiring a replaced package in a version, that is not provided by the replacing
         "foo/replaced": "2.0.0"
     }
 }
---RUN--
-install
---EXPECT-LOCK--
+--LOCK--
 {
     "packages": [
         { "name": "foo/original", "version": "1.0.0", "replace": {"foo/replaced": "1.0.0"}, "type": "library" },
@@ -34,8 +32,8 @@ install
     "platform": [],
     "platform-dev": []
 }
+--RUN--
+install
 --EXPECT-EXIT-CODE--
-0
+2
 --EXPECT--
-Installing foo/original (1.0.0)
-Installing foo/replaced (2.0.0)

+ 24 - 0
tests/Composer/Test/Fixtures/installer/replaced-packages-should-not-be-installed.test

@@ -0,0 +1,24 @@
+--TEST--
+Requiring a replaced package in a version, that is not provided by the replacing package, should result in a conflict
+--COMPOSER--
+{
+    "repositories": [
+        {
+            "type": "package",
+            "package": [
+                { "name": "foo/original", "version": "1.0.0", "replace": {"foo/replaced": "1.0.0"} },
+                { "name": "foo/replaced", "version": "1.0.0" },
+                { "name": "foo/replaced", "version": "2.0.0" }
+            ]
+        }
+    ],
+    "require": {
+        "foo/original": "1.0.0",
+        "foo/replaced": "2.0.0"
+    }
+}
+--RUN--
+install
+--EXPECT-EXIT-CODE--
+2
+--EXPECT--

+ 0 - 41
tests/Composer/Test/Fixtures/installer/replaced-packages-wrong-version-install-from-lock.test

@@ -1,41 +0,0 @@
---TEST--
-Requiring a replaced package in a version, that is not provided by the replacing package, should install correctly (although that is not a very smart idea) also when installing from lock
---COMPOSER--
-{
-    "repositories": [
-        {
-            "type": "package",
-            "package": [
-                { "name": "foo/original", "version": "1.0.0", "replace": {"foo/replaced": "1.0.0"} },
-                { "name": "foo/replaced", "version": "1.0.0" },
-                { "name": "foo/replaced", "version": "2.0.0" }
-            ]
-        }
-    ],
-    "require": {
-        "foo/original": "1.0.0",
-        "foo/replaced": "2.0.0"
-    }
-}
---LOCK--
-{
-    "packages": [
-        { "name": "foo/original", "version": "1.0.0", "replace": {"foo/replaced": "1.0.0"}, "type": "library" },
-        { "name": "foo/replaced", "version": "2.0.0", "type": "library" }
-    ],
-    "packages-dev": [],
-    "aliases": [],
-    "minimum-stability": "stable",
-    "stability-flags": {},
-    "prefer-stable": false,
-    "prefer-lowest": false,
-    "platform": [],
-    "platform-dev": []
-}
---RUN--
-install
---EXPECT-EXIT-CODE--
-0
---EXPECT--
-Installing foo/original (1.0.0)
-Installing foo/replaced (2.0.0)