[py-svn] pytest commit 4bdabf614c76: reverse options ordering in Parser class instead of on PluginManager

[commits-noreply at bitbucket.org]

# HG changeset patch -- Bitbucket.org
# Project pytest
# URL http://bitbucket.org/hpk42/pytest/overview
# User holger krekel <holger at 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