From 6fbe64aba6f901646e9f52e8cf683ed52805b67a Mon Sep 17 00:00:00 2001 From: joakin Date: Thu, 2 Mar 2017 13:26:06 +0100 Subject: [PATCH] Tests: Document why processLinks tests are integration tests Also, mark the test module as @integration. Change-Id: I83bf8fa3f4bda0dafbe2a2e458fe9dc9ce68f025 --- tests/qunit/ext.popups/processLinks.test.js | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/tests/qunit/ext.popups/processLinks.test.js b/tests/qunit/ext.popups/processLinks.test.js index 345312836..c847ea05f 100644 --- a/tests/qunit/ext.popups/processLinks.test.js +++ b/tests/qunit/ext.popups/processLinks.test.js @@ -1,6 +1,13 @@ ( function ( mw, $ ) { - QUnit.module( 'ext.popups/processLinks', { + /** + * processLinks QUnit tests remain integration tests given their dependency + * on a running MediaWiki instance with properly configured variables for + * 'wgContentNamespaces', 'wgArticlePath', and other configuration variables + * accessed by `mw.Title`. + */ + + QUnit.module( 'ext.popups/processLinks @integration', { setup: function () { this.config = new mw.Map(); this.config.set( {