chore: add tests for clarity & shave 7b

Author: lukeed

package.json

@@ -2,7 +2,7 @@
   "version": "2.0.0-next.0",
   "name": "resolve.exports",
   "repository": "lukeed/resolve.exports",
-  "description": "A tiny (959b), correct, general-purpose, and configurable \"exports\" and \"imports\" resolver without file-system reliance",
+  "description": "A tiny (952b), correct, general-purpose, and configurable \"exports\" and \"imports\" resolver without file-system reliance",
   "module": "dist/index.mjs",
   "main": "dist/index.js",
   "types": "index.d.ts",

readme.md

@@ -1,6 +1,6 @@
 # resolve.exports [![CI](https://github.com/lukeed/resolve.exports/workflows/CI/badge.svg)](https://github.com/lukeed/resolve.exports/actions) [![codecov](https://codecov.io/gh/lukeed/resolve.exports/branch/master/graph/badge.svg?token=4P7d4Omw2h)](https://codecov.io/gh/lukeed/resolve.exports)
 
-> A tiny (959b), correct, general-purpose, and configurable `"exports"` and `"imports"` resolver without file-system reliance
+> A tiny (952b), correct, general-purpose, and configurable `"exports"` and `"imports"` resolver without file-system reliance
 
 ***Why?***
 

src/index.ts

@@ -28,7 +28,8 @@ export function resolve(pkg: t.Package, input?: string, options?: t.Options): st
 	// let entry = input && input !== '.'
 	// 	? toEntry(pkg.name, input)
 	// 	: '.';
-	let entry = toEntry(pkg.name, input || '.');
-	if (entry[0] === '#') return imports(pkg, entry, options);
-	if (entry[0] === '.') return exports(pkg, entry, options);
+	input = toEntry(pkg.name, input || '.');
+	return input[0] === '#'
+		? imports(pkg, input, options)
+		: exports(pkg, input, options);
 }

test/index.ts

@@ -96,6 +96,42 @@ describe('$.resolve', it => {
 			assert.is((err as Error).message, `Missing "#bar" specifier in "foobar" package`);
 		}
 	});
+
+	it('should run `$.export` if given "external" identifier', () => {
+		let pkg: Package = {
+			"name": "foobar",
+			"exports": {
+				".": "./foo.mjs"
+			}
+		};
+
+		try {
+			lib.resolve(pkg, 'external');
+			assert.unreachable();
+		} catch (err) {
+			assert.instance(err, Error);
+			// IMPORTANT: treats "external" as "./external"
+			assert.is((err as Error).message, `Missing "./external" specifier in "foobar" package`);
+		}
+	});
+
+	it('should run `$.export` if given "external/subpath" identifier', () => {
+		let pkg: Package = {
+			"name": "foobar",
+			"exports": {
+				".": "./foo.mjs"
+			}
+		};
+
+		try {
+			lib.resolve(pkg, 'external/subpath');
+			assert.unreachable();
+		} catch (err) {
+			assert.instance(err, Error);
+			// IMPORTANT: treats "external/subpath" as "./external/subpath"
+			assert.is((err as Error).message, `Missing "./external/subpath" specifier in "foobar" package`);
+		}
+	});
 });
 
 describe('$.imports', it => {