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.

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

feat(accountDeleteTransaction): Implement AccountDeleteTransaction E2E tests: TCK #230

Conversation

rwalworth
Copy link
Contributor

@rwalworth rwalworth commented Jul 24, 2024

Description:
This PR implements the tests documented in test-specfications/crypto-service/accountDeleteTransaction.md. Since some of the work was already done for AccountDeleteTransaction tests, this reworks those tests to match what is in the test specifications, in addition to adding any new tests. The docs were also updated to mark the tests as implemented.

Issues opened in SDKs that encompass the SDK server work should be linked back to the TCK's AccountDeleteTransaction development issue (#41) for easy tracking.

Related issue(s):

Fixes #223

Checklist

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

…tion

Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
@rwalworth rwalworth added the enhancement New feature or request label Jul 24, 2024
@rwalworth rwalworth self-assigned this Jul 24, 2024
@rwalworth rwalworth linked an issue Jul 24, 2024 that may be closed by this pull request
Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
Copy link
Contributor

@ivaylogarnev-limechain ivaylogarnev-limechain left a comment

Choose a reason for hiding this comment

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

All tests are passing in JS.

Copy link
Contributor

@0xivanov 0xivanov left a comment

Choose a reason for hiding this comment

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

Passing in Go and Java

@rwalworth rwalworth merged commit 45bfb98 into main Sep 9, 2024
9 checks passed
@rwalworth rwalworth deleted the 00223-tck-implement-json-rpc-method-for-accountdeletetransaction branch September 9, 2024 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TCK: Implement tests for AccountDeleteTransaction
3 participants