packagedb-cli-commits July 2014

packagedb-cli-commits@lists.stg.fedorahosted.org

1 participants
20 discussions

[packagedb-cli] Adjust the setup.py to fit the new structure of the project
by Pierre-YvesChibon 30 Jul '14

30 Jul '14

commit a1177dc87b7ce45511cf54c3a476bc64b282d84b Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Jul 30 11:53:12 2014 +0200 Adjust the setup.py to fit the new structure of the project setup.py | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) --- diff --git a/setup.py b/setup.py index e577b7e..6820719 100644 --- a/setup.py +++ b/setup.py @@ -14,10 +14,10 @@ setup( url='https://fedorahosted.org/packagedb-cli/', author='Pierre-Yves Chibon', author_email='pingou(a)pingoured.fr', - py_modules=['pkgdb2client', 'pkgdb2_cli', 'pkgdb2version'], + packages=['pkgdb2client'], entry_points={ 'console_scripts': [ - "pkgdb-cli=pkgdb2_cli:main", + "pkgdb-cli=pkgdb2client.cli:main", ] }, install_requires=[

1 0

[packagedb-cli] Rework the structure so that pkgdb2client is a traditional python module
by Pierre-YvesChibon 30 Jul '14

30 Jul '14

commit 7a64d7ee2be840f4e7488fc0080f3fc9a56d3343 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Jul 30 11:49:06 2014 +0200 Rework the structure so that pkgdb2client is a traditional python module pkgdb2client.py => pkgdb2client/__init__.py | 0 pkgdb2_cli.py => pkgdb2client/cli.py | 0 2 files changed, 0 insertions(+), 0 deletions(-) --- diff --git a/pkgdb2client.py b/pkgdb2client/__init__.py similarity index 100% rename from pkgdb2client.py rename to pkgdb2client/__init__.py diff --git a/pkgdb2_cli.py b/pkgdb2client/cli.py similarity index 100% rename from pkgdb2_cli.py rename to pkgdb2client/cli.py

1 0

[packagedb-cli] Release 2.5
by Pierre-YvesChibon 30 Jul '14

30 Jul '14

commit 8fee3d0d05e6d96962682cc5b9af5489c8998436 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Jul 30 10:59:03 2014 +0200 Release 2.5 README | 4 ++-- setup.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) --- diff --git a/README b/README index 5225623..73fbb8e 100644 --- a/README +++ b/README @@ -4,8 +4,8 @@ Fedora PackageDB-cli :Author: Pierre-Yves Chibon :Contact: pingou(a)fedoraproject.org -:Date: Thu Mar 06, 2014 -:Version: 2.4 +:Date: Wed July 30, 2014 +:Version: 2.5 The `PackageDB-cli` is a commande line interface to the PackageDB of Fedora. diff --git a/setup.py b/setup.py index 56b1537..e577b7e 100644 --- a/setup.py +++ b/setup.py @@ -8,7 +8,7 @@ from setuptools import setup setup( name='packagedb-cli', description='A command line tool to access the Fedora Package Database.', - version='2.4', + version='2.5', license='GPLv2+', download_url='https://fedorahosted.org/releases/p/a/packagedb-cli/', url='https://fedorahosted.org/packagedb-cli/',

1 0

[packagedb-cli] Adjust the test function names to run them in the order I want
by Pierre-YvesChibon 30 Jul '14

30 Jul '14

commit eea879adae472682fa8a594099e8e68ea2ebc94c Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Jul 30 10:49:56 2014 +0200 Adjust the test function names to run them in the order I want test.py | 18 +++++++++--------- 1 files changed, 9 insertions(+), 9 deletions(-) --- diff --git a/test.py b/test.py index 0e8123c..6067450 100644 --- a/test.py +++ b/test.py @@ -295,7 +295,7 @@ class TestPkgdDBAuth(unittest.TestCase): self.pkgdb.login(USERNAME, PASSWORD) @auth_only - def test_1_create_collection(self): + def test_01_create_collection(self): ''' Test the create_collection function. ''' out = self.pkgdb.create_collection( clt_name='Test', @@ -338,7 +338,7 @@ class TestPkgdDBAuth(unittest.TestCase): ['Collection "%s" created' % COL_NAME]) @auth_only - def test_2_create_package(self): + def test_02_create_package(self): ''' Test the create_package function. ''' out = self.pkgdb.create_package( @@ -363,7 +363,7 @@ class TestPkgdDBAuth(unittest.TestCase): ['Package created']) @auth_only - def test_3_orphan_packages(self): + def test_03_orphan_packages(self): ''' Test the orphan_packages function. ''' out = self.pkgdb.orphan_packages('guake', ['master', 'el6']) @@ -381,7 +381,7 @@ class TestPkgdDBAuth(unittest.TestCase): 'pingou to: orphan on branch: el6']) @auth_only - def test_4_unorphan_packages(self): + def test_04_unorphan_packages(self): ''' Test the unorphan_packages function. ''' out = self.pkgdb.unorphan_packages( @@ -398,7 +398,7 @@ class TestPkgdDBAuth(unittest.TestCase): 'Package guake has been unorphaned on el6 by pingou']) @auth_only - def test_5_retire_packages(self): + def test_05_retire_packages(self): ''' Test the retire_packages function. ''' out = self.pkgdb.retire_packages('guake', 'master') @@ -414,7 +414,7 @@ class TestPkgdDBAuth(unittest.TestCase): 'Retired on branch: master']) @auth_only - def test_6_unretire_packages(self): + def test_06_unretire_packages(self): ''' Test the unretire_packages function. ''' out = self.pkgdb.unretire_packages('guake', 'master') @@ -430,7 +430,7 @@ class TestPkgdDBAuth(unittest.TestCase): 'Approved on branch: master']) @auth_only - def test_7_update_acl(self): + def test_07_update_acl(self): ''' Test the update_acl function. ''' out = self.pkgdb.update_acl( @@ -448,7 +448,7 @@ class TestPkgdDBAuth(unittest.TestCase): 'Nothing to update on branch: el6 for acl: commit']) @auth_only - def test_8_update_collection_status(self): + def test_08_update_collection_status(self): ''' Test the update_collection_status function. ''' out = self.pkgdb.update_collection_status(COL_NAME, 'EOL') @@ -463,7 +463,7 @@ class TestPkgdDBAuth(unittest.TestCase): ['Collection updated from "Active" to "EOL"']) @auth_only - def test_9_update_package_poc(self): + def test_09_update_package_poc(self): ''' Test the update_package_poc function. ''' out = self.pkgdb.update_package_poc(

1 0

[packagedb-cli] Add unit-tests for the update_critpath method
by Pierre-YvesChibon 30 Jul '14

30 Jul '14

commit 318d68d8fa40dbd1b4a6732a9f854ecd534282a8 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Jul 30 10:29:48 2014 +0200 Add unit-tests for the update_critpath method test.py | 98 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 98 insertions(+), 0 deletions(-) --- diff --git a/test.py b/test.py index 0214831..0e8123c 100644 --- a/test.py +++ b/test.py @@ -496,6 +496,104 @@ class TestPkgdDBAuth(unittest.TestCase): 'user: pingou changed point of contact of package: guake from: ' 'ralph to: pingou on branch: el6']) + @auth_only + def test_10_update_critpath(self): + ''' Test the update_critpath function. ''' + + # Check before changing the critpath + out = self.pkgdb.get_package('guake') + self.assertEqual( + sorted(out.keys()), + ['output', 'packages']) + self.assertEqual(out['output'], 'ok') + self.assertEqual(len(out['packages']), 5) + critpaths = [el['critpath'] for el in out['packages']] + branches = [el['collection']['branchname'] for el in out['packages']] + self.assertEqual( + critpaths, + [False, False, False, False, False] + ) + self.assertEqual( + branches, + ['master', 'el6', 'f19', 'f20', 'f21'] + ) + + out = self.pkgdb.update_critpath( + 'guake', ['master', 'el6'], True) + + self.assertEqual( + sorted(out.keys()), + ['messages', 'output']) + + self.assertEqual(out['output'], 'ok') + self.assertEqual( + out['messages'], + ['guake: critpath updated on master to True', + 'guake: critpath updated on el6 to True'] + ) + + # Check after changing the critpath + out = self.pkgdb.get_package('guake') + self.assertEqual( + sorted(out.keys()), + ['output', 'packages']) + self.assertEqual(out['output'], 'ok') + self.assertEqual(len(out['packages']), 5) + critpaths = [el['critpath'] for el in out['packages']] + branches = [el['collection']['branchname'] for el in out['packages']] + self.assertEqual( + critpaths, + [True, True, False, False, False] + ) + self.assertEqual( + branches, + ['master', 'el6', 'f19', 'f20', 'f21'] + ) + + out = self.pkgdb.get_package('guake') + self.assertEqual( + sorted(out.keys()), + ['output', 'packages']) + self.assertEqual(out['output'], 'ok') + self.assertEqual(len(out['packages']), 5) + self.assertEqual( + out['packages'][0]['collection']['branchname'], 'master') + self.assertEqual( + out['packages'][0]['package']['name'], 'guake') + self.assertEqual( + out['packages'][0]['point_of_contact'], 'pingou') + + out = self.pkgdb.update_critpath( + 'guake', ['master', 'el6'], False) + + self.assertEqual( + sorted(out.keys()), + ['messages', 'output']) + + self.assertEqual(out['output'], 'ok') + self.assertEqual( + out['messages'], + ['guake: critpath updated on master to False', + 'guake: critpath updated on el6 to False'] + ) + + # Check after reste critpath to False + out = self.pkgdb.get_package('guake') + self.assertEqual( + sorted(out.keys()), + ['output', 'packages']) + self.assertEqual(out['output'], 'ok') + self.assertEqual(len(out['packages']), 5) + critpaths = [el['critpath'] for el in out['packages']] + branches = [el['collection']['branchname'] for el in out['packages']] + self.assertEqual( + critpaths, + [False, False, False, False, False] + ) + self.assertEqual( + branches, + ['master', 'el6', 'f19', 'f20', 'f21'] + ) def suite(): suite = unittest.TestSuite()

