test: add coverage for bypass_modes in rulesets#2802
Merged
nickfloyd merged 2 commits intointegrations:mainfrom Oct 22, 2025
Merged
test: add coverage for bypass_modes in rulesets#2802nickfloyd merged 2 commits intointegrations:mainfrom
nickfloyd merged 2 commits intointegrations:mainfrom
Conversation
Add test coverage for all three bypass_modes (always, pull_request,
exempt) for both repository and organization rulesets.
Repository Ruleset Tests (resource_github_repository_ruleset_test.go):
- Add test for all three bypass_modes with Team actors
- Add test for updating bypass_mode (always → exempt)
- Add test for different actor types with different bypass_modes
(Team/always, RepositoryRole/pull_request, OrganizationAdmin/exempt)
Organization Ruleset Tests (resource_github_organization_ruleset_test.go):
- Fix existing "Creates and updates organization using bypasses" test:
* Move bypass_actors from incorrect location (inside rules) to correct
location (at ruleset level)
* Add missing bypass_mode assertions
* Fix incorrect actor_type assertions (was checking for "0", "5" instead
of actual actor type strings)
* Correct OrganizationAdmin actor_id from 0 to 1
- Add test for all three bypass_modes
- Add test for updating bypass_mode
All tests now verify:
- actor_id (both dynamic Team IDs and static role IDs)
- actor_type (Team, RepositoryRole, OrganizationAdmin, DeployKey)
- bypass_mode (always, pull_request, exempt)
This addresses the review comment from PR integrations#2764 requesting test coverage
for bypass_modes, including the newly added "exempt" mode.
Fixes: integrations#2764 (review)
4 tasks
nickfloyd
approved these changes
Oct 22, 2025
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
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.
Add test coverage for all three bypass_modes (always, pull_request, exempt) for both repository and organization rulesets.
Repository Ruleset Tests (resource_github_repository_ruleset_test.go):
Organization Ruleset Tests (resource_github_organization_ruleset_test.go):
All tests now verify:
This addresses the review comment from PR #2764 requesting test coverage for bypass_modes, including the newly added "exempt" mode.
Fixes: #2764 (review)
Fixes #2755
Before the change?
After the change?
Pull request checklist
Does this introduce a breaking change?
Please see our docs on breaking changes to help!