You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@couchdb.apache.org by GitBox <gi...@apache.org> on 2021/11/05 03:40:53 UTC

[GitHub] [couchdb] kocolosk opened a new pull request #3818: Re enable most elixir tests redux

kocolosk opened a new pull request #3818:
URL: https://github.com/apache/couchdb/pull/3818


   ## Overview
   
   This is essentially a rebase of #3175, updated to skip additional incompatible elixir tests for "couchapp" functionality that have been added in the interim.
   
   There is a bug on `main` right now where the `_users` DB cannot be recreated once it has been deleted, and the cluster is rendered completely inoperable until CouchDB restarts. That's the reason for d8eea48 (and, I believe, for Paul's exclusion of other tests in that module). I've called that out separately so we can revert that commit once the fix is found.
   
   If anyone is interested in consolidating the `elixir`, `elixir-only`, and `elixir-suite` into a single target I'm all ears. The first two are identical save for the `MIX_ENV` and `COUCHDB_TEST_ADMIN_PARTY_OVERRIDE` environment settings, so I'm guessing we only need one target. The last one uses a skip file instead of the `@pending` tag; I could certainly imagine going that route but stuck with @davisp 's original approach for this PR at this time.
   
   ## Testing recommendations
   
   `make elixir`
   
   For me this yields
   
   > Finished in 86.2 seconds
   440 tests, 0 failures, 136 excluded, 1 skipped
   
   ## Related Issues or Pull Requests
   
   #3175
   
   ## Checklist
   
   - [x] Code is written and works correctly
   - [x] Changes are covered by tests
   - [ ] Any new configurable parameters are documented in `rel/overlay/etc/default.ini`
   - [ ] A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [couchdb] kocolosk merged pull request #3818: Re enable most elixir tests redux

Posted by GitBox <gi...@apache.org>.
kocolosk merged pull request #3818:
URL: https://github.com/apache/couchdb/pull/3818


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org