use AutoValue for ExecGroup
PiperOrigin-RevId: 316150023
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewForTesting.java b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewForTesting.java
index 6fc88cd..950249f 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewForTesting.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewForTesting.java
@@ -511,7 +511,7 @@
execGroup.getKey(),
ToolchainContextKey.key()
.configurationKey(BuildConfigurationValue.key(targetConfig))
- .requiredToolchainTypeLabels(execGroup.getValue().getRequiredToolchains())
+ .requiredToolchainTypeLabels(execGroup.getValue().requiredToolchains())
.build());
}
String targetUnloadedToolchainContextKey = "target-unloaded-toolchain-context";
diff --git a/src/test/java/com/google/devtools/build/lib/packages/RuleClassBuilderTest.java b/src/test/java/com/google/devtools/build/lib/packages/RuleClassBuilderTest.java
index 9cd63ce..faa0cc2 100644
--- a/src/test/java/com/google/devtools/build/lib/packages/RuleClassBuilderTest.java
+++ b/src/test/java/com/google/devtools/build/lib/packages/RuleClassBuilderTest.java
@@ -197,8 +197,8 @@
Label mockToolchainType = Label.parseAbsoluteUnchecked("//mock_toolchain_type");
Label mockConstraint = Label.parseAbsoluteUnchecked("//mock_constraint");
ExecGroup parentGroup =
- new ExecGroup(ImmutableSet.of(mockToolchainType), ImmutableSet.of(mockConstraint));
- ExecGroup childGroup = new ExecGroup(ImmutableSet.of(), ImmutableSet.of());
+ ExecGroup.create(ImmutableSet.of(mockToolchainType), ImmutableSet.of(mockConstraint));
+ ExecGroup childGroup = ExecGroup.create(ImmutableSet.of(), ImmutableSet.of());
RuleClass parent =
new RuleClass.Builder("$parent", RuleClassType.ABSTRACT, false)
.add(attr("tags", STRING_LIST))
@@ -220,14 +220,16 @@
new RuleClass.Builder("ruleA", RuleClassType.NORMAL, false)
.factory(DUMMY_CONFIGURED_TARGET_FACTORY)
.addExecGroups(
- ImmutableMap.of("blueberry", new ExecGroup(ImmutableSet.of(), ImmutableSet.of())))
+ ImmutableMap.of(
+ "blueberry", ExecGroup.create(ImmutableSet.of(), ImmutableSet.of())))
.add(attr("tags", STRING_LIST))
.build();
RuleClass b =
new RuleClass.Builder("ruleB", RuleClassType.NORMAL, false)
.factory(DUMMY_CONFIGURED_TARGET_FACTORY)
.addExecGroups(
- ImmutableMap.of("blueberry", new ExecGroup(ImmutableSet.of(), ImmutableSet.of())))
+ ImmutableMap.of(
+ "blueberry", ExecGroup.create(ImmutableSet.of(), ImmutableSet.of())))
.add(attr("tags", STRING_LIST))
.build();
IllegalArgumentException e =
diff --git a/src/test/java/com/google/devtools/build/lib/packages/RuleClassTest.java b/src/test/java/com/google/devtools/build/lib/packages/RuleClassTest.java
index 8115483..fc9477f 100644
--- a/src/test/java/com/google/devtools/build/lib/packages/RuleClassTest.java
+++ b/src/test/java/com/google/devtools/build/lib/packages/RuleClassTest.java
@@ -1145,14 +1145,14 @@
ruleClassBuilder.addExecGroups(
ImmutableMap.of(
- "cherry", new ExecGroup(ImmutableSet.of(toolchain), ImmutableSet.of(constraint))));
+ "cherry", ExecGroup.create(ImmutableSet.of(toolchain), ImmutableSet.of(constraint))));
RuleClass ruleClass = ruleClassBuilder.build();
assertThat(ruleClass.getExecGroups()).hasSize(1);
- assertThat(ruleClass.getExecGroups().get("cherry").getRequiredToolchains())
+ assertThat(ruleClass.getExecGroups().get("cherry").requiredToolchains())
.containsExactly(toolchain);
- assertThat(ruleClass.getExecGroups().get("cherry").getExecutionPlatformConstraints())
+ assertThat(ruleClass.getExecGroups().get("cherry").execCompatibleWith())
.containsExactly(constraint);
}
diff --git a/src/test/java/com/google/devtools/build/lib/skylark/StarlarkRuleClassFunctionsTest.java b/src/test/java/com/google/devtools/build/lib/skylark/StarlarkRuleClassFunctionsTest.java
index ea1d36e..4e871d5 100644
--- a/src/test/java/com/google/devtools/build/lib/skylark/StarlarkRuleClassFunctionsTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skylark/StarlarkRuleClassFunctionsTest.java
@@ -1821,10 +1821,10 @@
")");
RuleClass plum = ((StarlarkRuleFunction) ev.lookup("plum")).getRuleClass();
assertThat(plum.getRequiredToolchains()).isEmpty();
- assertThat(plum.getExecGroups().get("group").getRequiredToolchains())
+ assertThat(plum.getExecGroups().get("group").requiredToolchains())
.containsExactly(makeLabel("//test:my_toolchain_type"));
assertThat(plum.getExecutionPlatformConstraints()).isEmpty();
- assertThat(plum.getExecGroups().get("group").getExecutionPlatformConstraints())
+ assertThat(plum.getExecGroups().get("group").execCompatibleWith())
.containsExactly(makeLabel("//constraint:cv1"), makeLabel("//constraint:cv2"));
}
@@ -1874,9 +1874,8 @@
" exec_compatible_with=['//constraint:cv1', '//constraint:cv2'],",
")");
ExecGroup group = ((ExecGroup) ev.lookup("group"));
- assertThat(group.getRequiredToolchains())
- .containsExactly(makeLabel("//test:my_toolchain_type"));
- assertThat(group.getExecutionPlatformConstraints())
+ assertThat(group.requiredToolchains()).containsExactly(makeLabel("//test:my_toolchain_type"));
+ assertThat(group.execCompatibleWith())
.containsExactly(makeLabel("//constraint:cv1"), makeLabel("//constraint:cv2"));
}
}
