Fix tracker

This commit is contained in:
René Pfeuffer
2019-09-16 17:49:21 +02:00
parent b2294b8ae1
commit 0243edf585
2 changed files with 2 additions and 8 deletions

View File

@@ -27,16 +27,10 @@ class PluginDependencyTracker {
} }
private void addDependency(String from, String to) { private void addDependency(String from, String to) {
plugins.computeIfAbsent(to, name -> createInitialDependencyCollection(from)); plugins.computeIfAbsent(to, name -> new HashSet<>()).add(from);
} }
private void removeDependency(String from, String to) { private void removeDependency(String from, String to) {
plugins.get(to).remove(from); plugins.get(to).remove(from);
} }
private Collection<String> createInitialDependencyCollection(String from) {
Collection<String> dependencies = new HashSet<>();
dependencies.add(from);
return dependencies;
}
} }

View File

@@ -73,7 +73,7 @@ class PluginDependencyTrackerTest {
PluginDependencyTracker pluginDependencyTracker = new PluginDependencyTracker(); PluginDependencyTracker pluginDependencyTracker = new PluginDependencyTracker();
pluginDependencyTracker.addInstalled(review); pluginDependencyTracker.addInstalled(review);
pluginDependencyTracker.addInstalled(jira); pluginDependencyTracker.addInstalled(jira);
pluginDependencyTracker.removeInstalled(jira); pluginDependencyTracker.removeInstalled(review);
boolean mayUninstall = pluginDependencyTracker.mayUninstall("scm-mail-plugin"); boolean mayUninstall = pluginDependencyTracker.mayUninstall("scm-mail-plugin");
assertThat(mayUninstall).isFalse(); assertThat(mayUninstall).isFalse();