gold and dollar flags are required for faucet script

[martinvol]

Description

After merging #1844 the default value to faucet cUSD and cGLD was zero, now they are required and have no default.

Tested

$ celotooljs account faucet --celo-env pilot --account 0xf601f32f6fee01ba148b4737d500a4991e7faa9b
celotooljs account faucet

command for fauceting an address with gold and/or dollar

Options:
  --version       Show version number                                                                                                                                   [boolean]
  --verbose       Whether to show a bunch of debugging output like stdout and stderr of shell commands                                                 [boolean] [default: false]
  --yesreally     Reply "yes" to prompts about changing staging/production (be careful!)                                                               [boolean] [default: false]
  --help          Show help                                                                                                                                             [boolean]
  --celo-env, -e  the environment in which you want to execute this command                                                                                            [required]
  --account       Account to faucet                                                                                                                           [string] [required]
  --dollar       Number of dollars to faucet                                                                                                                 [number] [required]
  --gold          Amount of gold to faucet                                                                                                                    [number] [required]

Missing required arguments: dollar, gold
Please specify dollars to faucet
Please specify gold to faucet
error Command failed with exit code 1.

$ celotooljs account faucet --celo-env pilot --account 0xf601f32f6fee01ba148b4737d500a4991e7faa9b --gold 10 --dollar 11
Port-forwarding to pilot validators 8545:8545 8546:8546 9200:9200
Port-forwarding to pilot validators 8545:8545 8546:8546 9200:9200


Using account: 0x387bCb16Bfcd37AccEcF5c9eB2938E30d3aB8BF2
Fauceting 10 Gold and 11 StableToken to 0xf601f32f6fee01ba148b4737d500a4991e7faa9b

Other changes

Changed logging, as it was printing the value in contract notation.

Related issues

Backwards compatibility

Nop

[jmrossy]

@asaj changed this to 0 recently (see git blame). We need to check with him before setting this back

[martinvol]

I thought it was accidental, if not I'll go ahead and change the documentation.

[codecov]

Codecov Report

Merging #1943 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1943      +/-   ##
==========================================
+ Coverage   74.43%   74.45%   +0.02%     
==========================================
  Files         281      281              
  Lines        7822     7817       -5     
  Branches      974      685     -289     
==========================================
- Hits         5822     5820       -2     
+ Misses       1883     1880       -3     
  Partials      117      117

Flag	Coverage Δ
#mobile	74.45% <ø> (+0.02%)	⬆️

Impacted Files	Coverage Δ
packages/mobile/src/import/ImportWalletEmpty.tsx	78.78% <0%> (-0.63%)	⬇️
packages/mobile/src/import/ImportWallet.tsx	95.83% <0%> (+5.44%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d6d0086...fb4503e. Read the comment docs.

approved by asa