Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(test): migrate test cmd #8138

Merged
merged 82 commits into from
Aug 23, 2022
Merged
Show file tree
Hide file tree
Changes from 14 commits
Commits
Show all changes
82 commits
Select commit Hold shift + click to select a range
6f20ccc
Update visualCallbackQunit.js
ShaMan123 Aug 9, 2022
d58f8ff
Create tests.css
ShaMan123 Aug 9, 2022
b21889b
testem
ShaMan123 Aug 9, 2022
d780c83
download goldens
ShaMan123 Aug 9, 2022
7487efc
runner
ShaMan123 Aug 9, 2022
e63df1d
migrate to testem
ShaMan123 Aug 9, 2022
902cf13
fix(ci): testem error
ShaMan123 Aug 9, 2022
dda1ef3
Update index.js
ShaMan123 Aug 9, 2022
2e38413
ci option
ShaMan123 Aug 9, 2022
75acb93
fix interactive test selection
ShaMan123 Aug 9, 2022
e416223
update workflows
ShaMan123 Aug 9, 2022
c0e7362
Update old-travis-reference.yml
ShaMan123 Aug 9, 2022
78ad4b2
cleanup
ShaMan123 Aug 9, 2022
6af11b4
Update index.js
ShaMan123 Aug 9, 2022
c7c4850
Update visualCallbackQunit.js
ShaMan123 Aug 9, 2022
204a4ca
create goldens
ShaMan123 Aug 9, 2022
ea3dcb4
busboy
ShaMan123 Aug 9, 2022
ea7094f
Update index.js
ShaMan123 Aug 9, 2022
62f26d0
create goldens only on chrome
ShaMan123 Aug 9, 2022
bfe82b1
Update tests.mustache
ShaMan123 Aug 9, 2022
6050218
Update index.js
ShaMan123 Aug 9, 2022
b30e170
Update visualCallbackQunit.js
ShaMan123 Aug 9, 2022
795e8ca
better
ShaMan123 Aug 9, 2022
cebb4a0
Update index.js
ShaMan123 Aug 9, 2022
a7d21dd
Update .gitignore
ShaMan123 Aug 9, 2022
7846cf6
Update index.js
ShaMan123 Aug 9, 2022
ee25842
Update index.js
ShaMan123 Aug 9, 2022
b330562
Update index.js
ShaMan123 Aug 9, 2022
e707a5f
Update index.js
ShaMan123 Aug 9, 2022
12789b8
Update visualTestLoop.js
ShaMan123 Aug 9, 2022
01a7c61
Update index.js
ShaMan123 Aug 9, 2022
d1bf80d
Update tests.mustache
ShaMan123 Aug 9, 2022
f0f9269
cleanup
ShaMan123 Aug 9, 2022
194098a
Update index.js
ShaMan123 Aug 9, 2022
71a1e2e
fix(): close http server on test end
ShaMan123 Aug 9, 2022
c133d0e
Delete old-travis-reference.yml
ShaMan123 Aug 9, 2022
aa23fc4
rename vars
ShaMan123 Aug 10, 2022
0d02092
Update rollup.config.js
ShaMan123 Aug 10, 2022
b26c423
rollup tests
ShaMan123 Aug 10, 2022
40dfa9f
rename opts
ShaMan123 Aug 10, 2022
c5b78b1
Update rollup.test.config.js
ShaMan123 Aug 10, 2022
f38b67a
Update rollup.test.config.js
ShaMan123 Aug 10, 2022
0b1abb7
Update rollup.test.config.js
ShaMan123 Aug 10, 2022
e8db53a
d
ShaMan123 Aug 10, 2022
de01665
index files
ShaMan123 Aug 10, 2022
8a40aff
move
ShaMan123 Aug 10, 2022
1ab8aa3
indexes
ShaMan123 Aug 10, 2022
fe89552
Update testem-visual.json
ShaMan123 Aug 10, 2022
f54ee83
rename option `--no-ci` => `--dev`
ShaMan123 Aug 10, 2022
c5b2357
Merge branch 'master' into testem-patch
ShaMan123 Aug 15, 2022
d924033
rename folder
ShaMan123 Aug 15, 2022
0e80994
Update index.js
ShaMan123 Aug 15, 2022
59d0ee2
rm `build-rollup` cmd
ShaMan123 Aug 16, 2022
9b4ed8e
cleanup
ShaMan123 Aug 16, 2022
549f80f
add dev cmd
ShaMan123 Aug 16, 2022
13b1881
cleanup cmd
ShaMan123 Aug 16, 2022
11dad69
Update package.json
ShaMan123 Aug 16, 2022
e2e1e0c
Revert "Update package.json"
ShaMan123 Aug 16, 2022
0b773c6
Revert "cleanup cmd"
ShaMan123 Aug 16, 2022
60e5d62
version
ShaMan123 Aug 16, 2022
2e18504
Squashed commit of the following:
ShaMan123 Aug 17, 2022
f46404f
docs
ShaMan123 Aug 17, 2022
70e556d
Update index.js
ShaMan123 Aug 17, 2022
92eced8
move
ShaMan123 Aug 17, 2022
e7303f7
fix null test
ShaMan123 Aug 17, 2022
b7c18a6
Merge branch 'master' into testem-patch
ShaMan123 Aug 17, 2022
7693c11
Update index.js
ShaMan123 Aug 18, 2022
a2df6f7
test dumps
ShaMan123 Aug 18, 2022
4e937e3
report to file
ShaMan123 Aug 18, 2022
16ac567
fix optionals + test out option
ShaMan123 Aug 18, 2022
4d47906
Merge branch 'master' into testem-patch
ShaMan123 Aug 19, 2022
31b70ad
Update CONTRIBUTING.md
ShaMan123 Aug 19, 2022
ca96690
Update CONTRIBUTING.md
ShaMan123 Aug 19, 2022
5cd35a7
Update CONTRIBUTING.md
ShaMan123 Aug 19, 2022
b9637ea
Update CONTRIBUTING.md
ShaMan123 Aug 19, 2022
992c697
Update CONTRIBUTING.md
ShaMan123 Aug 19, 2022
9f2252b
Update CONTRIBUTING.md
ShaMan123 Aug 19, 2022
647455c
Update CONTRIBUTING.md
ShaMan123 Aug 19, 2022
dffec2b
checkout
ShaMan123 Aug 19, 2022
4b9e5af
added 2 shortcuts
asturur Aug 22, 2022
5cac227
remove .fabric
asturur Aug 22, 2022
4cbf032
more rename
asturur Aug 22, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/browser-chrome.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,4 @@ jobs:
- name: Run headless test
uses: GabrielBB/xvfb-action@v1
with:
run: npm run testem:ci -- --port 8080 -f testem.json -l Chrome
run: npm run test -- -s unit -p 8080 -c chrome
2 changes: 1 addition & 1 deletion .github/workflows/browser-firefox.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,4 @@ jobs:
- name: Run headless test
uses: GabrielBB/xvfb-action@v1
with:
run: npm run testem:ci -- --port 8080 -f testem.json -l Firefox
run: npm run test -- -s unit -p 8080 -c firefox
2 changes: 1 addition & 1 deletion .github/workflows/node-unit-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,4 +26,4 @@ jobs:
node-version: ${{ matrix.node-version }}
- run: npm ci
- run: npm run build:fast
- run: npm run test -- --suite unit
- run: npm run test -- -s unit -p 8080 -c node
2 changes: 1 addition & 1 deletion .github/workflows/visual-node.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,4 +26,4 @@ jobs:
node-version: ${{ matrix.node-version }}
- run: npm ci
- run: npm run build -- -f -x gestures,accessors
- run: npm run test -- --suite visual
- run: npm run test -- -s visual -p 8080 -c node
2 changes: 1 addition & 1 deletion .github/workflows/visual-test-chrome.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,4 @@ jobs:
- name: Run headless test
uses: GabrielBB/xvfb-action@v1
with:
run: npm run testem:ci -- --port 8080 -f testem-visual.json -l Chrome
run: npm run test -- -s visual -p 8080 -c chrome
2 changes: 1 addition & 1 deletion .github/workflows/visual-test-firefox.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,4 @@ jobs:
- name: Run headless test
uses: GabrielBB/xvfb-action@v1
with:
run: npm run testem:ci -- --port 8080 -f testem-visual.json -l Firefox
run: npm run test -- -s visual -p 8080 -c firefox
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,4 @@ before_commit
/dist/fabric.require.js
/dist/fabric.min.js.gz
/scripts/cli_cache.json
/scripts/testem*.temp.json
8 changes: 4 additions & 4 deletions old-travis-reference.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,14 +48,14 @@ jobs:
apt:
packages: # avoid installing packages
- stage: Unit Tests
env: LAUNCHER=Chrome
script: npm run build:fast && testem ci --port 8080 -f testem.json -l $LAUNCHER
env: LAUNCHER=chrome
script: npm run build:fast && npm run test -- -s unit -p 8080 -c $LAUNCHER
addons:
apt:
packages: # avoid installing packages
- stage: Unit Tests
env: LAUNCHER=Firefox
script: npm run build:fast && testem ci --port 8080 -f testem.json -l $LAUNCHER
env: LAUNCHER=firefox
script: npm run build:fast && npm run test -- -s unit -p 8080 -c $LAUNCHER
ShaMan123 marked this conversation as resolved.
Show resolved Hide resolved
addons:
apt:
packages: # avoid installing packages
Expand Down
74 changes: 55 additions & 19 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 3 additions & 4 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,7 @@
"coverage:report": "nyc report --reporter=lcov --reporter=text",
"lint": "eslint --config .eslintrc.js src/**/*.ts",
"lint_tests": "eslint test/unit --config .eslintrc_tests && eslint test/visual --config .eslintrc_tests",
"all": "npm run build && npm run test -- --all && npm run lint && npm run lint_tests && npm run export",
"testem": "testem .",
"testem:ci": "testem ci"
"all": "npm run build && npm run test -- --all && npm run lint && npm run lint_tests && npm run export"
},
"optionalDependencies": {
"canvas": "^2.8.0",
Expand All @@ -71,16 +69,17 @@
"@types/node": "^17.0.21",
"@typescript-eslint/eslint-plugin": "^5.29.0",
"@typescript-eslint/parser": "^5.29.0",
"ansi-escape": "^1.1.0",
"auto-changelog": "^2.3.0",
"chalk": "^2.4.1",
"commander": "^9.1.0",
"deep-object-diff": "^1.1.7",
"eslint": "^8.21.0",
"fireworm": "^0.7.2",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixes testem ci error

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but what is it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the error?
or the package?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot what was the error...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no this package. How does it fix n error?

"fs-extra": "^10.0.1",
"fuzzy": "^0.1.3",
"inquirer": "^8.2.1",
"inquirer-checkbox-plus-prompt": "^1.0.1",
"kill-port": "^2.0.1",
"moment": "^2.29.1",
"nyc": "^15.1.0",
"pixelmatch": "^4.0.2",
Expand Down
Loading