diff --git a/.github/workflows/coverage-report.yml b/.github/workflows/coverage-report.yml index 35955505f5..be405bc093 100644 --- a/.github/workflows/coverage-report.yml +++ b/.github/workflows/coverage-report.yml @@ -29,7 +29,7 @@ jobs: github-token: ${{secrets.GITHUB_TOKEN}} script: | const fs = require('fs'); - const commentBody = '```' + fs.readFileSync('coverage.log', 'utf8') + '```'; + const commentBody = fs.readFileSync('coverage.log', 'utf8'); if (commentBody.length === 0) { return diff --git a/test/util/report.js b/test/util/report.js index 9971f96f64..ed5b92cbac 100644 --- a/test/util/report.js +++ b/test/util/report.js @@ -215,14 +215,14 @@ for (let example in exampleCoverage) { let exampleName = example; if (existingTestIds === missingTests) { - examplesWithNoTestsReport += exampleName + '\n'; + examplesWithNoTestsReport += '- ' + exampleName + '\n'; examplesWithNoTests++; } else if (missingTests) { - examplesMissingSomeTestsReport += exampleName + ':\n'; + examplesMissingSomeTestsReport += '- ' + exampleName + ':\n'; for (let testId of exampleCoverage[example].missingTests) { - examplesMissingSomeTestsReport += ' ' + testId + '\n'; + examplesMissingSomeTestsReport += ' - ' + testId + '\n'; } examplesMissingSomeTests += 1; @@ -230,18 +230,18 @@ for (let example in exampleCoverage) { } if (missingKeys || missingAttrs) { - missingTestIdsReport += exampleName + '\n'; + missingTestIdsReport += '- ' + exampleName + '\n'; if (missingKeys) { - missingTestIdsReport += ' "Keyboard Support" table(s):\n'; + missingTestIdsReport += ' - "Keyboard Support" table(s):\n'; for (let row of exampleCoverage[example].missingKeys) { - missingTestIdsReport += ' ' + row + '\n'; + missingTestIdsReport += ' - ' + row + '\n'; } } if (missingAttrs) { - missingTestIdsReport += ' "Attributes" table(s):\n'; + missingTestIdsReport += ' - "Attributes" table(s):\n'; for (let row of exampleCoverage[example].missingAttrs) { - missingTestIdsReport += ' ' + row + '\n'; + missingTestIdsReport += ' - ' + row + '\n'; } } } @@ -260,14 +260,15 @@ fs.readdirSync(testsPath).forEach(function (testFile) { } }); -console.log('\nExamples without any regression tests:\n'); +console.log('\n## Regression test coverage:\n'); +console.log('\n### Examples without any regression tests:\n'); console.log(examplesWithNoTestsReport || 'None found.\n'); -console.log('\nExamples missing some regression tests:\n'); +console.log('\n### Examples missing some regression tests:\n'); console.log(examplesMissingSomeTestsReport || 'None found.\n'); -console.log('\nExamples documentation table rows without data-test-ids:\n'); +console.log('\n### Examples documentation table rows without data-test-ids:\n'); console.log(missingTestIdsReport || 'None found.\n'); -console.log('SUMMARTY:\n'); +console.log('### SUMMARTY:\n'); console.log(' ' + exampleFiles.length + ' example pages found.'); console.log(' ' + examplesWithNoTests + ' example pages have no regression tests.'); console.log(' ' + examplesMissingSomeTests + ' example pages are missing approximately ' +