test: replace calls to API with Wiremock

lucasls · lucasls · commit 353e02d940c6 · 2022-12-20T17:51:29.000+01:00
diff --git a/pom.xml b/pom.xml
@@ -144,6 +144,13 @@
             <scope>test</scope>
         </dependency>
 
+        <dependency>
+            <groupId>com.github.tomakehurst</groupId>
+            <artifactId>wiremock-jre8-standalone</artifactId>
+            <version>2.35.0</version>
+            <scope>test</scope>
+        </dependency>
+
     </dependencies>
     <build>
         <pluginManagement>
diff --git a/src/test/java/com/mytaxi/apis/phrase/api/locale/PhraseLocaleAPITest.java b/src/test/java/com/mytaxi/apis/phrase/api/locale/PhraseLocaleAPITest.java
@@ -1,5 +1,6 @@
 package com.mytaxi.apis.phrase.api.locale;
 
+import com.github.tomakehurst.wiremock.junit.WireMockRule;
 import com.mytaxi.apis.phrase.api.locale.dto.PhraseLocaleDTO;
 import com.mytaxi.apis.phrase.config.TestConfig;
 import com.mytaxi.apis.phrase.domainobject.locale.PhraseBranch;
@@ -10,6 +11,7 @@
 import org.aeonbits.owner.ConfigFactory;
 import org.junit.Before;
 import org.junit.Ignore;
+import org.junit.Rule;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.ArgumentCaptor;
@@ -20,6 +22,7 @@
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.client.RestTemplate;
 
