fix(types): generate declaration files instead of importing from src (#24)

Author: jgoux

README.md

@@ -25,7 +25,9 @@ practices.</p>
 [![Code of Conduct](https://img.shields.io/badge/code%20of-conduct-ff69b4.svg?style=flat-square)](./CODE_OF_CONDUCT.md)
 
 <!-- ALL-CONTRIBUTORS-BADGE:START - Do not remove or modify this section -->
+
 [![All Contributors](https://img.shields.io/badge/all_contributors-5-orange.svg?style=flat-square)](#contributors-)
+
 <!-- ALL-CONTRIBUTORS-BADGE:END -->
 
 [![Watch on GitHub](https://img.shields.io/github/watchers/crutchcorn/cli-testing-library.svg?style=social)](https://github.com/crutchcorn/cli-testing-library/watchers)

docs/api.md

@@ -185,7 +185,7 @@ exit code it closed with.
 ```javascript
 const instance = render('command')
 
-await waitFor(() => expect(instance.hasExit()).toMatchObject({ exitCode: 1 }))
+await waitFor(() => expect(instance.hasExit()).toMatchObject({exitCode: 1}))
 ```
 
 This method returns `null` if still running, but `{exitCode: number}` if it has
@@ -206,8 +206,8 @@ They're defined as:
 
 ```typescript
 interface TestInstance {
-  stdoutArr: Array<{contents: Buffer | string, timestamp: number}>
-  stderrArr: Array<{contents: Buffer | string, timestamp: number}>
+  stdoutArr: Array<{contents: Buffer | string; timestamp: number}>
+  stderrArr: Array<{contents: Buffer | string; timestamp: number}>
 }
 ```
 

src/__tests__/execute-scripts/log-err.js

@@ -1,3 +1,3 @@
-console.log("Log here");
-console.warn("Warn here");
-console.error("Error here");
+console.log('Log here')
+console.warn('Warn here')
+console.error('Error here')

src/config.ts

@@ -27,17 +27,15 @@ let config: InternalConfig = {
 
   // called when getBy* queries fail. (message, container) => Error
   getInstanceError(message, testInstance: TestInstance | undefined) {
-    let instanceWarning: string = "";
+    let instanceWarning: string = ''
     if (testInstance) {
-      const stdallArrStr = testInstance.getStdallStr();
+      const stdallArrStr = testInstance.getStdallStr()
       instanceWarning = `\n${stdallArrStr}`
     } else {
-      instanceWarning = "";
+      instanceWarning = ''
     }
     const error = new Error(
-      [message, instanceWarning]
-        .filter(Boolean)
-        .join('\n\n'),
+      [message, instanceWarning].filter(Boolean).join('\n\n'),
     )
     error.name = 'TestingLibraryElementError'
     return error

src/index.js

@@ -4,7 +4,10 @@ import {cleanup} from './pure'
 // or teardown then we'll automatically run cleanup afterEach test
 // this ensures that tests run in isolation from each other
 // if you don't like this then set the CTL_SKIP_AUTO_CLEANUP env variable to 'true'.
-if (typeof process === 'undefined' || !(process.env && process.env.CTL_SKIP_AUTO_CLEANUP)) {
+if (
+  typeof process === 'undefined' ||
+  !(process.env && process.env.CTL_SKIP_AUTO_CLEANUP)
+) {
   // ignore teardown() in code coverage because Jest does not support it
   /* istanbul ignore else */
   if (typeof afterEach === 'function') {

src/matchers/to-be-in-the-console.js

@@ -8,7 +8,9 @@ export function toBeInTheConsole(instance) {
   }
 
   const errormessage = instance
-    ? getDefaultNormalizer()(instance.stdoutArr.map(obj => obj.contents).join('\n'))
+    ? getDefaultNormalizer()(
+        instance.stdoutArr.map(obj => obj.contents).join('\n'),
+      )
     : null
 
   return {

src/matchers/to-have-errormessage.js

@@ -11,7 +11,9 @@ export function toHaveErrorMessage(testInstance, checkWith) {
 
   const expectsErrorMessage = checkWith !== undefined
 
-  const errormessage = getDefaultNormalizer()(testInstance.stderrArr.map(obj => obj.contents).join('\n'))
+  const errormessage = getDefaultNormalizer()(
+    testInstance.stderrArr.map(obj => obj.contents).join('\n'),
+  )
 
   return {
     pass: expectsErrorMessage

src/pretty-cli.js

@@ -15,7 +15,7 @@ function prettyCLI(testInstance, maxLength) {
     throw new TypeError(`Expected an instance but got ${testInstance}`)
   }
 
-  const outStr = testInstance.getStdallStr();
+  const outStr = testInstance.getStdallStr()
 
   // eslint-disable-next-line no-negated-condition
   return maxLength !== undefined && outStr.length > maxLength

src/pure.ts

@@ -1,4 +1,5 @@
 import childProcess from 'child_process'
+import {performance} from 'perf_hooks'
 import stripFinalNewline from 'strip-final-newline'
 import {RenderOptions, RenderResult, TestInstance} from '../types/pure'
 import {_runObservers} from './mutation-observer'
@@ -8,7 +9,6 @@ import {bindObjectFnsToInstance, setCurrentInstance} from './helpers'
 import {fireEvent} from './events'
 import {getConfig} from './config'
 import {logCLI} from './pretty-cli'
-import {performance} from 'perf_hooks'
 
 const mountedInstances = new Set<TestInstance>()
 
@@ -48,8 +48,8 @@ async function render(
     },
     // An array of strings gathered from stdout when unable to do
     // `await stdout` because of inquirer interactive prompts
-    stdoutArr: [] as Array<{contents: Buffer | string, timestamp: number}>,
-    stderrArr: [] as Array<{contents: Buffer | string, timestamp: number}>,
+    stdoutArr: [] as Array<{contents: Buffer | string; timestamp: number}>,
+    stderrArr: [] as Array<{contents: Buffer | string; timestamp: number}>,
     hasExit() {
       return this.__exitCode === null ? null : {exitCode: this.__exitCode}
     },
@@ -66,7 +66,10 @@ async function render(
     }
 
     const resStr = stripFinalNewline(result as string)
-    execOutputAPI.stdoutArr.push({contents: resStr, timestamp: performance.now()})
+    execOutputAPI.stdoutArr.push({
+      contents: resStr,
+      timestamp: performance.now(),
+    })
     _runObservers()
   })
 
@@ -77,7 +80,10 @@ async function render(
     }
 
     const resStr = stripFinalNewline(result as string)
-    execOutputAPI.stderrArr.push({contents: resStr, timestamp: performance.now()})
+    execOutputAPI.stderrArr.push({
+      contents: resStr,
+      timestamp: performance.now(),
+    })
     _runObservers()
   })
 
@@ -104,12 +110,12 @@ async function render(
 
   await execOutputAPI._isReady
 
-
   function getStdallStr(this: Omit<TestInstance, 'getStdallStr'>) {
-    return this.stderrArr.concat(this.stdoutArr)
-        .sort((a,b) => a.timestamp < b.timestamp ? -1 : 1)
-        .map(obj => obj.contents)
-        .join('\n');
+    return this.stderrArr
+      .concat(this.stdoutArr)
+      .sort((a, b) => (a.timestamp < b.timestamp ? -1 : 1))
+      .map(obj => obj.contents)
+      .join('\n')
   }
 
   return Object.assign(

src/suggestions.js

@@ -61,7 +61,9 @@ function canSuggest(currentMethod, requestedMethod, data) {
 }
 
 export function getSuggestedQuery(instance, variant = 'get', method) {
-  const textContent = normalize(instance.stdoutArr.map(obj => obj.contents).join('\n'))
+  const textContent = normalize(
+    instance.stdoutArr.map(obj => obj.contents).join('\n'),
+  )
   if (canSuggest('Text', method, textContent)) {
     return makeSuggestion('Text', instance, textContent, {variant})
   }