Merge pull request #1870 from SUI-Components/no-coverage-inline

FEAT (sui-test & sui-studio) binary option: no coverage inline
SUI-Components · Nov 14, 2024 · e0bf9e4 · e0bf9e4
2 parents 4af7fd9 + 5cefcf4
commit e0bf9e4
Show file tree

Hide file tree

Showing 4 changed files with 87 additions and 38 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/sui-studio/bin/sui-studio-test.js b/packages/sui-studio/bin/sui-studio-test.js
@@ -13,6 +13,7 @@ program
   .option('-W, --watch', 'Watch mode')
   .option('-T, --timeout <timeout>', 'Timeout')
   .option('--coverage', 'Create coverage', false)
+  .option('--no-coverage-inline', 'Save the coverage summary in a text file', false)
   .on('--help', () => {
     console.log('  Examples:')
     console.log('')
@@ -23,7 +24,7 @@ program
   })
   .parse(process.argv)
 
-const {coverage, watch, ci, headless, timeout} = program.opts()
+const {coverage, coverageInline, watch, ci, headless, timeout} = program.opts()
 
 const relPath = path.relative(
   process.cwd(),
@@ -39,6 +40,7 @@ const run = async () => {
           '--pattern',
           path.join(relPath, 'node_modules', '@s-ui', 'studio', 'src', 'runtime-mocha', 'index.js'),
           coverage && '--coverage',
+          !coverageInline && '--no-coverage-inline',
           watch && '--watch',
           ci && '--ci',
           headless && '--headless',

diff --git a/packages/sui-test/bin/karma/index.js b/packages/sui-test/bin/karma/index.js
@@ -6,7 +6,17 @@ const {
 const config = require('./config.js')
 const CWD = process.cwd()
 
-module.exports = async ({ci, coverage, headless, ignorePattern, pattern, srcPattern, timeout, watch}) => {
+module.exports = async ({
+  ci,
+  coverage,
+  coverageInline,
+  headless,
+  ignorePattern,
+  pattern,
+  srcPattern,
+  timeout,
+  watch
+}) => {
   if (timeout) config.browserDisconnectTimeout = timeout
   if (ignorePattern) config.exclude = [ignorePattern]
 
@@ -38,7 +48,13 @@ module.exports = async ({ci, coverage, headless, ignorePattern, pattern, srcPatt
         {type: 'cobertura', subdir: '.', file: 'coverage.xml'},
         {type: 'html', subdir: 'report-html'},
         {type: 'json-summary', subdir: '.', file: 'coverage.json'},
-        {type: 'text-summary'}
+        {
+          type: 'text-summary',
+          ...(!coverageInline && {
+            subdir: '.',
+            file: 'coverage.txt'
+          })
+        }
       ]
     }
   }

diff --git a/packages/sui-test/bin/sui-test-browser.js b/packages/sui-test/bin/sui-test-browser.js
@@ -14,6 +14,7 @@ program
   .option('--src-pattern <srcPattern>', 'Define the source directory', false)
   .option('-T, --timeout <ms>', 'Timeout', 2000)
   .option('--coverage', 'Run the coverage preprocessor', false)
+  .option('--no-coverage-inline', 'Save the coverage summary in a text file', false)
   .on('--help', () => {
     console.log('  Description:')
     console.log('')
@@ -22,15 +23,19 @@ program
     console.log('  Examples:')
     console.log('')
     console.log('    $ sui-test browser')
+    console.log('    $ sui-test browser --headless')
+    console.log('    $ sui-test browser --coverage')
+    console.log('    $ sui-test browser --coverage --no-coverage-inline')
     console.log('')
   })
   .parse(process.argv)
 
-const {ci, coverage, headless, ignorePattern, pattern, srcPattern, timeout, watch} = program.opts()
+const {ci, coverage, coverageInline, headless, ignorePattern, pattern, srcPattern, timeout, watch} = program.opts()
 
 runner({
   ci,
   coverage,
+  coverageInline,
   headless,
   ignorePattern,
   pattern,