Skip to content

Conversation

@Daanvdplas
Copy link
Collaborator

@Daanvdplas Daanvdplas commented Nov 10, 2025

When deploying a contract the signer was no prompted if not provided and alice was always used. This fixes that.

Closes #721
Closes #732

@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

❌ Patch coverage is 82.81250% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.06%. Comparing base (8a050b2) to head (139aa75).

Files with missing lines Patch % Lines
crates/pop-cli/src/common/contracts.rs 85.96% 0 Missing and 8 partials ⚠️
crates/pop-cli/src/commands/up/contract.rs 33.33% 1 Missing and 1 partial ⚠️
crates/pop-cli/src/commands/call/contract.rs 66.66% 0 Missing and 1 partial ⚠️
@@           Coverage Diff           @@
##             main     #725   +/-   ##
=======================================
  Coverage   77.05%   77.06%           
=======================================
  Files         111      111           
  Lines       25422    25471   +49     
  Branches    25422    25471   +49     
=======================================
+ Hits        19589    19629   +40     
- Misses       3772     3773    +1     
- Partials     2061     2069    +8     
Files with missing lines Coverage Δ
crates/pop-cli/src/commands/up/mod.rs 71.87% <100.00%> (ø)
crates/pop-cli/src/commands/call/contract.rs 71.68% <66.66%> (-0.11%) ⬇️
crates/pop-cli/src/commands/up/contract.rs 13.08% <33.33%> (-0.90%) ⬇️
crates/pop-cli/src/common/contracts.rs 76.05% <85.96%> (+2.24%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codecov
Copy link

codecov bot commented Nov 12, 2025

Codecov Report

❌ Patch coverage is 52.99539% with 102 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.27%. Comparing base (69c1987) to head (5b588c3).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
crates/pop-cli/src/commands/up/contract.rs 19.00% 77 Missing and 4 partials ⚠️
crates/pop-cli/src/common/contracts.rs 83.14% 0 Missing and 15 partials ⚠️
crates/pop-cli/src/commands/call/contract.rs 85.71% 1 Missing and 2 partials ⚠️
crates/pop-cli/src/common/wallet.rs 60.00% 2 Missing ⚠️
crates/pop-cli/src/commands/call/chain.rs 0.00% 0 Missing and 1 partial ⚠️
@@            Coverage Diff             @@
##             main     #725      +/-   ##
==========================================
- Coverage   76.28%   76.27%   -0.01%     
==========================================
  Files         115      115              
  Lines       25909    26016     +107     
  Branches    25909    26016     +107     
==========================================
+ Hits        19765    19845      +80     
- Misses       4015     4029      +14     
- Partials     2129     2142      +13     
Files with missing lines Coverage Δ
crates/pop-chains/src/call/metadata/mod.rs 89.55% <ø> (ø)
crates/pop-cli/src/commands/up/mod.rs 70.40% <100.00%> (ø)
crates/pop-common/src/lib.rs 83.82% <ø> (ø)
crates/pop-cli/src/commands/call/chain.rs 68.21% <0.00%> (ø)
crates/pop-cli/src/common/wallet.rs 11.32% <60.00%> (+3.32%) ⬆️
crates/pop-cli/src/commands/call/contract.rs 71.81% <85.71%> (+0.02%) ⬆️
crates/pop-cli/src/common/contracts.rs 77.34% <83.14%> (+3.53%) ⬆️
crates/pop-cli/src/commands/up/contract.rs 19.95% <19.00%> (+2.16%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Daanvdplas Daanvdplas requested a review from moliholy November 12, 2025 09:34
@moliholy moliholy force-pushed the daan/fix-pop_up_contract_signer branch from e0a35b6 to d1df96c Compare November 12, 2025 13:48
Copy link
Collaborator

@moliholy moliholy left a comment

Choose a reason for hiding this comment

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

LGTM now! We still have to decide what to do if --skip-confirm is on and the user needs to specify parameters for deploying or calling contracts. Still, I think we can leave it as-is for now.

@Daanvdplas
Copy link
Collaborator Author

Can we also check whether with the --skip-confirm flag the user is not prompted for confirmation on executing the message @moliholy

@moliholy
Copy link
Collaborator

Can we also check whether with the --skip-confirm flag the user is not prompted for confirmation on executing the message

That's the case. No confirmation is requested when deploying or executing if --skip-confirm is passed.

@moliholy moliholy merged commit 0abc898 into main Nov 14, 2025
19 checks passed
@moliholy moliholy deleted the daan/fix-pop_up_contract_signer branch November 14, 2025 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pop up flow inconsistent pop up doesn't ask for who or how to sign the transaction

4 participants