Tests: Remove grunt-contrib-qunit

There are no browser qunit tests right now, and if they are added at
some point they should use core's infrastructure to run the tests like
in mobilefrontend for example.

Bug: T160406
Change-Id: I4346a891bc1fdb9252ebef593312911031cf2287
This commit is contained in:
joakin 2017-04-25 14:06:02 +02:00
parent c9d325d01e
commit 841c69ba2e
2 changed files with 3 additions and 24 deletions

View file

@ -1,11 +1,9 @@
/* eslint-evn node */ /* eslint-evn node */
module.exports = function ( grunt ) { module.exports = function ( grunt ) {
var conf = grunt.file.readJSON( 'extension.json' ), var conf = grunt.file.readJSON( 'extension.json' );
QUNIT_URL_BASE = 'http://localhost:8080/wiki/Special:JavaScriptTest/qunit/plain';
grunt.loadNpmTasks( 'grunt-banana-checker' ); grunt.loadNpmTasks( 'grunt-banana-checker' );
grunt.loadNpmTasks( 'grunt-contrib-qunit' );
grunt.loadNpmTasks( 'grunt-contrib-watch' ); grunt.loadNpmTasks( 'grunt-contrib-watch' );
grunt.loadNpmTasks( 'grunt-eslint' ); grunt.loadNpmTasks( 'grunt-eslint' );
grunt.loadNpmTasks( 'grunt-jsonlint' ); grunt.loadNpmTasks( 'grunt-jsonlint' );
@ -39,19 +37,6 @@ module.exports = function ( grunt ) {
'!node_modules/**' '!node_modules/**'
] ]
}, },
qunit: {
all: {
options: {
timeout: 10000, // Using the filter query param takes longer
summaryOnly: true,
urls: [
// Execute any QUnit test in those module whose names begin with
// "ext.popups".
QUNIT_URL_BASE + '?filter=ext.popups'
]
}
}
},
stylelint: { stylelint: {
options: { options: {
syntax: 'less' syntax: 'less'
@ -66,13 +51,9 @@ module.exports = function ( grunt ) {
debounceDelay: 1000 debounceDelay: 1000
}, },
lint: { lint: {
files: [ 'resources/ext.popups/**/*.less', 'resources/**/*.js', 'tests/qunit/**/*.js' ], files: [ 'resources/ext.popups/**/*.less', 'resources/**/*.js' ],
tasks: [ 'lint' ] tasks: [ 'lint' ]
}, },
scripts: {
files: [ 'resources/**/*.js', 'tests/qunit/**/*.js' ],
tasks: [ 'test' ]
},
configFiles: { configFiles: {
files: [ 'Gruntfile.js' ], files: [ 'Gruntfile.js' ],
options: { options: {
@ -83,6 +64,5 @@ module.exports = function ( grunt ) {
} ); } );
grunt.registerTask( 'lint', [ 'eslint:all', 'stylelint', 'jsonlint', 'banana' ] ); grunt.registerTask( 'lint', [ 'eslint:all', 'stylelint', 'jsonlint', 'banana' ] );
grunt.registerTask( 'test', [ 'qunit' ] ); grunt.registerTask( 'default', [ 'lint' ] );
grunt.registerTask( 'default', [ 'lint', 'test' ] );
}; };

View file

@ -14,7 +14,6 @@
"grunt": "1.0.1", "grunt": "1.0.1",
"grunt-banana-checker": "0.5.0", "grunt-banana-checker": "0.5.0",
"grunt-cli": "^1.2.0", "grunt-cli": "^1.2.0",
"grunt-contrib-qunit": "^1.2.0",
"grunt-contrib-watch": "^1.0.0", "grunt-contrib-watch": "^1.0.0",
"grunt-eslint": "19.0.0", "grunt-eslint": "19.0.0",
"grunt-jsonlint": "1.1.0", "grunt-jsonlint": "1.1.0",