Skip to content

Improve test coverage to ~100% for @azure/core-paging#38179

Merged
deyaaeldeen merged 1 commit intomainfrom
deyaaeldeen/core-paging-test-coverage
Apr 17, 2026
Merged

Improve test coverage to ~100% for @azure/core-paging#38179
deyaaeldeen merged 1 commit intomainfrom
deyaaeldeen/core-paging-test-coverage

Conversation

@deyaaeldeen
Copy link
Copy Markdown
Member

@deyaaeldeen deyaaeldeen commented Apr 16, 2026

Adds tests to bring @azure/core-paging to ~100% test coverage.

Changes

  • Extended getPagedAsyncIterator tests with edge cases
  • 1 file changed, 41 lines added

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@deyaaeldeen deyaaeldeen marked this pull request as ready for review April 17, 2026 00:32
@deyaaeldeen deyaaeldeen requested a review from a team as a code owner April 17, 2026 00:32
Copilot AI review requested due to automatic review settings April 17, 2026 00:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR increases unit test coverage for @azure/core-paging by adding additional getPagedAsyncIterator test cases that exercise edge paths in the iterator implementation (multi-page toElements extraction and byPage() behavior when the first page is undefined).

Changes:

  • Added a test validating element extraction across multiple pages when PagedResult.toElements is provided.
  • Added a test ensuring byPage() completes immediately when getPage() returns undefined for the first page.

@deyaaeldeen deyaaeldeen merged commit 1198906 into main Apr 17, 2026
19 checks passed
@deyaaeldeen deyaaeldeen deleted the deyaaeldeen/core-paging-test-coverage branch April 17, 2026 21:39
deyaaeldeen added a commit that referenced this pull request Apr 18, 2026
Adds tests to bring `@azure/core-paging` to ~100% test coverage.


### Changes
- Extended getPagedAsyncIterator tests with edge cases
- 1 file changed, 41 lines added

Co-authored-by: Deyaaeldeen Almahallawi <deyaa@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants