From 3f16355e18cdceb37828a00a8181d9cc60815cd0 Mon Sep 17 00:00:00 2001
From: "Chris St. Pierre" <chris.a.st.pierre@gmail.com>
Date: Mon, 24 Sep 2012 14:51:36 -0400
Subject: fixed tests for pylint changes

---
 src/lib/Bcfg2/Server/Plugins/Metadata.py | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

(limited to 'src/lib/Bcfg2/Server/Plugins/Metadata.py')

diff --git a/src/lib/Bcfg2/Server/Plugins/Metadata.py b/src/lib/Bcfg2/Server/Plugins/Metadata.py
index e0904339f..468d1f190 100644
--- a/src/lib/Bcfg2/Server/Plugins/Metadata.py
+++ b/src/lib/Bcfg2/Server/Plugins/Metadata.py
@@ -1,6 +1,5 @@
-"""
-This file stores persistent metadata for the Bcfg2 Configuration Repository.
-"""
+""" This file stores persistent metadata for the Bcfg2 Configuration
+Repository. """
 
 import re
 import os
@@ -15,7 +14,7 @@ import Bcfg2.Server
 import Bcfg2.Server.Lint
 import Bcfg2.Server.Plugin
 import Bcfg2.Server.FileMonitor
-from Bcfg2.Compat import MutableMapping, all
+from Bcfg2.Compat import MutableMapping, all  # pylint: disable=W0622
 from Bcfg2.version import Bcfg2VersionInfo
 
 try:
@@ -26,6 +25,7 @@ except ImportError:
 
 logger = logging.getLogger(__name__)
 
+
 def locked(fd):
     """Aquire a lock on a file"""
     try:
@@ -104,13 +104,17 @@ class XMLMetadataConfig(Bcfg2.Server.Plugin.XMLFileBacked):
         self.pseudo_monitor = isinstance(metadata.core.fam,
                                          Bcfg2.Server.FileMonitor.Pseudo)
 
-    @property
-    def xdata(self):  # pylint: disable=E0202
+    def _get_xdata(self):
         if not self.data:
             raise Bcfg2.Server.Plugin.MetadataRuntimeError("%s has no data" %
                                                            self.basefile)
         return self.data
 
+    def _set_xdata(self, val):
+        self.data = val
+
+    xdata = property(_get_xdata, _set_xdata)
+
     @property
     def base_xdata(self):
         if not self.basedata:
@@ -738,11 +742,7 @@ class Metadata(Bcfg2.Server.Plugin.Metadata,
                 # clear the entire cache when we get an event for any
                 # metadata file
                 self.core.metadata_cache.expire()
-                try:
-                    proc = getattr(self, "_handle_%s_event" % aname)
-                except AttributeError:
-                    proc = self._handle_default_event  # pylint: disable=E1101
-                proc(event)
+                getattr(self, "_handle_%s_event" % aname)(event)
 
         if False not in list(self.states.values()) and self.debug_flag:
             # check that all groups are real and complete. this is
@@ -853,8 +853,8 @@ class Metadata(Bcfg2.Server.Plugin.Metadata,
         address = addresspair[0]
         if address in self.addresses:
             if len(self.addresses[address]) != 1:
-                err = "Address %s has multiple reverse assignments; a " + \
-                    "uuid must be used" % address
+                err = ("Address %s has multiple reverse assignments; a "
+                       "uuid must be used" % address)
                 self.logger.error(err)
                 raise Bcfg2.Server.Plugin.MetadataConsistencyError(err)
             return self.addresses[address][0]
-- 
cgit v1.2.3-1-g7c22