test: add tests for alby

[motorina0]

Summary

• add test data for alby
• update alby.py to follow the BaseWallet interface
• minor changes to the test framework

Two open questions for @bitkarrot:

• is the alby fee value negative?
  • see comment here
  • the alby doc example has zero fee for regular invoices (ambigous) but a positive value for keysend payments ("fee": 1)
• we do not get back the preimage for the outgoing payments. I can see in the docs that the generic endpoint (/invoices/{checkin_id}) does not return the pre-image, but the dedicated endpoints do (/invoices/incoming/{checkin_id}, (/invoices/outgoing/{checkin_id})).
  • see my comment here

See the "skip": true tests for more details.

[codecov]

Codecov Report

Attention: Patch coverage is 86.56716% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 59.97%. Comparing base (bfda0b6) to head (2f8c382).

Files	Patch %	Lines
lnbits/wallets/alby.py	86.56%	9 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##              dev    #2390      +/-   ##
==========================================
+ Coverage   59.46%   59.97%   +0.50%     
==========================================
  Files          65       65              
  Lines        9404     9441      +37     
==========================================
+ Hits         5592     5662      +70     
+ Misses       3812     3779      -33     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[motorina0]

cc: @bitkarrot ☝️

[bitkarrot]

the fee value returned by Alby is positive.

[bitkarrot]

@motorina0 ok gimme a day to check this out.