[MATH-505] TestUtils is thread-hostile

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.2, 2.0, 2.1
• Fix Version/s: 3.0
• Assignee: Sebb
• Reporter: Sebb
• Created: 01/Feb/11 00:28
• Updated: 24/Mar/12 16:16
• Resolved: 01/Feb/11 18:58

Description

TestUtils has several mutable static fields which are not synchronised, or volatile.

If one of the fields is updated by thread A, there is no guarantee that thread B will see the full update - it may see a partially updated object.

Furthermore, at least some of the static fields reference a mutable object, which can be changed whilst another thread is using it.

As far as I can tell, this class must only ever be used by a single thread otherwise the results will be unpredictable.