# HG changeset patch # User Lars Kruse # Date 1503664528 -7200 # Node ID 48a00daba2f22c1526c46c7b30444136d989b46a # Parent 9988ec8c33e56a79beae88c98107306e2a060bda codingstyle: replace comparison for equality against None with "is" expression Both style and correctness. Reported by flake8. diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/lib/db_manage.py --- a/kallithea/lib/db_manage.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/lib/db_manage.py Fri Aug 25 14:35:28 2017 +0200 @@ -319,8 +319,8 @@ """ for k, v, t in [('auth_plugins', 'kallithea.lib.auth_modules.auth_internal', 'list'), - ('auth_internal_enabled', 'True', 'bool')]: - if skip_existing and Setting.get_by_name(k) != None: + ('auth_internal_enabled', 'True', 'bool')]: + if skip_existing and Setting.get_by_name(k) is not None: log.debug('Skipping option %s', k) continue setting = Setting(k, v, t) diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/lib/ipaddr.py --- a/kallithea/lib/ipaddr.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/lib/ipaddr.py Fri Aug 25 14:35:28 2017 +0200 @@ -769,18 +769,17 @@ s1, s2 = s2.subnet() else: # If we got here, there's a bug somewhere. - assert True == False, ('Error performing exclusion: ' - 's1: %s s2: %s other: %s' % - (str(s1), str(s2), str(other))) + assert False, ('Error performing exclusion: ' + 's1: %s s2: %s other: %s' % + (str(s1), str(s2), str(other))) if s1 == other: ret_addrs.append(s2) elif s2 == other: ret_addrs.append(s1) else: # If we got here, there's a bug somewhere. - assert True == False, ('Error performing exclusion: ' - 's1: %s s2: %s other: %s' % - (str(s1), str(s2), str(other))) + assert False, ('Error performing exclusion: s1: %s s2: %s other: %s' + % (str(s1), str(s2), str(other))) return sorted(ret_addrs, key=_BaseNet._get_networks_key) diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/tests/api/api_base.py --- a/kallithea/tests/api/api_base.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/tests/api/api_base.py Fri Aug 25 14:35:28 2017 +0200 @@ -142,7 +142,7 @@ option1 = Optional(None) assert '' % None == repr(option1) - assert option1() == None + assert option1() is None assert 1 == Optional.extract(Optional(1)) assert 'trololo' == Optional.extract('trololo') @@ -998,7 +998,7 @@ response = api_call(self, params) repo = RepoModel().get_by_repo_name(repo_name) - assert repo != None + assert repo is not None ret = { 'msg': 'Created new repository `%s`' % repo_name, 'success': True, @@ -1017,7 +1017,7 @@ response = api_call(self, params) print params repo = RepoModel().get_by_repo_name(repo_name) - assert repo != None + assert repo is not None ret = { 'msg': 'Created new repository `%s`' % repo_name, 'success': True, @@ -1083,7 +1083,7 @@ response = api_call(self, params) repo = RepoModel().get_by_repo_name(repo_name) - assert repo != None + assert repo is not None ret = { 'msg': 'Created new repository `%s`' % repo_name, 'success': True, @@ -1103,7 +1103,7 @@ response = api_call(self, params) repo = RepoModel().get_by_repo_name(repo_name) - assert repo != None + assert repo is not None ret = { 'msg': 'Created new repository `%s`' % repo_name, 'success': True, diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/tests/functional/test_admin_repos.py --- a/kallithea/tests/functional/test_admin_repos.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/tests/functional/test_admin_repos.py Fri Aug 25 14:35:28 2017 +0200 @@ -389,7 +389,7 @@ deleted_repo = Session().query(Repository) \ .filter(Repository.repo_name == repo_name).scalar() - assert deleted_repo == None + assert deleted_repo is None assert os.path.isdir(os.path.join(Ui.get_by_key('paths', '/').ui_value, repo_name)) == False @@ -439,7 +439,7 @@ deleted_repo = Session().query(Repository) \ .filter(Repository.repo_name == repo_name_unicode).scalar() - assert deleted_repo == None + assert deleted_repo is None assert os.path.isdir(os.path.join(Ui.get_by_key('paths', '/').ui_value, repo_name_unicode)) == False @@ -612,7 +612,7 @@ 'Error creating repository %s' % repo_name) # repo must not be in db repo = Repository.get_by_repo_name(repo_name) - assert repo == None + assert repo is None # repo must not be in filesystem ! assert not os.path.isdir(os.path.join(Ui.get_by_key('paths', '/').ui_value, repo_name)) diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/tests/functional/test_admin_user_groups.py --- a/kallithea/tests/functional/test_admin_user_groups.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/tests/functional/test_admin_user_groups.py Fri Aug 25 14:35:28 2017 +0200 @@ -60,7 +60,7 @@ gr = Session().query(UserGroup) \ .filter(UserGroup.users_group_name == users_group_name).scalar() - assert gr == None + assert gr is None def test_default_perms_enable_repository_read_on_group(self): self.log_user() @@ -123,7 +123,7 @@ gr = Session().query(UserGroup) \ .filter(UserGroup.users_group_name == users_group_name).scalar() - assert gr == None + assert gr is None p = Permission.get_by_key('hg.create.repository') perms = UserGroupToPerm.query() \ .filter(UserGroupToPerm.users_group_id == ugid).all() @@ -191,7 +191,7 @@ .filter(UserGroup.users_group_name == users_group_name).scalar() - assert gr == None + assert gr is None p = Permission.get_by_key('hg.fork.repository') perms = UserGroupToPerm.query() \ .filter(UserGroupToPerm.users_group_id == ugid).all() diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/tests/functional/test_login.py --- a/kallithea/tests/functional/test_login.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/tests/functional/test_login.py Fri Aug 25 14:35:28 2017 +0200 @@ -343,7 +343,7 @@ assert ret.email == email assert ret.name == name assert ret.lastname == lastname - assert ret.api_key != None + assert ret.api_key is not None assert ret.admin == False #========================================================================== diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/tests/functional/test_pullrequests.py --- a/kallithea/tests/functional/test_pullrequests.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/tests/functional/test_pullrequests.py Fri Aug 25 14:35:28 2017 +0200 @@ -157,7 +157,7 @@ # location is of the form: # http://localhost/vcs_test_hg/pull-request/54/_/title m = re.search('/pull-request/(\d+)/', response.location) - assert m != None + assert m is not None pull_request_id = m.group(1) # update it @@ -193,7 +193,7 @@ # location is of the form: # http://localhost/vcs_test_hg/pull-request/54/_/title m = re.search('/pull-request/(\d+)/', response.location) - assert m != None + assert m is not None pull_request_id = m.group(1) # edit it diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/tests/models/test_permissions.py --- a/kallithea/tests/models/test_permissions.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/tests/models/test_permissions.py Fri Aug 25 14:35:28 2017 +0200 @@ -684,8 +684,8 @@ old = Permission.get_by_key(perm) new = Permission.get_by_key(modify_to) - assert old != None - assert new != None + assert old is not None + assert new is not None # now modify permissions p = UserToPerm.query() \ diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/tests/models/test_repo_groups.py --- a/kallithea/tests/models/test_repo_groups.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/tests/models/test_repo_groups.py Fri Aug 25 14:35:28 2017 +0200 @@ -85,13 +85,13 @@ sg1 = fixture.create_repo_group(u'deleteme') self.__delete_group(sg1.group_id) - assert RepoGroup.get(sg1.group_id) == None + assert RepoGroup.get(sg1.group_id) is None assert not self.__check_path('deteteme') sg1 = fixture.create_repo_group(u'deleteme', parent_group_id=self.g1.group_id) self.__delete_group(sg1.group_id) - assert RepoGroup.get(sg1.group_id) == None + assert RepoGroup.get(sg1.group_id) is None assert not self.__check_path('test1', 'deteteme') def test_rename_single_group(self): @@ -99,7 +99,7 @@ new_sg1 = _update_repo_group(sg1.group_id, u'after') assert self.__check_path('after') - assert RepoGroup.get_by_group_name(u'initial') == None + assert RepoGroup.get_by_group_name(u'initial') is None def test_update_group_parent(self): @@ -107,7 +107,7 @@ new_sg1 = _update_repo_group(sg1.group_id, u'after', parent_id=self.g1.group_id) assert self.__check_path('test1', 'after') - assert RepoGroup.get_by_group_name(u'test1/initial') == None + assert RepoGroup.get_by_group_name(u'test1/initial') is None new_sg1 = _update_repo_group(sg1.group_id, u'after', parent_id=self.g3.group_id) assert self.__check_path('test3', 'after') diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/tests/models/test_repos.py --- a/kallithea/tests/models/test_repos.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/tests/models/test_repos.py Fri Aug 25 14:35:28 2017 +0200 @@ -22,7 +22,7 @@ RepoModel().delete(repo=repo) Session().commit() - assert None == Repository.get_by_repo_name(repo_name=u'test-repo-1') + assert Repository.get_by_repo_name(repo_name=u'test-repo-1') is None def test_remove_repo_repo_raises_exc_when_attached_forks(self): repo = fixture.create_repo(name=u'test-repo-1') @@ -52,9 +52,9 @@ RepoModel().delete(repo=repo, forks='delete') Session().commit() - assert None == Repository.get_by_repo_name(repo_name=u'test-repo-1') - assert None == Repository.get_by_repo_name(repo_name=u'test-repo-fork-1') - assert None == Repository.get_by_repo_name(repo_name=u'test-repo-fork-fork-1') + assert Repository.get_by_repo_name(repo_name=u'test-repo-1') is None + assert Repository.get_by_repo_name(repo_name=u'test-repo-fork-1') is None + assert Repository.get_by_repo_name(repo_name=u'test-repo-fork-fork-1') is None def test_remove_repo_detach_forks(self): repo = fixture.create_repo(name=u'test-repo-1') @@ -71,9 +71,9 @@ Session().commit() try: - assert None == Repository.get_by_repo_name(repo_name=u'test-repo-1') - assert None != Repository.get_by_repo_name(repo_name=u'test-repo-fork-1') - assert None != Repository.get_by_repo_name(repo_name=u'test-repo-fork-fork-1') + assert Repository.get_by_repo_name(repo_name=u'test-repo-1') is None + assert Repository.get_by_repo_name(repo_name=u'test-repo-fork-1') is not None + assert Repository.get_by_repo_name(repo_name=u'test-repo-fork-fork-1') is not None finally: RepoModel().delete(repo=u'test-repo-fork-fork-1') RepoModel().delete(repo=u'test-repo-fork-1') diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/tests/models/test_users.py --- a/kallithea/tests/models/test_users.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/tests/models/test_users.py Fri Aug 25 14:35:28 2017 +0200 @@ -90,13 +90,12 @@ assert usr.user_id == u.user_id assert usr.username == u.username u = User.get_by_email(email='main_email3@example.com') - assert None == u + assert u is None u = User.get_by_email(email='main_e%ail@example.com') - assert None == u + assert u is None u = User.get_by_email(email='main_emai_@example.com') - assert None == u - + assert u is None UserModel().delete(usr.user_id) Session().commit() diff -r 9988ec8c33e5 -r 48a00daba2f2 kallithea/tests/other/test_validators.py --- a/kallithea/tests/other/test_validators.py Fri Aug 25 14:34:58 2017 +0200 +++ b/kallithea/tests/other/test_validators.py Fri Aug 25 14:35:28 2017 +0200 @@ -103,7 +103,7 @@ def test_ValidPassword(self): validator = v.ValidPassword() assert 'lol' == validator.to_python('lol') - assert None == validator.to_python(None) + assert validator.to_python(None) is None with pytest.raises(formencode.Invalid): validator.to_python('ąćżź')