databricks-sql-nodejs icon indicating copy to clipboard operation
databricks-sql-nodejs copied to clipboard

Convert unit tests to Typescript

Open kravets-levko opened this issue 1 year ago • 1 comments

Part of PECO-1390

Initially I planned to convert unit tests gradually, but there were issues with collecting coverage data. So I had to convert everything at once

kravets-levko avatar May 20 '24 21:05 kravets-levko

Codecov Report

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

Project coverage is 92.93%. Comparing base (3eed509) to head (6d2718b).

Files Patch % Lines
...nnection/auth/DatabricksOAuth/AuthorizationCode.ts 89.28% 2 Missing and 1 partial :warning:
lib/connection/auth/DatabricksOAuth/index.ts 80.00% 0 Missing and 1 partial :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #258      +/-   ##
==========================================
- Coverage   93.06%   92.93%   -0.13%     
==========================================
  Files          65       64       -1     
  Lines        1586     1572      -14     
  Branches      280      282       +2     
==========================================
- Hits         1476     1461      -15     
+ Misses         46       44       -2     
- Partials       64       67       +3     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar May 23 '24 12:05 codecov-commenter