Merge branch '2.x' into 3.x

Author: cowtowncoder

release-notes/VERSION-2.x

@@ -11,6 +11,7 @@ Project: jackson-datatype-joda
 #146: `DateTime` can't be serialized with its own zone
   (`WRITE_DATES_WITH_CONTEXT_TIME_ZONE` not respected)
  (fixed by Joo-Hyuk K)
+- Add unit test to verify goodness of SPI metadata for Modules
 - Generate SBOMs [JSTEP-14]
 
 2.19.1 (not yet released)

src/main/resources/META-INF/services/tools.jackson.databind.JacksonModule

@@ -17,4 +17,8 @@
     opens tools.jackson.datatype.joda.deser;
     opens tools.jackson.datatype.joda.ser;
     opens tools.jackson.datatype.joda.testutil.failure;
+
+    provides tools.jackson.databind.JacksonModule with
+        tools.jackson.datatype.joda.JodaModule;
+    uses tools.jackson.databind.JacksonModule;
 }

src/test/java/module-info.java

@@ -0,0 +1,36 @@
+package tools.jackson.datatype.joda;
+
+import org.junit.jupiter.api.Test;
+
+import tools.jackson.databind.JacksonModule;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+import java.util.ServiceLoader;
+
+public class ModuleSPIMetadataTest
+    extends JodaTestBase
+{
+    @Test
+    void testModuleSPIMetadata() {
+        ServiceLoader<JacksonModule> loader
+            = ServiceLoader.load(JacksonModule.class);
+        assertTrue(loader.iterator().hasNext(),
+                "Expected at least one `Module` implementation to be found via `ServiceLoader`");
+        final String exp = JodaModule.class.getName();
+        int count = 0;
+
+        try {
+            for (JacksonModule service : loader) {
+                ++count;
+                if (service.getClass().getName().equals(exp)) {
+                    return;
+                }
+            }
+        } catch (Throwable t) {
+            fail("Expected to find `"+exp+"` Module (found "+count+" so far), problem: "+t);
+        }
+        fail("Expected to find `"+exp+"` Module (found "+count+" others)");
+        assertEquals(1, count);
+    }
+}