Add function to remove app installation as a GitHub App #483
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Created TEMPLATE.ps1 for structuring Pester tests with common setup. - Added Teams.Tests.ps1 to test GitHub Teams API functionalities including team creation, retrieval, updating, and deletion. - Implemented Users.Tests.ps1 to validate user-related API calls such as fetching user details and updating user information. - Developed Variables.Tests.ps1 to test GitHub repository and organization variable management, including setting, updating, and removing variables.
There was a problem hiding this comment.
Pull Request Overview
This PR adds functionality for uninstalling GitHub App installations, along with test infrastructure and configuration changes. The main focus is implementing proper cleanup mechanisms for GitHub App installations in organization contexts.
- Implements
Remove-GitHubAppInstallationprivate function for deleting app installations via API - Enhances
Uninstall-GitHubApppublic function with improved parameter handling and installation deletion logic - Adds comprehensive test coverage for app installation cleanup in organization lifecycle tests
Reviewed Changes
Copilot reviewed 4 out of 17 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
src/functions/private/Apps/GitHub Apps/Remove-GitHubAppInstallation.ps1 |
New private function implementing GitHub API call to delete app installations |
src/functions/public/Apps/GitHub App/Uninstall-GitHubApp.ps1 |
Enhanced parameter set handling and added default parameter set for installation deletion |
tests/Organizations.Tests.ps1 |
Added pre-test and post-test cleanup logic for app installations in organization tests |
.github/PSModule.yml |
Uncommented configuration to skip various test components and documentation builds |
Comments suppressed due to low confidence (1)
tests/Organizations.Tests.ps1:174
- The test assertion should include a descriptive message to make test failures more informative. Consider adding '-Because' parameter to explain what this assertion validates.
$remainingInstallations | Should -BeNullOrEmpty
src/functions/private/Apps/GitHub Apps/Remove-GitHubAppInstallation.ps1
Outdated
Show resolved
Hide resolved
…or uninstalling GitHub Apps
MariusStorhaug
changed the title
MariusStorhaug
force-pushed
the
removeinstalls
branch
from
3a099c6 to
a9f8063
Compare
… enhance GitHubAppContext with TokenExpiresIn property
…n GitHub context formatting
… in GitHub context formatting
…ds in GitHub context formatting
…n GitHub context formatting
…HubUnsignedJWT function
…d 10 minutes for 'APP' in GitHub context formatting
…b App installations
…eOrganization function to clarify it uninstalls from enterprise-owned organizations
…parameter handling for improved clarity and functionality
…y in Uninstall-GitHubApp function
…p for improved context handling
…e parameter in Organizations.Tests.ps1
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Type of change
Checklist