From 5bdfcb5dea448ef47ab646a09b4f13214c8bc8ee Mon Sep 17 00:00:00 2001
From: Steven Murray <Steven.Murray@cern.ch>
Date: Fri, 13 Sep 2019 19:42:24 +0200
Subject: [PATCH] Corrected unit-tests within test_ctafstgcd.py

---
 python/eosfstgcd/test_ctafstgcd.py | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

diff --git a/python/eosfstgcd/test_ctafstgcd.py b/python/eosfstgcd/test_ctafstgcd.py
index 422f00ee6e..74061baa60 100755
--- a/python/eosfstgcd/test_ctafstgcd.py
+++ b/python/eosfstgcd/test_ctafstgcd.py
@@ -57,6 +57,7 @@ class MockDisk:
     self.nbgetfreebytes = 0
 
   def listdir(self, path):
+    print "listdir path={}".format(path)
     self.nblistdir = self.nblistdir + 1
 
     pathlist = self.pathtolist(path)
@@ -373,7 +374,7 @@ class GcTestCase(unittest.TestCase):
     self.assertEqual(2, disk.nblistdir) # ls of fs and subdir
     self.assertEqual(1, disk.nbisdir)
     self.assertEqual(0, disk.nbisfile)
-    self.assertEqual(1, disk.nbgetfreebytes) # subdir
+    self.assertEqual(0, disk.nbgetfreebytes)
     self.assertEqual(0, disk.nbgetfilesizeandctime)
     self.assertEqual(1, eos.nbfsls)
     self.assertEqual(0, eos.nbstagerrm)
@@ -415,10 +416,10 @@ class GcTestCase(unittest.TestCase):
     runonlyonce = True
     gc.run(runonlyonce)
 
-    self.assertEqual(1, disk.nblistdir)
+    self.assertEqual(2, disk.nblistdir)
     self.assertEqual(1, disk.nbisdir)
     self.assertEqual(0, disk.nbisfile)
-    self.assertEqual(1, disk.nbgetfreebytes) # subdir
+    self.assertEqual(0, disk.nbgetfreebytes) # subdir
     self.assertEqual(0, disk.nbgetfilesizeandctime)
     self.assertEqual(1, eos.nbfsls)
     self.assertEqual(0, eos.nbstagerrm)
@@ -461,11 +462,11 @@ class GcTestCase(unittest.TestCase):
     runonlyonce = True
     gc.run(runonlyonce)
 
-    self.assertEqual(1, disk.nblistdir)
+    self.assertEqual(2, disk.nblistdir)
     self.assertEqual(1, disk.nbisdir)
-    self.assertEqual(0, disk.nbisfile)
-    self.assertEqual(1, disk.nbgetfreebytes) # subdir
-    self.assertEqual(0, disk.nbgetfilesizeandctime)
+    self.assertEqual(1, disk.nbisfile)
+    self.assertEqual(1, disk.nbgetfreebytes)
+    self.assertEqual(1, disk.nbgetfilesizeandctime)
     self.assertEqual(1, eos.nbfsls)
     self.assertEqual(0, eos.nbstagerrm)
     self.assertEqual(0, eos.nbattrset)
@@ -512,7 +513,7 @@ class GcTestCase(unittest.TestCase):
     self.assertEqual(2, disk.nblistdir)
     self.assertEqual(1, disk.nbisdir)
     self.assertEqual(1, disk.nbisfile)
-    self.assertEqual(2, disk.nbgetfreebytes) # subdir and file
+    self.assertEqual(1, disk.nbgetfreebytes) # file
     self.assertEqual(1, disk.nbgetfilesizeandctime)
     self.assertEqual(1, eos.nbfsls)
     self.assertEqual(0, eos.nbstagerrm)
@@ -563,7 +564,7 @@ class GcTestCase(unittest.TestCase):
     self.assertEqual(2, disk.nblistdir)
     self.assertEqual(1, disk.nbisdir)
     self.assertEqual(1, disk.nbisfile)
-    self.assertEqual(2, disk.nbgetfreebytes) # subdir and file
+    self.assertEqual(1, disk.nbgetfreebytes) # file
     self.assertEqual(1, disk.nbgetfilesizeandctime)
     self.assertEqual(1, eos.nbfsls)
     self.assertEqual(1, eos.nbstagerrm)
@@ -611,11 +612,11 @@ class GcTestCase(unittest.TestCase):
     runonlyonce = True
     gc.run(runonlyonce)
 
-    self.assertEqual(1, disk.nblistdir)
+    self.assertEqual(2, disk.nblistdir)
     self.assertEqual(1, disk.nbisdir)
-    self.assertEqual(0, disk.nbisfile)
+    self.assertEqual(1, disk.nbisfile)
     self.assertEqual(1, disk.nbgetfreebytes)
-    self.assertEqual(0, disk.nbgetfilesizeandctime)
+    self.assertEqual(1, disk.nbgetfilesizeandctime)
     self.assertEqual(1, eos.nbfsls)
     self.assertEqual(0, eos.nbstagerrm)
     self.assertEqual(0, eos.nbattrset)
-- 
GitLab