🧪 test(none): improve coverage (#2599)

- **🔨 improve(patch): add @types/node to create bud app**
- **🧪 test(none): improve @roots/bud-api coverage**

config/vitest/config.unit.ts

@@ -7,6 +7,13 @@ export default defineConfig({
     ...shared,
     coverage: {
       enabled: false,
+      exclude: [`sources/@roots/bud-support/**/*`],
+      include: [
+        `sources/@roots/*/src/*.ts`,
+        `sources/@roots/*/src/*.tsx`,
+        `sources/@roots/*/src/**/*.ts`,
+        `sources/@roots/*/src/**/*.tsx`,
+      ],
     },
     exclude: [`tests/e2e`, `**/node_modules`],
     include: [

sources/@roots/bud-api/src/methods/compilePaths/index.ts

@@ -1,6 +1,49 @@
-export {compilePaths} from './compilePaths.js'
-import type {Rules} from '@roots/bud-framework'
+import type {Bud, Rules} from '@roots/bud-framework'
 
-type Source = Array<RegExp | string> | RegExp | string
+import {InputError} from '@roots/bud-support/errors'
+import isArray from '@roots/bud-support/isArray'
+import isString from '@roots/bud-support/isString'
+
+export type Source = Array<RegExp | string> | RegExp | string
 
 export type Parameters = [Source, Array<`${keyof Rules & string}`>?]
+
+export interface compilePaths {
+  (this: Bud, ...value: Parameters): Bud
+}
+
+export const compilePaths: compilePaths = function (sources, rules) {
+  const sourcesArray = isArray(sources) ? sources : [sources]
+
+  sourcesArray.forEach(source => {
+    if (!isString(source) && !(source instanceof RegExp)) {
+      throw new InputError(
+        `bud.compilePaths: source must be a string or a regular expression.`,
+      )
+    }
+  })
+
+  this.hooks.action(`build.before`, async bud => {
+    const keys: Array<`${keyof Rules & string}`> = (rules ??
+      Object.keys(bud.build.rules)) as `${keyof Rules & string}`[]
+
+    const matches = keys.map(key => {
+      const match = bud.build.getRule(key)
+
+      if (!match) {
+        throw new InputError(
+          `bud.compilePaths: \`${key}\` is not a valid rule name.`,
+        )
+      }
+
+      return match
+    })
+
+    matches.map(rule => {
+      bud.api.logger.log(`setting compile paths for ${rule.getTest()}`)
+      rule.setInclude(sourcesArray)
+    })
+  })
+
+  return this
+}

sources/@roots/bud-api/test/__snapshots__/bundle.test.ts.snap

@@ -1,6 +1,6 @@
 // Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
-exports[`bud.bundle > should set the bundle using a string 1`] = `
+exports[`@roots/bud-api/methods/bundle > should set the bundle using a string 1`] = `
 {
   "automaticNameDelimiter": "/",
   "cacheGroups": {
@@ -18,7 +18,7 @@ exports[`bud.bundle > should set the bundle using a string 1`] = `
 }
 `;
 
-exports[`bud.bundle > should set the bundle using a string name and a string test 1`] = `
+exports[`@roots/bud-api/methods/bundle > should set the bundle using a string name and a string test 1`] = `
 {
   "automaticNameDelimiter": "/",
   "cacheGroups": {
@@ -36,7 +36,7 @@ exports[`bud.bundle > should set the bundle using a string name and a string tes
 }
 `;
 
-exports[`bud.bundle > should set the bundle using a string name and array of strings test 1`] = `
+exports[`@roots/bud-api/methods/bundle > should set the bundle using a string name and array of strings test 1`] = `
 {
   "automaticNameDelimiter": "/",
   "cacheGroups": {
@@ -54,7 +54,7 @@ exports[`bud.bundle > should set the bundle using a string name and array of str
 }
 `;
 
-exports[`bud.bundle > should set the bundle using a string name and regular expression test 1`] = `
+exports[`@roots/bud-api/methods/bundle > should set the bundle using a string name and regular expression test 1`] = `
 {
   "automaticNameDelimiter": "/",
   "cacheGroups": {

sources/@roots/bud-api/test/__snapshots__/define.test.ts.snap

@@ -1,6 +1,6 @@
 // Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
-exports[`bud.define > matches snapshot 1`] = `
+exports[`@roots/bud-api/methods/define > matches snapshot 1`] = `
 {
   "APP_DESCRIPTION": ""test app description"",
   "APP_TITLE": ""bud.js test app"",

sources/@roots/bud-api/test/__snapshots__/watch.test.ts.snap

@@ -1,12 +1,12 @@
 // Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
-exports[`bud.watch > [Array, Options] parameters > should add watch files and options 1`] = `
+exports[`@roots/bud-api/methods/watch > [Array, Options] parameters > should add watch files and options 1`] = `
 [
   "1/*.js",
 ]
 `;
 
-exports[`bud.watch > in development > should add watch files 1`] = `
+exports[`@roots/bud-api/methods/watch > in development > should add watch files 1`] = `
 [
   "1/*.js",
 ]

sources/@roots/bud-api/test/alias.test.ts

@@ -4,7 +4,7 @@ import {beforeAll, describe, expect, it} from 'vitest'
 
 import {alias as aliasFn} from '../src/methods/alias/index.ts'
 
-describe(`bud.alias`, () => {
+describe(`@roots/bud-api/methods/alias`, () => {
   let bud: Bud
   let alias: typeof aliasFn
 

sources/@roots/bud-api/test/assets.test.ts

@@ -3,7 +3,7 @@ import {beforeEach, describe, expect, it} from 'vitest'
 
 import {assets} from '../src/methods/assets/index.ts'
 
-describe(`bud.assets`, () => {
+describe(`@roots/bud-api/methods/assets`, () => {
   let bud: Bud
   let assetsFn: typeof assets
 

sources/@roots/bud-api/test/bundle.test.ts

@@ -4,7 +4,7 @@ import {beforeEach, describe, expect, it} from 'vitest'
 
 import {bundle} from '../src/methods/bundle/index.ts'
 
-describe(`bud.bundle`, () => {
+describe(`@roots/bud-api/methods/bundle`, () => {
   let bud: Bud
   let instance: typeof bundle