diff options
author | Vikraman Choudhury <vikraman.choudhury@gmail.com> | 2011-07-11 03:41:01 +0530 |
---|---|---|
committer | Vikraman Choudhury <vikraman.choudhury@gmail.com> | 2011-07-11 03:41:01 +0530 |
commit | 77af65cd6469b4ac15cdcc16202d51c06869a350 (patch) | |
tree | 7fb5429357380cfe333a110fe00f67087e6c236c /server | |
parent | fix tabs (diff) | |
download | gentoostats-77af65cd6469b4ac15cdcc16202d51c06869a350.tar.gz gentoostats-77af65cd6469b4ac15cdcc16202d51c06869a350.tar.bz2 gentoostats-77af65cd6469b4ac15cdcc16202d51c06869a350.zip |
fixes after pylint
Diffstat (limited to 'server')
-rw-r--r-- | server/host.py | 2 | ||||
-rw-r--r-- | server/kwd.py | 2 | ||||
-rw-r--r-- | server/package.py | 46 | ||||
-rw-r--r-- | server/tests/test_host.py | 16 | ||||
-rw-r--r-- | server/tests/test_index.py | 2 |
5 files changed, 34 insertions, 34 deletions
diff --git a/server/host.py b/server/host.py index 03bc2ba..6d1f6f3 100644 --- a/server/host.py +++ b/server/host.py @@ -21,7 +21,7 @@ class Host(object): env = db.select('ENV', vars={'uuid':uuid}, where="UUID=$uuid") e = env[0] - for var in ['PLATFORM','LASTSYNC','PROFILE','ARCH','CHOST','CFLAGS','CXXFLAGS','FFLAGS','LDFLAGS','MAKEOPTS','SYNC']: + for var in ['PLATFORM', 'LASTSYNC', 'PROFILE', 'ARCH', 'CHOST', 'CFLAGS', 'CXXFLAGS', 'FFLAGS', 'LDFLAGS', 'MAKEOPTS', 'SYNC']: host_data[var] = e[var] host_data['FEATURES'] = list() diff --git a/server/kwd.py b/server/kwd.py index c5f3403..6cef893 100644 --- a/server/kwd.py +++ b/server/kwd.py @@ -6,5 +6,5 @@ class Keyword(object): keyword_count = db.query('SELECT KEYWORD, COUNT(DISTINCT IPKEY) AS PACKAGES, COUNT(DISTINCT UUID) AS HOSTS FROM GLOBAL_KEYWORDS NATURAL JOIN KEYWORDS NATURAL JOIN INSTALLED_PACKAGES GROUP BY KEYWORD') keyword_data = dict() for t in keyword_count: - keyword_data[t['KEYWORD']] = {'HOSTS':t['HOSTS'],'PACKAGES':t['PACKAGES']} + keyword_data[t['KEYWORD']] = {'HOSTS':t['HOSTS'], 'PACKAGES':t['PACKAGES']} return render.keyword(keyword_data) diff --git a/server/package.py b/server/package.py index 58e3066..adc2d94 100644 --- a/server/package.py +++ b/server/package.py @@ -24,10 +24,10 @@ class Package(object): return config.internalerror() def __GET(self, top): - p_query = db.query('SELECT COUNT(DISTINCT UUID) AS HOST_COUNT,\ - COUNT(DISTINCT CAT) AS C_COUNT,\ - COUNT(DISTINCT CAT,PKG) AS CP_COUNT,\ - COUNT(DISTINCT CAT,PKG,VER) AS CPV_COUNT\ + p_query = db.query('SELECT COUNT(DISTINCT UUID) AS HOST_COUNT, \ + COUNT(DISTINCT CAT) AS C_COUNT, \ + COUNT(DISTINCT CAT, PKG) AS CP_COUNT, \ + COUNT(DISTINCT CAT, PKG, VER) AS CPV_COUNT\ FROM INSTALLED_PACKAGES RIGHT OUTER JOIN PACKAGES\ ON INSTALLED_PACKAGES.PKEY = PACKAGES.PKEY') p_tuple = p_query[0] @@ -41,9 +41,9 @@ class Package(object): return render.package(p_data) def __GET_C(self, top, cat): - p_query = db.query('SELECT COUNT(DISTINCT UUID) AS HOST_COUNT,\ - COUNT(DISTINCT CAT,PKG) AS CP_COUNT,\ - COUNT(DISTINCT CAT,PKG,VER) AS CPV_COUNT\ + p_query = db.query('SELECT COUNT(DISTINCT UUID) AS HOST_COUNT, \ + COUNT(DISTINCT CAT, PKG) AS CP_COUNT, \ + COUNT(DISTINCT CAT, PKG, VER) AS CPV_COUNT\ FROM INSTALLED_PACKAGES RIGHT OUTER JOIN PACKAGES\ ON INSTALLED_PACKAGES.PKEY = PACKAGES.PKEY\ WHERE CAT=$cat', vars={'cat':cat}) @@ -57,11 +57,11 @@ class Package(object): return render.package_c(cat, p_data) def __GET_CP(self, top, cat, pkg): - p_query = db.query('SELECT COUNT(DISTINCT UUID) AS HOST_COUNT,\ - COUNT(DISTINCT CAT,PKG,VER) AS CPV_COUNT\ + p_query = db.query('SELECT COUNT(DISTINCT UUID) AS HOST_COUNT, \ + COUNT(DISTINCT CAT, PKG, VER) AS CPV_COUNT\ FROM INSTALLED_PACKAGES RIGHT OUTER JOIN PACKAGES\ ON INSTALLED_PACKAGES.PKEY = PACKAGES.PKEY\ - WHERE CAT=$cat AND PKG=$pkg', vars={'cat':cat,'pkg':pkg}) + WHERE CAT=$cat AND PKG=$pkg', vars={'cat':cat, 'pkg':pkg}) p_tuple = p_query[0] p_data = { 'HOST_COUNT':p_tuple['HOST_COUNT'], @@ -74,7 +74,7 @@ class Package(object): p_query = db.query('SELECT COUNT(DISTINCT UUID) AS HOST_COUNT\ FROM INSTALLED_PACKAGES RIGHT OUTER JOIN PACKAGES\ ON INSTALLED_PACKAGES.PKEY = PACKAGES.PKEY\ - WHERE CAT=$cat AND PKG=$pkg AND VER=$ver', vars={'cat':cat,'pkg':pkg,'ver':ver}) + WHERE CAT=$cat AND PKG=$pkg AND VER=$ver', vars={'cat':cat, 'pkg':pkg, 'ver':ver}) p_tuple = p_query[0] p_data = { 'HOST_COUNT':p_tuple['HOST_COUNT'], @@ -84,12 +84,12 @@ class Package(object): def __top(self, count, *args): t_list = list() if len(args) == 0: - tc_query = db.query('SELECT CAT,COUNT(DISTINCT UUID) AS HOST_COUNT\ + tc_query = db.query('SELECT CAT, COUNT(DISTINCT UUID) AS HOST_COUNT\ FROM INSTALLED_PACKAGES RIGHT OUTER JOIN PACKAGES\ ON INSTALLED_PACKAGES.PKEY = PACKAGES.PKEY\ GROUP BY CAT\ - ORDER BY HOST_COUNT DESC,CAT') - for idx in range(0,count): + ORDER BY HOST_COUNT DESC, CAT') + for idx in range(0, count): try: tc_tuple = tc_query[idx] t_list.append({ @@ -100,14 +100,14 @@ class Package(object): break elif len(args) == 1: - tcp_query = db.query('SELECT CAT,PKG,COUNT(DISTINCT UUID) AS HOST_COUNT\ + tcp_query = db.query('SELECT CAT, PKG, COUNT(DISTINCT UUID) AS HOST_COUNT\ FROM INSTALLED_PACKAGES RIGHT OUTER JOIN PACKAGES\ ON INSTALLED_PACKAGES.PKEY = PACKAGES.PKEY\ WHERE CAT=$cat\ - GROUP BY CAT,PKG\ - ORDER BY HOST_COUNT DESC,CAT,PKG', + GROUP BY CAT, PKG\ + ORDER BY HOST_COUNT DESC, CAT, PKG', vars={'cat':args[0]}) - for idx in range(0,count): + for idx in range(0, count): try: tcp_tuple = tcp_query[idx] t_list.append({ @@ -119,14 +119,14 @@ class Package(object): break elif len(args) == 2: - tcpv_query = db.query('SELECT CAT,PKG,VER,COUNT(DISTINCT UUID) AS HOST_COUNT\ + tcpv_query = db.query('SELECT CAT, PKG, VER, COUNT(DISTINCT UUID) AS HOST_COUNT\ FROM INSTALLED_PACKAGES RIGHT OUTER JOIN PACKAGES\ ON INSTALLED_PACKAGES.PKEY = PACKAGES.PKEY\ WHERE CAT=$cat AND PKG=$pkg\ - GROUP BY CAT,PKG,VER\ - ORDER BY HOST_COUNT DESC,CAT,PKG,VER', - vars={'cat':args[0],'pkg':args[1]}) - for idx in range(0,count): + GROUP BY CAT, PKG, VER\ + ORDER BY HOST_COUNT DESC, CAT, PKG, VER', + vars={'cat':args[0], 'pkg':args[1]}) + for idx in range(0, count): try: tcpv_tuple = tcpv_query[idx] t_list.append({ diff --git a/server/tests/test_host.py b/server/tests/test_host.py index a5e723b..b781a09 100644 --- a/server/tests/test_host.py +++ b/server/tests/test_host.py @@ -38,7 +38,7 @@ class TestHost(unittest.TestCase): self.assertEqual(self.b.status, 500) # post with empty payload payload = { - 'AUTH':{'UUID':str_uuid,'PASSWD':'test'}, + 'AUTH':{'UUID':str_uuid, 'PASSWD':'test'}, 'PROTOCOL':1 } data = json.JSONEncoder().encode(payload) @@ -51,11 +51,11 @@ class TestHost(unittest.TestCase): uri = '/host/' + str_uuid # different uuid in payload payload = { - 'AUTH':{'UUID':str(uuid.uuid4()),'PASSWD':'test'}, + 'AUTH':{'UUID':str(uuid.uuid4()), 'PASSWD':'test'}, 'PROTOCOL':1 } data = json.JSONEncoder().encode(payload) - self.b.open(uri,data) + self.b.open(uri, data) self.assertEqual(self.b.path, uri) self.assertEqual(self.b.status, 200) self.assertTrue('Invalid uuid' in self.b.data) @@ -64,18 +64,18 @@ class TestHost(unittest.TestCase): str_uuid = str(uuid.uuid4()) uri = '/host/' + str_uuid payload = { - 'AUTH':{'UUID':str_uuid,'PASSWD':'test'}, + 'AUTH':{'UUID':str_uuid, 'PASSWD':'test'}, 'PROTOCOL':1 } - for var in ['PLATFORM','PROFILE','LASTSYNC']: + for var in ['PLATFORM', 'PROFILE', 'LASTSYNC']: payload[var] = 'Unknown' - for var in ['ARCH','CHOST','CFLAGS','CXXFLAGS','FFLAGS','LDFLAGS','MAKEOPTS','SYNC']: + for var in ['ARCH', 'CHOST', 'CFLAGS', 'CXXFLAGS', 'FFLAGS', 'LDFLAGS', 'MAKEOPTS', 'SYNC']: payload[var] = None - for var in ['ACCEPT_KEYWORDS','LANG','GENTOO_MIRRORS','FEATURES','USE']: + for var in ['ACCEPT_KEYWORDS', 'LANG', 'GENTOO_MIRRORS', 'FEATURES', 'USE']: payload[var] = [] payload['PACKAGES'] = {} data = json.JSONEncoder().encode(payload) - self.b.open(uri,data) + self.b.open(uri, data) self.assertEqual(self.b.path, uri) self.assertEqual(self.b.status, 200) self.assertTrue('POST for ' + str_uuid + ' successful' in self.b.data) diff --git a/server/tests/test_index.py b/server/tests/test_index.py index 2796c1d..f77a60d 100644 --- a/server/tests/test_index.py +++ b/server/tests/test_index.py @@ -22,5 +22,5 @@ class TestIndex(unittest.TestCase): if line.startswith('Number of hosts'): words = line.split() count = int(words[-1].strip('</br>')) - self.assertGreaterEqual(count,0) + self.assertGreaterEqual(count, 0) break |