Skip to content
This repository has been archived by the owner on Aug 22, 2019. It is now read-only.

Commit

Permalink
Acceptance tests now end in .atest.js.
Browse files Browse the repository at this point in the history
They're different enough from normal tap tests that I don't think they
should have the .test.js extension, but I would like to maintain that
naming style, so .atest.js seemed reasonable.

Also imported the acceptance test function as 'acceptanceTest' instead
of 'test' in login.atest.js, to make sure they don't get confused with
standard tap tests.
  • Loading branch information
toolness committed Apr 19, 2013
1 parent b9e3540 commit 2f3fc46
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
process.env['BROWSERID_TEST_USER'] = '[email protected]';

var async = require('async');
var test = require('./acceptance-test-utils.js').test;
var acceptanceTest = require('./acceptance-test-utils.js').test;

test('Login works', function(t) {
acceptanceTest('Login works', function(t) {
async.series([
t.open('/'),
t.click('.js-browserid-link'),
Expand Down
2 changes: 1 addition & 1 deletion acceptance-test/run-tests.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ const test = require('tap').test;
var rootDir = path.dirname(__dirname);
var node = process.argv[0];
var testFiles = fs.readdirSync(__dirname).filter(function(filename) {
return filename.match(/-test\.js$/);
return filename.match(/\.atest\.js$/);
});

testFiles.forEach(function(filename) {
Expand Down

0 comments on commit 2f3fc46

Please sign in to comment.