Fixes from hamiltont's suggestion

toolset/benchmark/framework_test.py

@@ -294,13 +294,10 @@ class FrameworkTest:
       def output_result(result, reason, url):
         specific_rules_url = "http://frameworkbenchmarks.readthedocs.org/en/latest/Project-Information/Framework-Tests/#specific-test-requirements"
         color = Fore.GREEN
-        showRules = False
         if result.upper() == "WARN":
           color = Fore.YELLOW
-          showRules = True
         elif result.upper() == "FAIL":
           color = Fore.RED
-          showRules = True
 
         out.write(("   " + color + "%s" + Style.RESET_ALL + " for %s\n") % (result.upper(), url))
         print ("   " + color + "%s" + Style.RESET_ALL + " for %s") % (result.upper(), url)
@@ -308,8 +305,8 @@ class FrameworkTest:
           for line in reason.splitlines():
             out.write("     " + line + '\n')
             print "     " + line
-          if showRules:
-            out.write("     See %s" % specific_rules_url);
+          if not test.passed:
+            out.write("     See %s" % specific_rules_url)
             print "     See %s" % specific_rules_url
 
       [output_result(r1,r2,url) for (r1, r2, url) in results]