Make enhanced the default for us_street_api

Author: RyanLCox1

src/us_street/Client.js

@@ -3,7 +3,7 @@ const Lookup = require("./Lookup");
 const Batch = require("../Batch");
 const UndefinedLookupError = require("../Errors").UndefinedLookupError;
 const sendBatch = require("../util/sendBatch");
-const keyTranslationFormat = require("../util/apiToSDKKeyMap").usStreet;
+const buildUsStreetInputData = require("../util/buildUsStreetInputData");
 
 /**
  * This client sends lookups to the Smarty US Street API, <br>
@@ -28,15 +28,13 @@ class Client {
 		let batch;
 
 		if (dataIsLookup) {
-			if (data.maxCandidates == null && data.match == "enhanced")
-				data.maxCandidates = 5;
 			batch = new Batch();
 			batch.add(data);
 		} else {
 			batch = data;
 		}
 
-		return sendBatch(batch, this.sender, Candidate, keyTranslationFormat);
+		return sendBatch(batch, this.sender, Candidate, null, buildUsStreetInputData);
 	}
 }
 

src/util/apiToSDKKeyMap.js

@@ -12,7 +12,8 @@ module.exports = {
 		"match": "match",
 		"format": "format",
 		"candidates": "maxCandidates",
-		"county_source": "countySource"
+		"county_source": "countySource",
+		"input_id": "inputId"
 	},
 	usAutocompletePro: {
 		search: "search",

src/util/buildUsStreetInputData.js

@@ -0,0 +1,35 @@
+const buildInputData = require("./buildInputData");
+const keyTranslationFormat = require("./apiToSDKKeyMap").usStreet;
+
+module.exports = (lookup) => {
+	// Apply default match strategy and candidates logic per Go SDK behavior
+	let effectiveMatch = lookup.match;
+	let effectiveCandidates = lookup.maxCandidates;
+
+	// Default match strategy is "enhanced"
+	if (!effectiveMatch) {
+		effectiveMatch = "enhanced";
+	}
+
+	// If match is "strict", don't send match parameter or default candidates
+	if (effectiveMatch === "strict") {
+		effectiveMatch = undefined;
+		// Only send candidates if explicitly set
+		if (!effectiveCandidates) {
+			effectiveCandidates = undefined;
+		}
+	} else {
+		// For non-strict (including default "enhanced"), set default candidates to 5 if not specified
+		if (!effectiveCandidates) {
+			effectiveCandidates = 5;
+		}
+	}
+
+	// Create a lookup copy with effective values for serialization
+	const effectiveLookup = Object.assign({}, lookup, {
+		match: effectiveMatch,
+		maxCandidates: effectiveCandidates,
+	});
+
+	return buildInputData(effectiveLookup, keyTranslationFormat);
+};

src/util/sendBatch.js

@@ -2,7 +2,7 @@ const Request = require("../Request");
 const Errors = require("../Errors");
 const buildInputData = require("../util/buildInputData");
 
-module.exports = (batch, sender, Result, keyTranslationFormat) => {
+module.exports = (batch, sender, Result, keyTranslationFormat, customBuildInputData) => {
 	if (batch.isEmpty()) throw new Errors.BatchEmptyError;
 
 	let request = new Request();
@@ -22,6 +22,9 @@ module.exports = (batch, sender, Result, keyTranslationFormat) => {
 
 	function generateRequestPayload(batch) {
 		return batch.lookups.map((lookup) => {
+			if (customBuildInputData) {
+				return customBuildInputData(lookup);
+			}
 			return buildInputData(lookup, keyTranslationFormat);
 		});
 	}

tests/us_street/test_Client.js

@@ -42,6 +42,23 @@ describe("A US Street client", function () {
 			urbanization: "9",
 			match: "10",
 			candidates: "11",
+			input_id: "12",
+		};
+
+		client.send(lookup);
+
+		expect(mockSender.request.parameters).to.deep.equal(expectedParameters);
+	});
+
+	it("defaults to enhanced match with 5 candidates when no match type specified.", function () {
+		let mockSender = new MockSender();
+		const client = new Client(mockSender);
+		let lookup = new Lookup();
+		lookup.street = "123 Main St";
+		let expectedParameters = {
+			street: "123 Main St",
+			match: "enhanced",
+			candidates: 5,
 		};
 
 		client.send(lookup);
@@ -64,6 +81,53 @@ describe("A US Street client", function () {
 		expect(mockSender.request.parameters).to.deep.equal(expectedParameters);
 	});
 
+	it("sends no match or candidates when match type is strict.", function () {
+		let mockSender = new MockSender();
+		const client = new Client(mockSender);
+		let lookup = new Lookup();
+		lookup.street = "123 Main St";
+		lookup.match = "strict";
+		let expectedParameters = {
+			street: "123 Main St",
+		};
+
+		client.send(lookup);
+
+		expect(mockSender.request.parameters).to.deep.equal(expectedParameters);
+	});
+
+	it("sends candidates but not match when match is strict with explicit candidates.", function () {
+		let mockSender = new MockSender();
+		const client = new Client(mockSender);
+		let lookup = new Lookup();
+		lookup.street = "123 Main St";
+		lookup.match = "strict";
+		lookup.maxCandidates = 3;
+		let expectedParameters = {
+			street: "123 Main St",
+			candidates: 3,
+		};
+
+		client.send(lookup);
+
+		expect(mockSender.request.parameters).to.deep.equal(expectedParameters);
+	});
+
+	it("sends match invalid when explicitly set.", function () {
+		let mockSender = new MockSender();
+		const client = new Client(mockSender);
+		let lookup = new Lookup();
+		lookup.match = "invalid";
+		let expectedParameters = {
+			match: "invalid",
+			candidates: 5,
+		};
+
+		client.send(lookup);
+
+		expect(mockSender.request.parameters).to.deep.equal(expectedParameters);
+	});
+
 	it("doesn't send an empty batch.", function () {
 		let mockSender = new MockSender();
 		const client = new Client(mockSender);