Migrate to new account

[ryutamago]

Context

This PR completes the migration to the new account type.
255
In the next step, NewAddress enums are replaced with SmartFunctionHash for parts that should only use sf(e.g. set_code, the self address of ledger, sf runtime api etc)

Description

Replaced the old Account with the new Account. the KT1 address is now supported.

Manually testing the PR

• Added unit tests for smart function deployment.

• Added unit tests for rejecting non KT1 address for adding sf alias

• manually tested sf deployment and running.

cargo run --bin jstz -- deploy examples/counter.js --network dev
cargo run --bin jstz -- run tezos://KT1J6BkxBK96qQgJURrut4SJz65EQAA5vDhU/ --network dev --trace

• manually tested account alias for sf

cargo run --bin jstz -- account alias counter_sf KT1J6BkxBK96qQgJURrut4SJz65EQAA5vDhU 

Added smart function: counter_sf -> KT1J6BkxBK96qQgJURrut4SJz65EQAA5vDhU

• manually tested deposit / withdrawal to user acount

cargo run --bin jstz -- account create test

cargo run --bin jstz -- bridge deposit --from bootstrap1 --to test --amount 2 --network dev

cargo run --bin jstz -- account balance -a test --network dev

cargo run --bin jstz -- bridge withdraw --to tz1KqTpEZ7Yob7QbPE4Hy4Wo8fHG8LhKxZSx --amount 1 --network dev

• manually tested deposit/withdraw to a KT1 address

cargo run --bin jstz -- bridge deposit --from bootstrap1 --to counter_sf --amount 2 --network dev

cargo run --bin jstz -- account balance -a counter_sf --network dev

argo run --bin jstz -- bridge withdraw --to KT1J6BkxBK96qQgJURrut4SJz65EQAA5vDhU --amount 1 --network dev

• manually tested that adding sf alias is rejected for tz1 address

cargo run --bin jstz -- account alias sf tz1cmuyEpW2vg6JxVzzRman9bDJXsJadxD2h

error: invalid value 'tz1cmuyEpW2vg6JxVzzRman9bDJXsJadxD2h' for '<ADDRESS>': Invalid smart function address: tz1cmuyEpW2vg6JxVzzRman9bDJXsJadxD2h

• manually tested repl

>> jstz.Account.balance("tz1cD5CuvAALcxgypqBXcBQEA8dkLJivoFjU");
0

[codecov]

Codecov Report

Attention: Patch coverage is 62.50000% with 33 lines in your changes missing coverage. Please review.

Project coverage is 49.33%. Comparing base (4fb8290) to head (5b69707).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/jstz_node/src/services/accounts.rs	0.00%	18 Missing ⚠️
crates/jstz_proto/src/executor/smart_function.rs	83.33%	1 Missing and 4 partials ⚠️
crates/jstz_proto/src/executor/fa_deposit.rs	78.57%	3 Missing ⚠️
crates/jstz_cli/src/account.rs	71.42%	1 Missing and 1 partial ⚠️
crates/jstz_cli/src/repl/debug_api/account.rs	0.00%	2 Missing ⚠️
crates/jstz_kernel/src/inbox.rs	66.66%	2 Missing ⚠️
crates/jstz_kernel/src/lib.rs	75.00%	1 Missing ⚠️

Files with missing lines	Coverage Δ
crates/jstz_cli/src/deploy.rs	0.00% <ø> (ø)
crates/jstz_cli/src/jstz.rs	0.00% <ø> (ø)
crates/jstz_kernel/src/parsing.rs	96.66% <ø> (ø)
crates/jstz_mock/src/host.rs	97.43% <ø> (ø)
crates/jstz_mock/src/message/fa_deposit.rs	100.00% <100.00%> (ø)
crates/jstz_proto/src/api/ledger.rs	32.67% <ø> (ø)
crates/jstz_proto/src/api/smart_function.rs	79.89% <100.00%> (+0.05%)	⬆️
crates/jstz_proto/src/context/mod.rs	32.15% <ø> (ø)
crates/jstz_proto/src/context/ticket_table.rs	94.76% <ø> (ø)
crates/jstz_proto/src/executor/deposit.rs	100.00% <ø> (ø)
... and 10 more

... and 2 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

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

[huancheng-trili]

Thanks for the manual tests btw. They are great as a starting point for integration tests.