Merge changes Id885bde0,I6b9124e7,Ifdb65a25,I240cfb73
* changes:
Add method to test API to retrieve raw file content of code owner config
CodeOwnerResolver: Add null check for email in isEmailDomainAllowed
Do not reject '*' as email if allowed email domains are configured
CodeOwnerConfigValidatorIT: Test that code owner with allowed email domain can be added
diff --git a/java/com/google/gerrit/plugins/codeowners/restapi/CheckCodeOwnerConfigFiles.java b/java/com/google/gerrit/plugins/codeowners/restapi/CheckCodeOwnerConfigFiles.java
index fac3c36..e18ddb3 100644
--- a/java/com/google/gerrit/plugins/codeowners/restapi/CheckCodeOwnerConfigFiles.java
+++ b/java/com/google/gerrit/plugins/codeowners/restapi/CheckCodeOwnerConfigFiles.java
@@ -175,6 +175,10 @@
public static Optional<ConsistencyProblemInfo> createConsistencyProblemInfo(
CommitValidationMessage commitValidationMessage) {
switch (commitValidationMessage.getType()) {
+ case FATAL:
+ return Optional.of(
+ new ConsistencyProblemInfo(
+ ConsistencyProblemInfo.Status.FATAL, commitValidationMessage.getMessage()));
case ERROR:
return Optional.of(
new ConsistencyProblemInfo(
