Skip to content

Commit

Permalink
chore(Python): Merge in remaining changes (#550)
Browse files Browse the repository at this point in the history
  • Loading branch information
lucasmcdonald3 authored Oct 2, 2024
1 parent 7e0c431 commit c571f51
Show file tree
Hide file tree
Showing 3 changed files with 28 additions and 12 deletions.
24 changes: 17 additions & 7 deletions .releaserc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,12 @@ const Runtimes = {
// must be accounted for in `CheckDependencyReplacementResults`.
python: {
"AwsCryptographicMaterialProviders/runtimes/python/pyproject.toml": {
dependencies: ["AwsCryptographyPrimitives", "ComAmazonawsKms", "ComAmazonawsDynamodb", "StandardLibrary"],
dependencies: [
"AwsCryptographyPrimitives",
"ComAmazonawsKms",
"ComAmazonawsDynamodb",
"StandardLibrary",
],
},
"AwsCryptographyPrimitives/runtimes/python/pyproject.toml": {
dependencies: ["StandardLibrary"],
Expand Down Expand Up @@ -148,15 +153,17 @@ module.exports = {
to: 'version = "${nextRelease.version}"',
results: Object.keys(Runtimes.python).map(CheckResults),
countMatches: true,
},
},

// Now update the local filesystem dependencies to PyPI dependencies
// pinned to the minor MPL version
{
files: Object.keys(Runtimes.python),
from: "{path =.*",
to: "\"~${nextRelease.version}\"",
results: Object.keys(Runtimes.python).map(CheckDependencyReplacementResults),
to: '"~${nextRelease.version}"',
results: Object.keys(Runtimes.python).map(
CheckDependencyReplacementResults,
),
countMatches: true,
},
],
Expand All @@ -166,7 +173,8 @@ module.exports = {
// Re-transpile Python code to update .dtr files as part of the release commit
"@semantic-release/exec",
{
"prepareCmd": "make -C TestVectorsAwsCryptographicMaterialProviders transpile_python"
prepareCmd:
"make -C TestVectorsAwsCryptographicMaterialProviders transpile_python",
},
],
// Commit and push changes the changelog and versions bumps
Expand Down Expand Up @@ -235,6 +243,8 @@ function CheckDependencyReplacementResults(file) {
numReplacements: 1,
};
} else {
throw new Error(`No known dependency replacement result specification for file ${file}`)
throw new Error(
`No known dependency replacement result specification for file ${file}`,
);
}
}
}
14 changes: 10 additions & 4 deletions AwsCryptographyPrimitives/test/TestECDH.dfy
Original file line number Diff line number Diff line change
Expand Up @@ -128,10 +128,12 @@ module TestECDH {
const INFINITY_POINT_ERR_MSG_JAVA := "encoded key spec not recognized: Point at infinity"
const INFINITY_POINT_ERR_MSG_NET6 := "Point at infinity (Parameter 'q')"
const INFINITY_POINT_ERR_MSG_NET48 := "Point at infinity\r\nParameter name: q"
const INFINITY_POINT_ERR_MSG_PYTHON := "Unable to load EC key"

const OUT_OF_BOUNDS_ERR_MSG_JAVA := "encoded key spec not recognized: x value invalid for"
const OUT_OF_BOUNDS_ERR_MSG_NET6 := "value invalid for Fp field element (Parameter 'x')"
const OUT_OF_BOUNDS_ERR_MSG_NE48 := "value invalid for Fp field element\r\nParameter name: x"
const OUT_OF_BOUNDS_ERR_MSG_PYTHON := "Could not deserialize key data. The data may be in an incorrect format"

// Rust does not provide a separate error message for infinity or out of bounds
const BAD_X509_KEY_ERR_MSG_RUST := "Invalid X509 Public Key."
Expand Down Expand Up @@ -300,7 +302,8 @@ module TestECDH {
errMsg == BAD_X509_KEY_ERR_MSG_RUST ||
errMsg == INFINITY_POINT_ERR_MSG_JAVA ||
errMsg == INFINITY_POINT_ERR_MSG_NET6 ||
errMsg == INFINITY_POINT_ERR_MSG_NET48
errMsg == INFINITY_POINT_ERR_MSG_NET48 ||
seq_contains(errMsg, INFINITY_POINT_ERR_MSG_PYTHON)
);
}
}
Expand Down Expand Up @@ -349,7 +352,8 @@ module TestECDH {
seq_contains(errMsg, OUT_OF_BOUNDS_ERR_MSG_JAVA) ||
errMsg == BAD_X509_KEY_ERR_MSG_RUST ||
errMsg == OUT_OF_BOUNDS_ERR_MSG_NET6 ||
errMsg == OUT_OF_BOUNDS_ERR_MSG_NE48
errMsg == OUT_OF_BOUNDS_ERR_MSG_NE48 ||
seq_contains(errMsg, OUT_OF_BOUNDS_ERR_MSG_PYTHON)
);
}
}
Expand Down Expand Up @@ -517,7 +521,8 @@ module TestECDH {
errMsg == INFINITY_POINT_ERR_MSG_JAVA ||
errMsg == BAD_X509_KEY_ERR_MSG_RUST ||
errMsg == INFINITY_POINT_ERR_MSG_NET6 ||
errMsg == INFINITY_POINT_ERR_MSG_NET48
errMsg == INFINITY_POINT_ERR_MSG_NET48 ||
seq_contains(errMsg, INFINITY_POINT_ERR_MSG_PYTHON)
);
}
}
Expand All @@ -544,7 +549,8 @@ module TestECDH {
seq_contains(errMsg, OUT_OF_BOUNDS_ERR_MSG_JAVA) ||
errMsg == BAD_X509_KEY_ERR_MSG_RUST ||
errMsg == OUT_OF_BOUNDS_ERR_MSG_NET6 ||
errMsg == OUT_OF_BOUNDS_ERR_MSG_NE48
errMsg == OUT_OF_BOUNDS_ERR_MSG_NE48 ||
seq_contains(errMsg, OUT_OF_BOUNDS_ERR_MSG_PYTHON)
);
}

