feat: change isomorphic-unfetch for a more cross platform package (#63)

Author: yagopv

e2e/get-owned-safes.test.ts

@@ -5,10 +5,8 @@ describe('getOwnedSafes tests', () => {
   it('should get owned safes on rinkeby', async () => {
     const data = await getOwnedSafes(config.baseUrl, '4', '0x661E1CF4aAAf6a95C89EA8c81D120E6c62adDFf9')
 
-    expect(data.safes.sort()).toEqual([
-      '0x9B5dc27B104356516B05b02F6166a54F6D74e40B',
-      '0xb3b83bf204C458B461de9B0CD2739DB152b4fa5A',
-    ])
+    expect(data.safes).toContain('0x9B5dc27B104356516B05b02F6166a54F6D74e40B')
+    expect(data.safes).toContain('0xb3b83bf204C458B461de9B0CD2739DB152b4fa5A')
   })
 
   it('should return an empty array if no owned safes', async () => {

e2e/safe-collectibles-list.test.ts

@@ -7,13 +7,12 @@ describe('getCollectibles tests', () => {
     const data = await getCollectibles(config.baseUrl, '4', address)
 
     expect(data.length).toBeGreaterThanOrEqual(1)
-    expect(typeof data[0].address).toBe('string')
-    expect(typeof data[0].description).toBe('string')
-    expect(data[0].imageUri).toMatch(/^http/)
+    expect(typeof data[1].address).toBe('string')
+    expect(data[0].description).toBe(null)
     expect(data[0].metadata).toBeDefined()
-    expect(data[0].name.length).toBeGreaterThan(0)
+    expect(data[0].name).toBe(null)
     expect(data[0].tokenName.length).toBeGreaterThan(0)
     expect(data[0].tokenSymbol.length).toBeGreaterThan(0)
-    expect(data[0].uri.length).toBeGreaterThan(0)
+    expect(data[0].uri).toMatch(/^https?:\/\//)
   })
 })

e2e/safes-read.test.ts

@@ -12,18 +12,9 @@ describe('getSafeInfo tests', () => {
     // Nonce should match any positive integer number over 0
     expect(data.nonce).toBeGreaterThanOrEqual(0)
     expect(data.owners).toEqual([
-      {
-        value: '0x21D62C6894741DE97944D7844ED44D7782C66ABC',
-      },
-      {
-        value: '0x661E1CF4aAAf6a95C89EA8c81D120E6c62adDFf9',
-      },
-      {
-        value: '0x8814db983b821D65647C565fBf7c1092fC32437D',
-      },
-      {
-        value: '0x474e5Ded6b5D078163BFB8F6dBa355C3aA5478C8',
-      },
+      { value: '0x21D62C6894741DE97944D7844ED44D7782C66ABC' },
+      { value: '0x661E1CF4aAAf6a95C89EA8c81D120E6c62adDFf9' },
+      { value: '0x8814db983b821D65647C565fBf7c1092fC32437D' },
     ])
     expect(data.threshold).toBe(1)
     expect(data.version).toBe('1.1.1')

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gnosis.pm/safe-react-gateway-sdk",
-  "version": "2.9.0",
+  "version": "2.10.0",
   "main": "dist/index.min.js",
   "types": "dist/index.d.ts",
   "files": [
@@ -10,7 +10,7 @@
   "author": "katspaugh",
   "license": "MIT",
   "dependencies": {
-    "isomorphic-unfetch": "^3.1.0"
+    "cross-fetch": "^3.1.5"
   },
   "devDependencies": {
     "@babel/core": "^7.15.0",

src/utils.ts

@@ -1,4 +1,4 @@
-import fetch from 'isomorphic-unfetch'
+import fetch from 'cross-fetch'
 
 export type Params = Record<string, string | number | boolean | null>
 

tests/utils.test.ts

@@ -1,7 +1,7 @@
-import fetch from 'isomorphic-unfetch'
+import fetch from 'cross-fetch'
 import { fetchData, insertParams, stringifyQuery } from '../src/utils'
 
-jest.mock('isomorphic-unfetch')
+jest.mock('cross-fetch')
 
 const fetchMock = fetch as typeof fetch & jest.Mock
 

yarn.lock

@@ -1988,6 +1988,13 @@ core-js-compat@^3.14.0, core-js-compat@^3.16.0:
     browserslist "^4.17.0"
     semver "7.0.0"
 
+cross-fetch@^3.1.5:
+  version "3.1.5"
+  resolved "https://registry.yarnpkg.com/cross-fetch/-/cross-fetch-3.1.5.tgz#e1389f44d9e7ba767907f7af8454787952ab534f"
+  integrity sha512-lvb1SBsI0Z7GDwmuid+mU3kWVBwTVUbe7S0H52yaaAdQOXq2YktTCZdlAcNKFzE6QtRz0snpw9bNiPeOIkkQvw==
+  dependencies:
+    node-fetch "2.6.7"
+
 cross-spawn@^7.0.2, cross-spawn@^7.0.3:
   version "7.0.3"
   resolved "https://registry.yarnpkg.com/cross-spawn/-/cross-spawn-7.0.3.tgz#f73a85b9d5d41d045551c177e2882d4ac85728a6"
@@ -2933,14 +2940,6 @@ isobject@^3.0.1:
   resolved "https://registry.yarnpkg.com/isobject/-/isobject-3.0.1.tgz#4e431e92b11a9731636aa1f9c8d1ccbcfdab78df"
   integrity sha1-TkMekrEalzFjaqH5yNHMvP2reN8=
 
-isomorphic-unfetch@^3.1.0:
-  version "3.1.0"
-  resolved "https://registry.yarnpkg.com/isomorphic-unfetch/-/isomorphic-unfetch-3.1.0.tgz#87341d5f4f7b63843d468438128cb087b7c3e98f"
-  integrity sha512-geDJjpoZ8N0kWexiwkX8F9NkTsXhetLPVbZFQ+JTW239QNOwvB0gniuR1Wc6f0AMTn7/mFGyXvHTifrCp/GH8Q==
-  dependencies:
-    node-fetch "^2.6.1"
-    unfetch "^4.2.0"
-
 istanbul-lib-coverage@^3.0.0:
   version "3.0.0"
   resolved "https://registry.yarnpkg.com/istanbul-lib-coverage/-/istanbul-lib-coverage-3.0.0.tgz#f5944a37c70b550b02a78a5c3b2055b280cec8ec"
@@ -3662,10 +3661,12 @@ neo-async@^2.6.2:
   resolved "https://registry.yarnpkg.com/neo-async/-/neo-async-2.6.2.tgz#b4aafb93e3aeb2d8174ca53cf163ab7d7308305f"
   integrity sha512-Yd3UES5mWCSqR+qNT93S3UoYUkqAZ9lLg8a7g9rimsWmYGK8cVToA4/sF3RrshdyV3sAGMXVUmpMYOw+dLpOuw==
 
-node-fetch@^2.6.1:
-  version "2.6.1"
-  resolved "https://registry.yarnpkg.com/node-fetch/-/node-fetch-2.6.1.tgz#045bd323631f76ed2e2b55573394416b639a0052"
-  integrity sha512-V4aYg89jEoVRxRb2fJdAg8FHvI7cEyYdVAh94HH0UIK8oJxUfkjlDQN9RbMx+bEjP7+ggMiFRprSti032Oipxw==
+node-fetch@2.6.7:
+  version "2.6.7"
+  resolved "https://registry.yarnpkg.com/node-fetch/-/node-fetch-2.6.7.tgz#24de9fba827e3b4ae44dc8b20256a379160052ad"
+  integrity sha512-ZjMPFEfVx5j+y2yF35Kzx5sF7kDzxuDj6ziH4FFbOp87zKDZNx8yExJIb05OGF4Nlt9IHFIMBkRl41VdvcNdbQ==
+  dependencies:
+    whatwg-url "^5.0.0"
 
 node-int64@^0.4.0:
   version "0.4.0"
@@ -4489,6 +4490,11 @@ tr46@^2.1.0:
   dependencies:
     punycode "^2.1.1"
 
+tr46@~0.0.3:
+  version "0.0.3"
+  resolved "https://registry.yarnpkg.com/tr46/-/tr46-0.0.3.tgz#8184fd347dac9cdc185992f3a6622e14b9d9ab6a"
+  integrity sha1-gYT9NH2snNwYWZLzpmIuFLnZq2o=
+
 ts-loader@^9.2.5:
   version "9.2.5"
   resolved "https://registry.yarnpkg.com/ts-loader/-/ts-loader-9.2.5.tgz#127733a5e9243bf6dafcb8aa3b8a266d8041dca9"
@@ -4572,11 +4578,6 @@ unbox-primitive@^1.0.1:
     has-symbols "^1.0.2"
     which-boxed-primitive "^1.0.2"
 
-unfetch@^4.2.0:
-  version "4.2.0"
-  resolved "https://registry.yarnpkg.com/unfetch/-/unfetch-4.2.0.tgz#7e21b0ef7d363d8d9af0fb929a5555f6ef97a3be"
-  integrity sha512-F9p7yYCn6cIW9El1zi0HI6vqpeIvBsr3dSuRO6Xuppb1u5rXpCPmMvLSyECLhybr9isec8Ohl0hPekMVrEinDA==
-
 unicode-canonical-property-names-ecmascript@^1.0.4:
   version "1.0.4"
   resolved "https://registry.yarnpkg.com/unicode-canonical-property-names-ecmascript/-/unicode-canonical-property-names-ecmascript-1.0.4.tgz#2619800c4c825800efdd8343af7dd9933cbe2818"
@@ -4663,6 +4664,11 @@ watchpack@^2.2.0:
     glob-to-regexp "^0.4.1"
     graceful-fs "^4.1.2"
 
+webidl-conversions@^3.0.0:
+  version "3.0.1"
+  resolved "https://registry.yarnpkg.com/webidl-conversions/-/webidl-conversions-3.0.1.tgz#24534275e2a7bc6be7bc86611cc16ae0a5654871"
+  integrity sha1-JFNCdeKnvGvnvIZhHMFq4KVlSHE=
+
 webidl-conversions@^5.0.0:
   version "5.0.0"
   resolved "https://registry.yarnpkg.com/webidl-conversions/-/webidl-conversions-5.0.0.tgz#ae59c8a00b121543a2acc65c0434f57b0fc11aff"
@@ -4747,6 +4753,14 @@ whatwg-mimetype@^2.3.0:
   resolved "https://registry.yarnpkg.com/whatwg-mimetype/-/whatwg-mimetype-2.3.0.tgz#3d4b1e0312d2079879f826aff18dbeeca5960fbf"
   integrity sha512-M4yMwr6mAnQz76TbJm914+gPpB/nCwvZbJU28cUD6dR004SAxDLOOSUaB1JDRqLtaOV/vi0IC5lEAGFgrjGv/g==
 
+whatwg-url@^5.0.0:
+  version "5.0.0"
+  resolved "https://registry.yarnpkg.com/whatwg-url/-/whatwg-url-5.0.0.tgz#966454e8765462e37644d3626f6742ce8b70965d"
+  integrity sha1-lmRU6HZUYuN2RNNib2dCzotwll0=
+  dependencies:
+    tr46 "~0.0.3"
+    webidl-conversions "^3.0.0"
+
 whatwg-url@^8.0.0, whatwg-url@^8.5.0:
   version "8.7.0"
   resolved "https://registry.yarnpkg.com/whatwg-url/-/whatwg-url-8.7.0.tgz#656a78e510ff8f3937bc0bcbe9f5c0ac35941b77"