Commit 0cf2a96e authored by Damien Lespiau's avatar Damien Lespiau

tests: Introduce unit tests for the javascript side of things

Patchwork frontend needs a lot of work and, beside the seleniums tests
that are end-to-end, integration tests, there's room for proper JS unit
tests.

I've done that with karma + jasmine, it's fairly straightfoward and
actually quite nice to use. I'm a big fan of karma's "rerun the tests on
save" feature that made the development of patch_strip_series_markers()
much more fun.
Signed-off-by: 's avatarDamien Lespiau <damien.lespiau@intel.com>
parent 13cdbecf
// Karma configuration
// Generated on Wed Nov 04 2015 14:49:54 GMT+0000 (GMT)
module.exports = function(config) {
config.set({
// base path that will be used to resolve all patterns (eg. files, exclude)
basePath: '',
// frameworks to use
// available frameworks: https://npmjs.org/browse/keyword/karma-adapter
frameworks: ['jasmine'],
// list of files / patterns to load in the browser
files: [
'../htdocs/js/patchwork.js',
'test_patchwork.js'
],
// list of files to exclude
exclude: [
],
// preprocess matching files before serving them to the browser
// available preprocessors: https://npmjs.org/browse/keyword/karma-preprocessor
preprocessors: {
},
// test results reporter to use
// possible values: 'dots', 'progress'
// available reporters: https://npmjs.org/browse/keyword/karma-reporter
reporters: ['progress'],
// web server port
port: 9876,
// enable / disable colors in the output (reporters and logs)
colors: true,
// level of logging
// possible values: config.LOG_DISABLE || config.LOG_ERROR || config.LOG_WARN || config.LOG_INFO || config.LOG_DEBUG
logLevel: config.LOG_INFO,
// enable / disable watching file and executing tests whenever any file changes
autoWatch: true,
// start these browsers
// available browser launchers: https://npmjs.org/browse/keyword/karma-launcher
browsers: ['Chrome'],
// Continuous Integration mode
// if true, Karma captures browsers, runs the tests and exits
singleRun: false,
// Concurrency level
// how many browser should be started simultanous
concurrency: Infinity
});
};
#!/bin/sh -e
script_dir=$(cd `dirname $0`; pwd)
root_dir=`dirname $script_dir`
tests=$root_dir/tests
karma start $tests/karma.conf.js --single-run
describe("patch_strip_series_marker()", function() {
var fn = pw.patch_strip_series_marker;
it("should strip series markers", function() {
res = fn("[1/2] foo");
expect(res.order).toBe("1");
expect(res.name).toBe("foo");
});
it("infer order when there's no series markers", function() {
res = fn("foo");
expect(res.order).toBe("1");
expect(res.name).toBe("foo");
res = fn("[i-g-t] foo");
expect(res.order).toBe("1");
expect(res.name).toBe("[i-g-t] foo");
});
it("strip only series markers", function() {
res = fn("[v2,2/3] foo");
expect(res.order).toBe("2");
expect(res.name).toBe("[v2] foo");
});
it("don't fail if the series name contains '[' or ']'", function() {
res = fn("[2/3] f[o]o");
expect(res.order).toBe("2");
expect(res.name).toBe("f[o]o");
res = fn("f[o]o");
expect(res.order).toBe("1");
expect(res.name).toBe("f[o]o");
/* That's really pushing it, but we never know */
res = fn("f[2/3]o");
expect(res.order).toBe("1");
expect(res.name).toBe("f[2/3]o");
});
});
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment