Fix failing kicad tests

gerbonara/apertures.py

@@ -254,11 +254,11 @@ class RectangleAperture(Aperture):
 
     def to_macro(self, rotation=0):
         return ApertureMacroInstance(GenericMacros.rect,
-                [MM(self.w, self.unit),
+                (MM(self.w, self.unit),
                     MM(self.h, self.unit),
                     MM(self.hole_dia, self.unit) or 0,
                     0,
-                    rotation])
+                    rotation))
 
     def _params(self, unit=None):
         return _strip_right(
@@ -321,11 +321,11 @@ class ObroundAperture(Aperture):
             inst = replace(self, w=self.h, h=self.w, hole_dia=self.hole_dia)
 
         return ApertureMacroInstance(GenericMacros.obround,
-                [MM(inst.w, self.unit),
+                (MM(inst.w, self.unit),
                  MM(inst.h, self.unit),
                  MM(inst.hole_dia, self.unit) or 0,
                  0,
-                 inst.rotation + rotation])
+                 rotation))
 
     def _params(self, unit=None):
         return _strip_right(
@@ -369,7 +369,7 @@ class PolygonAperture(Aperture):
     @lru_cache()
     def rotated(self, angle=0):
         if angle != 0:
-            return replace(self, rotatio=self.rotation + angle)
+            return replace(self, rotation=self.rotation + angle)
         else:
             return self
 

gerbonara/cad/kicad/footprints.py

@@ -404,7 +404,7 @@ class Pad:
         yield go.Flash(self.at.x+ox, self.at.y+oy, aperture, unit=MM)
 
     def aperture(self, margin=None):
-        rotation = -math.radians(self.at.rotation)
+        rotation = math.radians(self.at.rotation)
         margin = margin or 0
 
         if self.shape == Atom.circle:
@@ -437,7 +437,7 @@ class Pad:
             if dx != 0:
                 x, y = y, x
                 dy = dx
-                rotation -= math.pi/2
+                rotation += math.pi/2
 
             if margin <= 0:
                 # Note: KiCad already uses MM units, so no conversion needed here.

gerbonara/utils.py

@@ -84,7 +84,7 @@ class LengthUnit:
         if unit == self or unit is None or value is None:
             return value
 
-        return value * unit.factor / self.factor
+        return value * unit.this_in_mm / self.this_in_mm
 
     def convert_to(self, unit, value):
         """ :py:meth:`.LengthUnit.convert_from` but in reverse. """