Skip to content

Return all limit#30

Merged
baristaTam merged 6 commits into
mainfrom
ReturnAll-Limit
Jun 1, 2026
Merged

Return all limit#30
baristaTam merged 6 commits into
mainfrom
ReturnAll-Limit

Conversation

@baristaTam
Copy link
Copy Markdown
Contributor

[1.2.1] 2026-06-01

Changed

  • Get-CDBItem -Returnall parameter changed to use limit of 200 to reduce errors from service per IAM's guidance

edthedev
edthedev previously approved these changes Jun 1, 2026
Comment thread test/UofICDB.Tests.ps1
Describe 'Get-CDBItemPermission'{
It 'Returns permissions for a given item'{
(Get-CDBItemPermission -id 1778).permissions | Should -Not -BeNullOrEmpty
(Get-CDBItemPermission -id 8777).permissions | Should -Not -BeNullOrEmpty
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should this have $TestId rather than the ID value?

Comment thread test/UofICDB.Tests.ps1

It 'Accepts pipeline input'{
{Get-CDBItem -Id 1778 | Get-CDBItemPermission} | Should -Not -Throw
{Get-CDBItem -Id 8777 | Get-CDBItemPermission} | Should -Not -Throw
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same here about $TestId.

@baristaTam baristaTam merged commit 7798ba1 into main Jun 1, 2026
4 checks passed
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.

4 participants