wiscsee icon indicating copy to clipboard operation
wiscsee copied to clipboard

add a missing def test_run(self) in test_demo.py

Open rakeshpandit opened this issue 8 years ago • 1 comments

I always have a def test_run(self): line added to demo file to prevent SimulateForSyntheticWorkload from running as I selectively run other workloads. Seems like a typo. Below is the diff. May be you also want to add it?

diff --git a/tests/test_demo.py b/tests/test_demo.py
index 1a161fb..8ae3361 100644
--- a/tests/test_demo.py
+++ b/tests/test_demo.py
@@ -58,6 +58,7 @@ class Test_TraceAndSimulateNKFTL(unittest.TestCase):
 
 
 class Test_SimulateForSyntheticWorkload(unittest.TestCase):
+    def test_run(self):
         class LocalExperiment(experiment.Experiment):
             def setup_workload(self):
                 self.conf['workload_src'] = config.LBAGENERATOR

rakeshpandit avatar Jul 28 '17 10:07 rakeshpandit

I'll fix it later :)

junhe avatar Aug 01 '17 13:08 junhe