Remove/Deprecate Coordinator in testing pkg

[colin-axner]

Summary

Remove or deprecate the Coordiantor in the testing pkg

Problem Definition

The Coordinator was originally created to have context on all of the chains. This was useful in managing a global time between the chains and allowing execution between chains.

Following proposed improvements in #3999, #1014, the coordinator type will likely not be very useful. At the moment it is a wrapper around the existing chains and the current global time. The test caller will already have access to all the chains, typically denoted via suite.chainA and suite.chainB and the global time can likely be accessed by all the chains without wrapping it in another type

Removing this extra layer of the testing pkg will simplify the concepts to understand when new developers approach the pkg.

Proposal

Following improvements in #1014 and #3999, we should consider deprecating or removing the Coordinator

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged/assigned