Scene System Testing [AARD-1938]

[AlexD717]

Task

Create unit tests for each element found in fission/src/systems/scene/.

AARD-1938

Symptom

No unit tests currently exist for elements in fission/src/systems/scene/.

Solution

Create unit test for each element in fission/src/systems/scene/.
Note: No unit tests were created for SceneObject.ts due to it being very simple and other tests already testing its functionality.

Verification

All unit tests pass

---

Before merging, ensure the following criteria are met:

• All acceptance criteria outlined in the ticket are met.
• Necessary test cases have been added and updated.
• A feature toggle or safe disable path has been added (if applicable).
• User-facing polish:
  • Ask: "Is this ready-looking?"
• Cross-linking between Jira and GitHub:
  • PR links to the relevant Jira issue.
  • Jira ticket has a comment referencing this PR.

[rutmanz]

I'm only about halfway done reading through this but I'm concerned that a lot of the tests here aren't really checking for things that matter. Like #1179, it kind of feels like these might be AI generated, and I'm not sure what a lot of them are doing

[AlexD717]

I'm only about halfway done reading through this but I'm concerned that a lot of the tests here aren't really checking for things that matter. Like #1179, it kind of feels like these might be AI generated, and I'm not sure what a lot of them are doing

I did use AI to help make these unit tests, and I thought that I looked through them and fixed all the mistakes. I have clearly missed some, so I will go through again and double check that everything is good.