@percy/client 1.6.4 → 1.7.2

package/dist/client.js

@@ -187,19 +187,25 @@ export class PercyClient {
     timeout = 10 * 60 * 1000,
     interval = 1000
   }, onProgress) {
-    if (commit && !project) {
+    if (!project && commit) {
       throw new Error('Missing project path for commit');
-    } else if (!commit && !build) {
-      throw new Error('Missing build ID or commit SHA');
+    } else if (!project && !build) {
+      throw new Error('Missing project path or build ID');
     } else if (project) {
       validateProjectPath(project);
     }
 
+    commit || (commit = this.env.git.sha);
+    if (!build && !commit) throw new Error('Missing build commit');
     let sha = commit && (git(`rev-parse ${commit}`) || commit);
 
-    let fetchData = async () => build ? (await this.getBuild(build)).data : (await this.getBuilds(project, {
-      sha
-    })).data[0];
+    let fetchData = async () => {
+      var _await$this$getBuilds;
+
+      return build ? (await this.getBuild(build)).data : (_await$this$getBuilds = (await this.getBuilds(project, {
+        sha
+      })).data) === null || _await$this$getBuilds === void 0 ? void 0 : _await$this$getBuilds[0];
+    };
 
     this.log.debug(`Waiting for build ${build || `${project} (${commit})`}...`); // recursively poll every second until the build finishes
 
@@ -213,7 +219,7 @@ export class PercyClient {
         if (updated) {
           t = Date.now(); // no new data within the timeout
         } else if (Date.now() - t >= timeout) {
-          throw new Error('Timeout exceeded without an update');
+          throw new Error(state == null ? 'Build not found' : 'Timeout exceeded with no updates');
         } // call progress every update after the first update
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@percy/client",
-  "version": "1.6.4",
+  "version": "1.7.2",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -31,8 +31,8 @@
     "test:coverage": "yarn test --coverage"
   },
   "dependencies": {
-    "@percy/env": "1.6.4",
-    "@percy/logger": "1.6.4"
+    "@percy/env": "1.7.2",
+    "@percy/logger": "1.7.2"
   },
-  "gitHead": "651c0fad62f128053723f9b8d565c69496dc51e4"
+  "gitHead": "d9da4a4f9eafc3c3ac361a50f679f030172ad362"
 }

package/test/helpers.js

@@ -79,17 +79,19 @@ export async function mockRequests(baseUrl, defaultReply = () => [200]) {
   if (!jasmine.isSpy(http.request)) {
     spyOn(http, 'request').and.callFake((...a) => new MockRequest(null, ...a));
     spyOn(http, 'get').and.callFake((...a) => new MockRequest(null, ...a).end());
+    mockRequests.spies = new Map();
   }
 
   let any = jasmine.anything();
-  let match = o => o.hostname === hostname &&
-    (o.path ?? o.pathname).startsWith(pathname);
+  let match = o => o.hostname === hostname && (o.path ?? o.pathname).startsWith(pathname);
   let reply = jasmine.createSpy('reply').and.callFake(defaultReply);
+  let spies = mockRequests.spies.get(baseUrl) ?? {};
 
-  http.request.withArgs({ asymmetricMatch: match })
-    .and.callFake((...a) => new MockRequest(reply, ...a));
-  http.get.withArgs({ asymmetricMatch: u => match(new URL(u)) }, any, any)
-    .and.callFake((...a) => new MockRequest(reply, ...a).end());
+  spies.request = spies.request ?? http.request.withArgs({ asymmetricMatch: match });
+  spies.request.and.callFake((...a) => new MockRequest(reply, ...a));
+  spies.get = spies.get ?? http.get.withArgs({ asymmetricMatch: u => match(new URL(u)) }, any, any);
+  spies.get.and.callFake((...a) => new MockRequest(reply, ...a).end());
+  mockRequests.spies.set(baseUrl, spies);
 
   return reply;
 }