1 0

[packagedb-cli] Adjust the unit-tests for the changes to F21
by Pierre-YvesChibon 30 Jul '14

30 Jul '14

commit 12eb52dd5cebd3a7e3d47f47bfe7c5a04818a618 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Jul 30 10:29:33 2014 +0200 Adjust the unit-tests for the changes to F21 test.py | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) --- diff --git a/test.py b/test.py index e2d3f75..0214831 100644 --- a/test.py +++ b/test.py @@ -82,7 +82,7 @@ class TestPkgdDB(unittest.TestCase): sorted(out.keys()), ['output', 'packages']) self.assertEqual(out['output'], 'ok') - self.assertEqual(len(out['packages']), 4) + self.assertEqual(len(out['packages']), 5) self.assertEqual( out['packages'][0]['collection']['branchname'], 'master') self.assertEqual( @@ -122,7 +122,7 @@ class TestPkgdDB(unittest.TestCase): sorted(out.keys()), ['acls', 'output', 'page', 'page_total']) self.assertEqual(len(out['acls']), 250) - self.assertEqual(out['page_total'], 5) + self.assertEqual(out['page_total'], 6) out = self.pkgdb.get_packager_acls('pingou', acls=['approveacls']) self.assertEqual( @@ -136,7 +136,7 @@ class TestPkgdDB(unittest.TestCase): sorted(out.keys()), ['acls', 'output', 'page', 'page_total']) self.assertTrue(len(out['acls']) >= 250) - self.assertEqual(out['page_total'], 5) + self.assertEqual(out['page_total'], 6) out = self.pkgdb.get_packager_acls('pingou', count=True) self.assertEqual( @@ -164,7 +164,7 @@ class TestPkgdDB(unittest.TestCase): out = self.pkgdb.get_packager_stats('pingou') self.assertEqual( sorted(out.keys()), - ['el5', 'el6', 'epel7', 'f19', 'f20', 'master', 'output']) + ['el5', 'el6', 'epel7', 'f19', 'f20', 'f21', 'master', 'output']) self.assertEqual( sorted(out['master'].keys()), ['co-maintainer', 'point of contact'])

