| kparal |
Add release_status() that returns release status of specified repo.
If repo is not top-parent repo, return release_status of its top parent.
py.test testing/
| Lint Skipped |
| Unit Tests Skipped |
| Path | Packages | |||
|---|---|---|---|---|
| M | conf/yumrepoinfo.conf.example (2 lines) | |||
| M | libtaskotron/yumrepoinfo.py (12 lines) | |||
| M | testing/test_yumrepoinfo.py (8 lines) |
| Commit | Tree | Parents | Author | Summary | Date |
|---|---|---|---|---|---|
| 3b14d22af424 | 5ab8419cbe9b | f4b534c12216 | Garret Raziel | add release_status() for repos in yumrepoinfo (Show More…) | Jul 31 2014, 9:41 AM |
| Show All 16 Lines | |||||
| 17 | arches = i386, x86_64 | 17 | arches = i386, x86_64 | ||
| 18 | 18 | | |||
| 19 | # a direct parent of this repo, essential for operations (dependencies) | 19 | # a direct parent of this repo, essential for operations (dependencies) | ||
| 20 | parent = | 20 | parent = | ||
| 21 | 21 | | |||
| 22 | # koji tag defaults to section name | 22 | # koji tag defaults to section name | ||
| 23 | tag = %(__name__)s | 23 | tag = %(__name__)s | ||
| 24 | 24 | | |||
| 25 | # release_status can be one of: EOL, stable, branched or rawhide | 25 | # release_status can be one of: obsolete, stable, branched or rawhide | ||
| 26 | # for non-top-parent repos this is an empty string | 26 | # for non-top-parent repos this is an empty string | ||
| 27 | release_status = | 27 | release_status = | ||
| 28 | 28 | | |||
| 29 | # Rawhide | 29 | # Rawhide | ||
| 30 | [rawhide] | 30 | [rawhide] | ||
| 31 | url = %(rawhideurl)s | 31 | url = %(rawhideurl)s | ||
| 32 | path = development/rawhide | 32 | path = development/rawhide | ||
| 33 | tag = f22 | 33 | tag = f22 | ||
| ▲ Show 20 Lines • Show All 50 Lines • Show Last 20 Lines | |||||
| Show First 20 Lines • Show All 192 Lines • ▼ Show 20 Line(s) | 185 | while True: | |||
|---|---|---|---|---|---|
| 193 | if parent in parents: | 193 | if parent in parents: | ||
| 194 | raise exc.TaskotronConfigError('Infinite parent loop detected ' | 194 | raise exc.TaskotronConfigError('Infinite parent loop detected ' | ||
| 195 | 'in yumrepoinfo: %s' % parents) | 195 | 'in yumrepoinfo: %s' % parents) | ||
| 196 | 196 | | |||
| 197 | parents.append(parent) | 197 | parents.append(parent) | ||
| 198 | repo = parent | 198 | repo = parent | ||
| 199 | 199 | | |||
| 200 | 200 | | |||
| 201 | def release_status(self, reponame): | ||||
| 202 | '''Return release status of specified repo. For non-top-parent repos, | ||||
| 203 | return release_status of top parent repo. | ||||
| 204 | | ||||
| 205 | :param str reponame: repository name | ||||
| 206 | :return: release status of specified repo, lowercased. One of: | ||||
| 207 | `rawhide`, `branched`, `stable`, `obsolete`. | ||||
| 208 | :rtype: str | ||||
| 209 | ''' | ||||
| 210 | return self.get(self.top_parent(reponame), 'release_status').lower() | ||||
| 211 | | ||||
| 212 | | ||||
| 201 | def _read(self): | 213 | def _read(self): | ||
| 202 | '''Read first available config file from the list of provided config | 214 | '''Read first available config file from the list of provided config | ||
| 203 | files.''' | 215 | files.''' | ||
| 204 | log.debug('Looking for yumrepoinfo config files in: %s', self.filelist) | 216 | log.debug('Looking for yumrepoinfo config files in: %s', self.filelist) | ||
| 205 | 217 | | |||
| 206 | for cfg in self.filelist: | 218 | for cfg in self.filelist: | ||
| 207 | if self.parser.read(cfg): | 219 | if self.parser.read(cfg): | ||
| 208 | log.debug('Successfully loaded yumrepoinfo config file: %s', | 220 | log.debug('Successfully loaded yumrepoinfo config file: %s', | ||
| Show All 20 Lines | |||||
| Show First 20 Lines • Show All 48 Lines • ▼ Show 20 Line(s) | |||||
| 49 | [f20-updates-testing] | 49 | [f20-updates-testing] | ||
| 50 | url = %(updatesurl)s | 50 | url = %(updatesurl)s | ||
| 51 | path = testing/20 | 51 | path = testing/20 | ||
| 52 | parent = f20-updates | 52 | parent = f20-updates | ||
| 53 | 53 | | |||
| 54 | [f15] | 54 | [f15] | ||
| 55 | url = not_really_an_url | 55 | url = not_really_an_url | ||
| 56 | path = 15 | 56 | path = 15 | ||
| 57 | release_status = EOL | 57 | release_status = obsolete | ||
| 58 | ''' | 58 | ''' | ||
| 59 | 59 | | |||
| 60 | 60 | | |||
| 61 | class TestYumRepoInfo(object): | 61 | class TestYumRepoInfo(object): | ||
| 62 | 62 | | |||
| 63 | @classmethod | 63 | @classmethod | ||
| 64 | def setup_class(cls): | 64 | def setup_class(cls): | ||
| 65 | '''One-time class initialization''' | 65 | '''One-time class initialization''' | ||
| ▲ Show 20 Lines • Show All 102 Lines • ▼ Show 20 Line(s) | |||||
| 168 | parent = repo1 | 168 | parent = repo1 | ||
| 169 | ''' | 169 | ''' | ||
| 170 | repoinfo = yumrepoinfo.YumRepoInfo(filelist=[]) | 170 | repoinfo = yumrepoinfo.YumRepoInfo(filelist=[]) | ||
| 171 | repoinfo.parser.readfp(StringIO.StringIO(cfg)) | 171 | repoinfo.parser.readfp(StringIO.StringIO(cfg)) | ||
| 172 | 172 | | |||
| 173 | with pytest.raises(exc.TaskotronConfigError): | 173 | with pytest.raises(exc.TaskotronConfigError): | ||
| 174 | repoinfo.top_parent('repo1') | 174 | repoinfo.top_parent('repo1') | ||
| 175 | 175 | | |||
| 176 | def test_repo_release_status(self): | ||||
| 177 | assert self.repoinfo.release_status('f21') == 'branched' | ||||
| 178 | assert self.repoinfo.release_status('f20') == 'stable' | ||||
| 179 | assert self.repoinfo.release_status('f20-updates-testing') == 'stable' | ||||
| 180 | assert self.repoinfo.release_status('f15') == 'obsolete' | ||||
| 181 | | ||||
| 176 | def test_get_yumrepoinfo(self, clean_singleton): | 182 | def test_get_yumrepoinfo(self, clean_singleton): | ||
| 177 | '''get_yumrepoinfo must work as a singleton''' | 183 | '''get_yumrepoinfo must work as a singleton''' | ||
| 178 | 184 | | |||
| 179 | repoinfo = yumrepoinfo.get_yumrepoinfo() | 185 | repoinfo = yumrepoinfo.get_yumrepoinfo() | ||
| 180 | # two calls with same arch must return the same instance | 186 | # two calls with same arch must return the same instance | ||
| 181 | assert repoinfo is yumrepoinfo.get_yumrepoinfo() | 187 | assert repoinfo is yumrepoinfo.get_yumrepoinfo() | ||
| 182 | # there should be a record in the singleton container | 188 | # there should be a record in the singleton container | ||
| 183 | assert len(yumrepoinfo._yumrepoinfo.keys()) > 0 | 189 | assert len(yumrepoinfo._yumrepoinfo.keys()) > 0 | ||
| 184 | 190 | | |||
| 185 | # the same must work for basearch conversion | 191 | # the same must work for basearch conversion | ||
| 186 | repoinfo_i686 = yumrepoinfo.get_yumrepoinfo(arch='i686') | 192 | repoinfo_i686 = yumrepoinfo.get_yumrepoinfo(arch='i686') | ||
| 187 | repoinfo_i386 = yumrepoinfo.get_yumrepoinfo(arch='i386') | 193 | repoinfo_i386 = yumrepoinfo.get_yumrepoinfo(arch='i386') | ||
| 188 | assert repoinfo_i686 is repoinfo_i386 | 194 | assert repoinfo_i686 is repoinfo_i386 | ||
| 189 | assert 'i386' in yumrepoinfo._yumrepoinfo | 195 | assert 'i386' in yumrepoinfo._yumrepoinfo | ||
| 190 | assert 'i686' not in yumrepoinfo._yumrepoinfo | 196 | assert 'i686' not in yumrepoinfo._yumrepoinfo | ||