couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Paul Davis <paul.joseph.da...@gmail.com>
Subject Re: [Patch] remove redundant delete db call in basics.js test
Date Mon, 18 May 2009 19:35:06 GMT
Just because we fixed the bug doesn't mean we should stop testing it.
Also, we should probably make an explicit test that the status is 412
even if deleteDB has an internal check.

On Mon, May 18, 2009 at 3:26 PM, Kevin Jackson <foamdino@gmail.com> wrote:
> Hi,
>
> As issue 100 [1] has been fixed in the trunk code, the test doesn't
> need to have an additional delete.  Please see attached patch.  (I've
> included inline as the mailing list software may strip attachments).
>
> Thanks,
> Kev
>
> Index: basics.js
> ===================================================================
> --- basics.js   (revision 776052)
> +++ basics.js   (working copy)
> @@ -18,9 +18,6 @@
>     var db = new CouchDB("test_suite_db");
>     db.deleteDb();
>
> -    // bug COUCHDB-100: DELETE on non-existent DB returns 500 instead of 404
> -    db.deleteDb();
> -
>     db.createDb();
>
>     // PUT on existing DB should return 412 instead of 500
>

Mime
View raw message