search

databricks / databricks-sql-nodejs

Databricks SQL Connector for Node.js
Apache License 2.0
24 stars 34 forks source link

Convert unit tests to Typescript #258

Closed kravets-levko closed 5 months ago

kravets-levko commented 6 months ago

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

codecov-commenter commented 6 months ago

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 ```diff @@ 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.