Skip to content

feat(simulator): set default port and enable direct client API invocation - #296

Merged
Peefy merged 1 commit into
pluto-lang:mainfrom
jianzs:feat-enhance-sim
Jul 26, 2024
Merged

feat(simulator): set default port and enable direct client API invocation#296
Peefy merged 1 commit into
pluto-lang:mainfrom
jianzs:feat-enhance-sim

Conversation

@jianzs

Copy link
Copy Markdown
Contributor

Set simulator's default port to 9001 to standardize the starting point.

Enable invocation of client APIs through simulator API formatted as http://<simulator_url>/<resource_id>/<method>. This allows users to test client APIs directly using curl, simplifying the testing process.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix#123", "re #123".):

  • N
  • Y

2. What is the scope of this PR (e.g. component or file name):

  • Simulator Adapter

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

  • Affects user behaviors
  • Contains syntax changes
  • Contains variable changes
  • Contains experimental features
  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

  • N
  • Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

  • Unit test
  • Integration test
  • Benchmark (add benchmark stats below)
  • Manual test (add detailed scripts or steps below)
  • Other

…tion
Set simulator's default port to 9001 to standardize the starting point.
Enable invocation of client APIs through simulator API formatted as `http://<simulator_url>/<resource_id>/<method>`. This allows users to test client APIs directly using curl, simplifying the testing process.
@jianzsjianzs added the adapter Adapter label Jul 26, 2024
@changeset-bot

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: bdea514

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@plutolang/simulator-adapterPatch
@plutolang/cliPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jianzsjianzs added the 💡feature This issue or pull request is related to a new feature. label Jul 26, 2024

@PeefyPeefy 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.

LGTM

@Peefy
Peefy merged commit 871ca7b into pluto-lang:mainJul 26, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adapterAdapter💡featureThis issue or pull request is related to a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@jianzs@Peefy