@webex/http-core 3.0.0-beta.7 → 3.0.0-beta.71

package/test/integration/spec/request.js

@@ -14,33 +14,33 @@ describe('http-core', function () {
   this.timeout(30000);
   describe('request()', () => {
     describe('GET', () => {
-      it('retrieves html', () => request({uri: makeLocalUrl('/')})
-        .then((res) => {
+      it('retrieves html', () =>
+        request({uri: makeLocalUrl('/')}).then((res) => {
           assert.statusCode(res, 200);
           assert.isString(res.body);
         }));
 
-      it('attaches the request options object to every response', () => request(makeLocalUrl('/'))
-        .then((res) => {
+      it('attaches the request options object to every response', () =>
+        request(makeLocalUrl('/')).then((res) => {
           assert.property(res, 'options');
           assert.property(res.options, 'uri');
           assert.equal(res.options.uri, makeLocalUrl('/'));
         }));
 
-      it('accepts `uri` as a separate parameter from `options`', () => request(makeLocalUrl('/'))
-        .then((res) => {
+      it('accepts `uri` as a separate parameter from `options`', () =>
+        request(makeLocalUrl('/')).then((res) => {
           assert.statusCode(res, 200);
           assert.isString(res.body);
         }));
 
-      it('normalizes `options.url` as `options.uri`', () => request({url: makeLocalUrl('/')})
-        .then((res) => {
+      it('normalizes `options.url` as `options.uri`', () =>
+        request({url: makeLocalUrl('/')}).then((res) => {
           assert.statusCode(res, 200);
           assert.isString(res.body);
         }));
 
-      it('retrieves JSON', () => request({uri: makeLocalUrl('/json/get')})
-        .then((res) => {
+      it('retrieves JSON', () =>
+        request({uri: makeLocalUrl('/json/get')}).then((res) => {
           assert.statusCode(res, 200);
           assert.isObject(res.body);
           assert.deepEqual(res.body, {isObject: true});
@@ -59,83 +59,95 @@ describe('http-core', function () {
       });
 
       describe('with responseType="buffer"', () => {
-        it('retrieves a file as a buffer', () => request({
-          uri: makeLocalUrl('/sample-image-small-one.png'),
-          responseType: 'buffer'
-        })
-          .then((res) => {
+        it('retrieves a file as a buffer', () =>
+          request({
+            uri: makeLocalUrl('/sample-image-small-one.png'),
+            responseType: 'buffer',
+          }).then((res) => {
             assert.isBufferLike(res.body);
           }));
       });
 
       describe('with responseType="blob"', () => {
-        it('retrieves a file as a file', () => request({
-          uri: makeLocalUrl('/sample-image-small-one.png'),
-          responseType: 'blob'
-        })
-          .then((res) => {
+        it('retrieves a file as a file', () =>
+          request({
+            uri: makeLocalUrl('/sample-image-small-one.png'),
+            responseType: 'blob',
+          }).then((res) => {
             assert.isBlobLike(res.body);
           }));
       });
 
-      it('makes CORS compatible calls', () => request({
-        uri: 'https://ds.ciscospark.com/v1/region/'
-      })
-        .then((res) => {
+      it('makes CORS compatible calls', () =>
+        request({
+          uri: 'https://ds.ciscospark.com/v1/region/',
+        }).then((res) => {
           assert.notEqual(res.statusCode, 0);
         }));
 
-      it('fails with a subtyped error', () => assert.isRejected(request({
-        uri: makeLocalUrl('/not-a-route')
-      }))
-        .then((err) => {
-          assert.instanceOf(err, HttpError);
-          assert.instanceOf(err, HttpError.BadRequest);
-        }));
+      it('fails with a subtyped error', () =>
+        assert
+          .isRejected(
+            request({
+              uri: makeLocalUrl('/not-a-route'),
+            })
+          )
+          .then((err) => {
+            assert.instanceOf(err, HttpError);
+            assert.instanceOf(err, HttpError.BadRequest);
+          }));
 
       // This is somewhat difficult to test in web browser, but network errors in
       // browser look like network errors in browsers, so testing it isn`t that
       // critical. That said, moving the error-reformatting logic out of the
       // environment-specific implementations may make this easier to stub.
-      nodeOnly(it)('makes network errors look mostly like HTTP errors', () => assert.isRejected(request('https://localhost:0/not-a-route'))
-        .then((err) => {
+      nodeOnly(it)('makes network errors look mostly like HTTP errors', () =>
+        assert.isRejected(request('https://localhost:0/not-a-route')).then((err) => {
           assert.instanceOf(err, HttpError.NetworkOrCORSError);
-        }));
+        })
+      );
 
-      it('passes cookies', () => request({
-        uri: makeLocalUrl('/cookies/set'),
-        jar: true
-      })
-        .then(() => request({
-          uri: makeLocalUrl('/cookies/expect'),
-          jar: true
-        }))
-        .then((res) => {
-          assert.statusCode(res, 200);
-        }));
+      it('passes cookies', () =>
+        request({
+          uri: makeLocalUrl('/cookies/set'),
+          jar: true,
+        })
+          .then(() =>
+            request({
+              uri: makeLocalUrl('/cookies/expect'),
+              jar: true,
+            })
+          )
+          .then((res) => {
+            assert.statusCode(res, 200);
+          }));
 
       // this test fails in Safari 8+
       nodeOnly(it)('passes cookies to endpoints on other origins', () => {
         const p1 = request({
           uri: `http://localhost:${process.env.CORS_PORT}/cookies/set`,
-          jar: true
+          jar: true,
         })
-          .then(() => request({
-            uri: `http://localhost:${process.env.CORS_PORT}/cookies/expect`,
-            jar: true
-          }))
+          .then(() =>
+            request({
+              uri: `http://localhost:${process.env.CORS_PORT}/cookies/expect`,
+              jar: true,
+            })
+          )
           .then((res) => {
             assert.statusCode(res, 200);
           });
 
         const p2 = request({
           uri: `http://localhost:${process.env.CORS_PORT}/cookies/set`,
-          withCredentials: true
+          withCredentials: true,
         })
-          .then(() => request({
-            uri: `http://localhost:${process.env.CORS_PORT}/cookies/expect`,
-            withCredentials: true
-          }))
+          .then(() =>
+            request({
+              uri: `http://localhost:${process.env.CORS_PORT}/cookies/expect`,
+              withCredentials: true,
+            })
+          )
           .then((res) => {
             assert.statusCode(res, 200);
           });
@@ -148,34 +160,32 @@ describe('http-core', function () {
           uri: makeLocalUrl('/requires-basic-auth'),
           auth: {
             pass: 'basicpass',
-            user: 'basicuser'
-          }
-        })
-          .then((res) => {
-            assert.statusCode(res, 200);
-          });
+            user: 'basicuser',
+          },
+        }).then((res) => {
+          assert.statusCode(res, 200);
+        });
 
         const p2 = request({
           uri: makeLocalUrl('/requires-basic-auth'),
           auth: {
             password: 'basicpass',
-            username: 'basicuser'
-          }
-        })
-          .then((res) => {
-            assert.statusCode(res, 200);
-          });
+            username: 'basicuser',
+          },
+        }).then((res) => {
+          assert.statusCode(res, 200);
+        });
 
         return Promise.all([p1, p2]);
       });
 
-      it('makes Bearer Auth authenticated requests', () => request({
-        uri: makeLocalUrl('/requires-bearer-auth'),
-        auth: {
-          bearer: 'bearertoken'
-        }
-      })
-        .then((res) => {
+      it('makes Bearer Auth authenticated requests', () =>
+        request({
+          uri: makeLocalUrl('/requires-bearer-auth'),
+          auth: {
+            bearer: 'bearertoken',
+          },
+        }).then((res) => {
           assert.statusCode(res, 200);
         }));
 
@@ -187,35 +197,34 @@ describe('http-core', function () {
           object: {
             string: 'this is a another string',
             boolean: 'true',
-            integer: '5'
-          }
+            integer: '5',
+          },
         };
 
         return request({
           uri: makeLocalUrl('/return-qs-as-object'),
-          qs
-        })
-          .then((res) => {
-            assert.deepEqual(res.body, qs);
-          });
+          qs,
+        }).then((res) => {
+          assert.deepEqual(res.body, qs);
+        });
       });
     });
 
     flaky(it, process.env.SKIP_FLAKY_TESTS)('submits files as multipart form data', () => {
-      file.fetch('sample-powerpoint-two-page.ppt')
-        .then((f) => request({
+      file.fetch('sample-powerpoint-two-page.ppt').then((f) =>
+        request({
           method: 'POST',
           uri: makeLocalUrl('/files/metadata'),
           formData: {
-            files: [f]
+            files: [f],
           },
-          json: true
+          json: true,
+        }).then((res) => {
+          // This asserts that the server received the file and was able to
+          // decode its filename.
+          assert.equal(res.body[0].originalname, f.name);
         })
-          .then((res) => {
-            // This asserts that the server received the file and was able to
-            // decode its filename.
-            assert.equal(res.body[0].originalname, f.name);
-          }));
+      );
     });
 
     ['PUT', 'PATCH', 'POST'].forEach((method) => {
@@ -226,19 +235,18 @@ describe('http-core', function () {
             object: {
               string: 'this is a another string',
               boolean: 'true',
-              integer: '5'
-            }
+              integer: '5',
+            },
           };
 
           return request({
             method,
             uri: makeLocalUrl('/json/set'),
-            body: payload
-          })
-            .then((res) => {
-              assert.statusCode(res, 200);
-              assert.deepEqual(res.body, payload);
-            });
+            body: payload,
+          }).then((res) => {
+            assert.statusCode(res, 200);
+            assert.deepEqual(res.body, payload);
+          });
         });
 
         it('submits urlencoded form data', () =>
@@ -249,41 +257,40 @@ describe('http-core', function () {
             uri: makeLocalUrl('/form/reflect'),
             form: {
               a: 1,
-              b: 2
-            }
-          })
-            .then((res) => {
-              assert.statusCode(res, 200);
-              assert.deepEqual(res.body, {
-                a: '1',
-                b: '2'
-              });
-            }));
-
-        it('submits files', () => file.fetch('/sample-image-small-one.png')
-          .then((f) => request({
-            method,
-            uri: makeLocalUrl('/files/reflect'),
-            body: f,
-            json: false,
-            // Note: setting response type so the `reflect()ed` response is in
-            // a form we can use, this is not needed for normal file uploads.
-            responseType: 'blob'
-          })
-            .then((res) => {
+              b: 2,
+            },
+          }).then((res) => {
+            assert.statusCode(res, 200);
+            assert.deepEqual(res.body, {
+              a: '1',
+              b: '2',
+            });
+          }));
+
+        it('submits files', () =>
+          file.fetch('/sample-image-small-one.png').then((f) =>
+            request({
+              method,
+              uri: makeLocalUrl('/files/reflect'),
+              body: f,
+              json: false,
+              // Note: setting response type so the `reflect()ed` response is in
+              // a form we can use, this is not needed for normal file uploads.
+              responseType: 'blob',
+            }).then((res) => {
               assert.equal(res.body.type, f.type);
 
-              return file.isMatchingFile(res.body, f)
-                .then((result) => assert.isTrue(result));
-            })));
+              return file.isMatchingFile(res.body, f).then((result) => assert.isTrue(result));
+            })
+          ));
 
-        (inBrowser ? it : it.skip)('emits upload progress events', () => file.fetch('/sample-image-small-one.png')
-          .then((f) => {
+        (inBrowser ? it : it.skip)('emits upload progress events', () =>
+          file.fetch('/sample-image-small-one.png').then((f) => {
             const options = {
               method,
               uri: makeLocalUrl('/files/reflect'),
               body: f,
-              json: false
+              json: false,
             };
 
             const promise = request(options);
@@ -295,7 +302,8 @@ describe('http-core', function () {
             return promise.then(() => {
               assert.called(spy);
             });
-          }));
+          })
+        );
       });
     });
   });

package/test/unit/spec/interceptors/http-status.js

@@ -11,9 +11,13 @@ describe('http-core', () => {
       let interceptor;
 
       beforeEach(() => {
-        interceptor = Reflect.apply(HttpStatusInterceptor.create, {
-          sessionId: 'mock-webex_uuid'
-        }, []);
+        interceptor = Reflect.apply(
+          HttpStatusInterceptor.create,
+          {
+            sessionId: 'mock-webex_uuid',
+          },
+          []
+        );
       });
 
       describe('#onResponse', () => {
@@ -21,21 +25,20 @@ describe('http-core', () => {
           const response = {
             statusCode: 404,
             body: {
-              errorCode: 2000002
-            }
+              errorCode: 2000002,
+            },
           };
 
-          return interceptor.onResponse({}, response)
-            .then((result) => {
-              assert.equal(result, response);
-            });
+          return interceptor.onResponse({}, response).then((result) => {
+            assert.equal(result, response);
+          });
         });
         it('rejects when locus redirect is not intended', () => {
           const response = {
             statusCode: 404,
             body: {
-              errorCode: 2000001
-            }
+              errorCode: 2000001,
+            },
           };
 
           assert.isRejected(interceptor.onResponse({}, response));