Cleanup minor formatting issues and refine validation logic across components and tests
This commit is contained in:
@@ -21015,9 +21015,7 @@ export function App({ store, initialStatusMessage }: AppProps) {
|
||||
<input
|
||||
data-testid="player-start-targeting-look-switch"
|
||||
type="checkbox"
|
||||
checked={
|
||||
playerStartAllowLookInputTargetSwitchDraft
|
||||
}
|
||||
checked={playerStartAllowLookInputTargetSwitchDraft}
|
||||
onChange={(event) => {
|
||||
const nextValue = event.currentTarget.checked;
|
||||
setPlayerStartAllowLookInputTargetSwitchDraft(
|
||||
|
||||
@@ -969,7 +969,9 @@ function validateWorldSettings(
|
||||
}
|
||||
|
||||
if (
|
||||
!isNonNegativeFiniteNumber(advancedRendering.dynamicGlobalIllumination.radius)
|
||||
!isNonNegativeFiniteNumber(
|
||||
advancedRendering.dynamicGlobalIllumination.radius
|
||||
)
|
||||
) {
|
||||
diagnostics.push(
|
||||
createDiagnostic(
|
||||
|
||||
@@ -185,8 +185,7 @@ describe("validateSceneDocument", () => {
|
||||
expect.arrayContaining([
|
||||
expect.objectContaining({
|
||||
code: "missing-control-actor-target",
|
||||
path:
|
||||
"sequences.sequences.sequence-missing-actor.effects.0.effect.target.actorId"
|
||||
path: "sequences.sequences.sequence-missing-actor.effects.0.effect.target.actorId"
|
||||
}),
|
||||
expect.objectContaining({
|
||||
code: "missing-control-actor-target",
|
||||
|
||||
@@ -882,9 +882,7 @@ describe("scene document JSON", () => {
|
||||
expect(migratedDocument.version).toBe(SCENE_DOCUMENT_VERSION);
|
||||
expect(
|
||||
migratedDocument.world.advancedRendering.dynamicGlobalIllumination
|
||||
).toEqual(
|
||||
emptyScene.world.advancedRendering.dynamicGlobalIllumination
|
||||
);
|
||||
).toEqual(emptyScene.world.advancedRendering.dynamicGlobalIllumination);
|
||||
});
|
||||
|
||||
it("defaults missing water reflection mode and clamps legacy foam limits during migration", () => {
|
||||
|
||||
Reference in New Issue
Block a user