From 9cd589029822bc34a26b96a542fea1c1c296979c Mon Sep 17 00:00:00 2001 From: Timo Tijhof Date: Thu, 11 Jul 2024 05:03:08 +0100 Subject: [PATCH] Test: Cover config.fixture with null or custom string, improve docs --- Gruntfile.js | 2 ++ docs/api/config/fixture.md | 18 +++++++++--------- test/config-fixture-null.html | 14 ++++++++++++++ test/config-fixture-null.js | 19 +++++++++++++++++++ test/config-fixture-string.html | 14 ++++++++++++++ test/config-fixture-string.js | 20 ++++++++++++++++++++ 6 files changed, 78 insertions(+), 9 deletions(-) create mode 100644 test/config-fixture-null.html create mode 100644 test/config-fixture-null.js create mode 100644 test/config-fixture-string.html create mode 100644 test/config-fixture-string.js diff --git a/Gruntfile.js b/Gruntfile.js index a8d041b44..84d2a11be 100644 --- a/Gruntfile.js +++ b/Gruntfile.js @@ -75,6 +75,8 @@ module.exports = function (grunt) { 'test/amd.html', 'test/autostart.html', + 'test/config-fixture-null.html', + 'test/config-fixture-string.html', 'test/dynamic-import.html', 'test/events-filters.html', 'test/events-in-test.html', diff --git a/docs/api/config/fixture.md b/docs/api/config/fixture.md index e94353414..e3c905457 100644 --- a/docs/api/config/fixture.md +++ b/docs/api/config/fixture.md @@ -1,7 +1,7 @@ --- layout: page-api title: QUnit.config.fixture -excerpt: HTML content to render before each test. +excerpt: Clean HTML playground before each test. groups: - config redirect_from: @@ -9,7 +9,7 @@ redirect_from: version_added: "1.0.0" --- -In browser environments, this HTML will be rendered in the `#qunit-fixture` element before each test. +In browser environments, QUnit can create a safe playground for HTML and DOM manipulation, that is automatically cleaned and restored to the default HTML before each test. @@ -24,7 +24,7 @@ In browser environments, this HTML will be rendered in the `#qunit-fixture` elem QUnit automatically resets the contents of `
`. This gives every test a fresh start, and automatically cleans up any additions or other changes from your tests. As long as you append or insert your elements inside the fixture, you will never have to manually clean up after your tests to keep them atomic. -By starting with an empty fixture in your test HTML file, you effectively give each test a clean start, as QUnit will automatically remove anything that was added there before the next test begins. +By starting with an empty fixture in your test HTML file, you effectively give each test a clean start, as QUnit will automatically remove anything that was added or staged there before the next test begins. If many of your tests require the same markup, you can also set it inside the fixture ahead of time. This reduces duplication between tests. QUnit guruantees that each test will start with a fresh copy of the original fixture, undoing any changes that happened during any previous tests. @@ -57,14 +57,14 @@ Starting with an empty fixture. Any additions are automatically removed. QUnit.test('example [first]', function (assert) { const fixture = document.querySelector('#qunit-fixture'); - const resultA = document.querySelectorAll('#first'); + const resultA = fixture.querySelectorAll('.first'); assert.strictEqual(resultA.length, 0, 'initial'); const div = document.createElement('div'); - div.id = 'first'; + div.className = 'first'; fixture.append(div); - const resultB = document.querySelectorAll('#first'); + const resultB = fixture.querySelectorAll('.first'); assert.strictEqual(resultB.length, 1, 'after append'); }); @@ -72,13 +72,13 @@ QUnit.test('example [second]', function (assert) { const fixture = document.querySelector('#qunit-fixture'); // The previous elements were automatically detached. - const result = document.querySelectorAll('#first'); + const result = fixture.querySelectorAll('.first'); assert.strictEqual(result.length, 0, 'initial is back to zero'); }); ``` -### qunit-fixture HTML +### default in HTML ```html @@ -148,7 +148,7 @@ QUnit.test('findText [em]', function (assert) { }); ``` -### dynamic qunit-fixture +### default in JavaScript ```html diff --git a/test/config-fixture-null.html b/test/config-fixture-null.html new file mode 100644 index 000000000..29b1d146d --- /dev/null +++ b/test/config-fixture-null.html @@ -0,0 +1,14 @@ + + + + + config-fixture-null + + + + + +
+
test markup
+ + diff --git a/test/config-fixture-null.js b/test/config-fixture-null.js new file mode 100644 index 000000000..8970f4c58 --- /dev/null +++ b/test/config-fixture-null.js @@ -0,0 +1,19 @@ +/* eslint-env browser */ +QUnit.module('QUnit.config [fixture=null]'); + +QUnit.config.reorder = false; +QUnit.config.fixture = null; + +QUnit.test('make dirty', function (assert) { + assert.strictEqual(QUnit.config.fixture, null); + + var fixture = document.querySelector('#qunit-fixture'); + fixture.textContent = 'dirty'; +}); + +QUnit.test('find dirty', function (assert) { + assert.strictEqual(QUnit.config.fixture, null); + + var fixture = document.querySelector('#qunit-fixture'); + assert.strictEqual(fixture.textContent, 'dirty'); +}); diff --git a/test/config-fixture-string.html b/test/config-fixture-string.html new file mode 100644 index 000000000..75d14b49b --- /dev/null +++ b/test/config-fixture-string.html @@ -0,0 +1,14 @@ + + + + + config-fixture-string + + + + + +
+
test markup
+ + diff --git a/test/config-fixture-string.js b/test/config-fixture-string.js new file mode 100644 index 000000000..bfaa8ae1c --- /dev/null +++ b/test/config-fixture-string.js @@ -0,0 +1,20 @@ +/* eslint-env browser */ +QUnit.module('QUnit.config [fixture=string]'); + +QUnit.config.reorder = false; +QUnit.config.fixture = '

Hi there, stranger!

'; + +QUnit.test('example [first]', function (assert) { + var fixture = document.querySelector('#qunit-fixture'); + + assert.strictEqual(fixture.textContent, 'Hi there, stranger!'); + + fixture.querySelector('strong').remove(); + + assert.strictEqual(fixture.textContent, 'Hi , stranger!'); +}); + +QUnit.test('example [second]', function (assert) { + var fixture = document.querySelector('#qunit-fixture'); + assert.strictEqual(fixture.textContent, 'Hi there, stranger!'); +});