Handle did:sov service omitting routing_keys attribute

[guijd3p]

when performing a test with acapy a routing_keys problem was discovered, it generates an empty ledger.

[Patrik-Stas]

Awesome, great to hear you've tested with acapy :-)

Yeah in general .unwrap() ideally shouldn't exist in the codebase (expect for tests), I've missed this in the review.

While we are at it, if you want to earn some bonus points, it'd be cool if you'd add test which test which would have catched the problematic scenario (eg routing_keys missing).

[codecov-commenter]

Codecov Report

Merging #676 (d65cea7) into main (27411f0) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #676      +/-   ##
==========================================
- Coverage   63.85%   63.84%   -0.01%     
==========================================
  Files         214      214              
  Lines       21000    21000              
  Branches     4703     4701       -2     
==========================================
- Hits        13409    13408       -1     
  Misses       3843     3843              
- Partials     3748     3749       +1     

Flag	Coverage Δ
unittests-aries-vcx	63.64% <100.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
aries_vcx/src/indy/ledger/transactions.rs	56.39% <100.00%> (ø)
...vcx/src/protocols/issuance/issuer/state_machine.rs	69.32% <0.00%> (-0.13%)	⬇️

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