test: fix main's three red checks (fixture fopen deprecation; DLSS toggle stub lacks switching; actionlint SC2012) #2092

Merged

NotRexed merged 2 commits from fix/main-red-tests into main

2026-11-11 02:57:41 +00:00

NotRexed commented

2026-11-11 02:15:49 +00:00

(Migrated from github.com)

Main has been red since #2046 and the builder track switch landed:

tests/test_pipeline_state_portable.py (Windows): the fixture compiles with `-Werror` and clang on Windows marks `fopen` deprecated. It now passes `-D_CRT_SECURE_NO_WARNINGS`, as the other fixture tests do.
tests/test_make_game_ngx.py (all OSes): `Builder.apply_mode` now calls `self.switching()`, and the test's stub had none. It gets `switching=lambda: False`.
.github/workflows/linux-appimage.yml (actionlint, from #2079): the object count used ls | wc -l (shellcheck SC2012). It counts with a glob array, as the loop above it already does.

Checked locally: both pass (fixture built with clang++ and Vulkan headers 1.4.345; test_make_game_ngx 17 tests OK). Unblocks #1999 and the other open PRs.

ðŸ¤– Generated with Claude Code

Main has been red since #2046 and the builder track switch landed: - **tests/test_pipeline_state_portable.py** (Windows): the fixture compiles with \`-Werror\` and clang on Windows marks \`fopen\` deprecated. It now passes \`-D_CRT_SECURE_NO_WARNINGS\`, as the other fixture tests do. - **tests/test_make_game_ngx.py** (all OSes): \`Builder.apply_mode\` now calls \`self.switching()\`, and the test's stub had none. It gets \`switching=lambda: False\`. - **.github/workflows/linux-appimage.yml** (actionlint, from #2079): the object count used `ls | wc -l` (shellcheck SC2012). It counts with a glob array, as the loop above it already does. Checked locally: both pass (fixture built with clang++ and Vulkan headers 1.4.345; test_make_game_ngx 17 tests OK). Unblocks #1999 and the other open PRs. ðŸ¤– Generated with [Claude Code](https://claude.com/claude-code)