Uh oh!
There was an error while loading. Please reload this page.
(#2415) - add LEVEL_PREFIX option for riakdown etc. - #2415
Conversation
nolanlawson
commented
Jun 23, 2014
Oh yeah, and read this guide to see why I need this option. |
nolanlawson
commented
Jun 25, 2014
All right, as @nlf pointed out to me, this test was actually pretty broken for the On the bright side, I discovered that the errors with MemDOWN were our own, so it looks like now we can remove memdown vs. memdown from the allowed failures. |
There was a problem hiding this comment.
Sorry for the churn on the following three tests. All I did was wrap them in an if statement to skip them if we're using any settings other than just the default leveldown with no prefix.
nolanlawson
commented
Jun 25, 2014
Got confirmation from @nlf that RiakDOWN now passes the test suite as both the client and the server. :) |
nolanlawson
commented
Jun 26, 2014
Tested sqlDOWN and the tests passed with it on the server and the client. Had to npm link pouchdb-server to this branch of pouchdb in order for it to work, but it works! |
There was a problem hiding this comment.
var createOpts = utils.extend(true, {}, opts, {adapter : backend.adapter});
nolanlawson
commented
Jun 27, 2014
All the recent PRs are failing in Travis for the ADAPTERS browser tests, with the error |
calvinmetcalf
commented
Jun 27, 2014
link to a failing build? |
nolanlawson
commented
Jun 27, 2014
NM, s'all good now. |
nolanlawson
commented
Aug 3, 2014
rebased |
nolanlawson
commented
Aug 3, 2014
Fixed unnecessary |
calvinmetcalf
commented
Aug 12, 2014
👍 |
Forgot to add this for riakdown etc. Gonna open this up for testing, though, so we can validate that it actually works for *DOWN authors.