fix: check_contracts_node handles skip_confirm

[al3mart]

There was a small bug where the skip_confirm was ignored.
Prompting the user to source sustrate-contracts-node.

I have added a very minimal test. It could be improved and made more robust if we were to expand MockCli to include checks for certain prompts not being presented in the CLI. But that is another story.

Thanks to @ltfschoen for reporting.

[sc-2094]

[codecov]

Codecov Report

Attention: Patch coverage is 89.47368% with 2 lines in your changes missing coverage. Please review.

Project coverage is 75.17%. Comparing base (cbf722a) to head (11af03e).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/pop-cli/src/common/contracts.rs	89.47%	0 Missing and 2 partials ⚠️

@@            Coverage Diff             @@
##             main     #396      +/-   ##
==========================================
+ Coverage   75.15%   75.17%   +0.01%     
==========================================
  Files          62       62              
  Lines       13820    13835      +15     
  Branches    13820    13835      +15     
==========================================
+ Hits        10386    10400      +14     
  Misses       2082     2082              
- Partials     1352     1353       +1     

Files with missing lines	Coverage Δ
crates/pop-cli/src/common/contracts.rs	62.67% <89.47%> (+3.62%)	⬆️

[AlexD10S]

LGTM! Thanks for fixing this