Fix Jackson SPI (#443)

* fix jackson SPI

* test

* move test

Author: SentryMan

jsonb-jackson/pom.xml

@@ -32,6 +32,13 @@
       <version>${project.version}</version>
     </dependency>
 
+    <dependency>
+      <groupId>io.avaje</groupId>
+      <artifactId>avaje-spi-service</artifactId>
+      <version>${spi.version}</version>
+      <optional>true</optional>
+    </dependency>
+
     <dependency>
       <groupId>io.avaje</groupId>
       <artifactId>junit</artifactId>

jsonb-jackson/src/main/java/io/avaje/jsonb/jackson/JacksonAdapterFactory.java

@@ -2,12 +2,14 @@
 
 import io.avaje.json.stream.JsonStream;
 import io.avaje.jsonb.spi.JsonStreamFactory;
+import io.avaje.spi.ServiceProvider;
 
 /**
  * Jackson Core based adapter.
  * <p>
  * Uses jackson code to do the underlying json parsing and generation.
  */
+@ServiceProvider
 public class JacksonAdapterFactory implements JsonStreamFactory {
 
   @Override

jsonb-jackson/src/main/java/module-info.java

@@ -2,8 +2,8 @@
 
   requires transitive io.avaje.jsonb;
   requires transitive com.fasterxml.jackson.core;
+  requires static io.avaje.spi;
 
   exports io.avaje.jsonb.jackson;
-
-  provides io.avaje.jsonb.spi.JsonStreamFactory with io.avaje.jsonb.jackson.JacksonAdapterFactory;
+  provides io.avaje.jsonb.spi.JsonbExtension with io.avaje.jsonb.jackson.JacksonAdapterFactory;
 }

jsonb-jackson/src/main/resources/META-INF/services/io.avaje.jsonb.spi.JsonStreamFactory

@@ -1,15 +1,19 @@
 package io.avaje.jsonb.jackson;
 
+import static org.assertj.core.api.Assertions.assertThat;
+
+import java.io.OutputStream;
+
+import org.example.Address;
+import org.example.MyComponent;
+import org.junit.jupiter.api.Test;
+
 import com.fasterxml.jackson.core.JsonFactory;
 import com.fasterxml.jackson.core.JsonParser;
 import com.fasterxml.jackson.core.SerializableString;
 import com.fasterxml.jackson.core.io.CharacterEscapes;
-import io.avaje.jsonb.Jsonb;
-import org.example.Address;
-import org.example.MyComponent;
-import org.junit.jupiter.api.Test;
 
-import static org.assertj.core.api.Assertions.assertThat;
+import io.avaje.jsonb.Jsonb;
 
 class JacksonAdapterTest {
 
@@ -22,6 +26,13 @@ class JacksonAdapterTest {
   public static final JsonFactory HUMAN_READABLE_JSON_FACTORY = JSON_FACTORY.copy()
     .setCharacterEscapes(new HumanReadableCharacterEscapes());
 
+  @Test
+  void isJackson() {
+    var jsonb = Jsonb.instance();
+    assertThat(jsonb.reader("")).isInstanceOf(JacksonReader.class);
+    assertThat(jsonb.writer(OutputStream.nullOutputStream())).isInstanceOf(JacksonWriter.class);
+  }
+
   @Test
   void custom_JsonFactory() {
 

jsonb-jackson/src/test/java/io/avaje/jsonb/jackson/JacksonAdapterTest.java

@@ -1,21 +1,22 @@
 package io.avaje.jsonb.jackson;
 
-import io.avaje.json.JsonAdapter;
-import io.avaje.json.JsonWriter;
-import io.avaje.jsonb.JsonType;
-import io.avaje.jsonb.Jsonb;
-import org.junit.jupiter.api.Test;
+import static org.assertj.core.api.Assertions.assertThat;
 
 import java.io.IOException;
 import java.io.StringWriter;
 import java.util.List;
 import java.util.Map;
 
-import static org.assertj.core.api.Assertions.assertThat;
+import org.junit.jupiter.api.Test;
+
+import io.avaje.json.JsonAdapter;
+import io.avaje.json.JsonWriter;
+import io.avaje.jsonb.JsonType;
+import io.avaje.jsonb.Jsonb;
 
 class ObjectAdapterTest {
 
-  Jsonb jsonb = Jsonb.builder().adapter(new JacksonAdapter()).build();
+  Jsonb jsonb = Jsonb.instance();
 
   JsonAdapter<Object> objectAdapter = jsonb.adapter(Object.class);