Skip to content

test: e2e REST API additional coverage #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

test: e2e REST API additional coverage #34

wants to merge 6 commits into from

Conversation

@sansan
Copy link
Contributor

@sansan sansan commented Oct 22, 2025 •
edited
Loading

Description

PR Highlights

  • bumps default REST stack used in latest to v8.0.0-alpha.3
  • adds a local compose profile plus helper scripts to spin REST using locally published image
  • extends asset REST client to cover transfer restrictions, venue filtering, and permission-group management endpoints
  • introduces protocol fee and treasury balance helpers with matching smoke tests to verify new endpoints
  • adds comprehensive REST E2E coverage for asset permission groups, transfer restriction workflows, and venue filtering flows
  • updates the STO SDK example to register investor signers on venues (otherwise it was failing)

@sansan sansan changed the title Feat/rest 7.2.0 alpha.6 test: e2e REST API additional coverage Oct 22, 2025
@sansan sansan force-pushed the feat/rest-7.2.0-alpha.6 branch from da7da0e to ce6b5ad Compare October 30, 2025 15:41
@sansan sansan marked this pull request as ready for review October 31, 2025 07:53
@sansan sansan requested review from a team as code owners October 31, 2025 07:53
polymath-eric
polymath-eric reviewed Oct 31, 2025
View reviewed changes
scripts/start-env-local.sh
@@ -0,0 +1,18 @@
#!/bin/bash
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am not seeing how this is different from the regular start-env.sh script. Why do we need a new script for this?

scripts/stop-env-local.sh
@@ -0,0 +1,14 @@
#!/bin/bash
set -e
Copy link
Collaborator

Choose a reason for hiding this comment

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

Doesn't this duplicate stop-env.sh?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@polymath-eric polymath-eric polymath-eric left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sansan @polymath-eric