Merge pull request #9 from contentstack/test/cs-44062-sanity

added: surefire plugin to generate test reports

Author: reeshika-h

changelog.md

@@ -0,0 +1,13 @@
+# Changelog
+
+A brief description of what changes project contains
+
+#### v1.0.1
+
+- added surefire plugin to generate test report
+
+## Introducing Release Of Java Marketplace SDK
+
+#### v1.0.0
+
+- Initial release for Marketplace SDK

pom.xml

@@ -5,7 +5,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>com.contentstack.sdk</groupId>
     <artifactId>marketplace</artifactId>
-    <version>1.0.0</version>
+    <version>1.0.1</version>
     <description>Contentstack Java Management SDK for Content Management API</description>
     <url>https://github.com/contentstack/contentstack-management-java/</url>
 
@@ -161,7 +161,7 @@
                 <artifactId>maven-surefire-plugin</artifactId>
                 <version>3.0.0-M5</version>
                 <configuration>
-                    <skipTests>true</skipTests>
+                    <reportsDirectory>${project.build.directory}/surefire-reports</reportsDirectory>
                     <testFailureIgnore>true</testFailureIgnore>
                 </configuration>
             </plugin>

src/test/java/com/contentstack/sdk/marketplace/MarketplaceTest.java

@@ -36,7 +36,7 @@ void testConstructorWithNullOrganizationUidAndWithValidHost() {
         //String organizationUid = "nullOrganizationUid123";
         String organizationUid = null;
         String host = "developerhub-api.contentstack.com";
-        assertThrows(IllegalArgumentException.class, () -> new Marketplace.Builder(organizationUid)
+        assertThrows(NullPointerException.class, () -> new Marketplace.Builder(organizationUid)
                 .host(host)
                 .build());
     }
@@ -63,7 +63,7 @@ void testConstructorWithNullOrganizationUid() {
     void testConstructorWithNullOrganizationUidNullCheck() {
         // Arrange
         String organizationUid = null;
-        assertThrows(IllegalArgumentException.class, () -> new Marketplace.Builder(organizationUid)
+        assertThrows(NullPointerException.class, () -> new Marketplace.Builder(organizationUid)
                 .build());
     }
 

src/test/java/com/contentstack/sdk/marketplace/installations/location/LocationTests.java

@@ -60,7 +60,7 @@ void testParamsAndHeaders() {
         Assertions.assertEquals(5, request.headers().size());
         Assertions.assertEquals("api.contentstack.io", request.url().host());
         Assertions.assertNull(request.body());
-        Assertions.assertEquals("https://api.contentstack.io/installations/orgId999999999/locations/configuration?param1=value1", request.url().toString());
+        Assertions.assertEquals("https://api.contentstack.io/installations/"+ ORG_UID +"/locations/configuration?param1=value1", request.url().toString());
     }
 
 

src/test/java/com/contentstack/sdk/marketplace/installations/webhook/WebhookTests.java

@@ -111,7 +111,7 @@ void testParamsAndHeaders() {
         Assertions.assertEquals(5, request.headers().size());
         Assertions.assertEquals("api.contentstack.io", request.url().host());
         Assertions.assertNull(request.body());
-        Assertions.assertEquals("https://api.contentstack.io/installations/installation_id/webhooks/orgId999999999/executions?limit=10&skip=5&sort=asc&order=order", request.url().toString());
+        Assertions.assertEquals("https://api.contentstack.io/installations/installation_id/webhooks/" + WEBHOOK_ID + "/executions?limit=10&skip=5&sort=asc&order=order", request.url().toString());
     }