From d64f31893c86c1b4fff71d682f1be881505b8f77 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Mar 06 2014 20:04:45 +0000 Subject: Change default of /api/packagers to return the list of all packagers if no pattern is provided --- diff --git a/pkgdb2/api/packagers.py b/pkgdb2/api/packagers.py index de40a04..8070f5d 100644 --- a/pkgdb2/api/packagers.py +++ b/pkgdb2/api/packagers.py @@ -198,7 +198,7 @@ List packagers httpcode = 200 output = {} - pattern = flask.request.args.get('pattern', None) or pattern + pattern = flask.request.args.get('pattern', pattern) or '*' if pattern: packagers = pkgdblib.search_packagers(SESSION, pattern=pattern, diff --git a/tests/test_flask_api_packagers.py b/tests/test_flask_api_packagers.py index f13e77f..c89b875 100644 --- a/tests/test_flask_api_packagers.py +++ b/tests/test_flask_api_packagers.py @@ -136,13 +136,13 @@ class FlaskApiPackagersTest(Modeltests): """ Test the api_packager_list function. """ output = self.app.get('/api/packagers/') - self.assertEqual(output.status_code, 500) + self.assertEqual(output.status_code, 200) data = json.loads(output.data) self.assertEqual( data, { - "output": "notok", - "error": "Invalid request", + "output": "ok", + "packagers": [], } )