Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

split the single gradle project to subprojects #1348

Merged
merged 4 commits into from Dec 12, 2020
Merged

Conversation

asolntsev
Copy link
Member

@asolntsev asolntsev commented Dec 9, 2020

  • extract junit4 subproject
  • extract testng subproject
  • extract grid subproject

Proposed changes

extract JUnit4-related code to modules/junit4 and TestNG-related code to modules/testng.
The goal is to avoid mixing junit.jar and testng.jar in common classpath. For example, AssertJ assumeThat works incorrectly if encounters both jars in classpath.

@@ -1,3 +1,7 @@
org.gradle.parallel=true
org.gradle.configureondemand=true
org.gradle.jvmargs=-Xmx256m -XX:+HeapDumpOnOutOfMemoryError -Dfile.encoding=UTF-8

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you're added org.gradle.caching=true can you please also add org.gradle.vfs.watch=true configuration?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

task libsTest(type: Sync) {
from configurations.testRuntimeClasspath - configurations.compileClasspath
into "$buildDir/lib-test"
subprojects {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can use or just think about it - composite builds

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, I will try composite builds.

@rosolko rosolko self-requested a review December 10, 2020 07:05
Copy link
Collaborator

@rosolko rosolko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leave some comments, check them please.

@ghost
Copy link

ghost commented Dec 10, 2020

Congratulations 🎉. DeepCode analyzed your code in 0.74 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

+ extract `junit4` subproject
+ extract `testng` subproject
+ extract `grid` subproject
for some reason, File.exists() != File.getAbsoluteFile().exists() ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants