chore: refactor test for increment and vup for vulnerability (#64)

* chore: refactor test for increment and vup for vulnerability

* docs: update README

Author: thunderz99

README.md

@@ -23,7 +23,7 @@ java-cosmos is a client for Azure CosmosDB 's SQL API (also called documentdb fo
 <dependency>
   <groupId>com.github.thunderz99</groupId>
   <artifactId>java-cosmos</artifactId>
-  <version>0.5.8</version>
+  <version>0.5.11</version>
 </dependency>
 ```
 
@@ -233,12 +233,16 @@ db.updatePartial("Collection", user1.id, Map.of("lastName", "Hanks", "status", "
     // support incrementing a number field 
     // see https://docs.microsoft.com/en-us/azure/cosmos-db/partial-document-update-getting-started?tabs=java
 
-    // increment age by 1
+    // increment age by 1. Supports int / long / double.
     var result = db.increment("Collection1", "id1", "/age", 1, "Users");
 
     // increment age by -5
     var result = db.increment("Collection1", "id1", "/age", -5, "Users");
 
+
+		// 404 Not Found Exception will be throw if the id does not exist
+    var result = db.increment("Collection1", "not exist id", "/name", 1, "Users");
+
     // 400 Bad Request Exception will be throw if the field is not an integer
     var result = db.increment("Collection1", "id1", "/name", 1, "Users");
 

pom.xml

@@ -4,7 +4,7 @@
     <groupId>com.github.thunderz99</groupId>
     <artifactId>java-cosmos</artifactId>
     <packaging>jar</packaging>
-    <version>0.5.10</version>
+    <version>0.5.11</version>
     <name>${project.groupId}:${project.artifactId}$</name>
     <description>A lightweight Azure CosmosDB client for Java</description>
     <url>https://github.com/thunderz99/java-cosmos</url>
@@ -72,14 +72,14 @@
         <dependency>
             <groupId>com.fasterxml.jackson.core</groupId>
             <artifactId>jackson-databind</artifactId>
-            <version>2.13.1</version>
+            <version>2.13.3</version>
         </dependency>
 
         <!-- https://mvnrepository.com/artifact/com.fasterxml.jackson.core/jackson-core -->
         <dependency>
             <groupId>com.fasterxml.jackson.core</groupId>
             <artifactId>jackson-core</artifactId>
-            <version>2.13.1</version>
+            <version>2.13.3</version>
         </dependency>
 
         <!-- https://mvnrepository.com/artifact/com.microsoft.azure/azure-documentdb -->
@@ -92,13 +92,13 @@
         <dependency>
             <groupId>com.azure</groupId>
             <artifactId>azure-cosmos</artifactId>
-            <version>4.29.1</version>
+            <version>4.30.1</version>
         </dependency>
 
         <dependency>
             <groupId>io.github.cdimascio</groupId>
             <artifactId>java-dotenv</artifactId>
-            <version>5.2.1</version>
+            <version>5.2.2</version>
         </dependency>
 
         <dependency>

src/test/java/io/github/thunderz99/cosmos/CosmosDatabaseTest.java

@@ -1203,30 +1203,39 @@ void increment_should_work() throws Exception {
 
 
         try {
-            var data1 = Map.of("id", id, "name", "John", "contents", Map.of("age", 20), "score", 85.5);
+            var data1 = Map.of("id", id, "name", "John", "contents", Map.of("age", 20), "score", 85.5, "number", 3_147_483_647L);
             db.upsert(coll, data1, partition).toMap();
             {
-                // increment by 1
+                // increment by 1, integer field
                 var inc1 = db.increment(coll, id, "/contents/age", 1, partition).toMap();
                 assertThat((Map<String, Object>) inc1.get("contents")).containsEntry("age", 21);
 
-                // increment by -3
+                // increment by -3, integer field
                 var inc2 = db.increment(coll, id, "/contents/age", -3, partition).toMap();
                 assertThat((Map<String, Object>) inc2.get("contents")).containsEntry("age", 18);
+
+                // increment by 1, long field
+                var inc3 = db.increment(coll, id, "/number", 1, partition).toMap();
+                assertThat(inc3).containsEntry("number", 3_147_483_648L);
+
+                // increment by 5, double field
+                var inc4 = db.increment(coll, id, "/score", 5, partition).toMap();
+                assertThat(inc4).containsEntry("score", 90.5);
+
             }
 
             {
                 // failed when incrementing a string field
                 assertThatThrownBy(() -> {
-                    db.increment(coll, id, "name", 5, partition);
+                    db.increment(coll, id, "/name", 5, partition);
                 }).isInstanceOfSatisfying(CosmosException.class, (e) -> {
                     assertThat(e.getStatusCode()).isEqualTo(400);
-                    assertThat(e.getMessage()).contains("inputs is invalid");
+                    assertThat(e.getMessage()).contains("is not a number");
                 });
             }
 
             {
-                // failed when incrementing a double field
+                // 400 will be thrown when path is not correct
                 assertThatThrownBy(() -> {
                     db.increment(coll, id, "score", 5, partition);
                 }).isInstanceOfSatisfying(CosmosException.class, (e) -> {
@@ -1235,6 +1244,16 @@ void increment_should_work() throws Exception {
                 });
             }
 
+            {
+                // 404 will be thrown when incrementing a not existing item
+                assertThatThrownBy(() -> {
+                    db.increment(coll, "not exist", "/number", 1, partition);
+                }).isInstanceOfSatisfying(CosmosException.class, (e) -> {
+                    assertThat(e.getStatusCode()).isEqualTo(404);
+                    assertThat(e.getMessage()).contains("Not Found");
+                });
+            }
+
         } finally {
             db.delete(coll, id, partition);
         }