jjrodrig opened a new pull request #1812: Migrate cluster with(out) quorum js tests as elixir tests URL: https://github.com/apache/couchdb/pull/1812 <!-- Thank you for your contribution! Please file this form by replacing the Markdown comments with your text. If a section needs no action - remove it. Also remember, that CouchDB uses the Review-Then-Commit (RTC) model of code collaboration. Positive feedback is represented +1 from committers and negative is a -1. The -1 also means veto, and needs to be addressed to proceed. Once there are no objections, the PR can be merged by a CouchDB committer. See: http://couchdb.apache.org/bylaws.html#decisions for more info. --> ## Overview This PR adds support for testing in different quorum conditions to the elixir test suite. The current JS quorum test suite is now included as part of the elixir test suite. Two new tasks have been included into Makefile/Makefile.win ``` make elixir-cluster-without-quorum ``` *elixir-cluster-without-quorum*: Executes tests over a cluster without quorum, a 3 node cluster with 2 nodes stopped ``` make elixir-cluster-with-quorum ``` *elixir-cluster-with-quorum*: Executes tests over a cluster with quorum, a 3 node cluster with 1 node stopped This PR integrates cluster with(out) quorum testing into the current elixir testing infrastructure. The tests to be executed in each quorum condition is selected by using elixir module tags. ```@moduletag :with_quorum_test``` and ```@moduletag :without_quorum_test``` are excluded from the current elixir test suite and are only considered by the associated makefile task. Any other test that requires to consider cluster quorum conditions can be included by using these tags ## Testing recommendations <!-- Describe how we can test your changes. Does it provides any behaviour that the end users could notice? --> Use the corresponding makefile tasks ``` make elixir-cluster-with-quorum make elixir-cluster-with-quorum ``` ## Related Issues or Pull Requests <!-- If your changes affects multiple components in different repositories please put links to those issues or pull requests here. --> PR #1800 include modifications in test/elixir/run and test/elixir/run.cmd used by this PR ## Checklist - [x] Code is written and works correctly; - [x] Changes are covered by tests; - [ ] Documentation reflects the changes;
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
