refactor: credential manifest enhancements to match wallet-rendering specs

[HeidiHan0000]

Summary:

• Refactored credentialmanifest.go
  • moved everything related to resolving the different types into a separate file called resolve.go to make it clearer as to what is coming from the credential manifest and wallet rendering specs and what is needed for displaying
  • moved common functions to common.go
• Changes to match specs:
  • Fixed display mapping object to use text property if paths is empty
    • validateDisplayMappingObject to check for if both the text field and the paths field is empty
    • update unit tests for this
  • ValidateCredentialApplication
    • Added check for case where manifest's format is empty but credential application's is not (credential application's formats should be a subset of credential manifest's)
    • Removed duplicate code for checking credential application against credential manifest
    • update unit tests
  • Added missing optional property to CredentialFulfillment
  • Added missing optional properties to schema format

Closes #3304
Closes #3274
Closes #3280
Closes #3281

[codecov]

Codecov Report

Merging #3309 (729f5e8) into main (d134910) will increase coverage by 0.00%.
The diff coverage is 89.65%.

@@           Coverage Diff           @@
##             main    #3309   +/-   ##
=======================================
  Coverage   88.18%   88.19%           
=======================================
  Files         317      318    +1     
  Lines       42945    42967   +22     
=======================================
+ Hits        37872    37894   +22     
  Misses       3727     3727           
  Partials     1346     1346           

Impacted Files	Coverage Δ
pkg/doc/cm/credentialfulfillment.go	86.36% <ø> (ø)
pkg/doc/cm/credentialapplication.go	92.68% <82.35%> (-1.99%)	⬇️
pkg/doc/cm/common.go	94.33% <87.50%> (+15.02%)	⬆️
pkg/doc/cm/resolve.go	90.96% <90.96%> (ø)
pkg/doc/cm/credentialmanifest.go	92.59% <100.00%> (+1.23%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us.

[DRK3]

Only minor grammar comments. I don't see any code issues. The comments I left were all on code that you just moved from somewhere else rather than new code, so it's up to you if you think it's worth fixing!

[HeidiHan0000]

@DRK3 I'll make those grammar changes, thanks for reviewing!

[HeidiHan0000]

@sudeshrshetty Could you please approve the CI tests again?