Fix types

test/browser.test.ts

@@ -1,6 +1,8 @@
 /* eslint-disable @typescript-eslint/no-var-requires */
 import { bold, red, gray, yellow } from 'nanocolors'
 
+import CssSyntaxError from '../lib/css-syntax-error.js'
+
 beforeEach(() => {
   jest.resetModules()
   jest.doMock('fs', () => ({}))
@@ -10,17 +12,17 @@ beforeEach(() => {
 it('shows code with colors', () => {
   let postcss = require('../lib/postcss.js')
 
-  let error
+  let error: CssSyntaxError | undefined
   try {
     postcss.parse('a{')
   } catch (e) {
-    if (e.name === 'CssSyntaxError') {
+    if (e instanceof CssSyntaxError) {
       error = e
     } else {
       throw e
     }
   }
-  expect(error.showSourceCode(true)).toEqual(
+  expect(error?.showSourceCode(true)).toEqual(
     bold(red('>')) +
       gray(' 1 | ') +
       'a' +
@@ -34,17 +36,17 @@ it('shows code with colors', () => {
 it('shows code without colors', () => {
   let postcss = require('../lib/postcss.js')
 
-  let error
+  let error: CssSyntaxError | undefined
   try {
     postcss.parse('a{')
   } catch (e) {
-    if (e.name === 'CssSyntaxError') {
+    if (e instanceof CssSyntaxError) {
       error = e
     } else {
       throw e
     }
   }
-  expect(error.showSourceCode(false)).toEqual('> 1 | a{\n' + '    | ^')
+  expect(error?.showSourceCode(false)).toEqual('> 1 | a{\n' + '    | ^')
 })
 
 it('generates source map without fs', () => {

test/container.test.ts

@@ -200,7 +200,7 @@ it('walk() adds CSS position to error stack', () => {
   let error = new Error('T')
   error.stack = 'Error: T\n    at b (b.js:2:4)\n    at a (a.js:2:1)'
   let root = parse(example, { from: '/c.css' })
-  let catched
+  let catched: any
   try {
     root.walk(() => {
       throw error

test/css-syntax-error.test.ts

@@ -11,12 +11,19 @@ import postcss, {
   Rule
 } from '../lib/postcss.js'
 
+function isSyntaxError(e: unknown): e is CssSyntaxError {
+  return e instanceof Error && e.name === 'CssSyntaxError'
+}
+
 async function catchError(cb: () => Promise<any>): Promise<CssSyntaxError> {
   try {
     await cb()
   } catch (e) {
-    if (e.name !== 'CssSyntaxError') throw e
-    return e
+    if (isSyntaxError(e)) {
+      return e
+    } else {
+      throw e
+    }
   }
   throw new Error('Error was not thrown')
 }
@@ -25,17 +32,16 @@ function parseError(
   css: string,
   opts?: Pick<ProcessOptions, 'map' | 'from'>
 ): CssSyntaxError {
-  let error
   try {
     postcss.parse(css, opts)
   } catch (e) {
-    if (e.name === 'CssSyntaxError') {
-      error = e
+    if (isSyntaxError(e)) {
+      return e
     } else {
       throw e
     }
   }
-  return error
+  throw new Error('Error was not thrown')
 }
 
 it('saves source', () => {

test/parse.test.ts

@@ -177,7 +177,7 @@ it('throws on just colon', () => {
 })
 
 it('does not suggest different parsers for CSS', () => {
-  let error
+  let error: any
   try {
     parse('a { one:: 1 }', { from: 'app.css' })
   } catch (e) {
@@ -205,7 +205,7 @@ it('suggests postcss-less for Less sources', () => {
 })
 
 it('should give the correct column of missed semicolon with !important', () => {
-  let error
+  let error: any
   try {
     parse('a { \n    color: red !important\n    background-color: black;\n}')
   } catch (e) {
@@ -215,7 +215,7 @@ it('should give the correct column of missed semicolon with !important', () => {
 })
 
 it('should give the correct column of missed semicolon without !important', () => {
-  let error
+  let error: any
   try {
     parse('a { \n    color: red\n    background-color: black;\n}')
   } catch (e) {

test/processor.test.ts

@@ -12,6 +12,7 @@ import postcss, {
   Parser,
   Stringifier
 } from '../lib/postcss.js'
+import CssSyntaxError from '../lib/css-syntax-error.js'
 import LazyResult from '../lib/lazy-result.js'
 import Processor from '../lib/processor.js'
 import Rule from '../lib/rule.js'
@@ -36,7 +37,7 @@ async function catchError(cb: () => Promise<any>): Promise<Error> {
   try {
     await cb()
   } catch (e) {
-    return e
+    if (e instanceof Error) return e
   }
   throw new Error('Error was not thrown')
 }
@@ -147,19 +148,19 @@ it('inlines maps from previous result', () => {
 })
 
 it('throws with file name', () => {
-  let error
+  let error: CssSyntaxError | undefined
   try {
     new Processor([() => {}]).process('a {', { from: 'a.css' }).css
   } catch (e) {
-    if (e.name === 'CssSyntaxError') {
+    if (e instanceof CssSyntaxError) {
       error = e
     } else {
       throw e
     }
   }
 
-  expect(error.file).toEqual(pathResolve('a.css'))
-  expect(error.message).toMatch(/a.css:1:1: Unclosed block$/)
+  expect(error?.file).toEqual(pathResolve('a.css'))
+  expect(error?.message).toMatch(/a.css:1:1: Unclosed block$/)
 })
 
 it('allows to replace Root', () => {
@@ -373,7 +374,7 @@ it('remembers errors', async () => {
     processing.root
   }).toThrow('test')
 
-  let asyncError
+  let asyncError: any
   try {
     await processing
   } catch (e) {

test/visitor.test.ts

@@ -514,7 +514,7 @@ it('adds plugin to error', async () => {
       throw rule.error('test')
     }
   }
-  let error
+  let error: any
   try {
     postcss([broken]).process('a{}', { from: 'broken.css' }).css
   } catch (e) {
@@ -533,7 +533,7 @@ it('adds plugin to async error', async () => {
       throw rule.error('test')
     }
   }
-  let error
+  let error: any
   try {
     await postcss([broken]).process('a{}', { from: 'broken.css' })
   } catch (e) {
@@ -551,7 +551,7 @@ it('adds sync plugin to async error', async () => {
       throw rule.error('test')
     }
   }
-  let error
+  let error: any
   try {
     await postcss([broken]).process('a{}', { from: 'broken.css' })
   } catch (e) {
@@ -569,7 +569,7 @@ it('adds node to error', async () => {
       throw new Error('test')
     }
   }
-  let error
+  let error: any
   try {
     postcss([broken]).process('a{}', { from: 'broken.css' }).css
   } catch (e) {
@@ -588,7 +588,7 @@ it('adds node to async error', async () => {
       throw new Error('test')
     }
   }
-  let error
+  let error: any
   try {
     await postcss([broken]).process('a{}', { from: 'broken.css' })
   } catch (e) {
@@ -604,7 +604,7 @@ it('shows error on sync call async plugins', () => {
     postcssPlugin: 'asyncPlugin',
     async Rule() {}
   }
-  let error
+  let error: any
   try {
     postcss([asyncPlugin]).process('a{}', { from: 'broken.css' }).css
   } catch (e) {
@@ -1400,7 +1400,7 @@ it('throws error from async OnceExit', async () => {
     from: 'a.css'
   })
 
-  let error
+  let error: any
   try {
     await result
   } catch (e) {