Simplify split transition semantics: noops now return the input build options.

This results in less special logic in the implementation and a simpler API.

PiperOrigin-RevId: 197772283

1 files changed, 3 insertions, 3 deletions

diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsWithTrimmedConfigurationsTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsWithTrimmedConfigurationsTest.java
index 1fc5f6222a..3f5f37c02b 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsWithTrimmedConfigurationsTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsWithTrimmedConfigurationsTest.java
@@ -62,10 +62,10 @@ public class ConfigurationsForTargetsWithTrimmedConfigurationsTest
     return super.defaultFlags().with(Flag.TRIMMED_CONFIGURATIONS);
   }
 
-  private static class EmptySplitTransition implements SplitTransition {
+  private static class NoopSplitTransition implements SplitTransition {
     @Override
     public List<BuildOptions> split(BuildOptions buildOptions) {
-      return ImmutableList.of();
+      return ImmutableList.of(buildOptions);
     }
   }
 
@@ -111,7 +111,7 @@ public class ConfigurationsForTargetsWithTrimmedConfigurationsTest
           "empty_split",
           attr("with_empty_transition", LABEL)
               .allowedFileTypes(FileTypeSet.ANY_FILE)
-              .cfg(new EmptySplitTransition()));
+              .cfg(new NoopSplitTransition()));
 
   /** Rule with a split transition on an attribute. */
   private static final MockRule ATTRIBUTE_TRANSITION_RULE = () ->