tealdbg: use associated group index instead of global

[barnjamin]

Summary

The groupIndex variable is a global set by a flag and defaults to 0. In the main.go file its checked and then set in DebugParams.

I believe this is a bug that affects any grouped transactions where the groupIndex meant to be evaluated is > 0 and we should really be using the groupIndex passed in the evaluation struct.

Test Plan

Existing tests

[codecov-commenter]

Codecov Report

Merging #3111 (a9425ed) into master (c2054e5) will increase coverage by 0.01%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #3111      +/-   ##
==========================================
+ Coverage   43.68%   43.70%   +0.01%     
==========================================
  Files         390      390              
  Lines       86681    86681              
==========================================
+ Hits        37868    37881      +13     
+ Misses      42796    42774      -22     
- Partials     6017     6026       +9     

Impacted Files	Coverage Δ
cmd/tealdbg/main.go	25.39% <0.00%> (ø)
cmd/tealdbg/local.go	71.96% <100.00%> (+1.24%)	⬆️
ledger/roundlru.go	90.56% <0.00%> (-5.67%)	⬇️
ledger/blockqueue.go	82.18% <0.00%> (-1.73%)	⬇️
catchup/service.go	69.59% <0.00%> (-1.51%)	⬇️
catchup/peerSelector.go	98.95% <0.00%> (-1.05%)	⬇️
data/abi/abi_type.go	92.03% <0.00%> (-1.00%)	⬇️
ledger/internal/eval.go	69.94% <0.00%> (-0.27%)	⬇️
network/wsPeer.go	72.53% <0.00%> (-0.27%)	⬇️
ledger/acctupdates.go	64.75% <0.00%> (-0.10%)	⬇️
... and 5 more

---

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 c2054e5...a9425ed. Read the comment docs.

[jasonpaulos]

Definitely a bug, thanks for fixing

[jannotti]

Looks good!