fix(ledger): remove poolConnected on pool close

[niall-shaw]

fixes issue where cannot reconnect to ledger after rotating wallet key

Signed-off-by: Niall Shaw [email protected]

[codecov-commenter]

Codecov Report

Merging #1011 (316719c) into main (60a8091) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1011      +/-   ##
==========================================
- Coverage   88.77%   88.75%   -0.03%     
==========================================
  Files         520      520              
  Lines       12084    12085       +1     
  Branches     1994     1994              
==========================================
- Hits        10728    10726       -2     
- Misses       1294     1297       +3     
  Partials       62       62              

Impacted Files	Coverage Δ
packages/core/src/modules/ledger/IndyPool.ts	80.72% <100.00%> (-2.21%)	⬇️
packages/core/src/agent/Agent.ts	94.82% <0.00%> (-0.58%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[niall-shaw]

Aims to fix #1012

[TimoGlastra]

Do we actually close the pools when the agent is shutdown?

[niall-shaw]

Do we actually close the pools when the agent is shutdown?

Yes, see the constructor:

  // Listen to stop$ (shutdown) and close pool
 agentConfig.stop$.subscribe(async () => {
      if (this._poolHandle) {
        await this.close()
      }
    })