Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests: add new RemovePluginTest #177

Merged
merged 1 commit into from
May 25, 2022
Merged

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Apr 18, 2022

Proposed Changes

Test that the plugin does not throw errors when the plugin is still in memory, but PHPCS and the plugin are uninstalled (Composer 1.x).

These tests verify:

  • That the plugin exits with error code 0 when PHPCS and the plugin are uninstalled due to a no-dev install.
  • That the plugin exits with error code 0 when the standard which required the plugin is uninstalled.

This test is about Composer and the plugin, so does not need to be tested against multiple PHPCS versions.
The behaviour also shouldn't differ between a global vs local Composer install, so only testing one type.

Note: the behaviour of Composer changed for this in Composer 2.0.0, but still not a bad thing to safeguard this for Composer 2.x as well, in case of regressions in Composer.

This test class covers the following bug previously reported:

Related Issues

Related to #92

Test that the plugin does not throw errors when the plugin is still in memory, but PHPCS and the plugin are uninstalled (Composer 1.x).

These tests verify:
- That the plugin exits with error code 0 when PHPCS and the plugin are uninstalled due to a no-dev install.
- That the plugin exits with error code 0 when the standard which required the plugin is uninstalled.

This test is about Composer and the plugin, so does not need to be tested against multiple PHPCS versions.
The behaviour also shouldn't differ between a global vs local Composer install, so only testing one type.

Note: the behaviour of Composer changed for this in Composer 2.0.0, but still not a bad thing to safeguard this for Composer 2.x as well, in case of regressions in Composer.

This test class covers the following bug previously reported:
* Dealerdirect/phpcodesniffer-composer-installer issue 103
* Dealerdirect/phpcodesniffer-composer-installer PR 104
@jrfnl jrfnl merged commit 04f7a40 into master May 25, 2022
@jrfnl jrfnl deleted the feature/tests-new-removeplugin-test branch May 25, 2022 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants