feat: add codecov integration with test results upload (#35)

* feat: add codecov integration with test results upload

- Add lcov reporter to nyc configuration for codecov compatibility
- Add codecov upload steps to CI workflow for coverage and test results
- Configure fail_ci_if_error: false to not block CI on codecov issues

Project already has .github/codecov.yml with project and patch coverage checks configured.

* fix: add coverage instrumentation env var and fail_ci_if_error to test results upload

- Add VSC_JUPYTER_INSTRUMENT_CODE_FOR_COVERAGE env var to test step to enable coverage generation
- Add fail_ci_if_error: false to test results upload for consistency with coverage upload

Addresses CodeRabbit automated review feedback on PR #35

* fix: set codecov uploads to fail CI on error

Change fail_ci_if_error from false to true for both coverage and test results uploads to ensure CI fails when codecov upload fails, as requested by user.

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>

Author: jamesbhobbs

.github/workflows/ci.yml

@@ -91,6 +91,23 @@ jobs:
 
       - name: Run tests
         run: npm test
+        env:
+          VSC_JUPYTER_INSTRUMENT_CODE_FOR_COVERAGE: true
+
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v5
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          files: coverage/lcov.info
+          fail_ci_if_error: true
+
+      - name: Upload test results to Codecov
+        if: '!cancelled()'
+        uses: codecov/test-results-action@v1
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          files: test-report.xml
+          fail_ci_if_error: true
 
       - name: Check dependencies
         run: npm run checkDependencies

src/test/coverage.node.ts

@@ -14,7 +14,7 @@ export function setupCoverage() {
         return;
     }
     const htmlReport = process.env.VSC_JUPYTER_INSTRUMENT_CODE_FOR_COVERAGE_HTML ? ['html'] : [];
-    const reports = htmlReport.concat(['text', 'text-summary']);
+    const reports = htmlReport.concat(['text', 'text-summary', 'lcov']);
     const NYC = require('nyc');
     const nyc = new NYC({
         cwd: path.join(EXTENSION_ROOT_DIR_FOR_TESTS),