Skip to content

Commit

Permalink
Move tests from okhttp-tests/src/test/java to okhttp/src/test/java
Browse files Browse the repository at this point in the history
The previous module structure was necessary because Maven wouldn't allow
mockwebserver to depend on okhttp/src and for okhttp/test to depend on
mockwebserver. With Gradle this constraint is lifted and we can fold
everything into a single module.

I'm in a rush to do this now because it's necessary for Kotlin 'internal'
tests to be in the same module.
  • Loading branch information
squarejesse committed Mar 26, 2019
1 parent b650c4e commit 9837fea
Show file tree
Hide file tree
Showing 95 changed files with 8 additions and 14 deletions.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
13 changes: 0 additions & 13 deletions okhttp-tests/build.gradle

This file was deleted.

8 changes: 8 additions & 0 deletions okhttp/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,16 @@ dependencies {
compileOnly deps.jsr305
compileOnly deps.animalSniffer

testImplementation deps.okio
testImplementation project(':okhttp-testing-support')
testImplementation project(':okhttp-tls')
testImplementation project(':okhttp-urlconnection')
testImplementation project(':mockwebserver')
testImplementation project(':okhttp-logging-interceptor')
testImplementation deps.conscrypt
testImplementation deps.junit
testImplementation deps.assertj
testCompileOnly deps.jsr305
}

apply plugin: 'me.champeau.gradle.japicmp'
Expand Down
File renamed without changes.
File renamed without changes.
1 change: 0 additions & 1 deletion settings.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ include ':okhttp-hpacktests'
include ':okhttp-logging-interceptor'
include ':okhttp-sse'
include ':okhttp-testing-support'
include ':okhttp-tests'
include ':okhttp-tls'
include ':okhttp-urlconnection'
include ':samples:crawler'
Expand Down

0 comments on commit 9837fea

Please sign in to comment.