Skip to content

Commit afe5dcf

Browse files
ShubhamChaturvedi7josecorellalucasmcdonald3RitvikKapilaShubham Chaturvedi
authored
fix(Go): Adds high unicode char Test Vectors (#1327)
Co-authored-by: Jose Corella <joseeduardocorella@gmail.com> Co-authored-by: Lucas McDonald <lucasmcdonald3@gmail.com> Co-authored-by: Ritvik Kapila <61410899+RitvikKapila@users.noreply.github.com> Co-authored-by: Shubham Chaturvedi <scchatur@amazon.com> Co-authored-by: José Corella <39066999+josecorella@users.noreply.github.com>
1 parent 7f59a7e commit afe5dcf

58 files changed

Lines changed: 28728 additions & 28596 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/workflows/library_interop_tests.yml

Lines changed: 18 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
# TODO: Re-enable macOS after https://t.corp.amazon.com/P205755286
2727
# macos-13,
2828
]
29-
language: [java, net, python, rust, go]
29+
language: [java, net, rust, python, go]
3030
# https://taskei.amazon.dev/tasks/CrypTool-5284
3131
dotnet-version: ["6.0.x"]
3232
runs-on: ${{ matrix.os }}
@@ -46,11 +46,13 @@ jobs:
4646
role-to-assume: arn:aws:iam::370957321024:role/GitHub-CI-MPL-Dafny-Role-us-west-2
4747
role-session-name: InterOpTests
4848

49-
- uses: actions/checkout@v3
50-
# Not all submodules are needed.
51-
# We manually pull the submodule we DO need.
52-
- run: git submodule update --init libraries
53-
- run: git submodule update --init --recursive smithy-dafny
49+
- uses: actions/checkout@v4
50+
with:
51+
submodules: "true"
52+
- name: Init Submodules
53+
shell: bash
54+
run: |
55+
git submodule update --init --recursive smithy-dafny
5456
5557
# Set up runtimes
5658
- name: Setup .NET Core SDK ${{ matrix.dotnet-version }}
@@ -204,6 +206,7 @@ jobs:
204206
testInteroperablity:
205207
needs: generateEncryptVectors
206208
strategy:
209+
fail-fast: false
207210
matrix:
208211
library: [TestVectorsAwsCryptographicMaterialProviders]
209212
os: [
@@ -213,8 +216,8 @@ jobs:
213216
# TODO: Re-enable macOS after https://t.corp.amazon.com/P205755286
214217
# macos-13,
215218
]
216-
encrypting_language: [java, net, python, rust, go]
217-
decrypting_language: [java, net, python, rust, go]
219+
encrypting_language: [java, net, rust, python, go]
220+
decrypting_language: [java, net, rust, python, go]
218221
dotnet-version: ["6.0.x"]
219222
runs-on: ${{ matrix.os }}
220223
permissions:
@@ -233,11 +236,13 @@ jobs:
233236
role-to-assume: arn:aws:iam::370957321024:role/GitHub-CI-MPL-Dafny-Role-us-west-2
234237
role-session-name: InterOpTests
235238

236-
- uses: actions/checkout@v3
237-
# Not all submodules are needed.
238-
# We manually pull the submodule we DO need.
239-
- run: git submodule update --init libraries
240-
- run: git submodule update --init --recursive smithy-dafny
239+
- uses: actions/checkout@v4
240+
with:
241+
submodules: "true"
242+
- name: Init Submodules
243+
shell: bash
244+
run: |
245+
git submodule update --init --recursive smithy-dafny
241246
242247
# Set up runtimes
243248
- name: Setup .NET Core SDK ${{ matrix.dotnet-version }}

.github/workflows/library_net_tests.yml

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -95,14 +95,17 @@ jobs:
9595
# .NET Framework is a software development framework
9696
# for building and running applications on Windows.
9797
# https://dotnet.microsoft.com/en-us/learn/dotnet/what-is-dotnet-framework
98-
- name: Test ${{ matrix.library }} .NET Framework net48
99-
if: matrix.os == 'windows-latest'
100-
working-directory: ./${{ matrix.library }}
101-
shell: bash
102-
run: |
103-
nuget locals all -clear
104-
dotnet restore runtimes/net
105-
make test_net FRAMEWORK=net48
98+
99+
# .NET is broken on Windows
100+
# https://t.corp.amazon.com/P211278460
101+
# - name: Test ${{ matrix.library }} .NET Framework net48
102+
# if: matrix.os == 'windows-latest'
103+
# working-directory: ./${{ matrix.library }}
104+
# shell: bash
105+
# run: |
106+
# nuget locals all -clear
107+
# dotnet restore runtimes/net
108+
# make test_net FRAMEWORK=net48
106109

107110
- name: Test ${{ matrix.library }} .NET net6.0
108111
working-directory: ./${{ matrix.library }}

.github/workflows/library_rust_tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ jobs:
3939
id-token: write
4040
contents: read
4141
env:
42-
RUST_MIN_STACK: 104857600
42+
RUST_MIN_STACK: 404857600
4343
steps:
4444
- name: Support longpaths on Git checkout
4545
run: |

.github/workflows/pull.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@ jobs:
5454
uses: ./.github/workflows/library_interop_tests.yml
5555
with:
5656
dafny: ${{needs.getVersion.outputs.version}}
57+
secrets: inherit
5758
pr-ci-all-required:
5859
if: always()
5960
needs:

AwsCryptographicMaterialProviders/runtimes/go/ImplementationFromDafny-go/awscryptographykeystoresmithygenerated/to_dafny.go

Lines changed: 238 additions & 99 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

AwsCryptographicMaterialProviders/runtimes/go/ImplementationFromDafny-go/awscryptographykeystoresmithygenerated/to_native.go

Lines changed: 224 additions & 335 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

AwsCryptographicMaterialProviders/runtimes/go/ImplementationFromDafny-go/awscryptographykeystoresmithygeneratedtypes/types.go

Lines changed: 15 additions & 15 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)