Merge pull request #277 from eclipse-jnosql/include-tck-nosql

Include Jakarta NoSQL TCK at mongodb databases

Author: otaviojava

CHANGELOG.adoc

@@ -8,6 +8,16 @@ and this project adheres to https://semver.org/spec/v2.0.0.html[Semantic Version
 
 == [Unreleased]
 
+=== Added
+
+- Include TCK tests
+
+=== Changed
+
+- Update OrientDB driver to 3.2.36
+- Update Couchbase client 3.7.6
+- Update DynamoDB driver 2.29.45
+
 == [1.1.4] - 2024-10-24
 
 == Changes

LICENSE

@@ -13,7 +13,7 @@ b) in the case of each subsequent Contributor:
 i) changes to the Program, and
 ii) additions to the Program;
 where such changes and/or additions to the Program originate from and are distributed by that particular Contributor. A Contribution 'originates' from a Contributor if it was added to the Program by such Contributor itself or anyone acting on such Contributor's behalf. Contributions do not include additions to the Program which: (i) are separate modules of software distributed in conjunction with the Program under their own license agreement, and (ii) are not derivative works of the Program.
-"Contributor" means any person or entity that distributes the Program.
+"Contributor" means any contact or entity that distributes the Program.
 
 "Licensed Patents" mean patent claims licensable by a Contributor which are necessarily infringed by the use or sale of its Contribution alone or when combined with the Program.
 

README.adoc

@@ -1960,6 +1960,42 @@ class IntegrationSampleTest {
 }
 ----
 
+== Performing the Jakarta NoSQL TCK
+
+To perform the Jakarta NoSQL TCK you should activate the `tck` profile. This profile will download the TCK and run it.
+
+[IMPORTANT]
+====
+By default, the TCK will run in all modules that have the `tck.nosql.skip` property defined as `false`.
+====
+
+[source,bash]
+----
+mvn test -Ptck
+----
+
+To run the Jakarta NoSQL TCK only in a specific module, you can use the `-pl` option, for example:
+
+[source,bash]
+----
+mvn test -Ptck -pl jnosql-mongodb
+----
+
+
+[IMPORTANT]
+====
+By default, activating the `tck` profile does not disable the execution of the default tests. To skip the default tests you can use `-DskipTests`. It will skip the unit tests and run only the TCK tests.
+[source,bash]
+----
+mvn test -Ptck -DskipTests
+----
+====
+
+The JNoSQL Database API implementations that support Jakarta NoSQL TCK execution already:
+
+- link:#_mongodb[MongoDB]
+
+
 == Want to Contribute a New Driver?
 
 As an open-source project, you're free to create any driver, and you're welcome to join and participate in the process.

jnosql-arangodb/src/test/java/org/eclipse/jnosql/databases/arangodb/communication/ArangoDBDocumentManagerTest.java