1 0

[packagedb-cli] Created tag v2.5
by Pierre-YvesChibon 30 Jul '14

30 Jul '14

The signed tag 'v2.5' was created. Tagger: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Jul 30 10:59:27 2014 +0200 Release 2.5 Changes since the last tag 'v2.4': Chaoyi Zha (1): update client.py for critpath updater Pavol Babincak (1): Configure logging in cli instead of library Pierre-Yves Chibon (9): Fix logging exception after running self.login in call_api Small pep8 fix Adjust the update_critpath method as it accepts one or more branches Move the update_critpath at its right place in the file Small documentation update on the update_critpath method Adjust the unit-tests for the changes to F21 Add unit-tests for the update_critpath method Adjust the test function names to run them in the order I want Release 2.5

1 0

[packagedb-cli] Small documentation update on the update_critpath method
by Pierre-YvesChibon 30 Jul '14

30 Jul '14

commit 03883a1420c8d093f622e1a72aa5896b37d2d15e Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Jul 30 09:53:56 2014 +0200 Small documentation update on the update_critpath method pkgdb2client.py | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) --- diff --git a/pkgdb2client.py b/pkgdb2client.py index e65ac95..ad6307b 100644 --- a/pkgdb2client.py +++ b/pkgdb2client.py @@ -951,7 +951,7 @@ class PkgDB(object): data=args) def update_critpath(self, pkgname, branches, critpath=False): - ''' Set / Unset critpath status + ''' Set / Remove critpath status of a package :arg pkgname: The name of the package :type pkgname: str

