Skip to content

feat(nimbus): Add a test that loads all FML manifests#15090

Merged
freshstrangemusic merged 1 commit intomainfrom
freshstrangemusic/push-vxlnzootnlmm
Mar 31, 2026
Merged

feat(nimbus): Add a test that loads all FML manifests#15090
freshstrangemusic merged 1 commit intomainfrom
freshstrangemusic/push-vxlnzootnlmm

Conversation

@freshstrangemusic
Copy link
Copy Markdown
Member

Because:

  • we are constantly adding and updating feature manifests;
  • we are constantly updating our application-services dependency; and
  • we have no tests that validate that existing manifests are valid in these cases

this commit:

  • adds a test case per application that validates the vendored FML files are loadable.

Fixes #15020

@freshstrangemusic freshstrangemusic force-pushed the freshstrangemusic/push-vxlnzootnlmm branch 4 times, most recently from ae280f1 to edc92ff Compare March 30, 2026 20:15
Because:

- we are constantly adding and updating feature manifests; 
- we are constantly updating our application-services dependency; and
- we have no tests that validate that existing manifests are valid in
  these cases

this commit:

- adds a test case per application that validates the vendored FML files
  are loadable.

Fixes #15020
Copy link
Copy Markdown
Collaborator

@jaredlockhart jaredlockhart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome ty beth 🙏 I can't believe they're so fast XD

@freshstrangemusic freshstrangemusic added this pull request to the merge queue Mar 31, 2026
Merged via the queue into main with commit c7a0120 Mar 31, 2026
32 checks passed
@freshstrangemusic freshstrangemusic deleted the freshstrangemusic/push-vxlnzootnlmm branch March 31, 2026 17:23
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.

Add CI test that validates all FML manifests are parseable by FmlClient

2 participants