Expand Down
2 changes: 1 addition & 1 deletion smithy-dafny
Submodule smithy-dafny updated 84 files
+9 −1 SmithyDafnyMakefile.mk
+5 −6 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/client.rs
+113 −94 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/attribute_value.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/attribute_value_update.rs
+1 −1 ...aws-sdks/ddb-lite/runtimes/rust/src/conversions/batch_execute_statement/_batch_execute_statement_request.rs
+2 −2 ...ws-sdks/ddb-lite/runtimes/rust/src/conversions/batch_execute_statement/_batch_execute_statement_response.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/batch_get_item/_batch_get_item_request.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/batch_get_item/_batch_get_item_response.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/batch_statement_request.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/batch_statement_response.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/batch_write_item/_batch_write_item_request.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/batch_write_item/_batch_write_item_response.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/cancellation_reason.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/condition.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/condition_check.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/consumed_capacity.rs
+8 −8 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/create_table/_create_table_request.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/create_table/_create_table_response.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/delete.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/delete_item/_delete_item_request.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/delete_item/_delete_item_response.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/delete_request.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/describe_table/_describe_table_response.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/error/transaction_canceled_exception.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/execute_statement/_execute_statement_request.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/execute_statement/_execute_statement_response.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/execute_transaction/_execute_transaction_request.rs
+2 −2 ...Models/aws-sdks/ddb-lite/runtimes/rust/src/conversions/execute_transaction/_execute_transaction_response.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/expected_attribute_value.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/get.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/get_item/_get_item_request.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/get_item/_get_item_response.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/global_secondary_index.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/global_secondary_index_description.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/item_collection_metrics.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/item_response.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/keys_and_attributes.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/local_secondary_index.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/local_secondary_index_description.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/parameterized_statement.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/put.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/put_item/_put_item_request.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/put_item/_put_item_response.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/put_request.rs
+4 −4 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/query/_query_request.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/query/_query_response.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/replica_description.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/replica_global_secondary_index_description.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/restore_summary.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/scan/_scan_request.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/scan/_scan_response.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/stream_specification.rs
+12 −12 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/table_description.rs
+1 −1 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/transact_get_items/_transact_get_items_request.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/transact_get_items/_transact_get_items_response.rs
+4 −4 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/transact_write_item.rs
+1 −1 ...odels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/transact_write_items/_transact_write_items_request.rs
+2 −2 ...dels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/transact_write_items/_transact_write_items_response.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/update.rs
+4 −4 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/update_item/_update_item_request.rs
+3 −3 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/update_item/_update_item_response.rs
+2 −2 TestModels/aws-sdks/ddb-lite/runtimes/rust/src/conversions/write_request.rs
+5 −0 TestModels/aws-sdks/ddb/Makefile
+1 −0 TestModels/aws-sdks/ddb/runtimes/rust/Cargo.toml
+42 −0 TestModels/aws-sdks/ddb/runtimes/rust/src/ddb.rs
+1 −1 TestModels/aws-sdks/ddb/src/Index.dfy
+5 −6 TestModels/aws-sdks/kms-lite/runtimes/rust/src/client.rs
+1 −1 TestModels/aws-sdks/kms-lite/runtimes/rust/src/conversions/decrypt/_decrypt_request.rs
+1 −1 ...odels/aws-sdks/kms-lite/runtimes/rust/src/conversions/derive_shared_secret/_derive_shared_secret_request.rs
+1 −1 TestModels/aws-sdks/kms-lite/runtimes/rust/src/conversions/generate_data_key/_generate_data_key_request.rs
+5 −0 TestModels/aws-sdks/kms/Makefile
+0 −41 TestModels/aws-sdks/kms/Model/model.json
+3 −2 TestModels/aws-sdks/kms/runtimes/rust/Cargo.toml
+88 −0 TestModels/aws-sdks/kms/runtimes/rust/src/kms.rs
+1 −1 TestModels/aws-sdks/kms/src/Index.dfy
+22 −18 ...ython/codegen/smithy-python-codegen/src/main/java/software/amazon/smithy/python/codegen/UnionGenerator.java
+0 −2 codegen/smithy-dafny-codegen-test/src/test/java/software/amazon/polymorph/smithyrust/RustTestModels.java
+2 −32 codegen/smithy-dafny-codegen/src/main/java/software/amazon/polymorph/CodegenEngine.java
+134 −0 ...a/software/amazon/polymorph/smithypython/localservice/extensions/DafnyPythonLocalServiceUnionGenerator.java
+86 −6 ...y-dafny-codegen/src/main/java/software/amazon/polymorph/smithyrust/generator/AbstractRustShimGenerator.java
+172 −64 ...thy-dafny-codegen/src/main/java/software/amazon/polymorph/smithyrust/generator/RustAwsSdkShimGenerator.java
+3 −77 ...hy-dafny-codegen/src/main/java/software/amazon/polymorph/smithyrust/generator/RustLibraryShimGenerator.java
+2 −2 codegen/smithy-dafny-codegen/src/main/resources/templates/runtimes/rust/conversions/standard_structure.rs
+2 −2 codegen/smithy-dafny-codegen/src/main/resources/templates/runtimes/rust/conversions/union.rs

0 comments on commit c571f51

Please sign in to comment.