# HG changeset patch -- Bitbucket.org # Project pytest # URL http://bitbucket.org/hpk42/pytest/overview # User holger krekel <hol...@merlinux.eu> # Date 1288996651 -3600 # Node ID 4bdabf614c7658b422eba78d787f8bce41b81a2d # Parent 125c0136ba20fa7c651de76a4c679b1c59440487 reverse options ordering in Parser class instead of on PluginManager
--- a/pytest/plugin/config.py +++ b/pytest/plugin/config.py @@ -53,12 +53,12 @@ class Parser: def parse(self, args): self.optparser = optparser = MyOptionParser(self) - groups = self._groups + [self._anonymous] + groups = list(reversed(self._groups)) + [self._anonymous] for group in groups: if group.options: desc = group.description or group.name optgroup = py.std.optparse.OptionGroup(optparser, desc) - optgroup.add_options(group.options) + optgroup.add_options(reversed(group.options)) optparser.add_option_group(optgroup) return self.optparser.parse_args([str(x) for x in args]) @@ -304,7 +304,7 @@ class Config(object): self.pluginmanager.consider_env() self.pluginmanager.consider_preparse(args) self._setinitialconftest(args) - self.pluginmanager.do_addoption(self._parser) + self.hook.pytest_addoption(parser=self._parser) def _checkversion(self): minver = self.inicfg.get('minversion', None) --- a/pytest/_core.py +++ b/pytest/_core.py @@ -180,11 +180,6 @@ class PluginManager(object): for hint in self._hints: tw.line("hint: %s" % hint) - def do_addoption(self, parser): - mname = "pytest_addoption" - methods = reversed(self.listattr(mname)) - MultiCall(methods, {'parser': parser}).execute() - def do_configure(self, config): assert not hasattr(self, '_config') self._config = config _______________________________________________ py-svn mailing list py-svn@codespeak.net http://codespeak.net/mailman/listinfo/py-svn