Skip to content

Add test utility to create Nexus endpoint#1279

Open
Quinn-With-Two-Ns wants to merge 2 commits intotemporalio:mainfrom
Quinn-With-Two-Ns:SDK-3837
Open

Add test utility to create Nexus endpoint#1279
Quinn-With-Two-Ns wants to merge 2 commits intotemporalio:mainfrom
Quinn-With-Two-Ns:SDK-3837

Conversation

@Quinn-With-Two-Ns
Copy link

@Quinn-With-Two-Ns Quinn-With-Two-Ns commented Jan 16, 2026

Add test utility to create Nexus endpoint. This matches what we have in Java and Dotnet


Note

Low Risk
Mostly adds a small testing API surface and refactors tests to call it; minimal production impact aside from depending on operator-service Nexus RPCs being available in the test environment.

Overview
Adds WorkflowEnvironment.create_nexus_endpoint() and WorkflowEnvironment.delete_nexus_endpoint() to create/delete Nexus endpoints via the operator service, returning the created Endpoint.

Updates the test suite to use the new environment helper everywhere Nexus endpoints are needed, and removes the now-redundant tests.helpers.nexus.create_nexus_endpoint utility (leaving only make_nexus_endpoint_name).

Written by Cursor Bugbot for commit 9d85501. This will update automatically on new commits. Configure here.

@Quinn-With-Two-Ns Quinn-With-Two-Ns marked this pull request as ready for review January 16, 2026 03:22
@Quinn-With-Two-Ns Quinn-With-Two-Ns requested a review from a team as a code owner January 16, 2026 03:22
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

@tconley1428
Copy link
Contributor

It does have typechecking issues however.

@Quinn-With-Two-Ns Quinn-With-Two-Ns force-pushed the SDK-3837 branch 2 times, most recently from 5693a43 to 9ba31d0 Compare February 18, 2026 16:58
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.

4 participants

Comments