Merge branch 'main' into Parts-inside

Author: Beakerboy

src/extras/BuildingShapeUtils.js

@@ -24,7 +24,6 @@ class BuildingShapeUtils extends ShapeUtils {
     const elements = way.getElementsByTagName('nd');
 
     // Get the coordinates of all the nodes and add them to the shape outline.
-
     for (const element of elements) {
       ref = element.getAttribute('ref');
       nodes.push(nodelist[ref]);

test/utils.test.js

@@ -127,6 +127,13 @@ rectangle.lineTo(-4.332738077015795, 5.88221335051411);
 rectangle.lineTo(4.332747472106493, 5.88221335051411);
 rectangle.lineTo(4.332747472106493, -5.882209888874915);
 
+const rectangle = new Shape();
+rectangle.moveTo(-4.332738077015795, -5.882209888874915);
+rectangle.lineTo(-4.332738077015795, 5.88221335051411);
+rectangle.lineTo(4.332747472106493, 5.88221335051411);
+rectangle.lineTo(4.332747472106493, -5.882209888874915);
+rectangle.lineTo(-4.332738077015795, -5.882209888874915);
+
 test('Extents no Rotation', () => {
   expect(BuildingShapeUtils.extents(rightTriangle)).toStrictEqual([-1, -1, 1, 1]);
 });