exports and imports field continue resolving extensions, directories and aliases if fullySpecified is not specified

fixes #256

Author: sokra

lib/ResolverFactory.js

@@ -377,7 +377,7 @@ exports.createResolver = function (options) {
 				"internal",
 				conditionNames,
 				importsField,
-				"final-file",
+				"described-relative",
 				"internal-resolve"
 			)
 		);
@@ -444,7 +444,7 @@ exports.createResolver = function (options) {
 				"resolve-in-package",
 				conditionNames,
 				exportsField,
-				"final-file"
+				"described-relative"
 			)
 		);
 	});

test/exportsField.js

@@ -1314,6 +1314,13 @@ describe("ExportsFieldPlugin", () => {
 	const nodeFileSystem = new CachedInputFileSystem(fs, 4000);
 
 	const resolver = ResolverFactory.createResolver({
+		extensions: [".js"],
+		fileSystem: nodeFileSystem,
+		fullySpecified: true,
+		conditionNames: ["webpack"]
+	});
+
+	const commonjsResolver = ResolverFactory.createResolver({
 		extensions: [".js"],
 		fileSystem: nodeFileSystem,
 		conditionNames: ["webpack"]
@@ -1354,7 +1361,7 @@ describe("ExportsFieldPlugin", () => {
 		);
 	});
 
-	it("resolve using exports field, not a browser field #2", done => {
+	it("resolve using exports field and a browser alias field #2", done => {
 		const resolver = ResolverFactory.createResolver({
 			aliasFields: ["browser"],
 			extensions: [".js"],
@@ -1371,7 +1378,7 @@ describe("ExportsFieldPlugin", () => {
 				if (err) return done(err);
 				if (!result) throw new Error("No result");
 				result.should.equal(
-					path.resolve(fixture2, "node_modules/exports-field/lib/main.js")
+					path.resolve(fixture2, "node_modules/exports-field/lib/browser.js")
 				);
 				done();
 			}
@@ -1393,6 +1400,38 @@ describe("ExportsFieldPlugin", () => {
 		);
 	});
 
+	it("throw error if extension not provided", done => {
+		resolver.resolve(
+			{},
+			fixture2,
+			"exports-field/dist/main",
+			{},
+			(err, result) => {
+				if (!err) throw new Error(`expect error, got ${result}`);
+				err.should.be.instanceof(Error);
+				err.message.should.match(/Can't resolve/);
+				done();
+			}
+		);
+	});
+
+	it("should resolve extension without fullySpecified", done => {
+		commonjsResolver.resolve(
+			{},
+			fixture2,
+			"exports-field/dist/main",
+			{},
+			(err, result) => {
+				if (err) return done(err);
+				if (!result) throw new Error("No result");
+				result.should.equal(
+					path.resolve(fixture2, "node_modules/exports-field/lib/lib2/main.js")
+				);
+				done();
+			}
+		);
+	});
+
 	it("resolver should respect condition names", done => {
 		resolver.resolve(
 			{},

test/importsField.js

@@ -1325,10 +1325,16 @@ describe("ImportsFieldPlugin", () => {
 						"              existing directory .../node_modules/a",
 						"                using description file: .../node_modules/a/package.json (relative path: .)",
 						"                  using exports field: ./lib/lib2/index.js",
-						"                    .../node_modules/a/lib/lib2/index.js doesn't exist",
+						"                    no extension",
+						"                      .../node_modules/a/lib/lib2/index.js doesn't exist",
+						"                    .js",
+						"                      .../node_modules/a/lib/lib2/index.js.js doesn't exist",
+						"                    as directory",
+						"                      .../node_modules/a/lib/lib2/index.js doesn't exist",
 						"                  using exports field: ./lib/index.js",
-						"                    existing file: .../node_modules/a/lib/index.js",
-						"                      reporting result .../node_modules/a/lib/index.js"
+						"                    no extension",
+						"                      existing file: .../node_modules/a/lib/index.js",
+						"                        reporting result .../node_modules/a/lib/index.js"
 					]);
 				done();
 			}