Skip to content

Commit

Permalink
Demos: Move /test/integration to /demos
Browse files Browse the repository at this point in the history
  • Loading branch information
Krinkle committed Jul 15, 2024
1 parent 2b0ae95 commit c16cab9
Show file tree
Hide file tree
Showing 43 changed files with 20 additions and 23 deletions.
6 changes: 3 additions & 3 deletions .editorconfig
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@ charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = true

[test/cli/TapReporter.js]
[demos/grunt-contrib-qunit.js]
trim_trailing_whitespace = false
[test/integration/grunt-contrib-qunit.js]
[demos/nyc.js]
trim_trailing_whitespace = false
[test/integration/nyc.js]
[test/cli/TapReporter.js]
trim_trailing_whitespace = false

[*.css]
Expand Down
8 changes: 4 additions & 4 deletions .eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,14 +12,14 @@
"ignorePatterns": [
"__codeorigin/**",
"coverage/**",
"demos/*/qunit/**",
"demos/*/coverage/**",
"demos/*/package-lock.json",
"docs/.jekyll-cache/**",
"docs/_site/**",
"lib/**",
"qunit/**",
"test/cli/fixtures/sourcemap/*.min.js",
"test/integration/*/qunit/**",
"test/integration/*/coverage/**",
"test/integration/*/package-lock.json",
"temp/**"
],
"overrides": [
Expand Down Expand Up @@ -190,7 +190,7 @@
}
},
{
"files": ["test/integration/**/*.js"],
"files": ["demos/**/*.js"],
"env": {
"browser": true,
"es2017": true,
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/CI.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,10 @@ jobs:
node: 22.x
script: npm run test

- name: "Integration"
- name: "Demos"
os: ubuntu-20.04
node: 18.x
script: npm run test-integration
script: npm run test-demos

- name: "Windows: Node 18"
os: windows-latest
Expand Down
4 changes: 2 additions & 2 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
/.nyc_output/
/__codeorigin/
/coverage/
/demos/*/package-lock.json
/demos/*/node_modules/
/docs/.jekyll-cache/
/docs/_site/
/docs/Gemfile.lock
/test/benchmark/package-lock.json
/test/benchmark/node_modules/
/test/integration/*/package-lock.json
/test/integration/*/node_modules/
/node_modules/
/qunit/
/temp/
Expand Down
6 changes: 2 additions & 4 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -72,15 +72,13 @@ Fixes #1.
The subject line should use the [imperative mood](https://en.wikipedia.org/wiki/Imperative_mood),
and start with one of the following components:

* `All`
* `Assert`
* `Build`
* `CLI`
* `Core`
* `Demos`
* `Docs`
* `Dump`
* `HTML Reporter`
* `Release`
* `Tests`
* `Test`

See also [Commit message guidelines](https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines).
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
1 change: 1 addition & 0 deletions demos/grunt-contrib-qunit/qunit
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
"karma-chrome-launcher": "^3.2.0",
"karma-firefox-launcher": "2.1.3",
"karma-qunit": "^4.2.0",
"qunit": "file:../../.."
"qunit": "file:../.."
},
"scripts": {
"test": "karma start --no-colors"
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"private": true,
"devDependencies": {
"nyc": "15.1.0",
"qunit": "file:../../.."
"qunit": "file:../.."
},
"scripts": {
"test": "nyc qunit"
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
1 change: 1 addition & 0 deletions demos/testem/qunit
File renamed without changes.
File renamed without changes.
2 changes: 1 addition & 1 deletion docs/cli.md
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ Generate code coverage reports with [nyc](https://istanbul.js.org/):
}
```

See [/test/integration/nyc/](https://github.com/qunitjs/qunit/tree/main/test/integration/nyc) in the QUnit repo for a minimal example.
See [/demos/nyc/](https://github.com/qunitjs/qunit/tree/main/demos/nyc) in the QUnit repo for a minimal example.

For a more elaborate example showcasing a unified test coverage report for tests in both Node.js and a headless browser, see [Krinkle/example-node-and-browser-qunit](https://github.com/Krinkle/example-node-and-browser-qunit-ci/).

Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@
"test-cli": "node bin/qunit.js test/main/ test/cli/*.js",
"test": "npm run lint && npm run build && npm run test-main && npm run test-cli",
"test-nolint": "npm run build && npm run test-main && npm run test-cli",
"test-integration": "npm run build && node bin/qunit.js test/integration/*.js",
"test-demos": "npm run build && node bin/qunit.js demos/*.js",
"browserstack": "browserstack-runner -v",
"authors": "(echo 'Authors ordered by first contribution\n' && git log --format='%aN <%aE>' --no-merges --reverse | awk '!seen[$0]++') | tee AUTHORS.txt",
"coverage": "npm run build-coverage && npm run coverage-cli && npm run coverage-main && nyc report",
Expand Down
3 changes: 1 addition & 2 deletions test/cli/structure.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,8 +83,7 @@ QUnit.module('structure', () => {
.map(file => file.replace(/\\/g, '/'))
// Ignore file names containing "--", which are subresources (e.g. iframes).
// Ignore test/benchmark, which is unrelated.
// Ignore test/integration/grunt-contrib-qunit, which we manage separately.
.filter(file => !file.includes('--') && !file.includes('benchmark') && !file.includes('integration'))
.filter(file => !file.includes('--') && !file.includes('benchmark'))
.map(file => `test/${file}`);

QUnit.test('files', assert => {
Expand Down
1 change: 0 additions & 1 deletion test/integration/grunt-contrib-qunit/qunit

This file was deleted.

1 change: 0 additions & 1 deletion test/integration/testem/qunit

This file was deleted.

0 comments on commit c16cab9

Please sign in to comment.