(#341) Remove Region#scaled

Author: s1hofmann

lib/region.class.spec.ts

@@ -1,7 +1,7 @@
 import { Region } from "./region.class";
 
 describe("Region", () => {
-  it("should calcutate the correct area of a region", () => {
+  it("should calculate the correct area of a region", () => {
     const region = new Region(0, 0, 100, 100);
     const expected = 10000;
 
@@ -14,56 +14,4 @@ describe("Region", () => {
 
     expect(region.toString()).toEqual(expected);
   });
-
-  it("should scale and translate in x", () => {
-    const scaleFactor = 2.0;
-    const region = new Region(100, 100, 100, 100);
-    const result = Region.scaled(region, scaleFactor);
-
-    expect(result.left).toBeCloseTo(region.left * scaleFactor);
-    expect(result.width).toBeCloseTo(region.width * scaleFactor);
-    expect(result.top).toBeCloseTo(region.top);
-    expect(result.height).toBeCloseTo(region.height);
-  });
-
-  it("should scale and translate in y", () => {
-    const scaleFactor = 2.0;
-    const region = new Region(200, 250, 100, 100);
-    const result = Region.scaled(region, undefined, scaleFactor);
-
-    expect(result.left).toBeCloseTo(region.left);
-    expect(result.width).toBeCloseTo(region.width);
-    expect(result.top).toBeCloseTo(region.top * scaleFactor);
-    expect(result.height).toBeCloseTo(region.height * scaleFactor);
-  });
-
-  it("should scale and translate in both x and y", () => {
-    const scaleFactorX = 1.75;
-    const scaleFactorY = 2.5;
-    const region = new Region(300, 720, 100, 100);
-    const result = Region.scaled(region, scaleFactorX, scaleFactorY);
-
-    expect(result.left).toBeCloseTo(region.left * scaleFactorX);
-    expect(result.width).toBeCloseTo(region.width * scaleFactorX);
-    expect(result.top).toBeCloseTo(region.top * scaleFactorY);
-    expect(result.height).toBeCloseTo(region.height * scaleFactorY);
-  });
-
-  it("should throw an error when scaling to 0 in x", () => {
-    const scaleFactorX = 0.0;
-    const scaleFactorY = 2.5;
-    const region = new Region(300, 720, 100, 100);
-    expect(() => Region.scaled(region, scaleFactorX, scaleFactorY)).toThrow(
-      `Scaling to 0. Please check parameters: scaleX: ${scaleFactorX}, scaleY: ${scaleFactorY}`,
-    );
-  });
-
-  it("should throw an error when scaling to 0 in y", () => {
-    const scaleFactorX = 2.5;
-    const scaleFactorY = 0.0;
-    const region = new Region(300, 720, 100, 100);
-    expect(() => Region.scaled(region, scaleFactorX, scaleFactorY)).toThrow(
-      `Scaling to 0. Please check parameters: scaleX: ${scaleFactorX}, scaleY: ${scaleFactorY}`,
-    );
-  });
 });

lib/region.class.ts

@@ -1,22 +1,4 @@
 export class Region {
-  public static scaled(
-    region: Region,
-    scaleX: number = 1.0,
-    scaleY: number = 1.0,
-  ): Region {
-    if (scaleX === 0 || scaleY === 0) {
-      throw new Error(
-        `Scaling to 0. Please check parameters: scaleX: ${scaleX}, scaleY: ${scaleY}`,
-      );
-    }
-    return new Region(
-      region.left * scaleX,
-      region.top * scaleY,
-      region.width * scaleX,
-      region.height * scaleY,
-    );
-  }
-
   constructor(
     public left: number,
     public top: number,