vscode/test/smoke
2018-04-18 23:41:15 +02:00
..
src increase CSS test timeout 2018-04-18 23:41:15 +02:00
test green git smoke tests! 2018-04-11 10:54:42 +02:00
tools streamline build 2018-04-11 10:27:01 +02:00
.gitignore ignore generated d.ts 2018-04-10 10:57:11 +02:00
Audit.md Added markdown to trace all smoke test failures to understand their patterns. Resolves #27906 2017-06-26 14:37:27 +02:00
package.json remove spectron references 2018-04-11 14:05:22 +02:00
README.md improve smoketest docs 2018-04-18 12:01:17 +02:00
tsconfig.json further smoke test improvements 2017-09-04 14:54:26 +02:00
yarn.lock remove spectron references 2018-04-11 14:05:22 +02:00

VS Code Smoke Test

How to run

# Dev
yarn smoketest

# Build
yarn smoketest --build "path/to/code"

The script calls mocha, so all mocha arguments should work fine. For example, use -f Git to filter all tests except the Git tests.

A --verbose flag can be used to log to the console all the low level driver calls make to Code.

Screenshots can be captured when tests fail. In order to get them,you need to use the argument --screenshots SCREENSHOT_DIR.

Pitfalls

  • Beware of workbench state. The tests within a single suite will share the same state.

  • Beware of singletons. This evil can, and will, manifest itself under the form of FS paths, TCP ports, IPC handles. Whenever writing a test, or setting up more smoke test architecture, make sure it can run simultaneously with any other tests and even itself. All test suites should be able to run many times in parallel.

  • Beware of focus. Never depend on DOM elements having focus using .focused classes or :focus pseudo-classes, since they will lose that state as soon as another window appears on top of the running VS Code window. A safe approach which avoids this problem is to use the waitForActiveElement API. Many tests use this whenever they need to wait for a specific element to have focus.

  • Beware of timing. You need to read from or write to the DOM... but is it the right time to do that? Can you 100% guarantee that that input box will be visible at that point in time? Or are you just hoping that it will be so? Hope is your worst enemy in UI tests. Example: just because you triggered Quick Open with F1, it doesn't mean that it's open and you can just start typing; you must first wait for the input element to be in the DOM as well as be the current active element.

  • Beware of waiting. Never wait longer than a couple of seconds for anything, unless it's justified. Think of it as a human using Code. Would a human take 10 minutes to run through the Search viewlet smoke test? Then, the computer should even be faster. Don't use setTimeout just because. Think about what you should wait for in the DOM to be ready and wait for that instead.