From a139adaa2c7788c0f5c882fb2f767ef8449df3d7 Mon Sep 17 00:00:00 2001 From: Gorgious56 Date: Sun, 7 Jun 2026 02:05:53 +0200 Subject: [PATCH] Apply black formatting to satisfy lint-formatting CI MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Three files flagged by black --check on the lint-formatting job: * bim/module/geometry/operator.py — single-arg `.update(...)` rejoined onto one line under the 120-char budget. * test/bim/module/model/test_wall_gizmos.py — same join on a _make_path_rel call. * test/modal/test_modal.py — pre-existing baseline noise picked up via the upstream merge: PEP-8 blank-line separators between top- level functions, `0.68+` → `0.68 +`, double quotes, trailing whitespace stripped. No behavioural change; pure whitespace. Generated with the assistance of an AI coding tool. --- .../bonsai/bim/module/geometry/operator.py | 4 +--- .../test/bim/module/model/test_wall_gizmos.py | 4 +--- src/bonsai/test/modal/test_modal.py | 17 ++++++++++++----- 3 files changed, 14 insertions(+), 11 deletions(-) diff --git a/src/bonsai/bonsai/bim/module/geometry/operator.py b/src/bonsai/bonsai/bim/module/geometry/operator.py index 64d90af4be..3a6b911c15 100644 --- a/src/bonsai/bonsai/bim/module/geometry/operator.py +++ b/src/bonsai/bonsai/bim/module/geometry/operator.py @@ -1289,9 +1289,7 @@ class OverrideDuplicateMove(bpy.types.Operator): all_objects_to_select.add(part_obj) # Non-IFC duplicates aren't tracked in old_to_new but are left selected by duplicate_ifc_objects - all_objects_to_select.update( - obj for obj in context.selected_objects if not tool.Ifc.get_entity(obj) - ) + all_objects_to_select.update(obj for obj in context.selected_objects if not tool.Ifc.get_entity(obj)) # Deselect everything first bpy.ops.object.select_all(action="DESELECT") diff --git a/src/bonsai/test/bim/module/model/test_wall_gizmos.py b/src/bonsai/test/bim/module/model/test_wall_gizmos.py index f88ee3a6f9..2d1f8c568d 100644 --- a/src/bonsai/test/bim/module/model/test_wall_gizmos.py +++ b/src/bonsai/test/bim/module/model/test_wall_gizmos.py @@ -272,9 +272,7 @@ def test_iter_path_connections_includes_fillet_corner_partner(): # LAYER2 wall's perspective. self_elem = object() fillet_partner = object() - rel = _make_path_rel( - relating=self_elem, related=fillet_partner, relating_ct="ATEND", related_ct="ATSTART" - ) + rel = _make_path_rel(relating=self_elem, related=fillet_partner, relating_ct="ATEND", related_ct="ATSTART") elem = SimpleNamespace(ConnectedTo=[rel], ConnectedFrom=[]) result = _run_iter_path_connections(elem, partner_predicate=lambda e: e is fillet_partner) assert result == [(fillet_partner, "ATEND", "ATSTART")] diff --git a/src/bonsai/test/modal/test_modal.py b/src/bonsai/test/modal/test_modal.py index 07ce8b90ed..1ea6b1dbd5 100644 --- a/src/bonsai/test/modal/test_modal.py +++ b/src/bonsai/test/modal/test_modal.py @@ -114,11 +114,13 @@ def new_project(): props.template_file = "0" tool.Blender.get_addon_preferences().should_play_chaching_sound = False + def get_area_and_region(window): area = next(area for area in window.screen.areas if area.type == "VIEW_3D") region = next(region for region in area.regions if region.type == "WINDOW") return area, region + def test_snap_object_detection(window): yield from preset_event_simulate(window, "MOUSEMOVE", "NOTHING", 0, 0) area, region = get_area_and_region(window) @@ -161,6 +163,7 @@ def test_snap_object_detection(window): yield from preset_event_simulate(window, "RET", "TAP", x, y) yield "FINISHED" + def test_snap_partially_behind_camera(window): yield from preset_event_simulate(window, "MOUSEMOVE", "NOTHING", 0, 0) area, region = get_area_and_region(window) @@ -215,10 +218,11 @@ def test_snap_partially_behind_camera(window): yield from preset_event_simulate(window, "RET", "TAP", x, y) yield "FINISHED" + def test_snap_in_xray_mode(window): yield from preset_event_simulate(window, "MOUSEMOVE", "NOTHING", 0, 0) area, region = get_area_and_region(window) - x = round(area.width * 0.68+ area.x) + x = round(area.width * 0.68 + area.x) y = round(area.height * 0.54 + area.y) area.spaces[0].shading.show_xray = True @@ -244,10 +248,11 @@ def test_snap_in_xray_mode(window): assert_msg = "Object should be an IfcFurniture" assert snap_point.snap_object.split("/")[0] == "IfcFurniture", assert_msg _assert_pass(assert_msg) - + yield from preset_event_simulate(window, "RET", "TAP", x, y) yield "FINISHED" + def test_snap_far_from_origin(window): bpy.context.view_layer.objects.active = None bpy.ops.object.select_all(action="DESELECT") @@ -258,12 +263,11 @@ def test_snap_far_from_origin(window): yield from preset_event_simulate(window, "ESC", "TAP", x, y) - bpy.data.objects['IfcBuildingElementProxy/Cube'].select_set(True) + bpy.data.objects["IfcBuildingElementProxy/Cube"].select_set(True) yield from preset_event_simulate(window, "MOUSEMOVE", "NOTHING", x, y) with bpy.context.temp_override(area=area, region=region, space_data=area.spaces[0]): bpy.ops.view3d.view_selected() - measure_settings = tool.Project.get_measure_tool_settings() measure_settings.measurement_type = "POLYLINE" for obj in tool.Blender.get_selected_objects(): @@ -296,6 +300,7 @@ def test_snap_far_from_origin(window): yield from preset_event_simulate(window, "RET", "TAP", x, y) yield "FINISHED" + def test_snap_targets(window): yield from preset_event_simulate(window, "MOUSEMOVE", "NOTHING", 0, 0) area, region = get_area_and_region(window) @@ -315,7 +320,7 @@ def test_snap_targets(window): options.append((prop, props.rna_type.properties[prop].name)) for prop, name in options: - any(setattr(props, prop2, prop2 == prop) for prop2, _ in options) # set prop to true and others to false + any(setattr(props, prop2, prop2 == prop) for prop2, _ in options) # set prop to true and others to false measure_settings = tool.Project.get_measure_tool_settings() measure_settings.measurement_type = "POLYLINE" for obj in tool.Blender.get_selected_objects(): @@ -353,6 +358,7 @@ def test_snap_targets(window): assert name in snap_types _assert_pass(assert_msg) + def test_draw_polyline_wall(window, x, y): yield from preset_event_simulate(window, "ESC", "TAP", x, y) area, region = get_area_and_region(window) @@ -439,6 +445,7 @@ def run_tests(): _next() + if __name__ == "__main__": new_project() run_tests()