You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "Patrick Rhomberg (JIRA)" <ji...@apache.org> on 2018/10/01 21:41:00 UTC
[jira] [Assigned] (GEODE-5606)
CategoryWithParameterizedRunnerFactory can be removed
[ https://issues.apache.org/jira/browse/GEODE-5606?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Patrick Rhomberg reassigned GEODE-5606:
---------------------------------------
Assignee: Patrick Rhomberg
> CategoryWithParameterizedRunnerFactory can be removed
> -----------------------------------------------------
>
> Key: GEODE-5606
> URL: https://issues.apache.org/jira/browse/GEODE-5606
> Project: Geode
> Issue Type: Improvement
> Components: tests
> Reporter: Patrick Rhomberg
> Assignee: Patrick Rhomberg
> Priority: Major
> Labels: pull-request-available
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> Previously, this test runner was required for compatibility with any test using {{@RunWith(Parameterized.class)}}. See GEODE-4122 for details.
> The issue presented in GEODE-4122 et al that was solved by the {{CategoryWithParameterizedRunnerFactory}} is a non-issue in current versions of Gradle's JUnit plugin. This class and its annotated consumption can be safely removed.
> Steps taken to verify the above:
> * Check out {{0f8cd8e1ef12ab864f5b2ae7a0556156b249b432}} (fix for GEODE-4122)
> * Remove runner from some test using it, e.g., {{GatewayReceiverXmlParsingValidationsJUnitTest}}
> * Clean environment ({{./gradlew clean}} and delete {{out}} and {{build}} directories)
> * Run target test, e.g., via {{./gradlew :geode-core:integrationTest --tests GatewayReceiverXmlParsingValidationsJUnitTest}}
> * Inspect test report
> * Note that no test was executed.
> * Check out {{develop}}
> * Clean environment again.
> * Remove runner from the same test
> * Execute test via same gradle command
> * Inspect test report
> * Note that the test was indeed run
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)