Skip to content

Commit

Permalink
Clean up int-chrome tests, port some (microsoft#306)
Browse files Browse the repository at this point in the history
* Add hit condition tests

* Add tests for conditional breakpoints

* Delete unneeded int-chrome tests

* Add test that runs create-react-app

* Clean up testdata/ and remaining int-chrome references

* Fix env var spelling

* Fix "invalid condition" test

* Categorize tests correctly

* Disable framework tests for Windows

* Make React test a substring test

* Add new devops pipeline config
  • Loading branch information
roblourens authored Feb 6, 2020
1 parent 9991f30 commit c1bb814
Show file tree
Hide file tree
Showing 271 changed files with 397 additions and 48,625 deletions.
13 changes: 13 additions & 0 deletions .ci/common-validation.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
parameters:
runTests: true
runFrameworkTests: false

steps:
- task: NodeTool@0
Expand Down Expand Up @@ -36,6 +37,18 @@ steps:
env:
DISPLAY: ':99.0'

- task: Npm@1
displayName: npm test (framework tests)
inputs:
command: custom
verbose: false
customCommand: test
timeoutInMinutes: 10
condition: eq(${{ parameters.runFrameworkTests }}, true)
env:
FRAMEWORK_TESTS: 1
DISPLAY: ':99.0'

- task: Gulp@0
displayName: gulp lint
inputs:
Expand Down
26 changes: 26 additions & 0 deletions .ci/slow-tests.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
# Run on a schedule
trigger: none
pr: none

jobs:
- job: macOS
pool:
vmImage: 'macOS-10.13'
steps:
- template: common-validation.yml
parameters:
runFrameworkTests: true

- job: Linux
pool:
vmImage: 'ubuntu-16.04'
steps:
- template: common-validation.yml
parameters:
runFrameworkTests: true

- job: Windows
pool:
vmImage: 'vs2017-win2016'
steps:
- template: common-validation.yml
2 changes: 1 addition & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
module.exports = {
ignorePatterns: [
'**/*.d.ts',
'src/int-chrome/**/*.ts',
'src/test/**/*.ts',
'demos/**/*',
'**/*.js',
'testWorkspace/**'
],
parser: '@typescript-eslint/parser',
extends: ['plugin:@typescript-eslint/recommended'],
Expand Down
14 changes: 0 additions & 14 deletions .vscode/launch.json
Original file line number Diff line number Diff line change
Expand Up @@ -85,20 +85,6 @@
],
"outFiles": ["${workspaceFolder}/out/src/test/**/*.js"],
// "preLaunchTask": "npm: watch"
},
{
"type": "node",
"request": "launch",
"name": "Run int tests",
"program": "${workspaceFolder}/node_modules/mocha/bin/_mocha",
"args": ["--timeout", "10000", "-u", "tdd", "--colors", "--reporter", "out/src/int-chrome/testSupport/loggingReporter.js", "./out/src/int-chrome/**/*.test.js",
// "--grep", "Variables scopes"
],
"outFiles": ["${workspaceFolder}/out/src/**/*.js"],
"skipFiles": ["<node_internals>/**"],
"env": {
"DISPLAY": ":1.0"
}
},
{
"name": "Reset Results",
Expand Down
3 changes: 1 addition & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,7 @@
"publish": "gulp publish",
"test": "gulp && npm-run-all --parallel test:golden test:lint",
"test:golden": "node ./out/src/test/runTest.js",
"test:lint": "gulp lint",
"intTest": "mocha --exit --timeout 20000 -s 3500 -u tdd --colors --reporter out/src/int-chrome/testSupport/loggingReporter.js \"./out/src/int-chrome/**/*.test.js\""
"test:lint": "gulp lint"
},
"dependencies": {
"@c4312/chromehash": "^0.2.0",
Expand Down
291 changes: 0 additions & 291 deletions src/int-chrome/adapter.test.ts

This file was deleted.

Loading

0 comments on commit c1bb814

Please sign in to comment.