Skip to content

test(dafny): beacon partition tests#2199

Merged
josecorella merged 11 commits into
ajewell/bucketsfrom
jocorell/partition-beacon-tests
Apr 30, 2026
Merged

test(dafny): beacon partition tests#2199
josecorella merged 11 commits into
ajewell/bucketsfrom
jocorell/partition-beacon-tests

Conversation

@josecorella

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown

Detected changes to the release files or to the check-files action

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown

Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS

@github-actions

Copy link
Copy Markdown

Detected changes to the release files or to the check-files action

@github-actions

Copy link
Copy Markdown

Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS

@github-actions

Copy link
Copy Markdown

Detected changes to the release files or to the check-files action

@github-actions

Copy link
Copy Markdown

Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS

@github-actions

Copy link
Copy Markdown

Detected changes to the release files or to the check-files action

@github-actions

Copy link
Copy Markdown

Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS

@josecorella josecorella changed the title test(dafny): add basic beacon partition boundary tests test(dafny): beacon partition tests Apr 23, 2026
@github-actions

Copy link
Copy Markdown

Detected changes to the release files or to the check-files action

@github-actions

Copy link
Copy Markdown

Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS

@github-actions

Copy link
Copy Markdown

Detected changes to the release files or to the check-files action

@github-actions

Copy link
Copy Markdown

Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS

Comment thread DynamoDbEncryption/dafny/DynamoDbEncryption/test/BeaconPartition.dfy Outdated
@github-actions

Copy link
Copy Markdown

@josecorella and @josecorella, I noticed you are updating the smithy model files.
Does this update need new or updated javadoc trait documentation?
Are you adding constraints inside list, map or union? Do you know about this issue: smithy-lang/smithy-dafny#491?

@github-actions

Copy link
Copy Markdown

@josecorella and @josecorella, I noticed you are updating the smithy model files.
Does this update need new or updated javadoc trait documentation?
Are you adding constraints inside list, map or union? Do you know about this issue: smithy-lang/smithy-dafny#491?

@github-actions

Copy link
Copy Markdown

@josecorella and @josecorella, I noticed you are updating the smithy model files.
Does this update need new or updated javadoc trait documentation?
Are you adding constraints inside list, map or union? Do you know about this issue: smithy-lang/smithy-dafny#491?

@github-actions

Copy link
Copy Markdown

@josecorella and @josecorella, I noticed you are updating the smithy model files.
Does this update need new or updated javadoc trait documentation?
Are you adding constraints inside list, map or union? Do you know about this issue: smithy-lang/smithy-dafny#491?

@github-actions

Copy link
Copy Markdown

Detected changes to the release files or to the check-files action

@github-actions

Copy link
Copy Markdown

Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS

@github-actions

Copy link
Copy Markdown

Detected changes to the release files or to the check-files action

@github-actions

Copy link
Copy Markdown

Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS

@josecorella
josecorella marked this pull request as ready for review April 28, 2026 18:59
@josecorella
josecorella requested a review from a team as a code owner April 28, 2026 18:59
src1
);
expect res1.Failure?;
expect_equal(res1.error, E("Constrained numberOfPartitions for std2 is 5 but it must be less than the maximumNumberOfPartitions 5"));

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: double space

Suggested change
expect_equal(res1.error, E("Constrained numberOfPartitions for std2 is 5 but it must be less than the maximumNumberOfPartitions 5"));
expect_equal(res1.error, E("Constrained numberOfPartitions for std2 is 5 but it must be less than the maximumNumberOfPartitions 5"));

@github-actions

Copy link
Copy Markdown

@josecorella and @rishav-karanjit, I noticed you are updating the smithy model files.
Does this update need new or updated javadoc trait documentation?
Are you adding constraints inside list, map or union? Do you know about this issue: smithy-lang/smithy-dafny#491?

@josecorella
josecorella merged commit 5789490 into ajewell/buckets Apr 30, 2026
39 of 45 checks passed
@josecorella
josecorella deleted the jocorell/partition-beacon-tests branch April 30, 2026 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants