Add FirebaseAuth identity token

[petrpavlik]

These changes are now available in 5.1.0

• Add support for Firebase Auth by introducing FirebaseAuthIdentityToken.
• Fix some unit tests not being run. Probably a regression cause by migration to the new testing framework.

Adding support for Firebase Authentication was okayed by @ptoffy, see here vapor/jwt#159.

I'll also follow up with a PR for /~https://github.com/vapor/jwt once this is merged in.

[codecov]

Codecov Report

Attention: Patch coverage is 96.77419% with 1 line in your changes missing coverage. Please review.

Project coverage is 83.68%. Comparing base (b0b58ec) to head (4d26e2c).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...rces/JWTKit/Vendor/FirebaseAuthIdentityToken.swift	96.77%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #207      +/-   ##
==========================================
+ Coverage   77.20%   83.68%   +6.47%     
==========================================
  Files          55       56       +1     
  Lines        1347     1483     +136     
==========================================
+ Hits         1040     1241     +201     
+ Misses        307      242      -65     

Files with missing lines	Coverage Δ
...rces/JWTKit/Vendor/FirebaseAuthIdentityToken.swift	96.77% <96.77%> (ø)

... and 25 files with indirect coverage changes

[ptoffy]

Thanks for this! Just missing some formatting. Feel free to run swift format --recursive --in-place . with /~https://github.com/vapor/contributing/blob/main/.swift-format in the project's directory. While you're at it could you add some names to the @Tests which were missing? Something similar to the other test suites should be fine

[petrpavlik]

@ptoffy done, please take another look.

Not sure though why the windows CI pass failed on some CCryptoBoringSSL (linker?) errors?

[ptoffy]

Yeah looks like this should fix it apple/swift-crypto@4518e5e but there hasn't been a release yet

[ptoffy]

Just a few whitespace nits to be in par with the rest of the repo. I would have thought swift-format would catch these but maybe we need to be stricter 🤷

[petrpavlik]

Thanks, I'll fix those. Yeah, I just ran swift-format before pushing the changes, relying on that I'd fix everything for me.

[ptoffy]

Thanks!