Fix rule type propagation

uber · Jun 23, 2017 · 04a7a28 · 04a7a28
1 parent 03e1038
commit 04a7a28
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 3 deletions.
diff --git a/buildSrc/src/main/groovy/com/uber/okbuck/composer/java/JavaLibraryRuleComposer.groovy b/buildSrc/src/main/groovy/com/uber/okbuck/composer/java/JavaLibraryRuleComposer.groovy
@@ -37,6 +37,7 @@ final class JavaLibraryRuleComposer extends JvmBuckRuleComposer {
         }
 
         new JavaLibraryRule(
+                ruleType,
                 src(target),
                 ["PUBLIC"],
                 deps,

diff --git a/buildSrc/src/main/groovy/com/uber/okbuck/composer/java/JavaTestRuleComposer.groovy b/buildSrc/src/main/groovy/com/uber/okbuck/composer/java/JavaTestRuleComposer.groovy
@@ -33,6 +33,7 @@ final class JavaTestRuleComposer extends JvmBuckRuleComposer {
         }
 
         new JavaTestRule(
+                ruleType,
                 test(target),
                 ["PUBLIC"],
                 deps,

diff --git a/buildSrc/src/main/groovy/com/uber/okbuck/rule/java/JavaLibraryRule.groovy b/buildSrc/src/main/groovy/com/uber/okbuck/rule/java/JavaLibraryRule.groovy
@@ -6,6 +6,7 @@ import com.uber.okbuck.core.model.jvm.TestOptions
 class JavaLibraryRule extends JavaRule {
 
     JavaLibraryRule(
+            RuleType ruleType,
             String name,
             List<String> visibility,
             List<String> deps,
@@ -19,8 +20,7 @@ class JavaLibraryRule extends JavaRule {
             List<String> postprocessClassesCommands,
             List<String> options,
             List<String> testTargets,
-            Set<String> extraOpts,
-            RuleType ruleType = RuleType.JAVA_LIBRARY) {
+            Set<String> extraOpts) {
 
         super(
                 ruleType,

diff --git a/buildSrc/src/main/groovy/com/uber/okbuck/rule/java/JavaTestRule.groovy b/buildSrc/src/main/groovy/com/uber/okbuck/rule/java/JavaTestRule.groovy
@@ -8,6 +8,7 @@ class JavaTestRule extends JavaRule {
     private static final List<String> JAVA_TEST_LABELS = ['unit', 'java']
 
     JavaTestRule(
+            RuleType ruleType,
             String name,
             List<String> visibility,
             List<String> deps,
@@ -22,7 +23,6 @@ class JavaTestRule extends JavaRule {
             List<String> options,
             TestOptions testOptions,
             Set<String> extraOpts,
-            RuleType ruleType = RuleType.JAVA_TEST,
             List<String> testLabels = JAVA_TEST_LABELS) {
         super(
                 ruleType,