Skip to content

Commit

Permalink
Merge pull request #57 from dervus/mocha
Browse files Browse the repository at this point in the history
Port the tests to Mocha. And some of the missed PyYAML tests.
  • Loading branch information
ixti committed Dec 20, 2012
2 parents fac1f14 + 499a5d6 commit ca5722f
Show file tree
Hide file tree
Showing 199 changed files with 2,002 additions and 934 deletions.
6 changes: 3 additions & 3 deletions Makefile
Expand Up @@ -33,12 +33,12 @@ lint:




test: lint test: lint
@if test ! `which vows` ; then \ @if test ! `which mocha` ; then \
echo "You need 'vows' installed in order to run tests." >&2 ; \ echo "You need 'mocha' installed in order to run tests." >&2 ; \
echo " $ make dev-deps" >&2 ; \ echo " $ make dev-deps" >&2 ; \
exit 128 ; \ exit 128 ; \
fi fi
NODE_ENV=test vows --spec NODE_ENV=test mocha -R spec




doc: doc:
Expand Down
10 changes: 10 additions & 0 deletions lib/js-yaml.js
Expand Up @@ -23,6 +23,16 @@ jsyaml.scan = function scan(stream, callback, Loader) {
}; };




jsyaml.parse = function parse(stream, callback, Loader) {
Loader = (Loader || _loader.SafeLoader);

var loader = new Loader(stream);
while (loader.checkEvent()) {
callback(loader.getEvent());
}
};


jsyaml.compose = function compose(stream, Loader) { jsyaml.compose = function compose(stream, Loader) {
Loader = (Loader || _loader.SafeLoader); Loader = (Loader || _loader.SafeLoader);


Expand Down
1 change: 1 addition & 0 deletions lib/js-yaml/reader.js
Expand Up @@ -109,6 +109,7 @@ Reader.prototype.checkPrintable = function checkPrintable(data) {




module.exports.Reader = Reader; module.exports.Reader = Reader;
module.exports.ReaderError = ReaderError;




//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
Expand Down
2 changes: 1 addition & 1 deletion package.json
Expand Up @@ -24,6 +24,6 @@
"scripts" : { "test": "make test" }, "scripts" : { "test": "make test" },


"dependencies" : { "argparse": "~ 0.1.3" }, "dependencies" : { "argparse": "~ 0.1.3" },
"devDependencies" : { "vows": "~ 0.6.0" }, "devDependencies" : { "mocha": ">= 1.7.4" },
"engines" : { "node": ">= 0.6.0" } "engines" : { "node": ">= 0.6.0" }
} }
4 changes: 0 additions & 4 deletions test/functional-test.js

This file was deleted.

14 changes: 14 additions & 0 deletions test/functional.js
@@ -0,0 +1,14 @@
'use strict';
/*global describe */


describe('Functional.', function () {
require('./functional/reader');
require('./functional/mark');
require('./functional/tokens');
require('./functional/errors');
require('./functional/structure');
require('./functional/resolver');
require('./functional/constructor');
require('./functional/canonical');
});

0 comments on commit ca5722f

Please sign in to comment.