From 001524aa8f7e9e9def363b535baf13bfd76a9f86 Mon Sep 17 00:00:00 2001 From: Scott Bahling Date: Mon, 3 Feb 2020 12:48:09 +0100 Subject: Use repository attribute instead of "repo" to be consistent with the api --- obsapi/repoflags.py | 4 ++-- test/test_repo_flags.py | 10 +++++----- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/obsapi/repoflags.py b/obsapi/repoflags.py index 21d57d1..ffea5e6 100644 --- a/obsapi/repoflags.py +++ b/obsapi/repoflags.py @@ -3,7 +3,7 @@ from lxml import etree from collections import namedtuple -RepoFlag = namedtuple('RepoFlag', 'setting repo arch') +RepoFlag = namedtuple('RepoFlag', 'setting repository arch') valid_flag_types = ('build', 'publish', 'useforbuild', 'debuginfo') if sys.version_info >= (3, 0): @@ -87,7 +87,7 @@ class RepoFlags(): setting = element.tag repo = element.get('repository', None) arch = element.get('arch', None) - yield(RepoFlag(setting=setting, repo=repo, arch=arch)) + yield(RepoFlag(setting=setting, repository=repo, arch=arch)) def set(self, setting, repo=None, arch=None): if setting not in ['enable', 'disable']: diff --git a/test/test_repo_flags.py b/test/test_repo_flags.py index 08a39d6..290a106 100644 --- a/test/test_repo_flags.py +++ b/test/test_repo_flags.py @@ -66,10 +66,10 @@ def test_repo_flags_xml_build_flags(): assert repo_flags.flag_type == 'build' for flag in repo_flags.flags: if flag.setting == 'enable': - assert flag.repo == 'SLE_12_SP3' + assert flag.repository == 'SLE_12_SP3' assert flag.arch == 'x86_64' if flag.setting == 'disable': - assert flag.repo is None + assert flag.repository is None assert flag.arch is None @@ -78,7 +78,7 @@ def test_repo_flags_xml_debuginfo_flags(): assert repo_flags.flag_type == 'debuginfo' for flag in repo_flags.flags: if flag.setting == 'enable': - assert flag.repo is None + assert flag.repository is None assert flag.arch is None @@ -101,7 +101,7 @@ def test_repo_flags_enable(flag_type, repo, arch): repoflags.enable(repo=repo, arch=arch) for flag in repoflags.flags: assert flag.setting == 'enable' - assert flag.repo == repo + assert flag.repository == repo assert flag.arch == arch @@ -111,7 +111,7 @@ def test_repo_flags_disable(flag_type, repo, arch): repoflags.disable(repo=repo, arch=arch) for flag in repoflags.flags: assert flag.setting == 'disable' - assert flag.repo == repo + assert flag.repository == repo assert flag.arch == arch -- cgit v1.2.3