chore: add functional test for hostname generator

Author: AllanZhengYP

codegen/smithy-aws-typescript-codegen/src/main/java/software/amazon/smithy/aws/typescript/codegen/EndpointGenerator.java

@@ -133,7 +133,7 @@ private void writeEndpointProviderFunction() {
         writer.openBlock("export const defaultRegionInfoProvider: RegionInfoProvider = (\n"
                          + "  region: string,\n"
                          + "  options?: any\n"
-                         + ") {", "};", () -> {
+                         + ") => {", "};", () -> {
             writer.openBlock("switch (region) {", "}", () -> {
                 writer.write("// First, try to match exact region names.");
                 for (Map.Entry<String, ObjectNode> entry : endpoints.entrySet()) {
@@ -144,7 +144,7 @@ private void writeEndpointProviderFunction() {
                 writer.write("// Next, try to match partition endpoints.");
                 writer.write("default:").indent();
                 partitions.values().forEach(partition -> {
-                    writer.openBlock("if (region in $L) {", "}", partition.regionVariableName, () -> {
+                    writer.openBlock("if ($L.has(region)) {", "}", partition.regionVariableName, () -> {
                         writePartitionEndpointResolver(partition);
                     });
                 });

package.json

@@ -14,8 +14,7 @@
     "build:smithy-client": "lerna run --scope '@aws-sdk/client-rds-data' --include-dependencies pretest",
     "pretest": "yarn build:crypto-dependencies && yarn build:smithy-client",
     "test": "jest --coverage --passWithNoTests",
-    "pretest-all": "lerna run pretest",
-    "test-all": "jest --coverage",
+    "test:functional": "jest --config tests/functional/jest.config.js --passWithNoTests",
     "integ-test": "cucumber.js"
   },
   "repository": {

tests/functional/endpoints/fixtures.js

@@ -0,0 +1,15 @@
+const { KNOWN_REGIONS } = require("./fixtures");
+const { getRegionInfoProvider } = require("../util/regioninfo-provider");
+
+describe("hostname for know regions", () => {
+  for (const region of Object.keys(KNOWN_REGIONS)) {
+    describe(region, () => {
+      for (const [service, hostname] of Object.entries(KNOWN_REGIONS[region])) {
+        it(`${service} should resolve to hostname ${hostname}`, () => {
+          const regionInfoProvider = getRegionInfoProvider(service);
+          expect(regionInfoProvider(region).hostname).toBe(hostname);
+        });
+      }
+    });
+  }
+});

tests/functional/endpoints/index.spec.js

@@ -0,0 +1,5 @@
+const base = require("../../jest.config.base");
+module.exports = {
+  ...base,
+  testMatch: ["./**/?(*.)+(spec|test).js?(x)"]
+};

tests/functional/jest.config.js

@@ -0,0 +1,5 @@
+module.exports.SIGNING_NAMES = {
+  "us-west-2": {
+    "rds-data": undefined
+  }
+};

tests/functional/signing-names/fixtures.js

@@ -0,0 +1,17 @@
+const { SIGNING_NAMES } = require("./fixtures");
+const { getRegionInfoProvider } = require("../util/regioninfo-provider");
+
+describe("signing names for know regions", () => {
+  for (const region of Object.keys(SIGNING_NAMES)) {
+    describe(region, () => {
+      for (const [service, signingName] of Object.entries(
+        SIGNING_NAMES[region]
+      )) {
+        it(`${service} should infer signing name ${signingName}`, () => {
+          const regionInfoProvider = getRegionInfoProvider(service);
+          expect(regionInfoProvider(region).signingService).toBe(signingName);
+        });
+      }
+    });
+  }
+});

tests/functional/signing-names/index.spec.js

@@ -0,0 +1,14 @@
+const { join } = require("path");
+module.exports.getRegionInfoProvider = fromService => {
+  const path = join(
+    "..",
+    "..",
+    "..",
+    "clients",
+    `client-${fromService}`,
+    "dist",
+    "cjs",
+    "endpoints"
+  );
+  return require(path).defaultRegionInfoProvider;
+};