Skip to content

Feature/view state service architecture#19

Merged
ReneSchwarzer merged 7 commits into
developfrom
feature/view-state-service-architecture
Jun 22, 2026
Merged

Feature/view state service architecture#19
ReneSchwarzer merged 7 commits into
developfrom
feature/view-state-service-architecture

Conversation

@ReneSchwarzer

Copy link
Copy Markdown
Member

This pull request introduces a comprehensive JavaScript headless testing suite for the View, State, and Service engine, along with updates to documentation and configuration files to reflect recent architectural changes. The main focus is on enabling robust, platform-independent testing of JavaScript engine modules using Node.js, and ensuring that the .NET test suite can invoke these JavaScript tests as part of the regular test run.

Key changes:

JavaScript Headless Testing Infrastructure

  • Added a new JsTest folder with extensive headless unit tests for engine modules and REST model helpers, including a detailed README.md explaining test coverage, requirements, and usage. Tests cover a wide range of client-side logic, such as comment composer, selection, dashboard, workflow editor, and more. [1] [2] [3]

  • Introduced UnitTestJavaScript.cs to the .NET test suite, which discovers and runs all JavaScript tests in the JsTest folder using Node.js. The runner is platform-independent, checks for Node.js availability and version, and integrates results into the xUnit pipeline, skipping tests gracefully if prerequisites are missing.

Configuration and Documentation Updates

  • Updated webexpress.config.xml to replace the legacy <limit> section with a <kestrel> section, aligning configuration with current server settings for connection and upload limits.

  • Updated documentation in tag.md and theme-selector.md to reflect renaming of server-side controls from ControlRestTag/ControlRestSelectionTheme to ControlDataTag/ControlDataSelectionTheme, ensuring consistency with the codebase. [1] [2] [3]

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

@ReneSchwarzer ReneSchwarzer merged commit 8fb5eee into develop Jun 22, 2026
1 check passed
@ReneSchwarzer ReneSchwarzer deleted the feature/view-state-service-architecture branch June 22, 2026 05:48
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