Skip to content

test: cover runtime platform protocol compatibility#1792

Merged
radu-mocanu merged 1 commit into
mainfrom
test/runtime-platform-protocol-compat
Jul 6, 2026
Merged

test: cover runtime platform protocol compatibility#1792
radu-mocanu merged 1 commit into
mainfrom
test/runtime-platform-protocol-compat

Conversation

@radu-mocanu

Copy link
Copy Markdown
Collaborator

Summary

  • validate the platform resume trigger handler against the runtime trigger protocol
  • exercise the real resumable runtime suspension path with the platform handler
  • cover timer trigger creation through the umbrella package dependency graph

Copilot AI review requested due to automatic review settings July 6, 2026 13:53

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.

Pull request overview

Adds new unit tests in the uipath package to validate compatibility between the platform resume trigger handler and the runtime’s resumable trigger protocol, exercising the suspend → trigger creation → persistence path for a timer-based resume trigger.

Changes:

  • Introduces a new unit test module that type-checks and exercises runtime/platform trigger protocol integration.
  • Adds an in-memory UiPathResumableStorageProtocol test double and a suspended runtime stub to drive the resumable runtime path.
  • Asserts basic properties of the produced timer resume trigger (interrupt id/type/name).

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

Comment thread packages/uipath/tests/cli/unit/test_runtime_protocol_compat.py Outdated
Comment thread packages/uipath/tests/cli/unit/test_runtime_protocol_compat.py
@radu-mocanu radu-mocanu force-pushed the test/runtime-platform-protocol-compat branch from a7bfb0a to 2bb496e Compare July 6, 2026 14:08
@radu-mocanu radu-mocanu force-pushed the test/runtime-platform-protocol-compat branch from 2bb496e to 2c7b6ad Compare July 6, 2026 14:08
@sonarqubecloud

sonarqubecloud Bot commented Jul 6, 2026

Copy link
Copy Markdown

@radu-mocanu radu-mocanu merged commit 8ae652b into main Jul 6, 2026
89 of 94 checks passed
@radu-mocanu radu-mocanu deleted the test/runtime-platform-protocol-compat branch July 6, 2026 14:28
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.

3 participants