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 InstallUpdateEventsTest and NonInstallUpdateEventsTest #174

Merged
merged 2 commits into from May 25, 2022

Commits on Apr 18, 2022

  1. Tests: add new InstallUpdateEventsTest

    Test that the plugin is hooked into the right events and doesn't get triggered when those events are blocked (`--no-plugins`).
    
    These tests verify that:
    - The plugin gets triggered on install/update/require/remove/reinstall events.
    - The plugin doesn't run when Composer is run with `--no-plugins`.
    - The plugin does run when Composer is run with `--no-scripts` (since Composer 2.1.2).
    - Can be run on-demand via a custom script.
    
    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 `composer reinstall` test will only run on Composer 2.2.6+ as prior to that release plugins would not run on that command.
    
    This test class covers the following bug previously reported:
    * Dealerdirect/phpcodesniffer-composer-installer issue 4
    * Dealerdirect/phpcodesniffer-composer-installer PR 5
    jrfnl committed Apr 18, 2022
    Copy the full SHA
    095384b View commit details
    Browse the repository at this point in the history
  2. Tests: add new NonInstallUpdateEventsTest

    This new test class tests that the plugin doesn't get triggered on events it isn't hooked into.
    jrfnl committed Apr 18, 2022
    Copy the full SHA
    203fa94 View commit details
    Browse the repository at this point in the history