+import static com.github.tomakehurst.wiremock.core.WireMockConfiguration.options;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
@@ -32,6 +35,8 @@
 @RunWith(MockitoJUnitRunner.class)
 public class PhraseLocaleAPITest
 {
+    @Rule
+    public WireMockRule wireMockRule = new WireMockRule(options().port(9999));
 
     private RestTemplate restTemplate;
 
@@ -58,7 +63,10 @@ public void testListLocales_integration() throws Exception
     {
         // GIVEN
         String authToken = cfg.authToken();
-        PhraseLocaleAPI phraseAppApiV2 = new PhraseLocaleAPI(authToken);
+        String host = cfg.host();
+        String scheme = cfg.scheme();
+
+        PhraseLocaleAPI phraseAppApiV2 = new PhraseLocaleAPI(authToken, scheme, host);
 
         String projectId = cfg.projectId();
         List<String> branches = cfg.branches();
@@ -101,8 +109,10 @@ public void listLocales_getTwoTimes_statusCode200and304_XRateLimitRemainingNotCh
         String authToken = cfg.authToken();
         String projectId = cfg.projectId();
         List<String> branches = cfg.branches();
+        String host = cfg.host();
+        String scheme = cfg.scheme();
 
-        PhraseLocaleAPI phraseLocaleAPI = Mockito.spy(new PhraseLocaleAPI(authToken));
+        PhraseLocaleAPI phraseLocaleAPI = Mockito.spy(new PhraseLocaleAPI(authToken, scheme, host));
 
         // WHEN doing two request
         List<PhraseProject> phraseProject1 = phraseLocaleAPI.listLocales(Collections.singletonList(projectId), branches);
diff --git a/src/test/java/com/mytaxi/apis/phrase/api/locale/PhraseLocaleAPITestNew.java b/src/test/java/com/mytaxi/apis/phrase/api/locale/PhraseLocaleAPITestNew.java
@@ -1,5 +1,6 @@
 package com.mytaxi.apis.phrase.api.locale;
 
+import com.github.tomakehurst.wiremock.junit.WireMockRule;
 import com.mytaxi.apis.phrase.api.locale.dto.PhraseLocaleDTO;
 import com.mytaxi.apis.phrase.config.TestConfig;
 import com.mytaxi.apis.phrase.domainobject.locale.PhraseBranch;
@@ -10,6 +11,7 @@
 import org.aeonbits.owner.ConfigFactory;
 import org.junit.Before;
 import org.junit.Ignore;
+import org.junit.Rule;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.ArgumentCaptor;
@@ -20,6 +22,7 @@
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.client.RestTemplate;
 
+import static com.github.tomakehurst.wiremock.core.WireMockConfiguration.options;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
@@ -35,6 +38,8 @@
 @RunWith(MockitoJUnitRunner.class)
 public class PhraseLocaleAPITestNew
 {
+    @Rule
+    public WireMockRule wireMockRule = new WireMockRule(options().port(9999));
 
     private RestTemplate restTemplate;
 
diff --git a/src/test/java/com/mytaxi/apis/phrase/api/localedownload/PhraseLocaleDownloadAPITest.java b/src/test/java/com/mytaxi/apis/phrase/api/localedownload/PhraseLocaleDownloadAPITest.java
@@ -1,11 +1,13 @@
 package com.mytaxi.apis.phrase.api.localedownload;
 
+import com.github.tomakehurst.wiremock.junit.WireMockRule;
 import com.mytaxi.apis.phrase.api.format.Format;
 import com.mytaxi.apis.phrase.api.format.JavaPropertiesFormat;
 import com.mytaxi.apis.phrase.config.TestConfig;
 import java.util.List;
 import org.aeonbits.owner.ConfigFactory;
 import org.junit.Before;
+import org.junit.Rule;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.ArgumentCaptor;
@@ -15,9 +17,9 @@
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 
+import static com.github.tomakehurst.wiremock.core.WireMockConfiguration.options;
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.*;
 import static org.mockito.Matchers.anyString;
 import static org.mockito.Matchers.eq;
 import static org.mockito.Mockito.times;
@@ -26,6 +28,8 @@
 @RunWith(MockitoJUnitRunner.class)
 public class PhraseLocaleDownloadAPITest
 {
+    @Rule
+    public WireMockRule wireMockRule = new WireMockRule(options().port(9999));
 
     @Captor
     ArgumentCaptor<ResponseEntity<byte[]>> responseEntityCaptor;
@@ -47,8 +51,10 @@ public void tesDownloadLocales_integration() throws Exception
         String authToken = cfg.authToken();
         String projectId = cfg.projectId();
         String localeIdDe = cfg.localeIdDe();
+        String host = cfg.host();
+        String scheme = cfg.scheme();
 
-        PhraseLocaleDownloadAPI localeDownloadAPI = new PhraseLocaleDownloadAPI(authToken);
+        PhraseLocaleDownloadAPI localeDownloadAPI = new PhraseLocaleDownloadAPI(authToken, scheme, host);
 
         // WHEN
         byte[] fileBytes = localeDownloadAPI.downloadLocale(projectId, localeIdDe);
@@ -64,8 +70,10 @@ public void tesDownloadLocales_etag()
         String authToken = cfg.authToken();
         String projectId = cfg.projectId();
         String localeIdDe = cfg.localeIdDe();
+        String host = cfg.host();
+        String scheme = cfg.scheme();
 
-        PhraseLocaleDownloadAPI localeDownloadAPI = Mockito.spy(new PhraseLocaleDownloadAPI(authToken));
+        PhraseLocaleDownloadAPI localeDownloadAPI = Mockito.spy(new PhraseLocaleDownloadAPI(authToken, scheme, host));
 
         // WHEN doing two request
         byte[] fileBytes1 = localeDownloadAPI.downloadLocale(projectId, localeIdDe);
@@ -74,7 +82,7 @@ public void tesDownloadLocales_etag()
         // THEN assert same result
         assertNotNull(fileBytes1);
         assertNotNull(fileBytes2);
-        assertEquals(fileBytes1, fileBytes2);
+        assertArrayEquals(fileBytes1, fileBytes2);
 
         // AND assert status codes 200 and 304 to verify E-Tag handling
         verify(localeDownloadAPI, times(2)).handleResponse(eq(projectId), anyString(), responseEntityCaptor.capture());
@@ -94,8 +102,10 @@ public void tesDownloadLocales_withFormatOptions() throws Exception
         String projectId = cfg.projectId();
         String localeIdDe = cfg.localeIdDe();
         List<String> branches = cfg.branches();
+        String host = cfg.host();
+        String scheme = cfg.scheme();
 
-        PhraseLocaleDownloadAPI localeDownloadAPI = new PhraseLocaleDownloadAPI(authToken);
+        PhraseLocaleDownloadAPI localeDownloadAPI = new PhraseLocaleDownloadAPI(authToken, scheme, host);
 
         Format format = JavaPropertiesFormat.newBuilder()
             .setEscapeSingleQuotes(false)
diff --git a/src/test/java/com/mytaxi/apis/phrase/api/localedownload/PhraseLocaleDownloadAPITestNew.java b/src/test/java/com/mytaxi/apis/phrase/api/localedownload/PhraseLocaleDownloadAPITestNew.java
@@ -1,20 +1,25 @@
 package com.mytaxi.apis.phrase.api.localedownload;
 
+import com.github.tomakehurst.wiremock.junit.WireMockRule;
 import com.mytaxi.apis.phrase.api.format.Format;
 import com.mytaxi.apis.phrase.api.format.JavaPropertiesFormat;
 import com.mytaxi.apis.phrase.config.TestConfig;
 import java.util.List;
 import org.aeonbits.owner.ConfigFactory;
 import org.junit.Before;
+import org.junit.Rule;
 import org.junit.Test;
 
+import static com.github.tomakehurst.wiremock.core.WireMockConfiguration.options;
 import static org.junit.Assert.assertNotNull;
 
 /**
  * Created by m.winkelmann on 24.11.15.
  */
 public class PhraseLocaleDownloadAPITestNew
 {
+    @Rule
+    public WireMockRule wireMockRule = new WireMockRule(options().port(9999));
 
     private TestConfig cfg;
 
diff --git a/src/test/java/com/mytaxi/apis/phrase/api/translation/PhraseTranslationAPITest.java b/src/test/java/com/mytaxi/apis/phrase/api/translation/PhraseTranslationAPITest.java
@@ -1,5 +1,6 @@
 package com.mytaxi.apis.phrase.api.translation;
 
+import com.github.tomakehurst.wiremock.junit.WireMockRule;
 import com.mytaxi.apis.phrase.api.locale.dto.PhraseLocaleDTO;
 import com.mytaxi.apis.phrase.api.translation.dto.PhraseKeyDTO;
 import com.mytaxi.apis.phrase.api.translation.dto.PhraseTranslationDTO;
@@ -12,6 +13,7 @@
 import org.aeonbits.owner.ConfigFactory;
 import org.junit.Assert;
 import org.junit.Before;
+import org.junit.Rule;
 import org.junit.Test;
 import org.mockito.Matchers;
 import org.mockito.Mockito;
@@ -22,11 +24,16 @@
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.client.RestTemplate;
 
+import static com.github.tomakehurst.wiremock.core.WireMockConfiguration.options;
+
 /**
  * Created by m.winkelmann on 05.11.15.
  */
 public class PhraseTranslationAPITest
 {
+    @Rule
+    public WireMockRule wireMockRule = new WireMockRule(options().port(9999));
+
     private RestTemplate restTemplate;
 
     private PhraseTranslationAPI translationAPI;
@@ -118,7 +125,10 @@ public void testListTranslations() throws Exception
     {
         // GIVEN
         String authToken = cfg.authToken();
-        PhraseTranslationAPI translationAPI = new PhraseTranslationAPI(authToken);
+        String host = cfg.host();
+        String scheme = cfg.scheme();
+
+        PhraseTranslationAPI translationAPI = new PhraseTranslationAPI(authToken, scheme, host);
 
         String projectId = cfg.projectId();
 
diff --git a/src/test/java/com/mytaxi/apis/phrase/api/translation/PhraseTranslationAPITestNew.java b/src/test/java/com/mytaxi/apis/phrase/api/translation/PhraseTranslationAPITestNew.java
@@ -1,5 +1,6 @@
 package com.mytaxi.apis.phrase.api.translation;
 
+import com.github.tomakehurst.wiremock.junit.WireMockRule;
 import com.mytaxi.apis.phrase.api.locale.dto.PhraseLocaleDTO;
 import com.mytaxi.apis.phrase.api.translation.dto.PhraseKeyDTO;
 import com.mytaxi.apis.phrase.api.translation.dto.PhraseTranslationDTO;
@@ -12,6 +13,7 @@
 import org.aeonbits.owner.ConfigFactory;
 import org.junit.Assert;
 import org.junit.Before;
+import org.junit.Rule;
 import org.junit.Test;
 import org.mockito.Matchers;
 import org.mockito.Mockito;
@@ -22,11 +24,15 @@
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.client.RestTemplate;
 
+import static com.github.tomakehurst.wiremock.core.WireMockConfiguration.options;
+
 /**
  * Created by m.winkelmann on 05.11.15.
  */
 public class PhraseTranslationAPITestNew
 {
+    @Rule
+    public WireMockRule wireMockRule = new WireMockRule(options().port(9999));
     private RestTemplate restTemplate;
 
     private PhraseTranslationAPI translationAPI;
diff --git a/src/test/java/com/mytaxi/apis/phrase/config/TestConfig.java b/src/test/java/com/mytaxi/apis/phrase/config/TestConfig.java
@@ -8,21 +8,21 @@
  */
 public interface TestConfig extends Config
 {
-    @DefaultValue("${ENV_PHRASE_AUTHTOKEN}")
+    @DefaultValue("token-1234")
     String authToken();
 
-    @DefaultValue("${ENV_PHRASE_PROJECTID}")
+    @DefaultValue("proj-1234")
     String projectId();
 
-    @DefaultValue("${ENV_PHRASE_BRANCHES}")
+    @DefaultValue("master")
     List<String> branches();
 
-    @DefaultValue("${ENV_PHRASE_LOCALEID_DE}")
+    @DefaultValue("loc-1234")
     String localeIdDe();
 
-    @DefaultValue("${ENV_PHRASE_HOST}")
+    @DefaultValue("localhost:9999")
     String host();
 
-    @DefaultValue("${ENV_PHRASE_SCHEME}")
+    @DefaultValue("http")
     String scheme();
 }
diff --git a/src/test/java/com/mytaxi/apis/phrase/tasks/PhraseAppSyncTaskTest.java b/src/test/java/com/mytaxi/apis/phrase/tasks/PhraseAppSyncTaskTest.java
@@ -1,5 +1,6 @@
 package com.mytaxi.apis.phrase.tasks;
 
+import com.github.tomakehurst.wiremock.junit.WireMockRule;
 import com.google.common.collect.ImmutableList;
 import com.mytaxi.apis.phrase.api.format.Format;
 import com.mytaxi.apis.phrase.api.format.JavaPropertiesFormat;
@@ -16,14 +17,18 @@
 import org.aeonbits.owner.ConfigFactory;
 import org.junit.Before;
 import org.junit.Ignore;
+import org.junit.Rule;
 import org.junit.Test;
 
+import static com.github.tomakehurst.wiremock.core.WireMockConfiguration.options;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
 
 public class PhraseAppSyncTaskTest
 {
+    @Rule
+    public WireMockRule wireMockRule = new WireMockRule(options().port(9999));
     private TestConfig cfg;
 
 
@@ -75,7 +80,7 @@ private void testRun(Format format, List<ExpectedEntry> expectedEntries) throws
         String uuid = UUID.randomUUID().toString();
 
         //
-        PhraseAppSyncTask phraseAppSyncTask = new PhraseAppSyncTask(cfg.authToken(), cfg.projectId());
+        PhraseAppSyncTask phraseAppSyncTask = new PhraseAppSyncTask(cfg.authToken(), cfg.projectId(), cfg.scheme(), cfg.host());
         phraseAppSyncTask.setMessagesFoldername(uuid);
         phraseAppSyncTask.setFormat(format);
         phraseAppSyncTask.run();
diff --git a/src/test/resources/__files/get-locale-properties-res.properties b/src/test/resources/__files/get-locale-properties-res.properties
@@ -0,0 +1 @@
+key=value
diff --git a/src/test/resources/__files/get-locales-res.json b/src/test/resources/__files/get-locales-res.json
@@ -0,0 +1,20 @@
+[
+  {
+    "id": "loc-1234",
+    "name": "DE",
+    "code": "de",
+    "default": false,
+    "rtl": false,
+    "createdAt": "2019-05-23T10:40:03.000+00:00",
+    "updatedAt": "2022-12-15T10:58:09.000+00:00"
+  },
+  {
+    "id": "loc-5678",
+    "name": "EN_Source",
+    "code": "en",
+    "default": true,
+    "rtl": false,
+    "createdAt": "2019-05-23T10:38:04.000+00:00",
+    "updatedAt": "2022-12-15T16:02:13.000+00:00"
+  }
+]
diff --git a/src/test/resources/__files/get-translations-res.json b/src/test/resources/__files/get-translations-res.json
@@ -0,0 +1,15 @@
+[
+  {
+    "id": "4e80a956a92a5b3f65b6dc6edd85927f",
+    "content": "Buchung stornieren",
+    "locale": {
+      "id": "loc-1234",
+      "name": "DE",
+      "code": "de"
+    },
+    "key": {
+      "id": "b4977ac14b42876f364fac19dbc1acf6",
+      "name": "booking_fare.fare_confirmation_dialog.btn_cancel_text"
+    }
+  }
+]
diff --git a/src/test/resources/mappings/mappings.json b/src/test/resources/mappings/mappings.json
