Merge pull request #20 from transifex/retry_fetch

Retry languages/translations if status is 202
transifex · Sep 3, 2020 · 3557576 · 3557576
2 parents e7050b7 + 7a8476d
commit 3557576
Show file tree

Hide file tree

Showing 2 changed files with 55 additions and 10 deletions.
diff --git a/packages/native/src/TxNative.js b/packages/native/src/TxNative.js
@@ -136,11 +136,18 @@ export default class TxNative {
     // contact CDS
     try {
       sendEvent(FETCHING_TRANSLATIONS, localeCode, this);
-      const response = await axios.get(`${this.cdsHost}/content/${localeCode}`, {
-        headers: {
-          Authorization: `Bearer ${this.token}`,
-        },
-      });
+      let response;
+      let lastResponseStatus = 202;
+      while (lastResponseStatus === 202) {
+        /* eslint-disable no-await-in-loop */
+        response = await axios.get(`${this.cdsHost}/content/${localeCode}`, {
+          headers: {
+            Authorization: `Bearer ${this.token}`,
+          },
+        });
+        /* eslint-enable no-await-in-loop */
+        lastResponseStatus = response.status;
+      }
 
       const { data } = response;
       if (data && data.data) {
@@ -191,11 +198,18 @@ export default class TxNative {
     // contact CDS
     try {
       sendEvent(FETCHING_LOCALES, null, this);
-      const response = await axios.get(`${this.cdsHost}/languages`, {
-        headers: {
-          Authorization: `Bearer ${this.token}`,
-        },
-      });
+      let response;
+      let lastResponseStatus = 202;
+      while (lastResponseStatus === 202) {
+        /* eslint-disable no-await-in-loop */
+        response = await axios.get(`${this.cdsHost}/languages`, {
+          headers: {
+            Authorization: `Bearer ${this.token}`,
+          },
+        });
+        /* eslint-enable no-await-in-loop */
+        lastResponseStatus = response.status;
+      }
 
       const { data } = response;
       if (data && data.data) {

diff --git a/packages/native/tests/tx.test.js b/packages/native/tests/tx.test.js
@@ -258,4 +258,35 @@ describe('tx instance', () => {
       foo: 'bar',
     });
   });
+
+  it('retries fetching languages', async () => {
+    tx.init({ token: 'abcd' });
+    nock(tx.cdsHost)
+      .get('/languages')
+      .twice()
+      .reply(202)
+      .get('/languages')
+      .reply(200, {
+        data: [{
+          name: 'Greek',
+          code: 'el',
+          localized_name: 'Ελληνικά',
+          rtl: false,
+        }],
+      });
+    const locales = await tx.getRemoteLocales({ refresh: true });
+    expect(locales).to.deep.equal(['el']);
+  });
+
+  it('retries fetching translations', async () => {
+    tx.init({ token: 'abcd' });
+    nock(tx.cdsHost)
+      .get('/content/el')
+      .twice()
+      .reply(202)
+      .get('/content/el')
+      .reply(200, { data: { source: { string: 'translation' } } });
+    await tx.fetchTranslations('el');
+    expect(tx.cache.get('source', 'el')).to.equal('translation');
+  });
 });