Skip to content

add tests on Ordered Multiple/Collection with rdflib v2.3.8#752

Merged
bourgeoa merged 3 commits into
mainfrom
ordered
May 16, 2026
Merged

add tests on Ordered Multiple/Collection with rdflib v2.3.8#752
bourgeoa merged 3 commits into
mainfrom
ordered

Conversation

@bourgeoa
Copy link
Copy Markdown
Contributor

@bourgeoa bourgeoa commented May 16, 2026

Ordered Multiple/Collection field does not serialize as proper RDF list issue #737
Should be resolved with rdflib v2.3.8

Copilot AI review requested due to automatic review settings May 16, 2026 17:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds unit tests covering the ordered Multiple/Collection rendering flow in widgets/forms, validating behaviour relied upon by the rdflib v2.3.8 / solid-logic v4.0.7 upgrade. Also bumps the corresponding peerDependency ranges.

Changes:

  • Bump rdflib peer dep to ^2.3.8 and solid-logic to ^4.0.7.
  • Add a new test suite test/unit/widgets/forms/multiple.test.ts covering Collection recovery, refresh wiring, and add-button end-to-end behaviour.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
package.json Updates rdflib and solid-logic peerDependency versions.
test/unit/widgets/forms/multiple.test.ts New tests for ordered Multiple field: Collection reuse, refresh method, and add-button creating a single list head.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bourgeoa bourgeoa merged commit f853c89 into main May 16, 2026
8 checks passed
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.

2 participants