fix(gh-1162): ensure NitriteId serialization is compatible with previous versions

Author: DarkAtra

.gitignore

@@ -452,6 +452,7 @@ test.log
 /build
 !no2-old.db
 !no2-v3.db
+!no2-v4.3.0.db
 .diffblue
 infer-out
 secring.gpg

nitrite-mvstore-adapter/src/test/java/org/dizitart/no2/NitriteTest.java

@@ -53,6 +53,7 @@
 import java.io.InputStream;
 import java.nio.file.Files;
 import java.nio.file.Paths;
+import java.nio.file.StandardCopyOption;
 import java.text.ParseException;
 import java.text.SimpleDateFormat;
 import java.util.*;
@@ -622,7 +623,7 @@ public void run() {
                     log.error("Error in thread", e);
                 }
             }
-        };
+        }
 
         Thread t0 = new Thread(new ThreadRunner());
         Thread t1 = new Thread(new ThreadRunner());
@@ -682,6 +683,32 @@ public void testReadOnlyMode() {
         deleteDb(fileName);
     }
 
+    @Test
+    public void testIssue1162() throws IOException {
+
+        // setup no2-v4.3.0.db as a temp file
+        var databasePath = Files.createTempFile("temp-no2-v4.3.0", ".db");
+        var templateDb = Objects.requireNonNull(NitriteTest.class.getResourceAsStream("/no2-v4.3.0.db"));
+        Files.copy(templateDb, databasePath, StandardCopyOption.REPLACE_EXISTING);
+
+        var module = MVStoreModule.withConfig()
+            .filePath(databasePath.toAbsolutePath().toFile())
+            .build();
+
+        var database = Nitrite.builder()
+            .loadModule(module)
+            .openOrCreate();
+
+        try (database) {
+            var collection = database.getCollection("myCollection");
+            assertEquals(1, collection.size());
+
+            var firstPerson = collection.find().firstOrNull();
+            assertNotNull(firstPerson);
+            assertEquals(1970829645337976832L, firstPerson.getId().getIdValue());
+        }
+    }
+
     @Data
     @AllArgsConstructor
     @NoArgsConstructor

nitrite-mvstore-adapter/src/test/resources/no2-v4.3.0.db

@@ -16,14 +16,15 @@
 
 package org.dizitart.no2.collection;
 
-import lombok.EqualsAndHashCode;
 import lombok.Getter;
+
 import org.dizitart.no2.exceptions.InvalidIdException;
 
 import java.io.IOException;
 import java.io.ObjectInputStream;
 import java.io.ObjectOutputStream;
 import java.io.Serializable;
+import java.util.Objects;
 
 import static org.dizitart.no2.common.Constants.ID_PREFIX;
 import static org.dizitart.no2.common.Constants.ID_SUFFIX;
@@ -42,13 +43,12 @@
  * @since 1.0
  */
 @Getter
-@EqualsAndHashCode
 public final class NitriteId implements Comparable<NitriteId>, Serializable {
     private static final long serialVersionUID = 1477462375L;
     private static final SnowflakeIdGenerator generator = new SnowflakeIdGenerator();
 
     /** The underlying value of the NitriteId. */
-    private long idValue;
+    private transient long idValue;
 
     private NitriteId() {
         this.idValue = generator.getId();
@@ -120,6 +120,19 @@ public int compareTo(NitriteId other) {
         return Long.compare(idValue, other.idValue);
     }
 
+    @Override
+    public boolean equals(Object o) {
+        if (!(o instanceof NitriteId)) {
+            return false;
+        }
+        return idValue == ((NitriteId) o).idValue;
+    }
+
+    @Override
+    public int hashCode() {
+        return Objects.hashCode(idValue);
+    }
+
     @Override
     public String toString() {
         return ID_PREFIX + idValue + ID_SUFFIX;