Browse Source

verify Arrrrrg... maps are hard in python

Mike Smith 11 years ago
parent
commit
57fb4b7801
1 changed files with 6 additions and 12 deletions
  1. 6 12
      toolset/benchmark/framework_test.py

+ 6 - 12
toolset/benchmark/framework_test.py

@@ -829,18 +829,12 @@ def parse_config(config, directory, benchmarker):
       
       runTests = dict()
 
-      if (benchmarker.type == "all" or benchmarker.type == "json") and value.get("json_url", False):
-        runTests["json"] = True
-      if (benchmarker.type == "all" or benchmarker.type == "db") and value.get("db_url", False):
-        runTests["db"] = True
-      if (benchmarker.type == "all" or benchmarker.type == "query") and value.get("query_url", False):
-        runTests["query"] = True
-      if (benchmarker.type == "all" or benchmarker.type == "fortune") and value.get("fortune_url", False):
-        runTests["fortune"] = True
-      if (benchmarker.type == "all" or benchmarker.type == "update") and value.get("update_url", False):
-        runTests["update"] = True
-      if (benchmarker.type == "all" or benchmarker.type == "plaintext") and value.get("plaintext_url", False):
-        runTests["plaintext"] = True
+      runTests["json"] = (benchmarker.type == "all" or benchmarker.type == "json") and value.get("json_url", False)
+      runTests["db"] = (benchmarker.type == "all" or benchmarker.type == "db") and value.get("db_url", False)
+      runTests["query"] = (benchmarker.type == "all" or benchmarker.type == "query") and value.get("query_url", False)
+      runTests["fortune"] = (benchmarker.type == "all" or benchmarker.type == "fortune") and value.get("fortune_url", False)
+      runTests["update"] = (benchmarker.type == "all" or benchmarker.type == "update") and value.get("update_url", False)
+      runTests["plaintext"] = (benchmarker.type == "all" or benchmarker.type == "plaintext") and value.get("plaintext_url", False)
 
       # if the test uses the 'defualt' keywork, then we don't 
       # append anything to it's name. All configs should only have 1 default