vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #282 from coreos-inc/logout500

Browse source 
Fix 500 on logout
This commit is contained in:
Jimmy Zelinskie 2015-07-28 15:48:31 -04:00
commit 3a9337686e
2 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
endpoints/api/repository.py
View file

@ -110,7 +110,7 @@ class RepositoryList(ApiResource):
# If starred (and only starred) repositories were requested, then load them directly. # If starred (and only starred) repositories were requested, then load them directly.
if starred and namespace is None and not public: if starred and namespace is None and not public:
if not username: if not username:
return [] return [], set()
repositories = model.repository.get_user_starred_repositories(get_authenticated_user(), repositories = model.repository.get_user_starred_repositories(get_authenticated_user(),
limit=limit, limit=limit,

8
test/test_api_usage.py
View file

@ -1366,6 +1366,14 @@ class TestListRepos(ApiTestCase):
for repo in json['repositories']: for repo in json['repositories']:
self.assertEquals(ORGANIZATION, repo['namespace']) self.assertEquals(ORGANIZATION, repo['namespace'])
def test_listrepos_starred_nouser(self):
json = self.getJsonResponse(RepositoryList,
params=dict(last_modified=True,
popularity=True,
starred=True))
self.assertEquals(len(json['repositories']), 0)
def test_listrepos_starred(self): def test_listrepos_starred(self):
self.login(ADMIN_ACCESS_USER) self.login(ADMIN_ACCESS_USER)