diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2015-07-20 08:40:23 -0400 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2015-07-20 08:40:23 -0400 |
commit | c283ccda4cdd2eea3a22a06619d0460cbc8440cb (patch) | |
tree | f19d4e1c62df04808620667f0fa28be7a8397876 /coverage/data.py | |
parent | b53778a41bb8f799008adc3284c8f1da86fe860c (diff) | |
download | python-coveragepy-git-c283ccda4cdd2eea3a22a06619d0460cbc8440cb.tar.gz |
Be more strict when recording plugin names
Diffstat (limited to 'coverage/data.py')
-rw-r--r-- | coverage/data.py | 15 |
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`. |