Github Actions workflow improvements

[pczeglik-iohk]

Improve code coverage and clean-up output log for cardano package.

With this change we eliminate from outpu errors like this one 👇

Failed to collect coverage from /Users/piotrczeglik/Work/PROJECTS/LACE/lace/packages/cardano/src/wallet/util/calculate-deposit-reclaim.ts
ERROR: Invalid file coverage object, missing keys, found:data
STACK: Error: Invalid file coverage object, missing keys, found:data
    at assertValidObject (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/istanbul-lib-coverage/lib/file-coverage.js:36:15)
    at new FileCoverage (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/istanbul-lib-coverage/lib/file-coverage.js:78:9)
    at CoverageMap.addFileCoverage (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/istanbul-lib-coverage/lib/coverage-map.js:109:21)
    at /Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/@jest/reporters/build/CoverageReporter.js:279:33
    at async Promise.all (index 4)
    at CoverageReporter._addUntestedFiles (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/@jest/reporters/build/CoverageReporter.js:296:7)
    at CoverageReporter.onRunComplete (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/@jest/reporters/build/CoverageReporter.js:171:5)
    at ReporterDispatcher.onRunComplete (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/@jest/core/build/ReporterDispatcher.js:71:9)
    at TestScheduler.scheduleTests (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/@jest/core/build/TestScheduler.js:306:5)
    at runJest (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/@jest/core/build/runJest.js:367:19)
    at _run10000 (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/@jest/core/build/cli/index.js:343:7)
    at runCLI (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/@jest/core/build/cli/index.js:198:3)
    at Object.run (/Users/piotrczeglik/Work/PROJECTS/LACE/lace/node_modules/jest-cli/build/run.js:130:37)

as we have a bunch of files which are not imported in tests nor have a test:

    'src/wallet/lib/config.ts',
    'src/wallet/util/calculate-deposit-reclaim.ts',
    'src/wallet/lib/get-total-minimum-coins.ts',
    'src/wallet/lib/build-transaction-props.ts',
    'src/wallet/util/observable.ts',
    'src/wallet/lib/providers.ts',
    'src/wallet/lib/set-missing-coins.ts',
    'src/ui/components/Voting/CatalystRegisterStep.tsx',
    'src/wallet/lib/hardware-wallet.ts',
    'src/ui/components/Voting/CurrentCatalystFund.tsx',
    'src/ui/components/Voting/DownloadCatalystStep.tsx',
    'src/ui/components/Voting/VotingParticipation.tsx',
    'src/ui/components/Voting/CatalystScanStep.tsx',
    'src/ui/components/Voting/CatalystPinStep.tsx',
    'src/ui/components/Voting/WaitForNextFundCard.tsx',
    'src/ui/components/Voting/CatalystConfirmationStep.tsx'

This is what coveralls shows currently on main:

This is what coveralls shows after this change:

[pczeglik-iohk]

this code has been moved to common createJestConfig file

[github-actions]

Allure report

allure-report-publisher generated test report!

smokeTests: ✅ test report for 00540f34

	passed	failed	skipped	flaky	total	result
Total	30	0	0	0	30	✅

[coveralls]

Changes unknown
when pulling 00540f3 on test/cardano-package-unit-test-improvements
into ** on main**.

[pczeglik-iohk]

To get rid of warnings like that 👇 in our pipelines:

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/labeler@v4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

[pczeglik-iohk]

this is what we see in Checks Tab for each PR. Those message clouds are mostly related to deprecated actions.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud