Weirdness with decompose and concurrency.

[Matt Farmer]

Hey folks,

So I was working on developing some simple idiot-light tests for the WePay-Scala module that we open sourced recently, and I was noticing some weird behavior. Specifically that, for no reason I can explain, Extraction.decompose fails to read the case class properly. I've only been able to trigger this behavior while running my unit tests, and it doesn't happen every time - leading me to believe it's some sort of concurrency issue caused by how scalatest executes my tests.

You guys can take a look at the code I'm working on for these tests here: https://github.com/farmdawgnation/wepay-scala/tree/behavioral-development (direct link to the branch I'm working on).

The decompose that it's complaining about is this one: https://github.com/farmdawgnation/wepay-scala/blob/behavioral-development/src/main/scala/me/frmr/wepay/api/WePayResource.scala#L142

Additionally, you can see the failing test output on Travis CI: http://travis-ci.org/#!/farmdawgnation/wepay-scala/jobs/2388029

Am I doing something obscenely stupid here, is there a bug in the decompose code, or none of the above?

Thanks in advance for taking a look at this.

Cheers,

Matt Farmer

[Jeppe Nejsum Madsen]

Just took a quick look at the Meta.Memo class and it doesn't look like
the memoize method is thread-safe.

I believe this is a bug, but better have Joni explain if there are
good reasons for this :-)

/Jeppe

[Richard Dallaway]

By default SBT runs tests in parallel. If you have any state anywhere you have the risk of tests treading on each other.  If that's the problem, you can add this to build.sbt:

parallelExecution in Test := false

Richard

--
--
Lift, the simply functional web framework: http://liftweb.net
Code: http://github.com/lift
Discussion: http://groups.google.com/group/liftweb
Stuck? Help us help you: https://www.assembla.com/wiki/show/liftweb/Posting_example_code
 
 
 

[Matt Farmer]

Thanks for the tip, that'll solve my immediate problem, but I am still interested to hear what the verdict on memoize is.

--
Matt Farmer
Software Engineer, OpenStudy
http://openstudy.com

[Matt Farmer]

Hey folks, just curious: has anything happened with this since last week?

--

Matt Farmer

Software Engineer, OpenStudy

http://openstudy.com