Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[Translation] Move ProviderTestCase out of the Tests namespace #40911

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

Merged
merged 1 commit into from
Apr 22, 2021

Conversation

derrabus
Copy link
Member

Q A
Branch? 5.x
Bug fix? yes
New feature? no
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

Because the Tests namespace is stripped from our distribution packages, the new Loco translation bridge cannot use the abstract ProviderTestCase class. This PR suggests to move it into a dedicated Test namespace to make it reusable for the Loco bridge and potential future bridges.

Copy link
Member

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@nicolas-grekas
Copy link
Member

Let's make the class internal?

@derrabus derrabus force-pushed the bugfix/provider-test-case branch from f1faf7a to c1ec38f Compare April 22, 2021 12:42
@derrabus
Copy link
Member Author

Let's make the class internal?

@nicolas-grekas
Copy link
Member

Thank you @derrabus.

@nicolas-grekas nicolas-grekas merged commit ba2a354 into symfony:5.x Apr 22, 2021
@derrabus derrabus deleted the bugfix/provider-test-case branch April 22, 2021 12:45
nicolas-grekas added a commit that referenced this pull request Apr 22, 2021
…sts namespace (derrabus)

This PR was merged into the 5.3-dev branch.

Discussion
----------

[Translation] Move ProviderFactoryTestCase out of the Tests namespace

| Q             | A
| ------------- | ---
| Branch?       | 5.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | N/A
| License       | MIT
| Doc PR        | N/A

Same as #40911, different class. Sorry, I missed the seconds abstract class. 🙈

Commits
-------

e01499d [Translation] Move ProviderFactoryTestCase out of the Tests namespace
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.

4 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.