@toptal/davinci-monorepo 8.1.4-alpha-CRT-5891-create-coverage-command-2488e674.2 → 8.1.4-alpha-CRT-5891-create-coverage-command-23386c1d.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@toptal/davinci-monorepo",
-  "version": "8.1.4-alpha-CRT-5891-create-coverage-command-2488e674.2+2488e674",
+  "version": "8.1.4-alpha-CRT-5891-create-coverage-command-23386c1d.2+23386c1d",
   "description": "Monorepo utility tools",
   "publishConfig": {
     "access": "public"
@@ -35,7 +35,7 @@
   "dependencies": {
     "@nodelib/fs.walk": "^1.2.6",
     "@oclif/core": "^1.16.1",
-    "@toptal/davinci-cli-shared": "2.3.1-alpha-CRT-5891-create-coverage-command-2488e674.34+2488e674",
+    "@toptal/davinci-cli-shared": "2.3.1-alpha-CRT-5891-create-coverage-command-23386c1d.34+23386c1d",
     "chalk": "^4.1.2",
     "codeowners": "5.1.1",
     "dependency-cruiser": "^12.5.0",
@@ -53,5 +53,5 @@
   "devDependencies": {
     "@jest/globals": "^29.4.2"
   },
-  "gitHead": "2488e6747994019dbeab8517629432f68e9af814"
+  "gitHead": "23386c1d709c03d921ae81f6e8ede8bbaccd64aa"
 }

package/src/commands/coverage.js

@@ -4,8 +4,8 @@ import { groupCoverageByCodeownersCommand } from './group-coverage-by-codeowners
 export const coverageCommand = program => {
   const coverage = program.createCommand('coverage').description('TODO ..')
 
-  generateTeamCoverageCommand(coverageCommand)
-  groupCoverageByCodeownersCommand(coverageCommand)
+  generateTeamCoverageCommand(coverage)
+  groupCoverageByCodeownersCommand(coverage)
 
   return coverage
 }

package/src/commands/generate-team-coverage.js

@@ -2,6 +2,7 @@ import path from 'path'
 import * as url from 'url'
 import execa from 'execa'
 import fs from 'fs'
+import { print } from '@toptal/davinci-cli-shared'
 
 import Codeowners from './codeowners/core/codeowners/codeowners.js'
 
@@ -14,7 +15,9 @@ const sanitizeNycPercentage = percentage =>
   percentage === 'Unknown' ? 0 : percentage
 
 // eslint-disable-next-line max-statements
-const generateTeamCoverage = async () => {
+const generateTeamCoverage = async options => {
+  print.header('generateTeamCoverage')
+
   const codeowners = new Codeowners(rootProjectPath)
 
   const ownerEntries = codeowners.ownerEntries
@@ -66,15 +69,12 @@ const generateTeamCoverage = async () => {
   const results = []
   const created_at = new Date().toISOString()
 
-  // TODO pass it as param
-  const repo = 'client-portal'
-
   for await (const team of teams) {
     const sanitizedTeam = sanitizeTeamName(team)
     const result = {
       created_at,
       team,
-      repo,
+      repo: options.repository,
     }
 
     const jestCoverageSummary = JSON.parse(
@@ -127,12 +127,14 @@ const generateTeamCoverage = async () => {
 }
 
 export const generateTeamCoverageCommand = program => {
+  print.header('generateTeamCoverageCommand')
+
   return program
-    .createCommand('generate-team-coverage')
+    .command('generate-team-coverage')
     .description('TODO: generate-team-coverage')
     .requiredOption(
       '-r, --repository <repository_name>',
       'specify repository name'
     )
-    .action((_, options) => generateTeamCoverage(options))
+    .action(options => generateTeamCoverage(options))
 }

package/src/commands/group-coverage-by-codeowners.js

@@ -1,6 +1,7 @@
 import path from 'path'
 import fs from 'fs'
 import * as url from 'url'
+import { print } from '@toptal/davinci-cli-shared'
 
 import Codeowners from './codeowners/core/codeowners/codeowners.js'
 
@@ -74,8 +75,8 @@ const getAllTeams = codeowners => {
   return uniqueTeams
 }
 
-const groupCoverageByCodeowners = async options => {
-  console.log(options)
+const groupCoverageByCodeowners = async () => {
+  print.header('groupCoverageByCodeowners')
 
   const jestReport = readJestCoverageReport()
   const cypressReport = readCypressCoverageReport()
@@ -88,12 +89,14 @@ const groupCoverageByCodeowners = async options => {
 }
 
 export const groupCoverageByCodeownersCommand = program => {
+  print.header('groupCoverageByCodeownersCommand')
+
   return program
-    .createCommand('group-coverage-by-codeowners')
+    .command('group-coverage-by-codeowners')
     .description('TODO: group-coverage-by-codeowners')
     .requiredOption(
       '-r, --repository <repository_name>',
       'specify repository name'
     )
-    .action((_, options) => groupCoverageByCodeowners(options))
+    .action(groupCoverageByCodeowners)
 }

package/src/commands/__snapshots__/group-coverage-by-codeowners.test.js.snap

@@ -1,61 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`groupCoverageByCodeownersCommand has the correct command structure 1`] = `
-"{
-  "_events": {},
-  "_eventsCount": 1,
-  "commands": [],
-  "options": [
-    {
-      "flags": "-r, --repository <repository_name>",
-      "description": "specify repository name",
-      "required": true,
-      "optional": false,
-      "variadic": false,
-      "mandatory": true,
-      "short": "-r",
-      "long": "--repository",
-      "negate": false,
-      "hidden": false,
-      "conflictsWith": []
-    }
-  ],
-  "parent": null,
-  "_allowUnknownOption": false,
-  "_allowExcessArguments": true,
-  "_args": [],
-  "args": [],
-  "rawArgs": [],
-  "processedArgs": [],
-  "_scriptPath": null,
-  "_name": "group-coverage-by-codeowners",
-  "_optionValues": {},
-  "_optionValueSources": {},
-  "_storeOptionsAsProperties": false,
-  "_executableHandler": false,
-  "_executableFile": null,
-  "_executableDir": null,
-  "_defaultCommandName": null,
-  "_exitCallback": null,
-  "_aliases": [],
-  "_combineFlagAndOptionalValue": true,
-  "_description": "TODO: group-coverage-by-codeowners",
-  "_summary": "",
-  "_enablePositionalOptions": false,
-  "_passThroughOptions": false,
-  "_lifeCycleHooks": {},
-  "_showHelpAfterError": false,
-  "_showSuggestionAfterError": true,
-  "_outputConfiguration": {},
-  "_hidden": false,
-  "_hasHelpOption": true,
-  "_helpFlags": "-h, --help",
-  "_helpDescription": "display help for command",
-  "_helpShortFlag": "-h",
-  "_helpLongFlag": "--help",
-  "_helpCommandName": "help",
-  "_helpCommandnameAndArgs": "help [command]",
-  "_helpCommandDescription": "display help for command",
-  "_helpConfiguration": {}
-}"
-`;

package/src/commands/group-coverage-by-codeowners.test.js

@@ -1,23 +0,0 @@
-import { jest } from '@jest/globals'
-// eslint-disable-next-line import/no-extraneous-dependencies
-import { Command } from 'commander'
-
-import { groupCoverageByCodeownersCommand } from './group-coverage-by-codeowners.js'
-
-describe('groupCoverageByCodeownersCommand', () => {
-  let program
-
-  beforeEach(() => {
-    program = new Command()
-  })
-
-  afterEach(() => {
-    jest.clearAllMocks()
-  })
-
-  it('has the correct command structure', () => {
-    const command = groupCoverageByCodeownersCommand(program)
-
-    expect(JSON.stringify(command, null, 2)).toMatchSnapshot()
-  })
-})