Dan Smith created GEODE-833:
-------------------------------
Summary: Test jars like junit are part of the compile dependencies
for gemfire-core
Key: GEODE-833
URL: https://issues.apache.org/jira/browse/GEODE-833
Project: Geode
Issue Type: Bug
Reporter: Dan Smith
Tests that were checked into gemfire-core/src/main with junit dependencies were
still compiling.
It looks like junit is getting added to the src compile classpath because of
this dependency in gemfire-core/build.gradle
provided project(path: ':gemfire-junit', configuration: 'testOutput')
I think this should be testCompile.
{noformat}
provided - a dependency that is provided externally at runtime
+--- com.google.guava:guava:15.0
+--- org.apache.hadoop:hadoop-common:2.4.1
+--- org.apache.hadoop:hadoop-annotations:2.4.1
+--- org.apache.hadoop:hadoop-hdfs:2.4.1
+--- org.apache.hadoop:hadoop-mapreduce-client-core:2.4.1
+--- org.springframework.data:spring-data-gemfire:1.5.1.RELEASE
\--- project :gemfire-junit
+--- org.springframework:spring-aop:3.2.12.RELEASE
+--- org.springframework:spring-beans:3.2.12.RELEASE
+--- org.springframework:spring-context:3.2.12.RELEASE
+--- org.springframework:spring-context-support:3.2.12.RELEASE
+--- org.springframework:spring-core:3.2.12.RELEASE
+--- org.springframework:spring-expression:3.2.12.RELEASE
+--- org.springframework:spring-web:3.2.12.RELEASE
+--- org.springframework:spring-webmvc:3.2.12.RELEASE
+--- com.github.stephenc.findbugs:findbugs-annotations:1.3.9-1
+--- com.github.stefanbirkner:system-rules:1.15.0
+--- com.jayway.awaitility:awaitility:1.7.0
+--- edu.umd.cs.mtc:multithreadedtc:1.01
+--- eu.codearte.catch-exception:catch-exception:1.4.4
+--- eu.codearte.catch-exception:catch-throwable:1.4.4
+--- junit:junit:4.12
+--- org.assertj:assertj-core:3.2.0
+--- org.mockito:mockito-core:1.10.19
+--- org.hamcrest:hamcrest-all:1.3
+--- org.jmock:jmock:2.8.1
+--- org.jmock:jmock-junit4:2.8.1
+--- org.jmock:jmock-legacy:2.8.1
\--- pl.pragmatists:JUnitParams:1.0.4
{noformat}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
