8000 [phpunit-bridge] Cannot autoload class "\Symfony\Bridge\PhpUnit\SymfonyTestsListener" · Issue #26430 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[phpunit-bridge] Cannot autoload class "\Symfony\Bridge\PhpUnit\SymfonyTestsListener" #26430

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

Closed
yakobe opened this issue Mar 6, 2018 · 3 comments

Comments

@yakobe
Copy link
Contributor
yakobe commented Mar 6, 2018
Q A
Bug report? yes
Feature request? no
BC Break report? yes
RFC? yes/no
Symfony version 3.4.5

If you run install composer install --classmap-authoritative then running phpunit fails with the error Class "\Symfony\Bridge\PhpUnit\SymfonyTestsListener" does not exist.

The cause seems to be that the class is now just an alias and not a concrete class (see symfony/phpunit-bridge@48139af#diff-7f2565259188dda68869a0605b70c219L33) so composer cannot add this to the class map.

I am nut using simple-phpunit but a the phpunit defined in my composer require-dev because i wish to use johnkary/phpunit-speedtrap.

@yakobe
Copy link
Contributor Author
yakobe commented Mar 6, 2018

Perhaps it would be enough to just declare the class as before but wrap it in if (false) statement?

@nicolas-grekas
Copy link
Member

Perhaps it would be enough to just declare the class as before but wrap it in if (false) statement?

looks like a good idea, would you mind sending a PR for this?

@yakobe
Copy link
Contributor Author
yakobe commented Mar 16, 2018

I have added a PR to 3.4. I hope that is correct?

yakobe added a commit to yakobe/symfony that referenced this issue Mar 16, 2018
yakobe added a commit to yakobe/symfony that referenced this issue Mar 19, 2018
nicolas-grekas added a commit that referenced this issue Mar 19, 2018
…pUnit\SymfonyTestsListener" (Jake Bishop)

This PR was merged into the 3.4 branch.

Discussion
----------

[Bridge\PhpUnit] Cannot autoload class "\Symfony\Bridge\PhpUnit\SymfonyTestsListener"

| Q             | A
| ------------- | ---
| Branch?       | 3.4 <!-- see below -->
| Bug fix?      | yes
| New feature?  | no <!-- don't forget to update src/**/CHANGELOG.md files -->
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- don't forget to update UPGRADE-*.md files -->
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | #26430
| License       | MIT
| Doc PR        | - <!-- required for new features -->

Allows composer to discover the class for autoloading when using `composer install --classmap-authoritative`

Commits
-------

6b6fdab [Bridge\PhpUnit] Fix #26430 Cannot autoload listeners
nicolas-grekas added a commit that referenced this issue Mar 19, 2018
* 3.4: (22 commits)
  [DI] Add tests for EnvVarProcessor
  [Bridge\PhpUnit] Fix #26430 Cannot autoload listeners
  Make sure we always render errors. Eventhough labels are disabled
  Make sure form errors is valid HTML
  [HttpKernel] Allow generators in registerBundle
  [TwigBundle] document TwigRenderer BC break in UPGRADE-3.4
  Extra line to bootstrap 3 horizontal layout
  [Serializer] Remove const override
  Update Client.php
  [PhpUnitBridge] Ability to use different composer.json file
  [DomCrawler] FormField: remove an useless return statement
  [Config] ReflectionClassResource check abstract ServiceSubscriberInterface and EventSubscriberInterface
  Display the Welcome Page when there is no homepage defined
  fix regression when extending the Container class without a constructor
  fix the updating of timestamp in the MemcachedSessionHandler
  [SecurityBundle] Make extra character non mandatory in regex
  [Config] Add characters to the regex
  bumped Symfony version to 3.4.7
  updated VERSION for 3.4.6
  updated CHANGELOG for 3.4.6
  ...
nicolas-grekas added a commit that referenced this issue Mar 19, 2018
* 4.0: (28 commits)
  [DI] Add tests for EnvVarProcessor
  typo
  [Bridge\PhpUnit] Fix #26430 Cannot autoload listeners
  Make sure we always render errors. Eventhough labels are disabled
  Make sure form errors is valid HTML
  [HttpKernel] Allow generators in registerBundle
  [TwigBundle] document TwigRenderer BC break in UPGRADE-3.4
  Extra line to bootstrap 3 horizontal layout
  [Serializer] Remove const override
  Update Client.php
  [PhpUnitBridge] Ability to use different composer.json file
  [DomCrawler] FormField: remove an useless return statement
  [Config] ReflectionClassResource check abstract ServiceSubscriberInterface and EventSubscriberInterface
  Display the Welcome Page when there is no homepage defined
  [DI] Remove dead dumper check
  Make KernelInterface docblock more fit for bundle-less environment
  fix regression when extending the Container class without a constructor
  fix the updating of timestamp in the MemcachedSessionHandler
  [SecurityBundle] Make extra character non mandatory in regex
  [Config] Add characters to the regex
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants
0