From 7c332c1b0f4649ab773665e8c45f2729be267adf Mon Sep 17 00:00:00 2001 From: abrichr Date: Wed, 2 Sep 2026 01:16:10 -0400 Subject: [PATCH] fix: rustfmt overlay coords and skip POSIX 0600 on Windows rustfmt wants PhysicalPosition args on one line. NTFS does not report POSIX 0o600, so keep that assertion off Windows. The node table file still has to exist on every OS. --- src-tauri/src/commands.rs | 3 +-- tests/test_engine/test_authoring_runner.py | 8 ++++++-- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/src-tauri/src/commands.rs b/src-tauri/src/commands.rs index b2c4f76..d31e0cd 100644 --- a/src-tauri/src/commands.rs +++ b/src-tauri/src/commands.rs @@ -157,8 +157,7 @@ pub fn set_control_overlay_layout(app: AppHandle, layout: String) -> Result<(), .map_err(|error| format!("could not stage the control overlay: {error}"))?; window .set_position(Position::Physical(PhysicalPosition::new( - position.x, - position.y, + position.x, position.y, ))) .map_err(|error| format!("could not place the overlay stage: {error}"))?; } diff --git a/tests/test_engine/test_authoring_runner.py b/tests/test_engine/test_authoring_runner.py index 7f8585b..512e1f2 100644 --- a/tests/test_engine/test_authoring_runner.py +++ b/tests/test_engine/test_authoring_runner.py @@ -3,6 +3,7 @@ from __future__ import annotations import json +import os import stat from pathlib import Path from types import SimpleNamespace @@ -403,8 +404,11 @@ def test_node_table_is_mode_0600(tmp_path: Path) -> None: runner.allow() runner._observe() assert runner._node_table is not None - mode = stat.S_IMODE(runner._node_table._path.stat().st_mode) - assert mode == 0o600 + path = runner._node_table._path + assert path.is_file() + # NTFS does not report POSIX 0o600. Production already skips fchmod there. + if os.name != "nt": + assert stat.S_IMODE(path.stat().st_mode) == 0o600 def test_projector_drops_value_title_and_six_digit_names(tmp_path: Path) -> None: