[CALCITE-4821] Move utility test classes into calcite-testkit and unpublish -test.jar

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.28.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Vladimir Sitnikov
• Created: 03/Oct/21 10:36
• Updated: 19/Apr/22 19:03
• Resolved: 05/Oct/21 09:48

Description

-test.jar make it hard to support build configuration, and it blocks migration to Gradle 7.
As we move the commonly used classes to its own -testkit module, it would make it easier to reuse test helpers, it would make easier to recognize the Test API vs @Test code, and it would make certain classes smaller. E.g. CatchAllSchema is currently located in ReflectiveSchemaTest, and if we move it to a top-level class, then ReflectiveSchemaTest would be smaller.

Issue Links

• is depended upon by: CALCITE-4829 Bump Gradle to 7.2 and test with Java 17 at GitHub Actions
• relates to: CALCITE-5098 Migrate Avatica to Gradle 7.4.2, and support Java 16 and 17
• links to: GitHub Pull Request #2558