diff --git a/java/dagger/internal/codegen/compileroption/ProcessingEnvironmentCompilerOptions.java b/java/dagger/internal/codegen/compileroption/ProcessingEnvironmentCompilerOptions.java index 0fc9caad8d7..81380055a91 100644 --- a/java/dagger/internal/codegen/compileroption/ProcessingEnvironmentCompilerOptions.java +++ b/java/dagger/internal/codegen/compileroption/ProcessingEnvironmentCompilerOptions.java @@ -404,7 +404,7 @@ public ImmutableList aliases() { * How to report that an explicit binding in a subcomponent conflicts with an {@code @Inject} * constructor used in an ancestor component. */ - EXPLICIT_BINDING_CONFLICTS_WITH_INJECT(WARNING, ERROR, NONE), + EXPLICIT_BINDING_CONFLICTS_WITH_INJECT(ERROR, WARNING, NONE), ; final ValidationType defaultType; diff --git a/javatests/dagger/internal/codegen/DuplicateBindingsValidationTest.java b/javatests/dagger/internal/codegen/DuplicateBindingsValidationTest.java index b45357cd6d6..c352a2810e0 100644 --- a/javatests/dagger/internal/codegen/DuplicateBindingsValidationTest.java +++ b/javatests/dagger/internal/codegen/DuplicateBindingsValidationTest.java @@ -1052,9 +1052,8 @@ public void childProvidesConflictsWithParentInjects() { CompilerTests.daggerCompiler(foo, injected1, injected2, provided1, provided2) .compile( subject -> { - subject.hasErrorCount(0); - subject.hasWarningCount(1); - subject.hasWarningContaining( + subject.hasErrorCount(1); + subject.hasErrorContaining( message( "Foo is bound multiple times:", " @Inject Foo(Set) [Injected1]",