bug symfony#52765 [Translation] Remove @internal from abstract testcases (OskarStark)

fabpot · fabpot · commit f59ffb4ed7ef · 2023-11-29T09:14:36.000+01:00
This PR was merged into the 6.4 branch. Discussion ---------- [Translation] Remove ``@internal`` from abstract testcases | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | -- | License | MIT The test cases are in the `Test` namespace and therefore under BC promise. All third-party providers should benefit from it. We do the same for Mailer and Notifier. cc `@m2mtech` as this PR is inspired by your comment in the copied class 😃 in https://github.com/m2mtech/weblate-translation-provider/blob/fa38b3c50b5db832e993a0035f516ee0885482b1/tests/ProviderFactoryTestCase.php#L1-L5 `@nicolas`-grekas or do you want me to target `5.4`? Commits ------- b7d0b69 [Translation] Remove ``@internal`` from abstract testcases
diff --git a/src/Symfony/Component/Translation/Test/ProviderFactoryTestCase.php b/src/Symfony/Component/Translation/Test/ProviderFactoryTestCase.php
@@ -28,8 +28,6 @@
  * A test case to ease testing a translation provider factory.
  *
  * @author Mathieu Santostefano <msantostefano@protonmail.com>
- *
- * @internal
  */
 abstract class ProviderFactoryTestCase extends TestCase
 {
diff --git a/src/Symfony/Component/Translation/Test/ProviderTestCase.php b/src/Symfony/Component/Translation/Test/ProviderTestCase.php
@@ -25,8 +25,6 @@
  * A test case to ease testing a translation provider.
  *
  * @author Mathieu Santostefano <msantostefano@protonmail.com>
- *
- * @internal
  */
 abstract class ProviderTestCase extends TestCase
 {

Original file line number	Diff line number	Diff line change
`@@ -28,8 +28,6 @@`
`28`	`28`	`* A test case to ease testing a translation provider factory.`
`29`	`29`	`*`
`30`	`30`	`* @author Mathieu Santostefano <[email protected]>`
`31`		`- *`
`32`		`- * @internal`
`33`	`31`	`*/`
`34`	`32`	`abstract class ProviderFactoryTestCase extends TestCase`
`35`	`33`	`{`
Original file line number	Diff line number	Diff line change
`@@ -25,8 +25,6 @@`
`25`	`25`	`* A test case to ease testing a translation provider.`
`26`	`26`	`*`
`27`	`27`	`* @author Mathieu Santostefano <[email protected]>`
`28`		`- *`
`29`		`- * @internal`
`30`	`28`	`*/`
`31`	`29`	`abstract class ProviderTestCase extends TestCase`
`32`	`30`	`{`