Browse Source

Change update to upgrade for package update operations which are upgrades, refs #8594

Jordi Boggiano 5 years ago
parent
commit
9609729de7
37 changed files with 63 additions and 63 deletions
  1. 1 1
      doc/01-basic-usage.md
  2. 1 1
      src/Composer/Command/SelfUpdateCommand.php
  3. 1 1
      src/Composer/DependencyResolver/Operation/UpdateOperation.php
  4. 4 4
      tests/Composer/Test/Fixtures/installer/github-issues-4795-2.test
  5. 1 1
      tests/Composer/Test/Fixtures/installer/install-from-lock-removes-package.test
  6. 2 2
      tests/Composer/Test/Fixtures/installer/partial-update-forces-dev-reference-from-lock-for-non-updated-packages.test
  7. 1 1
      tests/Composer/Test/Fixtures/installer/partial-update-from-lock.test
  8. 2 2
      tests/Composer/Test/Fixtures/installer/partial-update-installs-from-lock-even-missing.test
  9. 1 1
      tests/Composer/Test/Fixtures/installer/root-requirements-do-not-affect-locked-versions.test
  10. 1 1
      tests/Composer/Test/Fixtures/installer/update-alias-lock.test
  11. 1 1
      tests/Composer/Test/Fixtures/installer/update-alias.test
  12. 2 2
      tests/Composer/Test/Fixtures/installer/update-all-dry-run.test
  13. 2 2
      tests/Composer/Test/Fixtures/installer/update-all.test
  14. 3 3
      tests/Composer/Test/Fixtures/installer/update-changes-url.test
  15. 1 1
      tests/Composer/Test/Fixtures/installer/update-dev-ignores-providers.test
  16. 1 1
      tests/Composer/Test/Fixtures/installer/update-dev-packages-updates-repo-url.test
  17. 1 1
      tests/Composer/Test/Fixtures/installer/update-dev-to-new-ref-picks-up-changes.test
  18. 1 1
      tests/Composer/Test/Fixtures/installer/update-ignore-platform-package-requirements.test
  19. 1 1
      tests/Composer/Test/Fixtures/installer/update-installed-reference.test
  20. 2 2
      tests/Composer/Test/Fixtures/installer/update-no-dev-still-resolves-dev.test
  21. 1 1
      tests/Composer/Test/Fixtures/installer/update-picks-up-change-of-vcs-type.test
  22. 1 1
      tests/Composer/Test/Fixtures/installer/update-prefer-lowest-stable.test
  23. 1 1
      tests/Composer/Test/Fixtures/installer/update-reference-picks-latest.test
  24. 1 1
      tests/Composer/Test/Fixtures/installer/update-reference.test
  25. 2 2
      tests/Composer/Test/Fixtures/installer/update-whitelist-locked-require.test
  26. 3 3
      tests/Composer/Test/Fixtures/installer/update-whitelist-patterns-with-all-dependencies.test
  27. 3 3
      tests/Composer/Test/Fixtures/installer/update-whitelist-patterns-with-dependencies.test
  28. 3 3
      tests/Composer/Test/Fixtures/installer/update-whitelist-patterns-with-root-dependencies.test
  29. 2 2
      tests/Composer/Test/Fixtures/installer/update-whitelist-patterns-without-dependencies.test
  30. 4 4
      tests/Composer/Test/Fixtures/installer/update-whitelist-patterns.test
  31. 2 2
      tests/Composer/Test/Fixtures/installer/update-whitelist-reads-lock.test
  32. 1 1
      tests/Composer/Test/Fixtures/installer/update-whitelist-removes-unused.test
  33. 2 2
      tests/Composer/Test/Fixtures/installer/update-whitelist-with-dependencies.test
  34. 1 1
      tests/Composer/Test/Fixtures/installer/update-whitelist.test
  35. 4 4
      tests/Composer/Test/Fixtures/installer/update-with-all-dependencies.test
  36. 1 1
      tests/Composer/Test/Fixtures/installer/updating-dev-from-lock-removes-old-deps.test
  37. 1 1
      tests/Composer/Test/Fixtures/installer/updating-dev-updates-url-and-reference.test

