diff options
author | rozza@x103086.gcapmedia.com <rozza@x103086.gcapmedia.com> | 2009-11-20 10:08:52 +0000 |
---|---|---|
committer | rozza@x103086.gcapmedia.com <rozza@x103086.gcapmedia.com> | 2009-11-20 10:08:52 +0000 |
commit | b09acaa84c8624d67c9c1e271a780fbd2122c2c6 (patch) | |
tree | f362c5291a2f176798dbdcaef8102186ecd29121 /coverage/codeunit.py | |
parent | e9f83b2bb467204bb6bbbfa0550b8b9fd19c9c7b (diff) | |
download | python-coveragepy-git-b09acaa84c8624d67c9c1e271a780fbd2122c2c6.tar.gz |
Moving file filter logic out so it can be monkey patched easily
Diffstat (limited to 'coverage/codeunit.py')
-rw-r--r-- | coverage/codeunit.py | 27 |
1 files changed, 17 insertions, 10 deletions
diff --git a/coverage/codeunit.py b/coverage/codeunit.py index e3107052..28fa0551 100644 --- a/coverage/codeunit.py +++ b/coverage/codeunit.py @@ -34,19 +34,26 @@ def code_unit_factory(morfs, file_locator, omit_prefixes=None): code_units = [CodeUnit(morf, file_locator) for morf in morfs] if omit_prefixes: - prefixes = [file_locator.abs_file(p) for p in omit_prefixes] - filtered = [] - for cu in code_units: - for prefix in prefixes: - if cu.filename.startswith(prefix): - break - else: - filtered.append(cu) - - code_units = filtered + code_units = omit_filter(omit_prefixes, code_units) return code_units +def omit_filter(omit_prefixes, code_units): + """ + The filtering method removing any unwanted code_units + + Refactored out so you can easily monkeypatch if needs be + """ + prefixes = [file_locator.abs_file(p) for p in omit_prefixes] + filtered = [] + for cu in code_units: + for prefix in prefixes: + if cu.filename.startswith(prefix): + break + else: + filtered.append(cu) + + return filtered class CodeUnit(object): """Code unit: a filename or module. |