@@ -202,7 +202,7 @@ void shouldCount() {
     void shouldReadFromDifferentBaseDocumentUsingInstance() {
         entityManager.insert(getEntity());
         ArangoDB arangoDB = DefaultArangoDBDocumentManager.class.cast(entityManager).getArangoDB();
-        arangoDB.db(DATABASE).collection(COLLECTION_NAME).insertDocument(new Person());
+        arangoDB.db(DATABASE).collection(COLLECTION_NAME).insertDocument(new Human());
         SelectQuery select = select().from(COLLECTION_NAME).build();
         List<CommunicationEntity> entities = entityManager.select(select).collect(Collectors.toList());
         assertFalse(entities.isEmpty());

jnosql-arangodb/src/test/java/org/eclipse/jnosql/databases/arangodb/communication/Person.java renamed to jnosql-arangodb/src/test/java/org/eclipse/jnosql/databases/arangodb/communication/Human.java

@@ -14,7 +14,7 @@
  */
 package org.eclipse.jnosql.databases.arangodb.communication;
 
-public class Person {
+public class Human {
 
     private String name = "Ada";
 

jnosql-arangodb/src/test/java/org/eclipse/jnosql/databases/arangodb/integration/ArangoDBTemplateIntegrationTest.java

@@ -48,7 +48,7 @@
 
 @EnableAutoWeld
 @AddPackages(value = {Database.class, EntityConverter.class, DocumentTemplate.class})
-@AddPackages(Book.class)
+@AddPackages(Magazine.class)
 @AddPackages(ArangoDBTemplate.class)
 @AddExtensions({EntityMetadataExtension.class,
         DocumentExtension.class})
@@ -68,101 +68,101 @@ class ArangoDBTemplateIntegrationTest {
 
     @BeforeEach
     void setUp() {
-        this.template.delete(Book.class).execute();
+        this.template.delete(Magazine.class).execute();
     }
 
     @Test
     void shouldInsert() {
-        Book book = new Book(randomUUID().toString(), "Effective Java", 1);
-        template.insert(book);
-        Optional<Book> optional = template.find(Book.class, book.id());
+        Magazine magazine = new Magazine(randomUUID().toString(), "Effective Java", 1);
+        template.insert(magazine);
+        Optional<Magazine> optional = template.find(Magazine.class, magazine.id());
         assertThat(optional).isNotNull().isNotEmpty()
-                .get().isEqualTo(book);
+                .get().isEqualTo(magazine);
     }
 
     @Test
     void shouldUpdate() {
-        Book book = new Book(randomUUID().toString(), "Effective Java", 1);
-        assertThat(template.insert(book))
+        Magazine magazine = new Magazine(randomUUID().toString(), "Effective Java", 1);
+        assertThat(template.insert(magazine))
                 .isNotNull()
-                .isEqualTo(book);
+                .isEqualTo(magazine);
 
-        Book updated = new Book(book.id(), book.title() + " updated", 2);
+        Magazine updated = new Magazine(magazine.id(), magazine.title() + " updated", 2);
 
         assertThat(template.update(updated))
                 .isNotNull()
-                .isNotEqualTo(book);
+                .isNotEqualTo(magazine);
 
-        assertThat(template.find(Book.class, book.id()))
+        assertThat(template.find(Magazine.class, magazine.id()))
                 .isNotNull().get().isEqualTo(updated);
 
     }
 
     @Test
     void shouldFindById() {
-        Book book = new Book(randomUUID().toString(), "Effective Java", 1);
-        assertThat(template.insert(book))
+        Magazine magazine = new Magazine(randomUUID().toString(), "Effective Java", 1);
+        assertThat(template.insert(magazine))
                 .isNotNull()
-                .isEqualTo(book);
+                .isEqualTo(magazine);
 
-        assertThat(template.find(Book.class, book.id()))
-                .isNotNull().get().isEqualTo(book);
+        assertThat(template.find(Magazine.class, magazine.id()))
+                .isNotNull().get().isEqualTo(magazine);
     }
 
     @Test
     void shouldDelete() {
-        Book book = new Book(randomUUID().toString(), "Effective Java", 1);
-        assertThat(template.insert(book))
+        Magazine magazine = new Magazine(randomUUID().toString(), "Effective Java", 1);
+        assertThat(template.insert(magazine))
                 .isNotNull()
-                .isEqualTo(book);
+                .isEqualTo(magazine);
 
-        template.delete(Book.class, book.id());
-        assertThat(template.find(Book.class, book.id()))
+        template.delete(Magazine.class, magazine.id());
+        assertThat(template.find(Magazine.class, magazine.id()))
                 .isNotNull().isEmpty();
     }
 
     @Test
     void shouldDeleteAll() {
         for (int index = 0; index < 20; index++) {
-            Book book = new Book(randomUUID().toString(), "Effective Java", 1);
-            assertThat(template.insert(book))
+            Magazine magazine = new Magazine(randomUUID().toString(), "Effective Java", 1);
+            assertThat(template.insert(magazine))
                     .isNotNull()
-                    .isEqualTo(book);
+                    .isEqualTo(magazine);
         }
 
-        template.delete(Book.class).execute();
-        assertThat(template.select(Book.class).result()).isEmpty();
+        template.delete(Magazine.class).execute();
+        assertThat(template.select(Magazine.class).result()).isEmpty();
     }
 
 
     @Test
     void shouldUpdateNullValues() {
-        var book = new Book(randomUUID().toString(), "Effective Java", 1);
+        var book = new Magazine(randomUUID().toString(), "Effective Java", 1);
         template.insert(book);
-        template.update(new Book(book.id(), null, 2));
-        Optional<Book> optional = template.select(Book.class).where("id")
+        template.update(new Magazine(book.id(), null, 2));
+        Optional<Magazine> optional = template.select(Magazine.class).where("id")
                 .eq(book.id()).singleResult();
         SoftAssertions.assertSoftly(softly -> {
             softly.assertThat(optional).isPresent();
-            softly.assertThat(optional).get().extracting(Book::title).isNull();
-            softly.assertThat(optional).get().extracting(Book::edition).isEqualTo(2);
+            softly.assertThat(optional).get().extracting(Magazine::title).isNull();
+            softly.assertThat(optional).get().extracting(Magazine::edition).isEqualTo(2);
         });
     }
 
     @Test
     void shouldExecuteLimit() {
 
         for (int index = 1; index < 10; index++) {
-            var book = new Book(randomUUID().toString(), "Effective Java", index);
+            var book = new Magazine(randomUUID().toString(), "Effective Java", index);
             template.insert(book);
         }
 
-        List<Book> books = template.select(Book.class).orderBy("edition")
+        List<Magazine> magazines = template.select(Magazine.class).orderBy("edition")
                 .asc().limit(4).result();
 
         SoftAssertions.assertSoftly(soft -> {
-            soft.assertThat(books).hasSize(4);
-            var editions = books.stream().map(Book::edition).toList();
+            soft.assertThat(magazines).hasSize(4);
+            var editions = magazines.stream().map(Magazine::edition).toList();
             soft.assertThat(editions).hasSize(4).contains(1, 2, 3, 4);
         });
 
@@ -171,52 +171,52 @@ void shouldExecuteLimit() {
     @Test
     void shouldExecuteSkip() {
         for (int index = 1; index < 10; index++) {
-            var book = new Book(randomUUID().toString(), "Effective Java", index);
+            var book = new Magazine(randomUUID().toString(), "Effective Java", index);
             template.insert(book);
         }
 
-        List<Book> books = template.select(Book.class).orderBy("edition")
+        List<Magazine> magazines = template.select(Magazine.class).orderBy("edition")
                 .asc().skip(4).result();
 
         SoftAssertions.assertSoftly(soft -> {
-            soft.assertThat(books).hasSize(5);
-            var editions = books.stream().map(Book::edition).toList();
+            soft.assertThat(magazines).hasSize(5);
+            var editions = magazines.stream().map(Magazine::edition).toList();
             soft.assertThat(editions).hasSize(5).contains(5, 6, 7, 8, 9);
         });
     }
 
     @Test
     void shouldExecuteLimitStart() {
         for (int index = 1; index < 10; index++) {
-            var book = new Book(randomUUID().toString(), "Effective Java", index);
+            var book = new Magazine(randomUUID().toString(), "Effective Java", index);
             template.insert(book);
         }
 
-        List<Book> books = template.select(Book.class).orderBy("edition")
+        List<Magazine> magazines = template.select(Magazine.class).orderBy("edition")
                 .asc().skip(4).limit(3).result();
 
         SoftAssertions.assertSoftly(soft -> {
-            soft.assertThat(books).hasSize(3);
-            var editions = books.stream().map(Book::edition).toList();
+            soft.assertThat(magazines).hasSize(3);
+            var editions = magazines.stream().map(Magazine::edition).toList();
             soft.assertThat(editions).hasSize(3).contains(5, 6, 7);
         });
     }
 
     @Test
     void shouldSelectCursorSize() {
         for (int index = 1; index < 10; index++) {
-            var book = new Book(randomUUID().toString(), "Effective Java", index);
+            var book = new Magazine(randomUUID().toString(), "Effective Java", index);
             template.insert(book);
         }
-        var select = SelectQuery.select().from("Book").orderBy("edition").asc()
+        var select = SelectQuery.select().from("Magazine").orderBy("edition").asc()
                 .skip(4).limit(3).build();
         var pageRequest = PageRequest.ofSize(3);
-        CursoredPage<Book> entities = template.selectCursor(select, pageRequest);
+        CursoredPage<Magazine> entities = template.selectCursor(select, pageRequest);
 
         SoftAssertions.assertSoftly(soft -> {
             var content = entities.content();
             soft.assertThat(content).hasSize(3);
-            var editions = content.stream().map(Book::edition).toList();
+            var editions = content.stream().map(Magazine::edition).toList();
             soft.assertThat(editions).hasSize(3).contains(1, 2, 3);
         });
     }

jnosql-arangodb/src/test/java/org/eclipse/jnosql/databases/arangodb/integration/Book.java renamed to jnosql-arangodb/src/test/java/org/eclipse/jnosql/databases/arangodb/integration/Magazine.java

@@ -21,7 +21,7 @@
 import java.util.Objects;
 
 @Entity
-public class Book {
+public class Magazine {
 
     @Id("_key")
     private String id;
@@ -32,13 +32,13 @@ public class Book {
     @Column("edition")
     private int edition;
 
-    public Book(String id, String title, int edition) {
+    public Magazine(String id, String title, int edition) {
         this.id = id;
         this.title = title;
         this.edition = edition;
     }
 
-    Book() {
+    Magazine() {
     }
 
     public String id() {
@@ -57,10 +57,10 @@ public int edition() {
     public boolean equals(Object o) {
         if (this == o) return true;
         if (o == null || getClass() != o.getClass()) return false;
-        Book book = (Book) o;
-        return edition == book.edition
-                && Objects.equals(id, book.id)
-                && Objects.equals(title, book.title);
+        Magazine magazine = (Magazine) o;
+        return edition == magazine.edition
+                && Objects.equals(id, magazine.id)
+                && Objects.equals(title, magazine.title);
     }
 
     @Override