Skip to content

Commit aecc64d

Browse files
authored
Fix tests on .com (#12131)
1 parent 971d94a commit aecc64d

File tree

3 files changed

+18
-1
lines changed

3 files changed

+18
-1
lines changed

readthedocs/oauth/services/github.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -452,7 +452,7 @@ def remove_webhook(self, project):
452452
return False
453453
return True
454454

455-
def remove_ssh_key(self, project):
455+
def remove_ssh_key(self, project) -> bool:
456456
"""
457457
Remove the SSH key from the GitHub repository associated with the project.
458458

readthedocs/profiles/tests/test_views.py

+15
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
MESSAGE_OAUTH_WEBHOOK_NOT_REMOVED,
2828
)
2929
from readthedocs.oauth.services.github import GitHubService
30+
from readthedocs.organizations.models import Organization
3031
from readthedocs.projects.models import Project
3132

3233

@@ -175,6 +176,20 @@ def setUp(self):
175176
repo="https://github.com/user/repo-e",
176177
)
177178

179+
# Make tests work on .com.
180+
if settings.RTD_ALLOW_ORGANIZATIONS:
181+
self.organization = get(
182+
Organization,
183+
owners=[self.user],
184+
projects=[
185+
self.project_with_remote_repository,
186+
self.project_with_remote_repository_no_admin,
187+
self.project_with_remote_repository_no_member,
188+
self.project_with_remote_organization,
189+
self.project_without_remote_repository,
190+
],
191+
)
192+
178193
self.url = reverse("migrate_to_github_app")
179194
self.client.force_login(self.user)
180195

readthedocs/rtd_tests/tests/test_project_forms.py

+2
Original file line numberDiff line numberDiff line change
@@ -560,6 +560,8 @@ def test_change_default_branch_from_tag_to_branch_and_vice_versa(self, trigger_b
560560
"versioning_scheme": self.project.versioning_scheme,
561561
"language": self.project.language,
562562
"default_branch": branch.verbose_name,
563+
"privacy_level": self.project.privacy_level,
564+
"external_builds_privacy_level": self.project.external_builds_privacy_level,
563565
}
564566
form = UpdateProjectForm(data, instance=self.project)
565567
assert form.is_valid()

0 commit comments

Comments
 (0)