1 0

[packagedb-cli] Move the update_critpath at its right place in the file
by Pierre-YvesChibon 30 Jul '14

30 Jul '14

commit 187de64c841d1308ac649ca24ddcca400285997c Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Jul 30 09:53:15 2014 +0200 Move the update_critpath at its right place in the file pkgdb2client.py | 40 ++++++++++++++++++++-------------------- 1 files changed, 20 insertions(+), 20 deletions(-) --- diff --git a/pkgdb2client.py b/pkgdb2client.py index dddcdc6..e65ac95 100644 --- a/pkgdb2client.py +++ b/pkgdb2client.py @@ -401,26 +401,6 @@ class PkgDB(object): return self.handle_api_call('/collection/new/', data=args) - def update_critpath(self, pkgname, branches, critpath=False): - ''' Set / Unset critpath status - - :arg pkgname: The name of the package - :type pkgname: str - :arg branches: One or more branch names for the collections in - which to update the critpath status of the package - :type branches: str or list - :arg critpath: A boolean corresponding to the critpath status to - set - :type critpath: bool - - ''' - args = { - 'pkgnames': pkgname, - 'branches': branches, - 'critpath': critpath - } - return self.handle_api_call('/package/critpath/', data=args) - def create_package( self, pkgname, summary, description, review_url, status, shouldopen, branches, poc, upstream_url, critpath=False): @@ -970,6 +950,26 @@ class PkgDB(object): return self.handle_api_call('/collection/{0}/status/'.format(branch), data=args) + def update_critpath(self, pkgname, branches, critpath=False): + ''' Set / Unset critpath status + + :arg pkgname: The name of the package + :type pkgname: str + :arg branches: One or more branch names for the collections in + which to update the critpath status of the package + :type branches: str or list + :arg critpath: A boolean corresponding to the critpath status to + set + :type critpath: bool + + ''' + args = { + 'pkgnames': pkgname, + 'branches': branches, + 'critpath': critpath + } + return self.handle_api_call('/package/critpath/', data=args) + def update_package_poc(self, pkgnames, branches, poc): ''' Update the point of contact of the specified packages on the specified branches.

1 0

[packagedb-cli] Adjust the update_critpath method as it accepts one or more branches
by Pierre-YvesChibon 30 Jul '14

30 Jul '14

commit 8f31b46a9e12059540f24721af0057c7261583e6 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Jul 30 09:52:26 2014 +0200 Adjust the update_critpath method as it accepts one or more branches pkgdb2client.py | 12 +++++++----- 1 files changed, 7 insertions(+), 5 deletions(-) --- diff --git a/pkgdb2client.py b/pkgdb2client.py index a791b6e..dddcdc6 100644 --- a/pkgdb2client.py +++ b/pkgdb2client.py @@ -401,20 +401,22 @@ class PkgDB(object): return self.handle_api_call('/collection/new/', data=args) - def update_critpath(self, pkgname, branch=None, critpath=False): + def update_critpath(self, pkgname, branches, critpath=False): ''' Set / Unset critpath status :arg pkgname: The name of the package :type pkgname: str - :arg branch: The branch of the package - :type branch: str - :arg critpath: Whether to set critpath status or not + :arg branches: One or more branch names for the collections in + which to update the critpath status of the package + :type branches: str or list + :arg critpath: A boolean corresponding to the critpath status to + set :type critpath: bool ''' args = { 'pkgnames': pkgname, - 'branches': branch, + 'branches': branches, 'critpath': critpath } return self.handle_api_call('/package/critpath/', data=args)

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

packagedb-cli-commits July 2014