[ 
https://issues.apache.org/jira/browse/BEAM-4539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Kenneth Knowles resolved BEAM-4539.
-----------------------------------
       Resolution: Invalid
    Fix Version/s: Not applicable

Actually src/test/java is a path for _test suites_ not _test utilities_. These 
are test utilities that are meant to be used by other modules. One workaround 
is to make them optional dependencies so they are not pulled in, assuming the 
user will depend on them in their own test suite.

But since hamcrest and junit are both trivially tiny and self-contained there's 
no point.

> TestBigQuery and TestBigQueryOptions are in the wrong path
> ----------------------------------------------------------
>
>                 Key: BEAM-4539
>                 URL: https://issues.apache.org/jira/browse/BEAM-4539
>             Project: Beam
>          Issue Type: Bug
>          Components: io-java-gcp
>            Reporter: Ismaël Mejía
>            Assignee: Anton Kedin
>            Priority: Major
>             Fix For: Not applicable
>
>
> {{TestBigQuery}} and {{TestBigQueryOptions}} are currently in the 
> src/main/java dir but are used only for tests so they should be part of the 
> test artifact in src/test/java. As a side effect this makes the main GCPIO 
> jar depend on both JUnit and Hamcrest which should be avoided..



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to