summaryrefslogtreecommitdiff
path: root/coverage/data.py
diff options
context:
space:
mode:
authorNed Batchelder <ned@nedbatchelder.com>2015-07-20 08:40:23 -0400
committerNed Batchelder <ned@nedbatchelder.com>2015-07-20 08:40:23 -0400
commitc283ccda4cdd2eea3a22a06619d0460cbc8440cb (patch)
treef19d4e1c62df04808620667f0fa28be7a8397876 /coverage/data.py
parentb53778a41bb8f799008adc3284c8f1da86fe860c (diff)
downloadpython-coveragepy-git-c283ccda4cdd2eea3a22a06619d0460cbc8440cb.tar.gz
Be more strict when recording plugin names
Diffstat (limited to 'coverage/data.py')
-rw-r--r--coverage/data.py15
1 files changed, 14 insertions, 1 deletions
diff --git a/coverage/data.py b/coverage/data.py
index e5c37cb7..b1fcd212 100644
--- a/coverage/data.py
+++ b/coverage/data.py
@@ -216,7 +216,20 @@ class CoverageData(object):
`plugin_data` is { filename: plugin_name, ... }
"""
- self._plugins.update(plugin_data)
+ existing_files = self._arcs or self._lines
+ for filename, plugin_name in iitems(plugin_data):
+ if filename not in existing_files:
+ raise CoverageException(
+ "Can't add plugin data for unmeasured file '%s'" % (filename,)
+ )
+ existing_plugin = self._plugins.get(filename)
+ if existing_plugin is not None and plugin_name != existing_plugin:
+ raise CoverageException(
+ "Conflicting plugin name for '%s': %r vs %r" % (
+ filename, existing_plugin, plugin_name,
+ )
+ )
+ self._plugins[filename] = plugin_name
def update(self, other_data, aliases=None):
"""Update this data with data from another `CoverageData`.