fix: change how render width, height and scale works for html_template expectedPackages

Author: nytamin

shared/packages/api/src/expectationApi.ts

@@ -354,7 +354,12 @@ export namespace Expectation {
 				renderer?: {
 					width?: number
 					height?: number
-					zoom?: number
+					/**
+					 * Scale the rendered width and height with this value, and also zoom the content accordingly.
+					 * For example, if the width is 1920 and scale is 0.5, the width will be scaled to 960.
+					 * (Defaults to 1)
+					 */
+					scale?: number
 					/** Background color, #RRGGBB, CSS-string, "transparent" or "default" (defaults to "default") */
 					background?: string
 					userAgent?: string

shared/packages/worker/src/worker/workers/genericWorker/expectationHandlers/renderHTML.ts

@@ -679,20 +679,24 @@ class HTMLRenderer {
 	}
 
 	private spawnHTMLRendererProcess() {
+		let width = this.exp.endRequirement.version.renderer?.width
+		let height = this.exp.endRequirement.version.renderer?.height
+		const scale = this.exp.endRequirement.version.renderer?.scale
+		if (width !== undefined && height !== undefined && scale !== undefined) {
+			width = Math.floor(width * scale)
+			height = Math.floor(height * scale)
+		}
+
 		const args = compact<string>([
 			`--`,
 			`--url=${this.url}`,
-			this.exp.endRequirement.version.renderer?.width !== undefined &&
-				`--width=${this.exp.endRequirement.version.renderer?.width}`,
-			this.exp.endRequirement.version.renderer?.height !== undefined &&
-				`--height=${this.exp.endRequirement.version.renderer?.height}`,
-			this.exp.endRequirement.version.renderer?.zoom !== undefined &&
-				`--zoom=${this.exp.endRequirement.version.renderer?.zoom}`,
+			width !== undefined && `--width=${width}`,
+			height !== undefined && `--height=${height}`,
+			scale !== undefined && `--zoom=${scale}`,
 			`--outputPath=${this.outputPath}`,
 			`--background=${this.exp.endRequirement.version.renderer?.background ?? 'default'}`,
 			`--interactive=true`,
 		])
-		this.worker.logger.info('AAAA ' + JSON.stringify(args))
 		this.htmlRendererProcess = spawn(getHtmlRendererExecutable(), args, {
 			windowsVerbatimArguments: true, // To fix an issue with arguments on Windows
 		})