Require `TransitionFactory` subclasses to implement `transitionType`.
This guarantees that it is accurate, the default of `ANY` was too broad.
Part of #22248.
PiperOrigin-RevId: 632513554
Change-Id: I90b66b45b640ba84b658f23cb56a9016dd853310
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/CircularDependencyTest.java b/src/test/java/com/google/devtools/build/lib/analysis/CircularDependencyTest.java
index 7866b61..1fe61d8 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/CircularDependencyTest.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/CircularDependencyTest.java
@@ -306,7 +306,7 @@
.mandatory()
.allowedFileTypes()
.cfg(
- new TransitionFactory<AttributeTransitionData>() {
+ new TransitionFactory<>() {
@Override
public SplitTransition create(AttributeTransitionData data) {
return new SplitTransition() {
@@ -333,6 +333,11 @@
}
@Override
+ public TransitionType transitionType() {
+ return TransitionType.ATTRIBUTE;
+ }
+
+ @Override
public boolean isSplit() {
return true;
}