Commons-events passes tests...

[Bryce L Nordgren <bn...@fs.fed.us>]

My original checkout of commons-events failed two out of 316 tests packaged
with it.  It took a couple of hours, but here's the diagnosis:

I was depending on commons-collections 3.1 and the test in commons-events
was finding a bug in HashBag (actually AbstractMapBag) w.r.t. the return
value of "remove()".  I updated the dependency to commons-collections-3.2
(installed manually in my maven repo) and the problem vanished.  (I did
have to fix some control logic in ObservableBag, but it was really
cosmetic.)

Note that even though the buggy return value was fixed, the unit tests for
commons-collections (3.2) do not examine the return value of the "remove()"
method.  I don't know whether this is a bug or not. :)

Now the patient looks fine!!

Bryce


---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org