Use temp files to store tests output, make it easy to compare results if output changes

.gitignore

@@ -14,3 +14,4 @@ lol*
 .idea*
 coverage_html_report/
 .coverage
+tests/testdata/outputs_current/

README_DEV.md

@@ -52,6 +52,12 @@ pip3 install -r requirements.txt
 
 It may cost you about 3 minutes to run all the tests.
 
+The tests may save their outputs content to `./tests/testdata/outputs_current/`, you can compare the files between it and `./tests/testdata/outputs_expected/` to see the difference if there is any changes.
+
+If you think the changes are expected, you can just copy them to `outputs_expected` and commit them as new expected outputs.
+
+The `./tests/testdata/outputs_current/` directory is deleted and recreated in `all_tests.sh` and `coverage_report.sh` each time.
+
 ### Generating test coverage report 
 
 ```bash

all_tests.sh

@@ -1,3 +1,5 @@
 #!/bin/sh
 
+rm -rf ./tests/testdata/outputs_current/
+mkdir -p ./tests/testdata/outputs_current/
 python3 -m unittest discover -p "*_test.py"

coverage_report.sh

@@ -1,6 +1,9 @@
 #!/bin/sh
 
+rm -rf ./tests/testdata/outputs_current/
+mkdir -p ./tests/testdata/outputs_current/
 rm -rf coverage_html_report
+
 coverage run -m unittest discover -p "*_test.py"
 coverage html
 open coverage_html_report/index.html

tests/__init__.py

@@ -5,6 +5,10 @@ TESTS_DIR = Path(__file__).parent
 PROJECT_DIR = TESTS_DIR.parent
 TESTDATA = TESTS_DIR / "testdata"
 TESTDATA_INPUTS = TESTDATA / "inputs"
-TESTDATA_OUTPUTS = TESTDATA / "outputs"
+TESTDATA_OUTPUTS_EXPECTED = TESTDATA / "outputs_expected"
+TESTDATA_OUTPUTS_CURRENT = TESTDATA / "outputs_current"
 
 os.environ['MYTHRIL_DIR'] = str(TESTS_DIR / "mythril_dir")
+
+def compare_files_error_message(expected, current):
+    return "Expected output changes, compare the following files to see differences: \n- {}\n- {}\n".format(str(expected), str(current))

tests/disassembler_test.py

@@ -18,12 +18,12 @@ class DisassemblerTestCase(TestCase):
 
     def test_easm_from_solidity_files(self):
         for input_file in TESTDATA_INPUTS.iterdir():
-            code = _compile_to_code(input_file)
+            output_expected = TESTDATA_OUTPUTS_EXPECTED / (input_file.name + ".easm")
+            output_current = TESTDATA_OUTPUTS_CURRENT / (input_file.name + ".easm")
 
+            code = _compile_to_code(input_file)
             disassembly = Disassembly(code)
 
-            # Useful for generating output file
+            output_current.write_text(disassembly.get_easm())
-            # (TESTDATA_OUTPUTS / (input_file.name + ".easm")).write_text(disassembly.get_easm())
 
-            expected_easm = (TESTDATA_OUTPUTS / (input_file.name + ".easm")).read_text()
+            self.assertEqual(output_expected.read_text(), output_current.read_text(), compare_files_error_message(output_expected, output_current))
-            self.assertEqual(disassembly.get_easm(), expected_easm, "{} returns invalid easm".format(str(input_file)))

tests/graph_test.py

@@ -1,8 +1,6 @@
 from unittest import TestCase
 
 from mythril.analysis.callgraph import generate_graph
-from mythril.analysis.report import Report
-from mythril.analysis.security import fire_lasers
 from mythril.analysis.symbolic import SymExecWrapper
 from mythril.ether import util
 from mythril.ether.soliditycontract import SolidityContract
@@ -12,21 +10,13 @@ class GraphTest(TestCase):
 
     def test_generate_graph(self):
         for input_file in TESTDATA_INPUTS.iterdir():
+            output_expected = TESTDATA_OUTPUTS_EXPECTED / (input_file.name + ".graph.html")
+            output_current = TESTDATA_OUTPUTS_CURRENT / (input_file.name + ".graph.html")
+
             contract = SolidityContract(str(input_file), name=None, solc_args=None)
             sym = SymExecWrapper(contract, address=(util.get_indexed_address(0)))
-            issues = fire_lasers(sym)
-
-            for issue in issues:
-                issue.add_code_info(contract)
-
-            report = Report()
-            for issue in issues:
-                report.append_issue(issue)
 
             html = generate_graph(sym)
+            output_current.write_text(html)
 
-            # Useful for generating output file
+            self.assertEqual(output_expected.read_text(), output_current.read_text(), compare_files_error_message(output_expected, output_current))
-            # (TESTDATA_OUTPUTS / (input_file.name + ".graph.html")).write_text(html)
-
-            expected = (TESTDATA_OUTPUTS / (input_file.name + ".graph.html")).read_text()
-            self.assertEqual(html, expected, "{}: graph html is changed".format(str(input_file)))

tests/report_test.py

@@ -5,6 +5,7 @@ from mythril.analysis.security import fire_lasers
 from mythril.analysis.symbolic import SymExecWrapper
 from mythril.ether import util
 from mythril.ether.soliditycontract import SolidityContract
+
 import json
 from tests import *
 
@@ -17,30 +18,49 @@ def _fix_debug_data(json_str):
         issue["debug"] = "<DEBUG-DATA>"
     return json.dumps(read_json, indent=4)
 
+def _generate_report(input_file):
+    contract = SolidityContract(str(input_file), name=None, solc_args=None)
+    sym = SymExecWrapper(contract, address=(util.get_indexed_address(0)))
+    issues = fire_lasers(sym)
+
+    report = Report()
+    for issue in issues:
+        issue.add_code_info(contract)
+        report.append_issue(issue)
+
+    return report
+
 class AnalysisReportTest(TestCase):
 
-    def test_reports(self):
+    def test_json_reports(self):
-        for input_file in (TESTDATA / "inputs").iterdir():
+        for input_file in TESTDATA_INPUTS.iterdir():
-            contract = SolidityContract(str(input_file), name=None, solc_args=None)
+            output_expected = TESTDATA_OUTPUTS_EXPECTED / (input_file.name + ".json")
-            sym = SymExecWrapper(contract, address=(util.get_indexed_address(0)))
+            output_current = TESTDATA_OUTPUTS_CURRENT / (input_file.name + ".json")
-            issues = fire_lasers(sym)
+
+            report = _generate_report(input_file)
+
+            output_current.write_text(_fix_path(_fix_debug_data(report.as_json())).strip())
+
+            self.assertEqual(output_expected.read_text(), output_expected.read_text(), compare_files_error_message(output_expected, output_current))
+
+    def test_markdown_reports(self):
+        for input_file in TESTDATA_INPUTS.iterdir():
+            output_expected = TESTDATA_OUTPUTS_EXPECTED / (input_file.name + ".markdown")
+            output_current = TESTDATA_OUTPUTS_CURRENT / (input_file.name + ".markdown")
+
+            report = _generate_report(input_file)
+
+            output_current.write_text(_fix_path(report.as_markdown()))
 
-            for issue in issues:
+            self.assertEqual(output_expected.read_text(), output_current.read_text(), compare_files_error_message(output_expected, output_current))
-                issue.add_code_info(contract)
 
-            report = Report()
+    def test_text_reports(self):
-            for issue in issues:
+        for input_file in TESTDATA_INPUTS.iterdir():
-                report.append_issue(issue)
+            output_expected = TESTDATA_OUTPUTS_EXPECTED / (input_file.name + ".text")
+            output_current = TESTDATA_OUTPUTS_CURRENT / (input_file.name + ".text")
 
-            # Useful for generating output file
+            report = _generate_report(input_file)
-            # (TESTDATA_OUTPUTS / (input_file.name + ".text")).write_text(_fix_path(report.as_text()))
-            # (TESTDATA_OUTPUTS / (input_file.name + ".json")).write_text(_fix_path(_fix_debug_data(report.as_json())))
-            # (TESTDATA_OUTPUTS / (input_file.name + ".markdown")).write_text(_fix_path(report.as_markdown()))
 
-            text = (TESTDATA / "outputs" / (input_file.name + ".text")).read_text()
+            output_current.write_text(_fix_path(report.as_text()))
-            json_report = (TESTDATA / "outputs" / (input_file.name + ".json")).read_text()
-            markdown = (TESTDATA / "outputs" / (input_file.name + ".markdown")).read_text()
 
-            self.assertEqual(_fix_path(report.as_text()), text, "{}: text report is changed".format(str(input_file)))
+            self.assertEqual(output_expected.read_text(), output_current.read_text(), compare_files_error_message(output_expected, output_current))
-            self.assertEqual(_fix_path(report.as_markdown()), markdown, "{}: markdown report is changed".format(str(input_file)))
-            self.assertEqual(_fix_path(_fix_debug_data(report.as_json())).strip(), json_report.strip(), "{}: json report is changed".format(str(input_file)))