Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(shared-tree): Add simple LazyOptionalField unit tests #17475

Merged
merged 41 commits into from Sep 28, 2023

Conversation

Josmithr
Copy link
Contributor

@Josmithr Josmithr commented Sep 26, 2023

Adds an intro suite of unit tests for LazyOptionalField. Specifically, adds tests for the following properties and methods:

  • is
  • length
  • map
  • mapBoxed

@github-actions github-actions bot added area: dds Issues related to distributed data structures area: dds: tree base: main PRs targeted against main branch labels Sep 26, 2023
@Josmithr Josmithr requested review from noencke and a team September 27, 2023 18:05
@Josmithr Josmithr marked this pull request as ready for review September 27, 2023 18:05
@Josmithr Josmithr requested a review from a team as a code owner September 27, 2023 18:05
Co-authored-by: Noah Encke <78610362+noencke@users.noreply.github.com>
@noencke
Copy link
Contributor

noencke commented Sep 28, 2023

/azp run Build - client packages

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@noencke noencke merged commit f198a8f into microsoft:main Sep 28, 2023
28 checks passed
@Josmithr Josmithr deleted the shared-tree/lazy-field-tests branch September 28, 2023 17:10
DLehenbauer pushed a commit that referenced this pull request Sep 29, 2023
This PR implements sequence mutation methods. Tests will be added in a
follow-up PR when #17475 is complete.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: dds: tree area: dds Issues related to distributed data structures base: main PRs targeted against main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants