fixed one way to get a ConcurrentModificationException:

[shark-horse]

Using one object namesCollectorsLock to lock both names / collectors maps. Using a seperate object instead of one of the maps just to prevent confusion.

Stack trace which I previously encountered with version 0.6.0:

java.util.ConcurrentModificationException
	at java.util.HashMap$HashIterator.nextNode(HashMap.java:1445)
	at java.util.HashMap$EntryIterator.next(HashMap.java:1479)
	at java.util.HashMap$EntryIterator.next(HashMap.java:1477)
	at io.prometheus.client.CollectorRegistry$MetricFamilySamplesEnumeration.includedCollectorIterator(CollectorRegistry.java:156)
	at io.prometheus.client.CollectorRegistry$MetricFamilySamplesEnumeration.<init>(CollectorRegistry.java:146)
	at io.prometheus.client.CollectorRegistry.filteredMetricFamilySamples(CollectorRegistry.java:134)
	at com.procentive.yawl.logic.metrics.prometheus.MetricFamilySamplesIterable.getEnumeration(MetricFamilySamplesIterable.java:59)
	at com.procentive.yawl.logic.metrics.prometheus.MetricFamilySamplesIterable.iterator(MetricFamilySamplesIterable.java:41)
	at com.procentive.yawl.logic.metrics.prometheus.MetricFamilySamplesAssertions.reportSamples(MetricFamilySamplesAssertions.java:94)
	at com.procentive.yawl.logic.metrics.prometheus.MetricFamilySamplesAssertions.assert_hasMetrics(MetricFamilySamplesAssertions.java:41)

[brian-brazil]

Can you add the DCO please?

[shark-horse]

@brian-brazil Sorry, I don't see anything in this repo about what the agreement is for this repo. Can you please confirm what I would be agreeing to? The prometheus/prometheus wiki says says it's the Developer Certificate of Origin, but this repo says nothing about it that I found.

[brian-brazil]

Yes, it's that.

[shark-horse]

Thanks; done!

[brian-brazil]

Thanks!