Parameterise browser zoom tests

ScottLogic · Jun 19, 2024 · 8abe6a2 · 8abe6a2
1 parent 1165f10
commit 8abe6a2
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/src/app/carbon-estimation/carbon-estimation.component.spec.ts b/src/app/carbon-estimation/carbon-estimation.component.spec.ts
@@ -87,14 +87,16 @@ describe('CarbonEstimationComponent', () => {
     });
   });
 
-  it('should scale chart height according to browser zoom factor', () => {
-    spyOn(component.chart as ChartComponent, 'updateOptions');
-    spyOnProperty(component.detailsPanel.nativeElement, 'clientHeight').and.returnValue(200);
+  [0.5, 1, 2, 5].forEach(browserZoomFactor => {
+    it(`should scale chart height appropriately if the browser zoom factor is ${browserZoomFactor}`, () => {
+      spyOn(component.chart as ChartComponent, 'updateOptions');
+      spyOnProperty(component.detailsPanel.nativeElement, 'clientHeight').and.returnValue(200);
 
-    component.onResize(1500, 1000, 2000, 2);
+      component.onResize(1500, 1000, 2000, browserZoomFactor);
 
-    expect(component.chart?.updateOptions).toHaveBeenCalledOnceWith({
-      chart: { height: (1500 - estimatorBaseHeight - 200) * 2 },
+      expect(component.chart?.updateOptions).toHaveBeenCalledOnceWith({
+        chart: { height: (1500 - estimatorBaseHeight - 200) * browserZoomFactor },
+      });
     });
   });