do not to unverified if already in verified

This commit is contained in:
Zlatin Balevsky
2018-07-10 19:58:31 +01:00
parent 6781944bdf
commit c017a937ee
2 changed files with 13 additions and 1 deletions

View File

@@ -26,7 +26,9 @@ class HostPool {
} }
synchronized def addUnverified(host) { synchronized def addUnverified(host) {
unverified.add(host) if (!verified.contains(host)) {
unverified.add(host)
}
} }
synchronized def getUnverified(int max) { synchronized def getUnverified(int max) {

View File

@@ -160,4 +160,14 @@ class HostPoolTest {
assert hp.getVerified(10,true).isEmpty() assert hp.getVerified(10,true).isEmpty()
assert hp.getUnverified(10).size() == 1 assert hp.getUnverified(10).size() == 1
} }
@Test
void doNotAddIfVerified() {
hp.addUnverified(freeBoth)
hp.verify(freeBoth)
assert hp.getUnverified(1).isEmpty()
hp.addUnverified(freeBoth)
assert hp.getUnverified(1).isEmpty()
}
} }