Skip to content
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

feat(tck): add TCK server #344

Merged
merged 8 commits into from
Jun 18, 2024
Merged

feat(tck): add TCK server #344

merged 8 commits into from
Jun 18, 2024

Conversation

rwalworth
Copy link
Contributor

@rwalworth rwalworth commented Jun 7, 2024 •
edited
Loading

Description:
This PR adds the structures and processing necessary to serve TCK tests. It is currently configured to listen for HTTP requests on port 80. swift run HederaTCK is the command to run the server.

Related issue(s):

Fixes #319

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@rwalworth
feat(tck): first draft of TCK server with setup and reset methods
71117cf 
Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
@rwalworth rwalworth requested a review from RickyLB June 7, 2024 19:56
@rwalworth rwalworth self-assigned this Jun 7, 2024
@rwalworth rwalworth linked an issue Jun 7, 2024 that may be closed by this pull request
Set-up TCK server #319
Closed
@rwalworth
feat(tck): clean up and better organize JSON RPC processing. Add util…
1b00a77 
…ity methods

Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
@rwalworth
refactor: capitalize RFC 2119 words
add063a 
Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
@rwalworth rwalworth marked this pull request as ready for review June 13, 2024 21:02
@rwalworth rwalworth requested review from a team as code owners June 13, 2024 21:02
@rwalworth rwalworth changed the title feat(tck): first draft of TCK server with setup and reset methods feat(tck): add TCK server Jun 13, 2024
@rwalworth rwalworth added the enhancement New feature or request label Jun 13, 2024
@rwalworth
refactor: remove debugging artifacts
18326ee 
Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
@rwalworth
refactor: apply swift-format
b9ac0a8 
Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
@rwalworth
refactor: task lint
12eb164 
Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
@rwalworth
refactor: fix build
c90b5b7 
Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jun 17, 2024

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

RickyLB
RickyLB approved these changes Jun 18, 2024
View reviewed changes
Copy link
Contributor

@RickyLB RickyLB left a comment

Choose a reason for hiding this comment

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

LGTM!

@RickyLB RickyLB merged commit 47ee636 into main Jun 18, 2024
11 checks passed
@RickyLB RickyLB deleted the 00319-set-up-tck-server branch June 18, 2024 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RickyLB RickyLB RickyLB approved these changes

@tomzhenghedera tomzhenghedera Awaiting requested review from tomzhenghedera tomzhenghedera is a code owner automatically assigned from hashgraph/release-engineering

@rbarkerSL rbarkerSL Awaiting requested review from rbarkerSL rbarkerSL is a code owner automatically assigned from hashgraph/release-engineering-managers

Assignees

@rwalworth rwalworth

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Set-up TCK server
2 participants
@rwalworth @RickyLB