fix: clamp image dimensions to min and max (#460)

When we render on our side, the full image should still be captured.
percy · Jan 31, 2020 · ca68b0f · ca68b0f
1 parent 06df6ff
commit ca68b0f
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/services/image-snapshot-service.ts b/src/services/image-snapshot-service.ts
@@ -93,8 +93,9 @@ export default class ImageSnapshotService extends PercyClientService {
   ): Promise<any> {
     return this.percyClient.createSnapshot(this.buildId, resources, {
       name,
-      widths: [width],
-      minimumHeight: height,
+      // clamp between 10px - 2000px
+      widths: [Math.max(10, Math.min(width, 2000))],
+      minimumHeight: Math.max(10, Math.min(height, 2000)),
       clientInfo: 'percy-upload',
     }).then(async (response: any) => {
       await this.percyClient.uploadMissingResources(this.buildId, response, resources)