Extract logic from "dao act"

[macor161]

🦅 Pull Request

Ref #874 and #947

🚨 Test instructions

dao act

TODO

• Refactor execHandler
• Test execHandler

[codecov]

Codecov Report

Merging #970 into develop will increase coverage by 0.59%.
The diff coverage is 28.07%.

@@             Coverage Diff             @@
##           develop     #970      +/-   ##
===========================================
+ Coverage    20.22%   20.82%   +0.59%     
===========================================
  Files          106      108       +2     
  Lines         2442     2449       +7     
===========================================
+ Hits           494      510      +16     
+ Misses        1948     1939       -9

Impacted Files	Coverage Δ
packages/aragon-cli/src/commands/dao_cmds/exec.js	0% <0%> (ø)	⬆️
packages/aragon-cli/src/commands/dao_cmds/act.js	0% <0%> (ø)	⬆️
packages/aragon-cli/src/lib/dao/exec.js	0% <0%> (ø)
...gon-cli/src/commands/dao_cmds/utils/execHandler.js	0% <0%> (ø)	⬆️
packages/aragon-cli/src/lib/dao/encodeActCall.js	100% <100%> (ø)
packages/aragon-cli/src/lib/getAppKernel.js	100% <100%> (ø)

---

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 d70b41b...7f21223. Read the comment docs.

[dapplion]

lgtm

[theethernaut]

⚠️

@macor161 I think that at this point, we should be able to at least isolate the call to execHandler in lib/ and write a test for it. Otherwise dao act ends up not being tested as a whole...

[macor161]

@ajsantander Agreed, I will refactor it in this PR.

[0xGabi]

Separate the logic on the execHandler make it much more clear 💪

I have included a couple of commits with changes of #928

I think we can merge this one and focus on integration test on a new PR