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

test: validate contract and admin key in hollow account finalized as contract #1142

Conversation

nadezhdapopovaa
Copy link
Contributor

Description:
Added test for admin key and contract id validation (should be the same) after deploying a contract with hollow account.

Related issue(s):

Fixes #1007

@quiet-node
Copy link
Member

Let’s not forget the assignees and additional PR details.

@nadezhdapopovaa nadezhdapopovaa self-assigned this Jan 6, 2025
@nadezhdapopovaa nadezhdapopovaa added the enhancement New feature or request label Jan 6, 2025
@nadezhdapopovaa nadezhdapopovaa added this to the 0.12.0 milestone Jan 6, 2025
@quiet-node
Copy link
Member

@lukelee-sl please review the PR when you have a chance to see if it aligns with your vision for this feature.

konstantinabl
konstantinabl previously approved these changes Jan 7, 2025
Copy link
Contributor

@konstantinabl konstantinabl left a comment

Choose a reason for hiding this comment

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

Good job! LGTM

@nadezhdapopovaa nadezhdapopovaa changed the title test: validate contract and admin key in hollow account finalized as … test: validate contract and admin key in hollow account finalized as contract Jan 8, 2025
…out finalizing a hollow account

Signed-off-by: Nadezhda Popova <[email protected]>
Copy link

github-actions bot commented Jan 16, 2025

Test Results

 17 files  +1   83 suites  ±0   12m 10s ⏱️ -4s
294 tests ±0  288 ✅ ±0  6 💤 ±0  0 ❌ ±0 
303 runs  ±0  296 ✅ ±0  7 💤 ±0  0 ❌ ±0 

Results for commit 626ee16. ± Comparison against base commit b3f254f.

♻️ This comment has been updated with latest results.

natanasow
natanasow previously approved these changes Jan 16, 2025
Copy link
Contributor

@natanasow natanasow left a comment

Choose a reason for hiding this comment

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

LGTM

…ct without finalizing a hollow account

Signed-off-by: Nadezhda Popova <[email protected]>
… contract without finalizing a hollow account

Signed-off-by: Nadezhda Popova <[email protected]>
Copy link
Member

@lukelee-sl lukelee-sl left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you very much!

@nadezhdapopovaa nadezhdapopovaa merged commit 74c78d1 into main Jan 17, 2025
35 of 36 checks passed
@nadezhdapopovaa nadezhdapopovaa deleted the 1007-create-test-to-validate-contract-admin-key-in-hollow-account-finalized-as-contract branch January 17, 2025 08:34
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
Status: Done
Development

Successfully merging this pull request may close these issues.

create test to validate contract admin key in hollow account finalized as contract
6 participants