Skip to content

Update two nightly tests for the 2.7.4 changes - #186

Open
Views2k wants to merge 1 commit into
mainfrom
fix/nightly-after-274
Open

Views2k wants to merge 1 commit into
mainfrom
fix/nightly-after-274

Conversation

@Views2k

@Views2k Views2k commented Nov 8, 2026

Copy link
Copy Markdown
Owner

Two nightly-only tests encode behavior that 2.7.4 changed on purpose; the PR gate selects tests by changed-file stem, so neither ran on the fixing PRs.

  • ElectricNativeRoutingTests.TireTemperatureGaugesShareTheNativeRootAndRemainAvailableForElectricLayouts asserted the analogue tire gauge's IsElectricMaterial binding path. Retry a missing stock gauge every 250 ms and notify electric-material bindings once #178 binds IsElectricPowertrain, which changes only with the powertrain; the expectation follows.
  • SupplementaryOperationFailureTests.SetupTimeoutReportsWhatTheReceiverActuallyObserved expected the 45-second guidance checkpoint to record a timeout outcome with the receiver's reason. Stop recording the setup test's 45 s checkpoint as a timeout #180 stopped recording the checkpoint (a player who keeps going has not failed the test). The theory becomes CancellingAfterTheGuidanceCheckpointRecordsNoTimeout: for the same receiver states, cancelling after the checkpoint records no timeout and no incident, only cancelled. The real timeout path (listener stopped) is covered by SetupTelemetryTests.

Verification

  • Test-AppBehavior.ps1 -Full on main plus Mask the game's thread-local slot operand in the gauge field templates #184 and Recognize the Steam 6.483 tune benchmark #185: Core 784, Telemetry 96, Update 184, Updater 76 passed; App 6125 of 6130 passed. The five failures were these two tests (four cases) and WpfStyleRuntimeTests.ApplicationStylesAndNativeControlsPassRuntimeChecks, which failed on the maintainer PC at a desktop-dependent assertion (CalmSidebarTests.AssertUniformDockItems label fit, once at HudNativeHostRecoveryTests.AssertHostRebuildOnAttach); see the release PR for its CI nightly result.
  • With this change: ElectricNativeRoutingTests, SupplementaryOperationFailureTests and SetupTelemetryTests: 90 passed, 0 failed.

The analogue tire temperature gauge binds IsElectricPowertrain since the
gauge views stopped re-evaluating NativeGaugeFrame.IsElectric on every
frame, and the setup wizard's Data Out test no longer records its guidance
checkpoint as a timeout. Both tests run only in the nightly suite, so the
pull request gate did not select them.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant