biraz daha verify işleri.

sandbox/metadata.xml valid değil. Hiç bir "path" tagı içermiyor. Onu da düzelteceğiz.

pisi/metadata.py

@@ -27,10 +27,10 @@ class SourceInfo:
         return node
 
     def verify(self):
-        ret = True
+        if not self.name: return False
-        ret &= self.name
         return True
 
+
 class PackageInfo(specfile.PackageInfo):
 
     def __init__(self, node=None):
@@ -50,9 +50,7 @@ class PackageInfo(specfile.PackageInfo):
         return node
 
     def verify(self):
-        ret = True
+        super(PackageInfo, self).verify()
-        ret &= specfile.PackageInfo.verify()
-        return True
 
 class MetaData(XmlFile):
     """Package metadata. Metadata is composed of Specfile and various
@@ -92,11 +90,9 @@ class MetaData(XmlFile):
         self.writexml(filename)
 
     def verify(self):
-        ret = True
+        if not hasattr(self, 'source'): return False
-        ret &= hasattr(self, 'source')
+        if not self.source.verify(): return False
-        if self.source:
+        
-            ret &= self.source.verify()
+        if not self.package: return False
-        ret &= self.package != None
+        if not self.package.verify(): return False
-        if self.package:
-            ret &= self.package.verify()
         return True

tests/metadatatests.py

@@ -25,5 +25,10 @@ class MetaDataTestCase(unittest.TestCase):
         md = self.testRead()
         md.write(os.path.join(config.tmp_dir(),'metadata-test.xml' ))
 
+    def testVerify(self):
+        md = self.testRead()
+        if not md.verify():
+            self.fail("Couldn't verify!")
+
 
 suite = unittest.makeSuite(MetaDataTestCase)

tests/sandbox/metadata.xml

@@ -4,6 +4,7 @@
 
   <Source>
   <Name>sandbox</Name>
+  <Summary>We should have something to say here.</Summary>
   <Packager>
     <Name>Meral</Name>
     <Email>meren@uludag.org.tr</Email>