[MNG-8142] Hidden bug: JDK profile activator throw NumberFormatEx (#1557

) If property `java.version` is in unexpected format, the activator throws `NumberFormatEx`, that in turn, is caught and reported by `DefaultProfileSelector` w/o any cause. These should be cleanly reported instead: report that `java.version` property is in "unexpected format", and also report why was there are failure to evaluate a property activation. Note 1: Maven allows `-Djava.version` override (!!!), this is exactly what IT MNG-3746 does, but the `NumberFormatEx` went unnoticed, was swallowed, no cause reported. Note 2: This bug was revealed by #1555 as it reported the issue, and later "asserted error free log" which was not error-free. Hence, this bug was simply revealed by improved logging on unrelated issue. --- https://issues.apache.org/jira/browse/MNG-8142
apache · Jun 6, 2024 · 200dc02 · 200dc02
1 parent 8650720
commit 200dc02
Show file tree

Hide file tree

Showing 4 changed files with 107 additions and 2 deletions.
diff --git a/maven-model-builder/src/main/java/org/apache/maven/model/profile/DefaultProfileSelector.java b/maven-model-builder/src/main/java/org/apache/maven/model/profile/DefaultProfileSelector.java
@@ -103,7 103,8 @@ private boolean isActive(Profile profile, ProfileActivationContext context, Mode
  }
  } catch (RuntimeException e) {
  problems.add(new ModelProblemCollectorRequest(Severity.ERROR, Version.BASE)
- .setMessage("Failed to determine activation for profile " profile.getId())
  .setMessage(
  "Failed to determine activation for profile " profile.getId() ": " e.getMessage())
  .setLocation(profile.getLocation(""))
  .setException(e));
  return false;

diff --git a/...r/src/main/java/org/apache/maven/model/profile/activation/JdkVersionProfileActivator.java b/...r/src/main/java/org/apache/maven/model/profile/activation/JdkVersionProfileActivator.java
@@ -74,7 74,16 @@ public boolean isActive(Profile profile, ProfileActivationContext context, Model
  if (jdk.startsWith("!")) {
  return !version.startsWith(jdk.substring(1));
  } else if (isRange(jdk)) {
- return isInRange(version, getRange(jdk));
  try {
  return isInRange(version, getRange(jdk));
  } catch (NumberFormatException e) {
  problems.add(new ModelProblemCollectorRequest(Severity.WARNING, Version.BASE)
  .setMessage("Failed to determine JDK activation for profile " profile.getId()
  " due invalid JDK version: '" version "'")
  .setLocation(profile.getLocation(""))
  .setException(e));
  return false;
  }
  } else {
  return version.startsWith(jdk);
  }

diff --git a/...odel-builder/src/test/java/org/apache/maven/model/profile/DefaultProfileSelectorTest.java b/...odel-builder/src/test/java/org/apache/maven/model/profile/DefaultProfileSelectorTest.java
@@ -0,0 1,72 @@
 /*
  * Licensed to the Apache Software Foundation (ASF) under one
  * or more contributor license agreements. See the NOTICE file
  * distributed with this work for additional information
  * regarding copyright ownership. The ASF licenses this file
  * to you under the Apache License, Version 2.0 (the
  * "License"); you may not use this file except in compliance
  * with the License. You may obtain a copy of the License at
  *
  * http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing,
  * software distributed under the License is distributed on an
  * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
  * KIND, either express or implied. See the License for the
  * specific language governing permissions and limitations
  * under the License.
  */
 package org.apache.maven.model.profile;
 
 import java.util.Collections;
 import java.util.List;
 
 import org.apache.maven.model.Activation;
 import org.apache.maven.model.Profile;
 import org.apache.maven.model.building.ModelProblemCollector;
 import org.apache.maven.model.building.SimpleProblemCollector;
 import org.apache.maven.model.profile.activation.ProfileActivator;
 import org.junit.Test;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
 
 /**
  * Tests {@link DefaultProfileSelector}.
  */
 public class DefaultProfileSelectorTest {
  private Profile newProfile(String id) {
  Activation activation = new Activation();
  Profile profile = new Profile();
  profile.setId(id);
  profile.setActivation(activation);
  return profile;
  }
 
  @Test
  public void testThrowingActivator() {
  DefaultProfileSelector selector = new DefaultProfileSelector();
  selector.addProfileActivator(new ProfileActivator() {
  @Override
  public boolean isActive(Profile profile, ProfileActivationContext context, ModelProblemCollector problems) {
  throw new RuntimeException("BOOM");
  }
 
  @Override
  public boolean presentInConfig(
  Profile profile, ProfileActivationContext context, ModelProblemCollector problems) {
  return true;
  }
  });
 
  List<Profile> profiles = Collections.singletonList(newProfile("one"));
  DefaultProfileActivationContext context = new DefaultProfileActivationContext();
  SimpleProblemCollector problems = new SimpleProblemCollector();
  List<Profile> active = selector.getActiveProfiles(profiles, context, problems);
  assertTrue(active.isEmpty());
  assertEquals(1, problems.getErrors().size());
  assertEquals(
  "Failed to determine activation for profile one: BOOM",
  problems.getErrors().get(0));
  }
 }
diff --git a/...c/test/java/org/apache/maven/model/profile/activation/JdkVersionProfileActivatorTest.java b/...c/test/java/org/apache/maven/model/profile/activation/JdkVersionProfileActivatorTest.java
@@ -22,6 22,8 @@
 
 import org.apache.maven.model.Activation;
 import org.apache.maven.model.Profile;
 import org.apache.maven.model.building.SimpleProblemCollector;
 import org.apache.maven.model.profile.ProfileActivationContext;
 
 /**
  * Tests {@link JdkVersionProfileActivator}.
@@ -161,4 163,25 @@ public void testVersionRangeExclusiveUpperBound() throws Exception {
  assertActivation(false, profile, newContext(null, newProperties("1.6.0_09")));
  assertActivation(false, profile, newContext(null, newProperties("1.6.0_09-b03")));
  }
 
  public void testRubbishJavaVersion() {
  Profile profile = newProfile("[1.8,)");
 
  assertActivationWithProblems(profile, newContext(null, newProperties("Pūteketeke")), "invalid JDK version");
  assertActivationWithProblems(profile, newContext(null, newProperties("rubbish")), "invalid JDK version");
  assertActivationWithProblems(profile, newContext(null, newProperties("1.a.0_09")), "invalid JDK version");
  assertActivationWithProblems(profile, newContext(null, newProperties("1.a.2.b")), "invalid JDK version");
  }
 
  private void assertActivationWithProblems(
  Profile profile, ProfileActivationContext context, String warningContains) {
  SimpleProblemCollector problems = new SimpleProblemCollector();
 
  assertEquals(false, activator.isActive(profile, context, problems));
 
  assertEquals(problems.getErrors().toString(), 0, problems.getErrors().size());
  assertEquals(
  problems.getWarnings().toString(), 1, problems.getWarnings().size());
  assertTrue(problems.getWarnings().get(0), problems.getWarnings().get(0).contains(warningContains));
  }
 }