Tags: winglang/wing
Tags
fix: structs can't parse json with null values (#7258) Fixes #7257 by accepting `null` for any optional field in structs and converting `null` to `undefined` during parsing from JSON to structs. ## Checklist - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted) - [x] Description explains motivation and solution - [x] Tests added (always) - [x] Docs updated (only required for features) - [x] Added `pr/e2e-full` label if this feature requires end-to-end testing *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*. --------- Signed-off-by: monada-bot[bot] <[email protected]> Co-authored-by: monada-bot[bot] <[email protected]>
chore: test harness interface (#7248) Extract common logic across Terraform and AWS CDK targets when running tests. This is an initial step towards allowing custom platforms to provide test running capabilities. ## Checklist - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted) - [x] Description explains motivation and solution - [x] Tests added (always) - [x] Docs updated (only required for features) - [x] Added `pr/e2e-full` label if this feature requires end-to-end testing *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
chore: extract common aws logic across cloud api (#7245) Create base classes for all cloud API resources to share common logic across AWS platforms ## Checklist - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted) - [x] Description explains motivation and solution - [x] Tests added (always) - [x] Docs updated (only required for features) - [x] Added `pr/e2e-full` label if this feature requires end-to-end testing *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*. --------- Signed-off-by: monada-bot[bot] <[email protected]> Co-authored-by: monada-bot[bot] <[email protected]>
fix(sdk): cloud.Service test when using tf-aws target (#7233) I try to run this test whit the command `wing test --platform tf-aws minimal.test.w` ```w bring cloud; let s = new cloud.Service(inflight () => { log("hello, service!"); return () => { log("stopping!"); }; }); test "start and stop" { assert(s.started()); s.stop(); assert(!s.started()); } ``` but I get this error ```shell $ wing test --platform tf-aws minimal.test.w ✔ Compiling minimal.test.w to tf-aws... ✔ terraform init ✖ terraform apply Command failed: terraform apply -auto-approve ╷ │ Error: creating ECS Cluster (Test.mfet8zI2Z4_cluster): InvalidParameterException: Cluster name must match ^[a-zA-Z0-9\-_]{1,255}$, but was Test.mfet8zI2Z4_cluster │ │ with aws_ecs_cluster.Testmfet8zI2Z4_env0_Service_ECSCluster_1D77BD98, │ on main.tf.json line 116, in resource.aws_ecs_cluster.Testmfet8zI2Z4_env0_Service_ECSCluster_1D77BD98: │ 116: } │ ╵ ✔ terraform destroy Tests 1 failed (1) Snapshots 1 skipped Test Files 1 failed (1) Duration 3m29.12s ``` When objects are generated for the test, the default is to generate the cluster name like this Test.mfet8zI2Z4_cluster. However, the cluster name cannot have a `.`, so I made a small adjustment to replace `.` with `_` to avoid this error ## Checklist - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted) - [x] Description explains motivation and solution - [ ] Tests added (always) - [ ] Docs updated (only required for features) - [ ] Added `pr/e2e-full` label if this feature requires end-to-end testing *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
fix(sdk): some fixes and minor refactoring (#7234) I made two small corrections and a refactoring in this PR: 1. There was an error in an environment variable related to the bucket name that was passed in the aws-cdk build. 2. When updating the lambda function log, I assigned the new enum to the old variable, but I have now corrected it to the proper variable. 3. It didn't make sense to have a file just to keep a global variable related to the Counter, so I removed the file and moved the global variable to the shared-aws/counter.ts file. ## Checklist - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted) - [x] Description explains motivation and solution - [ ] Tests added (always) - [ ] Docs updated (only required for features) - [ ] Added `pr/e2e-full` label if this feature requires end-to-end testing *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
PreviousNext