diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2010-09-03 23:20:39 -0400 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2010-09-03 23:20:39 -0400 |
commit | e3e843da657cf109f482a173f8b1eec301c91f38 (patch) | |
tree | 5084ad4d5d7cc85c8999a4b9099fc83eb61eeffc | |
parent | 023f6b5f6789230d7dc473361d75505bf86c60f3 (diff) | |
download | python-coveragepy-e3e843da657cf109f482a173f8b1eec301c91f38.tar.gz |
Now that we include 0% files in the data, they aren't 'executed', they are 'measured'.
-rw-r--r-- | coverage/data.py | 4 | ||||
-rw-r--r-- | coverage/report.py | 2 | ||||
-rw-r--r-- | lab/dataflow.txt | 2 | ||||
-rw-r--r-- | test/test_data.py | 18 |
4 files changed, 13 insertions, 13 deletions
diff --git a/coverage/data.py b/coverage/data.py index 30119d9..3d750c4 100644 --- a/coverage/data.py +++ b/coverage/data.py @@ -211,8 +211,8 @@ class CoverageData(object): """Ensure that `filename` appears in the data, empty if needed.""" self.lines.setdefault(filename, {}) - def executed_files(self): - """A list of all files that had been measured as executed.""" + def measured_files(self): + """A list of all files that had been measured.""" return list(self.lines.keys()) def executed_lines(self, filename): diff --git a/coverage/report.py b/coverage/report.py index c6725bb..0fb353a 100644 --- a/coverage/report.py +++ b/coverage/report.py @@ -31,7 +31,7 @@ class Reporter(object): CoverageConfig instance. """ - morfs = morfs or self.coverage.data.executed_files() + morfs = morfs or self.coverage.data.measured_files() file_locator = self.coverage.file_locator self.code_units = code_unit_factory(morfs, file_locator) diff --git a/lab/dataflow.txt b/lab/dataflow.txt index 5796206..b2b619a 100644 --- a/lab/dataflow.txt +++ b/lab/dataflow.txt @@ -15,7 +15,7 @@ Collector.get_line_data(): CoverageData.add_line_data( { filename: { lineno: None, .. }, ... } ) --> data.lines -CoverageData.executed_files(): +CoverageData.measured_files(): returns [filename, ...] called by: Reporter.find_code_units() diff --git a/test/test_data.py b/test/test_data.py index fd7e487..298078a 100644 --- a/test/test_data.py +++ b/test/test_data.py @@ -11,13 +11,13 @@ from coveragetest import CoverageTest DATA_1 = { 'a.py': {1:None, 2:None}, 'b.py': {3:None} } SUMMARY_1 = { 'a.py':2, 'b.py':1 } -EXECED_FILES_1 = [ 'a.py', 'b.py' ] +MEASURED_FILES_1 = [ 'a.py', 'b.py' ] A_PY_LINES_1 = [1,2] B_PY_LINES_1 = [3] DATA_2 = { 'a.py': {1:None, 5:None}, 'c.py': {17:None} } SUMMARY_1_2 = { 'a.py':3, 'b.py':1, 'c.py':1 } -EXECED_FILES_1_2 = [ 'a.py', 'b.py', 'c.py' ] +MEASURED_FILES_1_2 = [ 'a.py', 'b.py', 'c.py' ] ARC_DATA_3 = { 'x.py': {(1,2):None, (2,3):None}, 'y.py': {(17,23):None} } X_PY_ARCS_3 = [(1,2), (2,3)] @@ -30,9 +30,9 @@ class DataTest(CoverageTest): """Check that the summary of `covdata` is `summary`.""" self.assertEqual(covdata.summary(), summary) - def assert_executed_files(self, covdata, execed): - """Check that `covdata`'s executed files are `execed`.""" - self.assertSameElements(covdata.executed_files(), execed) + def assert_measured_files(self, covdata, measured): + """Check that `covdata`'s measured files are `measured`.""" + self.assertSameElements(covdata.measured_files(), measured) def test_reading_empty(self): covdata = CoverageData() @@ -43,13 +43,13 @@ class DataTest(CoverageTest): covdata = CoverageData() covdata.add_line_data(DATA_1) self.assert_summary(covdata, SUMMARY_1) - self.assert_executed_files(covdata, EXECED_FILES_1) + self.assert_measured_files(covdata, MEASURED_FILES_1) def test_touch_file(self): covdata = CoverageData() covdata.add_line_data(DATA_1) covdata.touch_file('x.py') - self.assert_executed_files(covdata, EXECED_FILES_1 + ['x.py']) + self.assert_measured_files(covdata, MEASURED_FILES_1 + ['x.py']) def test_writing_and_reading(self): covdata1 = CoverageData() @@ -72,7 +72,7 @@ class DataTest(CoverageTest): covdata3 = CoverageData() covdata3.combine_parallel_data() self.assert_summary(covdata3, SUMMARY_1_2) - self.assert_executed_files(covdata3, EXECED_FILES_1_2) + self.assert_measured_files(covdata3, MEASURED_FILES_1_2) def test_erasing(self): covdata1 = CoverageData() @@ -98,7 +98,7 @@ class DataTest(CoverageTest): fdata.close() lines = data['lines'] - self.assertSameElements(lines.keys(), EXECED_FILES_1) + self.assertSameElements(lines.keys(), MEASURED_FILES_1) self.assertSameElements(lines['a.py'], A_PY_LINES_1) self.assertSameElements(lines['b.py'], B_PY_LINES_1) # If not measuring branches, there's no arcs entry. |