Skip to content

Commit

Permalink
Make better sense out of the organization of the test suite
Browse files Browse the repository at this point in the history
  • Loading branch information
jwolski committed Oct 4, 2015
1 parent ae6c4b8 commit 7a8d631
Show file tree
Hide file tree
Showing 18 changed files with 40 additions and 40 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
// THE SOFTWARE.
'use strict';

var testRingpop = require('./lib/test-ringpop');
var testRingpop = require('../lib/test-ringpop');

testRingpop('full sync includes all members', function t(deps, assert) {
var membership = deps.membership;
Expand Down
4 changes: 2 additions & 2 deletions test/hashring_test.js → test/unit/hashring_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.
var HashRing = require('../lib/ring');
var RBTree = require('../lib/rbtree').RBTree;
var HashRing = require('../../lib/ring');
var RBTree = require('../../lib/rbtree').RBTree;

var test = require('tape');

Expand Down
10 changes: 5 additions & 5 deletions test/index-test.js → test/unit/index-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,12 +18,12 @@
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.
var _ = require('underscore');
var AdminMember = require('../server/admin/member.js');
var createJoinHandler = require('../server/protocol/join.js');
var mock = require('./mock');
var Ringpop = require('../index.js');
var AdminMember = require('../../server/admin/member.js');
var createJoinHandler = require('../../server/protocol/join.js');
var mock = require('../mock');
var Ringpop = require('../../index.js');
var test = require('tape');
var testRingpop = require('./lib/test-ringpop.js');
var testRingpop = require('../lib/test-ringpop.js');

var createAdminJoinHandler = AdminMember.memberJoin.handler;
var createAdminLeaveHandler = AdminMember.memberLeave.handler;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,9 @@
// THE SOFTWARE.
'use strict';

var Member = require('../lib/membership/member.js');
var mergeJoinResponses = require('../lib/swim/join-response-merge.js');
var testRingpop = require('./lib/test-ringpop.js');
var Member = require('../../lib/membership/member.js');
var mergeJoinResponses = require('../../lib/swim/join-response-merge.js');
var testRingpop = require('../lib/test-ringpop.js');

testRingpop('no responses results in empty array', function t(deps, assert) {
var result = mergeJoinResponses(deps.ringpop, deps.ringpop, null);
Expand Down
4 changes: 2 additions & 2 deletions test/join-sender-test.js → test/unit/join-sender-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@
// THE SOFTWARE.
'use strict';

var createJoiner = require('../lib/swim/join-sender.js').createJoiner;
var Ringpop = require('../index.js');
var createJoiner = require('../../lib/swim/join-sender.js').createJoiner;
var Ringpop = require('../../index.js');
var test = require('tape');

function assertThrows(assert, thrower, assertions) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@
// THE SOFTWARE.
'use strict';

var mergeMembershipChangesets = require('../lib/membership/merge.js');
var testRingpop = require('./lib/test-ringpop.js');
var mergeMembershipChangesets = require('../../lib/membership/merge.js');
var testRingpop = require('../lib/test-ringpop.js');

testRingpop('merges incarnation numbers', function t(deps, assert) {
var changesets = [firstChangeset(), secondChangeset()];
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.
var testRingpop = require('./lib/test-ringpop.js');
var testRingpop = require('../lib/test-ringpop.js');

testRingpop('iterates over two members correctly', function t(deps, assert) {
var membership = deps.membership;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@
// THE SOFTWARE.
'use strict';

var createMembershipSetListener = require('../lib/membership-set-listener.js');
var testRingpop = require('./lib/test-ringpop.js');
var createMembershipSetListener = require('../../lib/membership-set-listener.js');
var testRingpop = require('../lib/test-ringpop.js');

testRingpop('starts suspicion period for suspect', function t(deps, assert) {
var ringpop = deps.ringpop;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,10 @@
// THE SOFTWARE.
'use strict';

var MembershipUpdateRollup = require('../lib/membership/rollup.js');
var Ringpop = require('../index.js');
var MembershipUpdateRollup = require('../../lib/membership/rollup.js');
var Ringpop = require('../../index.js');
var test = require('tape');
var testRingpop = require('./lib/test-ringpop.js');
var testRingpop = require('../lib/test-ringpop.js');

var localMemberUpdate = {
address: '127.0.0.1:3000',
Expand Down
8 changes: 4 additions & 4 deletions test/proxy_req_test.js → test/unit/proxy_req_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@
var _ = require('underscore');
var test = require('tape');

var allocRingpop = require('./lib/alloc-ringpop.js');
var bootstrap = require('./lib/bootstrap.js');
var mocks = require('./mock');
var Ringpop = require('../index.js');
var allocRingpop = require('../lib/alloc-ringpop.js');
var bootstrap = require('../lib/bootstrap.js');
var mocks = require('../mock');
var Ringpop = require('../../index.js');

test('proxyReq() proxies the request', function t(assert) {
var left = allocRingpop('left');
Expand Down
4 changes: 2 additions & 2 deletions test/rbiterator_test.js → test/unit/rbiterator_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.
var RBTree = require('../lib/rbtree').RBTree;
var RBIterator = require('../lib/rbtree').RBIterator;
var RBTree = require('../../lib/rbtree').RBTree;
var RBIterator = require('../../lib/rbtree').RBIterator;
var test = require('tape');

test('construct a new RBIterator', function t(assert) {
Expand Down
4 changes: 2 additions & 2 deletions test/rbtree_test.js → test/unit/rbtree_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.
var RBTree = require('../lib/rbtree').RBTree;
var RingNode = require('../lib/rbtree').RingNode;
var RBTree = require('../../lib/rbtree').RBTree;
var RingNode = require('../../lib/rbtree').RingNode;
var test = require('tape');

test('construct a new RBTree', function t(assert) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,10 @@
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.
var allocRequest = require('./lib/alloc-request.js');
var mocks = require('./mock');
var RequestProxy = require('../lib/request-proxy/index.js');
var Ringpop = require('../index.js');
var allocRequest = require('../lib/alloc-request.js');
var mocks = require('../mock');
var RequestProxy = require('../../lib/request-proxy/index.js');
var Ringpop = require('../../index.js');
var test = require('tape');

function createRingpop() {
Expand Down
2 changes: 1 addition & 1 deletion test/ring-test.js → test/unit/ring-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
'use strict';

var _ = require('underscore');
var HashRing = require('../lib/ring.js');
var HashRing = require('../../lib/ring.js');
var test = require('tape');

function createServers(size) {
Expand Down
2 changes: 1 addition & 1 deletion test/ringnode_test.js → test/unit/ringnode_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.
var RingNode = require('../lib/rbtree').RingNode;
var RingNode = require('../../lib/rbtree').RingNode;
var test = require('tape');

test('construct a new red node with supplied values', function t(assert) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@

var test = require('tape');

var Ringpop = require('../index.js');
var Ringpop = require('../../index.js');

test('ringpop without app throws', function t(assert) {
assert.throws(function throwIt() {
Expand Down
4 changes: 2 additions & 2 deletions test/swim_test.js → test/unit/swim_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.

var Member = require('../lib/membership/member.js');
var testRingpop = require('./lib/test-ringpop.js');
var Member = require('../../lib/membership/member.js');
var testRingpop = require('../lib/test-ringpop.js');

testRingpop('starting and stopping gossip sets timer / unsets timers', function t(deps, assert) {
var gossip = deps.gossip;
Expand Down
4 changes: 2 additions & 2 deletions test/trace-test.js → test/unit/trace-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ var EventEmitter = require('events').EventEmitter;
var test = require('tape');
var Timer = require('time-mock');

var core = require('../lib/trace/core');
var TracerStore = require('../lib/trace/store');
var core = require('../../lib/trace/core');
var TracerStore = require('../../lib/trace/store');

var timer = Timer(1000);
// workaround timer id 0 by setting a throwaway, increments to 1
Expand Down

0 comments on commit 7a8d631

Please sign in to comment.