Some more work on creating a "clean" rcParams environment for running tests.

mdboom · mdboom · commit ffd40741be05 · 2009-09-09T20:01:03.000Z
svn path=/trunk/matplotlib/; revision=7729
diff --git a/lib/matplotlib/__init__.py b/lib/matplotlib/__init__.py
@@ -894,16 +894,20 @@ def test(verbosity=0):
     from testing.noseclasses import KnownFailure
     from nose.plugins.manager import PluginManager
 
+    # Store the current backend and all rcParams for restoration later
     backend = rcParams['backend']
+    original_params = rcParams.copy()
 
     use('Agg') # use Agg backend for these tests
 
-    # store the old values before overriding
-    overrides = 'font.family', 'text.hinting'
-    stored = dict([(k, rcParams[k]) for k in overrides])
-
+    # These settings *must* be hardcoded for running the comparison
+    # tests and are not necessarily the default values as specified in
+    # rcsetup.py
+    rcdefaults() # Start with all defaults
     rcParams['font.family'] = 'Bitstream Vera Sans'
     rcParams['text.hinting'] = False
+
+    # store the old values before overriding
     plugins = []
     plugins.append( KnownFailure() )
     plugins.extend( [plugin() for plugin in nose.plugins.builtin.plugins] )
@@ -914,11 +918,10 @@ def test(verbosity=0):
     success = nose.run( defaultTest=default_test_modules,
                         config=config,
                         )
-    # restore the old rc values
-    rcParams.update(stored)
 
-    # restore the old backend
+    # restore the old backend and rcParams
     use(backend)
+    rcParams.update(original_params)
     return success
 
 test.__test__ = False # nose: this function is not a test
diff --git a/lib/matplotlib/font_manager.py b/lib/matplotlib/font_manager.py
@@ -642,8 +642,7 @@ class FontProperties(object):
     The preferred usage of font sizes is to use the relative values,
     e.g.  'large', instead of absolute font sizes, e.g. 12.  This
     approach allows all text sizes to be made larger or smaller based
-    on the font manager's default font size, i.e. by using the
-    :meth:`FontManager.set_default_size` method.
+    on the font manager's default font size.
 
     This class will also accept a `fontconfig
     <http://www.fontconfig.org/>`_ pattern, if it is the only argument
@@ -1022,23 +1021,14 @@ def get_default_size(self):
         """
         Return the default font size.
         """
-        if self.default_size is None:
-            return rcParams['font.size']
-        return self.default_size
+        return rcParams['font.size']
 
     def set_default_weight(self, weight):
         """
         Set the default font weight.  The initial value is 'normal'.
         """
         self.__default_weight = weight
 
-    def set_default_size(self, size):
-        """
-        Set the default font size in points.  The initial value is set
-        by ``font.size`` in rc.
-        """
-        self.default_size = size
-
     def update_fonts(self, filenames):
         """
         Update the font dictionary with new font files.
diff --git a/test/matplotlibrc b/test/matplotlibrc
@@ -1,3 +1,3 @@
 backend      : Agg
-font.family         : sans-serif
-font.sans-serif     : Bitstream Vera Sans, Lucida Grande, Verdana, Geneva, Lucid, Arial, Helvetica, Avant Garde, sans-serif
+font.family  : Bitstream Vera Sans
+text.hinting : True
