Skip to content

Commit

Permalink
fix: find_tenant() now returns a tuple
Browse files Browse the repository at this point in the history
  • Loading branch information
atodorov committed Jul 22, 2020
1 parent 4fb59ec commit e8fd165
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion tcms_github_app/tests/test_utils.py
Expand Up @@ -59,4 +59,6 @@ def test_for_missing_installation_returns_none(self):
payload=json.loads(payload),
)

self.assertIsNone(utils.find_tenant(wh_payload))
tenant, app_inst = utils.find_tenant(wh_payload)
self.assertIsNone(tenant)
self.assertIsNone(app_inst)
2 changes: 1 addition & 1 deletion tcms_github_app/utils.py
Expand Up @@ -71,7 +71,7 @@ def find_tenant(data):
).first()

if not app_inst:
return None
return None, None

tenant = Tenant.objects.filter(pk=app_inst.tenant_pk).first()

Expand Down

0 comments on commit e8fd165

Please sign in to comment.