From 7b050af44ac99581e237072d82f22966a802a873 Mon Sep 17 00:00:00 2001 From: Bjørn Christian Seime Date: Fri, 29 Jul 2022 13:59:07 +0200 Subject: Convert bundle-plugin to junit5 --- .../container/plugin/osgi/ImportPackageTest.java | 40 ++++++++++------------ 1 file changed, 18 insertions(+), 22 deletions(-) (limited to 'bundle-plugin/src/test/java/com/yahoo/container/plugin/osgi/ImportPackageTest.java') diff --git a/bundle-plugin/src/test/java/com/yahoo/container/plugin/osgi/ImportPackageTest.java b/bundle-plugin/src/test/java/com/yahoo/container/plugin/osgi/ImportPackageTest.java index 29f2be838a1..aa74746bfff 100644 --- a/bundle-plugin/src/test/java/com/yahoo/container/plugin/osgi/ImportPackageTest.java +++ b/bundle-plugin/src/test/java/com/yahoo/container/plugin/osgi/ImportPackageTest.java @@ -4,9 +4,7 @@ package com.yahoo.container.plugin.osgi; import com.yahoo.container.plugin.osgi.ExportPackages.Export; import com.yahoo.container.plugin.osgi.ExportPackages.Parameter; import com.yahoo.container.plugin.osgi.ImportPackages.Import; -import org.junit.Rule; -import org.junit.Test; -import org.junit.rules.ExpectedException; +import org.junit.jupiter.api.Test; import java.util.Collections; import java.util.HashSet; @@ -15,8 +13,9 @@ import java.util.Map; import java.util.Optional; import java.util.Set; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertTrue; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertThrows; +import static org.junit.jupiter.api.Assertions.assertTrue; /** * @author Tony Vaagenes @@ -34,37 +33,37 @@ public class ImportPackageTest { } @Test - public void require_that_non_implemented_import_with_matching_export_is_included() { + void require_that_non_implemented_import_with_matching_export_is_included() { Set imports = calculateImports(referencedPackages, Set.of(), exports); assertEquals(1, imports.stream().filter(imp -> imp.packageName().equals(PACKAGE_NAME) && imp.version().isEmpty()).count()); } @Test - public void require_that_non_implemented_import_without_matching_export_is_excluded() { + void require_that_non_implemented_import_without_matching_export_is_excluded() { Set imports = calculateImports(referencedPackages, Set.of(), Map.of()); assertTrue(imports.isEmpty()); } @Test - public void require_that_implemented_import_with_matching_export_is_excluded() { + void require_that_implemented_import_with_matching_export_is_excluded() { Set imports = calculateImports(referencedPackages, referencedPackages, exports); assertTrue(imports.isEmpty()); } @Test - public void require_that_version_is_included() { + void require_that_version_is_included() { Set imports = calculateImports(referencedPackages, Set.of(), exportsWithVersion); assertEquals(1, imports.stream().filter(imp -> imp.packageName().equals(PACKAGE_NAME) && imp.version().equals("1.3")).count()); } @Test - public void require_that_all_versions_up_to_the_next_major_version_is_in_range() { + void require_that_all_versions_up_to_the_next_major_version_is_in_range() { assertEquals("[1.2,2)", new Import("foo", Optional.of("1.2")).importVersionRange().get()); } // TODO: Detecting guava packages should be based on bundle-symbolicName, not package name. @Test - public void require_that_for_guava_all_future_major_versions_are_in_range() { + void require_that_for_guava_all_future_major_versions_are_in_range() { Optional rangeWithInfiniteUpperLimit = Optional.of("[18.1," + ImportPackages.INFINITE_VERSION + ")"); assertEquals(rangeWithInfiniteUpperLimit, new Import("com.google.common", Optional.of("18.1")).importVersionRange()); assertEquals(rangeWithInfiniteUpperLimit, new Import("com.google.common.foo", Optional.of("18.1")).importVersionRange()); @@ -72,32 +71,29 @@ public class ImportPackageTest { } @Test - public void require_that_none_version_gives_non_version_range() { + void require_that_none_version_gives_non_version_range() { assertTrue(new Import("foo", Optional.empty()).importVersionRange().isEmpty()); } - @SuppressWarnings("deprecation") - @Rule - public ExpectedException expectedException = ExpectedException.none(); - @Test - public void require_that_exception_is_thrown_when_major_component_is_non_numeric() { - expectedException.expect(IllegalArgumentException.class); - new Import("foo", Optional.of("1notValid.2")); + void require_that_exception_is_thrown_when_major_component_is_non_numeric() { + assertThrows(IllegalArgumentException.class, () -> { + new Import("foo", Optional.of("1notValid.2")); + }); } @Test - public void require_that_osgi_import_supports_missing_version() { + void require_that_osgi_import_supports_missing_version() { assertEquals("com.yahoo.exported", new Import("com.yahoo.exported", Optional.empty()).asOsgiImport()); } @Test - public void require_that_osgi_import_version_range_includes_all_versions_from_the_current_up_to_the_next_major_version() { + void require_that_osgi_import_version_range_includes_all_versions_from_the_current_up_to_the_next_major_version() { assertEquals("com.yahoo.exported;version=\"[1.2,2)\"", new Import("com.yahoo.exported", Optional.of("1.2")).asOsgiImport()); } @Test - public void require_that_osgi_import_version_range_ignores_qualifier() { + void require_that_osgi_import_version_range_ignores_qualifier() { assertEquals("com.yahoo.exported;version=\"[1.2.3,2)\"", new Import("com.yahoo.exported", Optional.of("1.2.3.qualifier")).asOsgiImport()); } -- cgit v1.2.3