Fixing a race condition in a test and some linter warnings.

We can't control the order in which the threads are unlocked to initialize
Foo post-initialization, so I've removed the string argument from Foo.

Would probably be easier in the future to encode these as happens-before
relations and just check them.