Skip to content

Commit

Permalink
exclude single test that uses an AST from the same project
Browse files Browse the repository at this point in the history
- greclipse doesn't support having the AST in the same project as the test
- previously all tests were disabled by removing the test directory from source folder
- now only exclude the test that fails to compile in Eclipse
  • Loading branch information
lhotari committed May 29, 2014
1 parent bb09202 commit 899d095
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions grails-plugin-testing/build.gradle
Expand Up @@ -32,8 +32,8 @@ eclipse {
classpath {
file {
whenMerged { classpath ->
// greclipse doesn't support ASTs and tests in same project
classpath.entries.removeAll { entry -> entry.kind == 'src' && entry.path == "src/test/groovy" }
// greclipse doesn't support ASTs and tests in same project, exclude those tests
classpath.entries.find { entry -> entry.kind == 'src' && entry.path == "src/test/groovy" }?.excludes=["grails/test/mixin/MetaClassCleanupSpec.groovy"]
}
}
}
Expand Down

2 comments on commit 899d095

@osscontributor
Copy link
Member

Choose a reason for hiding this comment

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

This change is problematic because it results in 2 separate classpath entries for src/test/groovy in the generated .classpath. When the project is imported into GGTS the project will not build until one of them is removed.

@osscontributor
Copy link
Member

Choose a reason for hiding this comment

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

Hmmm... Now I am not sure about that. It was generating this for me...

    <classpathentry excluding="grails/test/mixin/MetaClassCleanupSpec.groovy" kind="src" path="src/test/groovy"/>
    <classpathentry kind="src" path="src/test/groovy"/>

But now it looks like it isn't. I am not sure what happened there.

Please sign in to comment.