Skip to content

Commit 64cd9c8

Browse files
authored
chore: update smithy to 1.45.0 (#1172)
also makes test failures more descriptive
1 parent 5c98400 commit 64cd9c8

File tree

3 files changed

+20
-11
lines changed

3 files changed

+20
-11
lines changed

build.gradle.kts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,10 @@ subprojects {
213213
// Log on passed, skipped, and failed test events if the `-Plog-tests` property is set.
214214
if (project.hasProperty("log-tests")) {
215215
tasks.test {
216-
testLogging.events("passed", "skipped", "failed")
216+
testLogging {
217+
events("passed", "skipped", "failed")
218+
exceptionFormat = org.gradle.api.tasks.testing.logging.TestExceptionFormat.FULL
219+
}
217220
}
218221
}
219222

gradle.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
smithyVersion=1.41.1
1+
smithyVersion=1.45.0
22
smithyGradleVersion=0.6.0

smithy-typescript-codegen/src/test/java/software/amazon/smithy/typescript/codegen/PackageJsonGeneratorTest.java

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -77,17 +77,23 @@ void expectPackageBrowserFieldToBeMerged() {
7777
assertTrue(manifest.getFileString(PackageJsonGenerator.PACKAGE_JSON_FILENAME).isPresent());
7878

7979
String packageJson = manifest.getFileString(PackageJsonGenerator.PACKAGE_JSON_FILENAME).get();
80-
81-
assertThat(packageJson, containsString("""
82-
"browser": {
83-
"example-browser": "example",
84-
"./dist-es/runtimeConfig": "./dist-es/runtimeConfig.browser"
85-
},
86-
"react-native": {
80+
ObjectNode packageJsonNode = Node.parse(packageJson).expectObjectNode();
81+
82+
Node expectedBrowserNode = Node.parse("""
83+
{
84+
"example-browser": "example",
85+
"./dist-es/runtimeConfig": "./dist-es/runtimeConfig.browser"
86+
}
87+
""");
88+
Node expectedReactNativeNode = Node.parse("""
89+
{
8790
"example-react-native": "example",
8891
"./dist-es/runtimeConfig": "./dist-es/runtimeConfig.native"
89-
}
90-
"""));
92+
}
93+
""");
94+
95+
Node.assertEquals(packageJsonNode.expectObjectMember("browser"), expectedBrowserNode);
96+
Node.assertEquals(packageJsonNode.expectObjectMember("react-native"), expectedReactNativeNode);
9197
}
9298

9399
@Test

0 commit comments

Comments
 (0)