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;
                         }