Skip to content

test: add focused unit tests for template semver and compatibility#279

Open
Pidoko257 wants to merge 1 commit into
Nanle-code:masterfrom
Pidoko257:add-unit-tests-template-compatibility-parsing-edge-cases
Open

test: add focused unit tests for template semver and compatibility#279
Pidoko257 wants to merge 1 commit into
Nanle-code:masterfrom
Pidoko257:add-unit-tests-template-compatibility-parsing-edge-cases

Conversation

@Pidoko257
Copy link
Copy Markdown

Closes #267

  • Add parse_semver edge cases: large numbers, 1/2 components, extra dots, whitespace, negative and alpha components
  • Add check_version_range payload verification: TooOld/TooNew field values, exact min/max boundary semantics, min-only and max-only paths, malformed running version, malformed max reason field
  • Add assert_template_compatible error message content checks for TooOld, TooNew, and MalformedMetadata variants

Fix pre-existing compilation errors that blocked the test suite:

  • install_template_package: wrong arity calling publish_template_versioned
  • template.rs install fn: wrong arity calling publish
  • plugin.rs: install_plugin called with 3 args instead of 5; removed references to non-existent installed_at field
  • registry.rs: removed reference to non-existent version field on InstalledPlugin

- Add parse_semver edge cases: large numbers, 1/2 components, extra
  dots, whitespace, negative and alpha components
- Add check_version_range payload verification: TooOld/TooNew field
  values, exact min/max boundary semantics, min-only and max-only
  paths, malformed running version, malformed max reason field
- Add assert_template_compatible error message content checks for
  TooOld, TooNew, and MalformedMetadata variants

Fix pre-existing compilation errors that blocked the test suite:
- install_template_package: wrong arity calling publish_template_versioned
- template.rs install fn: wrong arity calling publish
- plugin.rs: install_plugin called with 3 args instead of 5; removed
  references to non-existent installed_at field
- registry.rs: removed reference to non-existent version field on
  InstalledPlugin
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Jun 1, 2026

@Pidoko257 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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 unit tests for template compatibility parsing and edge cases

1 participant