Add tests that make sure the test assembly loading scripts can be interpreted and run #157

New issue

Closed

opened 2026-06-12 22:43:44 +00:00 by Vectornaut · 1 comment

Vectornaut commented

2026-06-12 22:43:44 +00:00

Member

In pull request #155, we stopped accepting and ignoring arguments for Sphere and Point descriptors. This revealed an error in the loading script for the "Pointed" test assembly by causing interpretation of the miswritten script to fail, as it always should have. I only noticed the interpretation failure when I happened to run the loading script by hand. Adding tests that make sure the test assembly loading scripts can be interpreted and run would help us catch stuff like this more reliably in the future.

In pull request #155, we stopped accepting and ignoring arguments for `Sphere` and `Point` descriptors. This revealed an error in the loading script for the "Pointed" test assembly by causing interpretation of the miswritten script to fail, as it always should have. I only noticed the interpretation failure when I happened to run the loading script by hand. Adding tests that make sure the test assembly loading scripts can be interpreted and run would help us catch stuff like this more reliably in the future.

Vectornaut added the

enhancement

labels

2026-06-12 22:44:35 +00:00

Vectornaut referenced this issue

2026-06-12 22:57:18 +00:00

Polish behavior and code in the action system #155

Vectornaut changed title from ~~Add tests that make sure the test assembly loadings scripts can be interpreted and run~~ to Add tests that make sure the test assembly loading scripts can be interpreted and run

2026-06-23 10:23:50 +00:00

Vectornaut referenced this issue

2026-06-23 18:43:29 +00:00

Correct the rest of the test assembly loading scripts #161

Vectornaut referenced this issue

2026-06-23 18:50:43 +00:00

Move the command language interpretation system into its own module #159