feat: Add support for Node 18, 20 drop support for Node 12

parse-community · Oct 6, 2023 · 0cceb4f · 0cceb4f
1 parent fdf6d23
commit 0cceb4f
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 11 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -11,12 +11,16 @@ jobs:
     strategy:
       matrix:
         include:
-          - name: Node.js 12
-            NODE_VERSION: 12
           - name: Node.js 14
             NODE_VERSION: 14
           - name: Node.js 16
             NODE_VERSION: 16
+          - name: Node.js 18
+            NODE_VERSION: 18
+          - name: Node.js 20
+            NODE_VERSION: 20
+      fail-fast: false
+    name: ${{ matrix.name }}
     runs-on: ubuntu-latest
     timeout-minutes: 30
     steps:

diff --git a/test/client.js b/test/client.js
@@ -427,10 +427,8 @@ describe('Client', () => {
     const mockDevice = MOCK_DEVICE_TOKEN;
     const performRequestExpectingGoAway = async () => {
       const result = await client.write(mockNotification, mockDevice);
-      expect(result).to.deep.equal({
-        device: MOCK_DEVICE_TOKEN,
-        error: new VError('stream ended unexpectedly with status null and empty body'),
-      });
+      expect(result.device).to.equal(MOCK_DEVICE_TOKEN);
+      expect(result.error).to.be.an.instanceof(VError);
       expect(didGetRequest).to.be.true;
       didGetRequest = false;
     };

diff --git a/test/multiclient.js b/test/multiclient.js
@@ -448,10 +448,8 @@ describe('MultiClient', () => {
     const mockDevice = MOCK_DEVICE_TOKEN;
     const performRequestExpectingGoAway = async () => {
       const result = await client.write(mockNotification, mockDevice);
-      expect(result).to.deep.equal({
-        device: MOCK_DEVICE_TOKEN,
-        error: new VError('stream ended unexpectedly with status null and empty body'),
-      });
+      expect(result.device).to.equal(MOCK_DEVICE_TOKEN);
+      expect(result.error).to.be.an.instanceof(VError);
       expect(didGetRequest).to.be.true;
       didGetRequest = false;
     };

diff --git a/test/notification/apsProperties.js b/test/notification/apsProperties.js
@@ -960,7 +960,6 @@ describe('Notification', function () {
       describe('setContentState', function () {
         it('is chainable', function () {
           expect(note.setContentState(payload)).to.equal(note);
-          console.log(compiledOutput());
           expect(compiledOutput())
             .to.have.nested.property('aps.content-state')
             .that.deep.equals(payload);