+ 1 - 1
doc/01-basic-usage.md

@@ -159,7 +159,7 @@ php composer.phar update
 > if the `composer.lock` has not been updated since changes were made to the
 > `composer.json` that might affect dependency resolution.
 
-If you only want to install or update one dependency, you can whitelist them:
+If you only want to install, upgrade or remove one dependency, you can whitelist them:
 
 ```sh
 php composer.phar update monolog/monolog [...]

+ 1 - 1
src/Composer/Command/SelfUpdateCommand.php

@@ -154,7 +154,7 @@ EOT
 
         $updatingToTag = !preg_match('{^[0-9a-f]{40}$}', $updateVersion);
 
-        $io->write(sprintf("Updating to version <info>%s</info> (%s channel).", $updateVersion, $versionsUtil->getChannel()));
+        $io->write(sprintf("Upgrading to version <info>%s</info> (%s channel).", $updateVersion, $versionsUtil->getChannel()));
         $remoteFilename = $baseUrl . ($updatingToTag ? "/download/{$updateVersion}/composer.phar" : '/composer.phar');
         $signature = $httpDownloader->get($remoteFilename.'.sig')->getBody();
         $io->writeError('   ', false);

+ 1 - 1
src/Composer/DependencyResolver/Operation/UpdateOperation.php

@@ -86,7 +86,7 @@ class UpdateOperation extends SolverOperation
             $toVersion = $this->targetPackage->getFullPrettyVersion(true, PackageInterface::DISPLAY_DIST_REF);
         }
 
-        $actionName = VersionParser::isUpgrade($this->initialPackage->getVersion(), $this->targetPackage->getVersion()) ? 'Updating' : 'Downgrading';
+        $actionName = VersionParser::isUpgrade($this->initialPackage->getVersion(), $this->targetPackage->getVersion()) ? 'Upgrading' : 'Downgrading';
 
         return $actionName.' '.$this->initialPackage->getPrettyName().' ('.$fromVersion.' => '.$toVersion.')';
     }

+ 4 - 4
tests/Composer/Test/Fixtures/installer/github-issues-4795-2.test

@@ -55,13 +55,13 @@ update a/a b/b --with-dependencies
 Loading composer repositories with package information
 Updating dependencies
 Lock file operations: 0 installs, 2 updates, 0 removals
-  - Updating a/a (1.0.0 => 1.1.0)
-  - Updating b/b (1.0.0 => 1.1.0)
+  - Upgrading a/a (1.0.0 => 1.1.0)
+  - Upgrading b/b (1.0.0 => 1.1.0)
 Writing lock file
 Installing dependencies from lock file (including require-dev)
 Package operations: 0 installs, 2 updates, 0 removals
 Generating autoload files
 
 --EXPECT--
-Updating a/a (1.0.0 => 1.1.0)
-Updating b/b (1.0.0 => 1.1.0)
+Upgrading a/a (1.0.0 => 1.1.0)
+Upgrading b/b (1.0.0 => 1.1.0)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/install-from-lock-removes-package.test

@@ -41,4 +41,4 @@ Install from a lock file that deleted a package
 install
 --EXPECT--
 Uninstalling old-dependency (1.0.0)
-Updating whitelisted (1.0.0 => 1.1.0)
+Upgrading whitelisted (1.0.0 => 1.1.0)

+ 2 - 2
tests/Composer/Test/Fixtures/installer/partial-update-forces-dev-reference-from-lock-for-non-updated-packages.test

@@ -93,5 +93,5 @@ update b/b
     "platform-dev": []
 }
 --EXPECT--
-Updating a/a (dev-master oldmaster-a => dev-master newmaster-a)
-Updating b/b (dev-master oldmaster-b => dev-master newmaster-b2)
+Upgrading a/a (dev-master oldmaster-a => dev-master newmaster-a)
+Upgrading b/b (dev-master oldmaster-b => dev-master newmaster-b2)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/partial-update-from-lock.test

@@ -74,7 +74,7 @@ update b/unstable
     "platform-dev": []
 }
 --EXPECT--
-Updating a/old (0.9.0 => 1.0.0)
+Upgrading a/old (0.9.0 => 1.0.0)
 Downgrading b/unstable (1.1.0-alpha => 1.0.0)
 Downgrading c/uptodate (2.0.0 => 1.0.0)
 Installing d/removed (1.0.0)

+ 2 - 2
tests/Composer/Test/Fixtures/installer/partial-update-installs-from-lock-even-missing.test

@@ -97,9 +97,9 @@ update b/b
     "platform-dev": []
 }
 --EXPECT--
-Updating a/a (dev-master oldmaster-a => dev-master newmaster-a)
+Upgrading a/a (dev-master oldmaster-a => dev-master newmaster-a)
 Marking a/a (2.2.x-dev newmaster-a) as installed, alias of a/a (dev-master newmaster-a)
-Updating b/b (dev-master oldmaster-b => dev-master newmaster-b2)
+Upgrading b/b (dev-master oldmaster-b => dev-master newmaster-b2)
 Marking b/b (2.3.x-dev newmaster-b2) as installed, alias of b/b (dev-master newmaster-b2)
 Marking a/a (2.1.x-dev oldmaster-a) as uninstalled, alias of a/a (dev-master oldmaster-a)
 Marking b/b (2.1.x-dev oldmaster-b) as uninstalled, alias of b/b (dev-master oldmaster-b)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/root-requirements-do-not-affect-locked-versions.test

@@ -38,4 +38,4 @@ The locked version will not get overwritten by an install
 --RUN--
 install
 --EXPECT--
-Updating foo/baz (1.0.0 => 2.0.0)
+Upgrading foo/baz (1.0.0 => 2.0.0)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-alias-lock.test

@@ -59,4 +59,4 @@ update
     "platform-dev": []
 }
 --EXPECT--
-Updating a/a (dev-master 1234 => dev-master master)
+Upgrading a/a (dev-master 1234 => dev-master master)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-alias.test

@@ -33,5 +33,5 @@ Update aliased package to non-aliased version
 --RUN--
 update
 --EXPECT--
-Updating a/a (dev-master master => dev-foo foo)
+Upgrading a/a (dev-master master => dev-foo foo)
 Marking a/a (1.0.x-dev master) as uninstalled, alias of a/a (dev-master master)

+ 2 - 2
tests/Composer/Test/Fixtures/installer/update-all-dry-run.test

@@ -44,6 +44,6 @@ Lock file operations: 3 installs, 0 updates, 0 removals
   - Locking a/c (1.0.0)
 Installing dependencies from lock file (including require-dev)
 Package operations: 0 installs, 2 updates, 0 removals
-  - Updating a/a (1.0.0 => 1.0.1)
-  - Updating a/b (1.0.0 => 2.0.0)
+  - Upgrading a/a (1.0.0 => 1.0.1)
+  - Upgrading a/b (1.0.0 => 2.0.0)
 --EXPECT--

+ 2 - 2
tests/Composer/Test/Fixtures/installer/update-all.test

@@ -36,5 +36,5 @@ Updates updateable packages
 --RUN--
 update
 --EXPECT--
-Updating a/a (1.0.0 => 1.0.1)
-Updating a/b (1.0.0 => 2.0.0)
+Upgrading a/a (1.0.0 => 1.0.1)
+Upgrading a/b (1.0.0 => 2.0.0)

+ 3 - 3
tests/Composer/Test/Fixtures/installer/update-changes-url.test

@@ -214,7 +214,7 @@ g/g is dev and installed in a different ref than the #ref, so it gets updated an
 --RUN--
 update a/a b/b d/d g/g
 --EXPECT--
-Updating a/a (dev-master 1111111 => dev-master 2222222)
-Updating b/b (2.0.3 1111111 => 2.0.3 2222222)
+Upgrading a/a (dev-master 1111111 => dev-master 2222222)
+Upgrading b/b (2.0.3 1111111 => 2.0.3 2222222)
 Installing e/e (dev-master 1111111)
-Updating g/g (dev-master 0000000 => dev-master 1111111)
+Upgrading g/g (dev-master 0000000 => dev-master 1111111)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-dev-ignores-providers.test

@@ -35,4 +35,4 @@ Updating a dev package selects its newest version but no providers
 --RUN--
 update
 --EXPECT--
-Updating a/installed (dev-master oldref => dev-master newref)
+Upgrading a/installed (dev-master oldref => dev-master newref)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-dev-packages-updates-repo-url.test

@@ -93,4 +93,4 @@ update
     "platform-dev": []
 }
 --EXPECT--
-Updating a/a (dev-master oldmaster => dev-master newmaster)
+Upgrading a/a (dev-master oldmaster => dev-master newmaster)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-dev-to-new-ref-picks-up-changes.test

@@ -38,4 +38,4 @@ Updating a dev package to its latest ref should pick up new dependencies
 update
 --EXPECT--
 Installing a/dependency (dev-master ref)
-Updating a/devpackage (dev-master oldref => dev-master newref)
+Upgrading a/devpackage (dev-master oldref => dev-master newref)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-ignore-platform-package-requirements.test

@@ -23,4 +23,4 @@ Update in ignore-platform-reqs mode
 --RUN--
 update --ignore-platform-reqs
 --EXPECT--
-Updating a/a (1.0.0 => 1.0.1)
+Upgrading a/a (1.0.0 => 1.0.1)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-installed-reference.test

@@ -28,4 +28,4 @@ Updating a dev package forcing it's reference should not do anything if the refe
 --RUN--
 update
 --EXPECT--
-Updating a/a (dev-master def000 => dev-master )
+Upgrading a/a (dev-master def000 => dev-master )

+ 2 - 2
tests/Composer/Test/Fixtures/installer/update-no-dev-still-resolves-dev.test

@@ -61,8 +61,8 @@ Updates with --no-dev but we still end up with a complete lock file including de
 update --no-dev
 --EXPECT--
 Uninstalling a/b (1.0.0)
-Updating a/a (1.0.0 => 1.0.1)
+Upgrading a/a (1.0.0 => 1.0.1)
 Installing a/c (1.0.0)
-Updating dev/pkg (dev-master old => dev-master new)
+Upgrading dev/pkg (dev-master old => dev-master new)
 Marking dev/pkg (1.1.x-dev new) as installed, alias of dev/pkg (dev-master new)
 Marking dev/pkg (1.0.x-dev old) as uninstalled, alias of dev/pkg (dev-master old)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-picks-up-change-of-vcs-type.test

@@ -62,4 +62,4 @@ update mirrors
     "platform-dev": []
 }
 --EXPECT--
-Updating a/a (1.0.0 old-hg-ref => 1.0.0 new-git-ref)
+Upgrading a/a (1.0.0 old-hg-ref => 1.0.0 new-git-ref)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-prefer-lowest-stable.test

@@ -36,5 +36,5 @@ Updates packages to their lowest stable version
 --RUN--
 update --prefer-lowest --prefer-stable
 --EXPECT--
-Updating a/a (1.0.0-rc1 => 1.0.1)
+Upgrading a/a (1.0.0-rc1 => 1.0.1)
 Downgrading a/b (1.0.1 => 1.0.0)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-reference-picks-latest.test

@@ -28,4 +28,4 @@ Updating a dev package should update to the latest available reference
 --RUN--
 update
 --EXPECT--
-Updating a/a (dev-master def000 => dev-master abc123)
+Upgrading a/a (dev-master def000 => dev-master abc123)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-reference.test

@@ -27,4 +27,4 @@ Updates a dev package forcing it's reference
 --RUN--
 install
 --EXPECT--
-Updating a/a (dev-master abc123 => dev-master def000)
+Upgrading a/a (dev-master abc123 => dev-master def000)

+ 2 - 2
tests/Composer/Test/Fixtures/installer/update-whitelist-locked-require.test

@@ -49,5 +49,5 @@ Update with a package whitelist only updates those packages if they are not pres
 --RUN--
 update whitelisted dependency
 --EXPECT--
-Updating dependency (1.0.0 => 1.1.0)
-Updating whitelisted (1.0.0 => 1.1.0)
+Upgrading dependency (1.0.0 => 1.1.0)
+Upgrading whitelisted (1.0.0 => 1.1.0)

+ 3 - 3
tests/Composer/Test/Fixtures/installer/update-whitelist-patterns-with-all-dependencies.test

@@ -60,6 +60,6 @@ Update with a package whitelist pattern and all-dependencies flag updates packag
 --RUN--
 update whitelisted-* --with-all-dependencies
 --EXPECT--
-Updating whitelisted-component1 (1.0.0 => 1.1.0)
-Updating dependency (1.0.0 => 1.1.0)
-Updating whitelisted-component2 (1.0.0 => 1.1.0)
+Upgrading whitelisted-component1 (1.0.0 => 1.1.0)
+Upgrading dependency (1.0.0 => 1.1.0)
+Upgrading whitelisted-component2 (1.0.0 => 1.1.0)

+ 3 - 3
tests/Composer/Test/Fixtures/installer/update-whitelist-patterns-with-dependencies.test

@@ -62,6 +62,6 @@ Update with a package whitelist only updates those packages and their dependenci
 --RUN--
 update whitelisted-* --with-dependencies
 --EXPECT--
-Updating whitelisted-component1 (1.0.0 => 1.1.0)
-Updating dependency (1.0.0 => 1.1.0)
-Updating whitelisted-component2 (1.0.0 => 1.1.0)
+Upgrading whitelisted-component1 (1.0.0 => 1.1.0)
+Upgrading dependency (1.0.0 => 1.1.0)
+Upgrading whitelisted-component2 (1.0.0 => 1.1.0)

+ 3 - 3
tests/Composer/Test/Fixtures/installer/update-whitelist-patterns-with-root-dependencies.test

@@ -72,6 +72,6 @@ Update with a package whitelist only updates those packages and their dependenci
 --RUN--
 update whitelisted-* --with-dependencies
 --EXPECT--
-Updating dependency (1.0.0 => 1.1.0)
-Updating whitelisted-component2 (1.0.0 => 1.1.0)
-Updating whitelisted-component1 (1.0.0 => 1.1.0)
+Upgrading dependency (1.0.0 => 1.1.0)
+Upgrading whitelisted-component2 (1.0.0 => 1.1.0)
+Upgrading whitelisted-component1 (1.0.0 => 1.1.0)

+ 2 - 2
tests/Composer/Test/Fixtures/installer/update-whitelist-patterns-without-dependencies.test

@@ -57,5 +57,5 @@ Update with a package whitelist only updates those packages matching the pattern
 --RUN--
 update whitelisted-*
 --EXPECT--
-Updating whitelisted-component1 (1.0.0 => 1.1.0)
-Updating whitelisted-component2 (1.0.0 => 1.1.0)
+Upgrading whitelisted-component1 (1.0.0 => 1.1.0)
+Upgrading whitelisted-component2 (1.0.0 => 1.1.0)

+ 4 - 4
tests/Composer/Test/Fixtures/installer/update-whitelist-patterns.test

@@ -63,7 +63,7 @@ Update with a package whitelist only updates those corresponding to the pattern
 --RUN--
 update vendor/Test* exact/Test-Package notexact/Test all/* no/reg?xp
 --EXPECT--
-Updating all/Package1 (1.0 => 2.0)
-Updating all/Package2 (1.0 => 2.0)
-Updating exact/Test-Package (1.0 => 2.0)
-Updating vendor/Test-Package (1.0 => 2.0)
+Upgrading all/Package1 (1.0 => 2.0)
+Upgrading all/Package2 (1.0 => 2.0)
+Upgrading exact/Test-Package (1.0 => 2.0)
+Upgrading vendor/Test-Package (1.0 => 2.0)

+ 2 - 2
tests/Composer/Test/Fixtures/installer/update-whitelist-reads-lock.test

@@ -43,6 +43,6 @@ Limited update takes rules from lock if available, and not from the installed re
 --RUN--
 update toupdate/installed
 --EXPECT--
-Updating old/installed (0.9.0 => 1.0.0)
-Updating toupdate/installed (1.0.0 => 1.1.0)
+Upgrading old/installed (0.9.0 => 1.0.0)
+Upgrading toupdate/installed (1.0.0 => 1.1.0)
 Installing toupdate/notinstalled (1.0.0)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-whitelist-removes-unused.test

@@ -45,4 +45,4 @@ Update with a package whitelist removes unused packages
 update --with-dependencies whitelisted
 --EXPECT--
 Uninstalling old-dependency (1.0.0)
-Updating whitelisted (1.0.0 => 1.1.0)
+Upgrading whitelisted (1.0.0 => 1.1.0)

+ 2 - 2
tests/Composer/Test/Fixtures/installer/update-whitelist-with-dependencies.test

@@ -52,5 +52,5 @@ Update with a package whitelist only updates those packages and their dependenci
 --RUN--
 update whitelisted --with-dependencies
 --EXPECT--
-Updating dependency (1.0.0 => 1.1.0)
-Updating whitelisted (1.0.0 => 1.1.0)
+Upgrading dependency (1.0.0 => 1.1.0)
+Upgrading whitelisted (1.0.0 => 1.1.0)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/update-whitelist.test

@@ -54,4 +54,4 @@ Update with a package whitelist only updates those packages listed as command ar
 --RUN--
 update whitelisted
 --EXPECT--
-Updating whitelisted (1.0.0 => 1.1.0)
+Upgrading whitelisted (1.0.0 => 1.1.0)

+ 4 - 4
tests/Composer/Test/Fixtures/installer/update-with-all-dependencies.test

@@ -50,13 +50,13 @@ update b/b --with-all-dependencies
 Loading composer repositories with package information
 Updating dependencies
 Lock file operations: 0 installs, 2 updates, 0 removals
-  - Updating a/a (1.0.0 => 1.1.0)
-  - Updating b/b (1.0.0 => 1.1.0)
+  - Upgrading a/a (1.0.0 => 1.1.0)
+  - Upgrading b/b (1.0.0 => 1.1.0)
 Writing lock file
 Installing dependencies from lock file (including require-dev)
 Package operations: 0 installs, 2 updates, 0 removals
 Generating autoload files
 
 --EXPECT--
-Updating a/a (1.0.0 => 1.1.0)
-Updating b/b (1.0.0 => 1.1.0)
+Upgrading a/a (1.0.0 => 1.1.0)
+Upgrading b/b (1.0.0 => 1.1.0)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/updating-dev-from-lock-removes-old-deps.test

@@ -42,4 +42,4 @@ Installing locked dev packages should remove old dependencies
 install
 --EXPECT--
 Uninstalling a/dependency (dev-master ref)
-Updating a/devpackage (dev-master oldref => dev-master newref)
+Upgrading a/devpackage (dev-master oldref => dev-master newref)

+ 1 - 1
tests/Composer/Test/Fixtures/installer/updating-dev-updates-url-and-reference.test

@@ -65,4 +65,4 @@ update
     "platform-dev": []
 }
 --EXPECT--
-Updating a/a (dev-master oldref => dev-master newref)
+Upgrading a/a (dev-master oldref => dev-master newref)