From a3889d63d2e61f6605e2e2a9bd0a0dd3ade797a0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christoph=20L=C3=A4ubrich?= <mail@laeubi-soft.de>
Date: Thu, 25 Aug 2022 05:28:15 +0200
Subject: [PATCH] Apply suggestions from code review

Co-authored-by: BJ Hargrave <bj@hargrave.dev>
---
 .../test/test/model/EETest.java               | 22 +++++++------------
 .../src/aQute/bnd/build/model/EE.java         |  2 +-
 2 files changed, 9 insertions(+), 15 deletions(-)

diff --git a/biz.aQute.bndlib.tests/test/test/model/EETest.java b/biz.aQute.bndlib.tests/test/test/model/EETest.java
index c9b251807a6..0aa013799d1 100644
--- a/biz.aQute.bndlib.tests/test/test/model/EETest.java
+++ b/biz.aQute.bndlib.tests/test/test/model/EETest.java
@@ -92,9 +92,9 @@ public void checkEEHasCompatible(EE ee) throws Exception {
 		assertThat(compatible).isNotEmpty();
 	}
 
-	@ParameterizedTest(name = "Validate valid release target foe EEs exist for {arguments}")
+	@ParameterizedTest(name = "Validate release target for {arguments}")
 	@ArgumentsSource(EEsArgumentsProvider.class)
-	@DisplayName("Validate each EEs has valid release target")
+	@DisplayName("Validate release target for each EE")
 	public void checkEEHasValidRelease(EE ee) throws Exception {
 		switch (ee) {
 			case OSGI_Minimum_1_0 :
@@ -105,30 +105,24 @@ public void checkEEHasValidRelease(EE ee) throws Exception {
 			case J2SE_1_4 :
 			case J2SE_1_5 :
 			case JRE_1_1 :
-				assertTrue(ee.getReleaseTarget()
-					.isEmpty());
+				assertThat(ee.getReleaseTarget()).isEmpty();
 				break;
 			case JavaSE_1_6 :
-				assertEquals(6, ee.getReleaseTarget()
-					.getAsInt());
+				assertThat(ee.getReleaseTarget()).hasValue(6);
 				break;
 			case JavaSE_1_7 :
-				assertEquals(7, ee.getReleaseTarget()
-					.getAsInt());
+				assertThat(ee.getReleaseTarget()).hasValue(7);
 				break;
 			case JavaSE_1_8 :
 			case JavaSE_compact1_1_8 :
 			case JavaSE_compact2_1_8 :
 			case JavaSE_compact3_1_8 :
-				assertEquals(8, ee.getReleaseTarget()
-					.getAsInt());
+				assertThat(ee.getReleaseTarget()).hasValue(8);
 				break;
 
 			default :
-				assertEquals(ee.getCapabilityVersion()
-					.getMajor(),
-					ee.getReleaseTarget()
-						.getAsInt());
+				assertThat(ee.getReleaseTarget()).hasValue(ee.getCapabilityVersion()
+					.getMajor());
 				break;
 		}
 	}
diff --git a/biz.aQute.bndlib/src/aQute/bnd/build/model/EE.java b/biz.aQute.bndlib/src/aQute/bnd/build/model/EE.java
index ee67090b80f..4f94fa4f4c0 100644
--- a/biz.aQute.bndlib/src/aQute/bnd/build/model/EE.java
+++ b/biz.aQute.bndlib/src/aQute/bnd/build/model/EE.java
@@ -152,7 +152,7 @@ public Version getCapabilityVersion() {
 	public OptionalInt getReleaseTarget() {
 		Version version = getCapabilityVersion();
 		int major = version.getMajor();
-		if (major > 8) {
+		if (major > 1) {
 			return OptionalInt.of(major);
 		}
 		if (major == 1 && version.getMinor() > 5) {