[SERVER-403] Fix clojure unit tests in preparation for ruby/puppet update Created: 2015/02/27 Updated: 2016/09/27 Resolved: 2015/03/10
|Remaining Estimate:||Not Specified|
|Time Spent:||Not Specified|
|Original Estimate:||Not Specified|
|Epic Link:||Green: Puppet 4.0 Changes|
|Sprint:||Server Emerald 2015-03-04, Server Emerald 2015-03-18|
This ticket is preparation for the puppet-server 2.0 release. At that point we'll lock our submodule to the puppet 4.0 tag.
We know ahead of time that lein -U test :all is failing with the current HEAD of the master branch of puppet. We need to resolve those test failures prior to, or during, the submodule update.
Here's what the failure looks like:
|Comment by Jeff McCune [ 2015/03/02 ]|
|Comment by Nate Wolfe [ 2015/03/03 ]|
The unit tests are all failing because of missing directories, which is a result of the potential bug described in
|Comment by Chris Price [ 2015/03/03 ]|
As discussed in
For the clojure tests, though, I don't think we can necessarily avoid specifying all of them. Because, as Josh Cooper pointed out in that ticket, there is no guarantee that the log/run dirs are children of the vardir, and we want to configure our tests to use quarantined directories that are neither the ones specified by the 'root/master' run mode nor the 'agent/user' run mode.
So... not sure I've thought this through enough to give a solid recommendation on a particular path forward, but I do think that I can see a possibility that it ends up being relatively reasonable to specify all of those paths for our clojure tests, but specify a smaller number of them in our packaging.
|Comment by Jeremy Barlow [ 2015/03/03 ]|
The work in
|Comment by Nate Wolfe [ 2015/03/09 ]|
I expect this ticket to be resolved upon merging