From eabc29d159dd82bb70a6a1de5ea3c4b478dd67bb Mon Sep 17 00:00:00 2001
From: Seth Falco <seth@falco.fun>
Date: Tue, 6 May 2025 08:05:09 +0100
Subject: [PATCH] fix: types for loadConfig and add more tests (#2123)

---
 lib/svgo-node.js               |  2 +-
 test-d/lib/svgo-node.test-d.ts | 13 ++++++++++++-
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/lib/svgo-node.js b/lib/svgo-node.js
index d539e304..236b3fcc 100644
--- a/lib/svgo-node.js
+++ b/lib/svgo-node.js
@@ -37,7 +37,7 @@ export * from './svgo.js';
  * You can also specify relative or absolute path and customize current working
  * directory.
  *
- * @type {<T extends string>(configFile?: T | null, cwd?: string) => Promise<T extends string ? import('./svgo.js').Config : import('./svgo.js').Config | null>}
+ * @type {<T extends string | null>(configFile?: T, cwd?: string) => Promise<T extends string ? import('./svgo.js').Config : import('./svgo.js').Config | null>}
  */
 export const loadConfig = async (configFile, cwd = process.cwd()) => {
   if (configFile != null) {
diff --git a/test-d/lib/svgo-node.test-d.ts b/test-d/lib/svgo-node.test-d.ts
index 82c7324d..4e246d9d 100644
--- a/test-d/lib/svgo-node.test-d.ts
+++ b/test-d/lib/svgo-node.test-d.ts
@@ -1,5 +1,16 @@
 import { expectType, expectAssignable } from 'tsd';
-import { DataUri, Output, optimize } from '../../types/lib/svgo-node.js';
+import {
+  type Config,
+  type DataUri,
+  type Output,
+  loadConfig,
+  optimize,
+} from '../../types/lib/svgo-node.js';
 
 expectType<Output>(optimize('<svg></svg>'));
 expectAssignable<DataUri>('enc');
+
+expectType<Promise<Config | null>>(loadConfig());
+expectType<Promise<Config | null>>(loadConfig(undefined));
+expectType<Promise<Config | null>>(loadConfig(null));
+expectType<Promise<Config>>(loadConfig('svgo.config.js'));