Skip to content

Test wallet integration #383

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
Merged
peterwht merged 29 commits into from
Dec 16, 2024
Merged

Test wallet integration #383

peterwht merged 29 commits into from
Dec 16, 2024

Conversation

@peterwht
Copy link
Contributor

@peterwht peterwht commented Dec 16, 2024

No description provided.

al3mart and others added 28 commits December 15, 2024 21:06
@al3mart @peterwht
chore: get_payload_works
65a1f1e
@al3mart @peterwht
test(wallet-integration): up contract get payload works
de25ba3
@al3mart @peterwht
get paylaod from server works
2197dbf
@al3mart @peterwht
test(wallet-integration): retrieve upload call data works
da6c43b
@al3mart @peterwht
test(wallet-integration): retrieve instantiate call data works
b6747ad
@al3mart @peterwht
style: better comments
60978e9
@al3mart @peterwht
test: try higher wait times in CI
4af7a70
@al3mart @peterwht
test(wallet-integration): bump sleep time
ce2b861
@al3mart @peterwht
test(wallet-integration): even more sleep time
b245a30
@al3mart @peterwht
test(wallet-integration): maybe a port problem ?
3f9ad4b
@al3mart @peterwht
revert 0075e94
88b445e
@al3mart @peterwht
test(wallet-integration): better unit tests
aedee39
@al3mart @peterwht
test(wallet-integration): wait for wallet signature
02ff358
@al3mart @peterwht
test(wallet-integration): assert on received payload
d64486f
@al3mart @peterwht
test(wallet-integration): use tokio spawn
63ad27c
@al3mart @peterwht
test(wallet-integration): add some waiting time
99c25a8
@al3mart @peterwht
test(wallet-integration): use cargo run
477554d
@al3mart @peterwht
style: nightly fmt
c3d1eaf
@al3mart @peterwht
test(wallet-integration): 500s sleep time
a473598
@al3mart @peterwht
test(wallet-integration): ignore some tests
2f41c3a
@al3mart @peterwht
test(wallet-integration): get instantiate call data
4874344
@al3mart @peterwht
test(wallet-integration): integration tests improvements
8232611
@al3mart @peterwht
test(wallet-integration): bump sleep time
e8b0da3
@al3mart @peterwht
test(wallet-integration): merge integration tests
a15d1ff
@al3mart @peterwht
test(wallet-integration): remove sign_call_data test
9cc6a19
@al3mart @peterwht
test(wallet-integration): use random free port
a3993cc
@al3mart @peterwht
test(wallet-integration): define gas_limit & proof_size
86b5a61
@peterwht
fix: merge issues
0ac034d
@codecov
Copy link

codecov bot commented Dec 16, 2024

Codecov Report

Attention: Patch coverage is 54.83871% with 98 lines in your changes missing coverage. Please review.

Please upload report for BASE (peter/feat-wallet-integration@a0a8397). Learn more about missing BASE report.

Files with missing lines Patch % Lines
crates/pop-contracts/src/up.rs 27.72% 68 Missing and 5 partials ⚠️
crates/pop-cli/src/commands/up/contract.rs 78.26% 0 Missing and 25 partials ⚠️ 
@@                       Coverage Diff                        @@
##             peter/feat-wallet-integration     #383   +/-   ##
================================================================
  Coverage                                 ?   75.19%           
================================================================
  Files                                    ?       60           
  Lines                                    ?    13525           
  Branches                                 ?    13525           
================================================================
  Hits                                     ?    10170           
  Misses                                   ?     2012           
  Partials                                 ?     1343
Files with missing lines Coverage Δ
crates/pop-cli/src/wallet_integration.rs 95.28% <100.00%> (ø)
crates/pop-cli/src/commands/up/contract.rs 34.79% <78.26%> (ø)
crates/pop-contracts/src/up.rs 61.25% <27.72%> (ø)

@peterwht peterwht changed the base branch from peter/feat-wallet-integration to main December 16, 2024 06:27
@peterwht peterwht changed the base branch from main to peter/feat-wallet-integration December 16, 2024 06:27
@peterwht peterwht merged commit 6b13134 into peter/feat-wallet-integration Dec 16, 2024
11 of 13 checks passed
@peterwht peterwht deleted the test-wallet-integration branch December 16, 2024 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

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

@peterwht @al3mart