fix: improve parseInvitation to handle parsing oob properly

elribonazo commented 2 months ago

Description:

Improvement in order to keep SDK's aligned, we are able to use parseOOBInvitation with the URL object directly or the standard parseInvitation with the uri string + added test coverage

agent.parseOOBInvitation(new URL(validOOB))
agent.parseInvitation(validOOB) //as string

Checklist:

[ ] My PR follows the contribution guidelines of this project
[ ] My PR is free of third-party dependencies that don't comply with the Allowlist
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked the PR title to follow the conventional commit specification

github-actions[bot] commented 2 months ago

Lines	Statements	Branches	Functions
	76.26% (2860/3750)	65.91% (1402/2127)	81.1% (734/905)

JUnit

Tests	Skipped	Failures	Errors	Time
528	6 :zzz:	0 :x:	0 :fire:	55.675s :stopwatch:

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 10664806424

Details

28 of 33 (84.85%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 72.462%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/edge-agent/Agent.Invitations.ts	11	13	84.62%
src/edge-agent/protocols/invitation/v2/DIDCommInvitationRunner.ts	14	17	82.35%
<!--	Total:	28	33	84.85%	-->

Totals
Change from base Build 10662917238:	0.2%
Covered Lines:	2795
Relevant Lines:	3665

💛 - Coveralls