Skip to content

Commit

Permalink
chore: rm non-test-related coffeescript references
Browse files Browse the repository at this point in the history
  • Loading branch information
flotwig committed Jun 10, 2020
1 parent 4cc301d commit 0e63263
Show file tree
Hide file tree
Showing 48 changed files with 49 additions and 404 deletions.
1 change: 0 additions & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,6 @@ Here is a list of the core packages in this repository with a short description,
| Folder Name | Package Name | Purpose |
| :------------------------------------ | :---------------------- | :--------------------------------------------------------------------------- |
| [cli](./cli) | `cypress` | The command-line tool that is packaged as an `npm` module. |
| [coffee](./packages/coffee) | `@packages/coffee` | A centralized version of CoffeeScript used for other packages. |
| [desktop-gui](./packages/desktop-gui) | `@packages/desktop-gui` | The front-end code for the Cypress Desktop GUI. |
| [driver](./packages/driver) | `@packages/driver` | The code that is used to drive the behavior of the API commands. |
| [electron](./packages/electron) | `@packages/electron` | The Cypress implementation of Electron. |
Expand Down
135 changes: 0 additions & 135 deletions coffeelint.json

This file was deleted.

18 changes: 8 additions & 10 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -48,20 +48,20 @@
"stop-only-all": "yarn stop-only --folder packages",
"pretest": "yarn ensure-deps",
"test": "yarn lerna exec yarn test --scope cypress --scope \"'@packages/{electron,extension,https-proxy,launcher,network,proxy,rewriter,reporter,runner,socket}'\"",
"test-debug": "lerna exec yarn test-debug --ignore \"'@packages/{coffee,desktop-gui,driver,root,static,web-config}'\"",
"test-debug": "lerna exec yarn test-debug --ignore \"'@packages/{desktop-gui,driver,root,static,web-config}'\"",
"pretest-e2e": "yarn ensure-deps",
"test-e2e": "lerna exec yarn test-e2e --ignore \"'@packages/{coffee,desktop-gui,driver,root,static,web-config}'\"",
"test-integration": "lerna exec yarn test-integration --ignore \"'@packages/{coffee,desktop-gui,driver,root,static,web-config}'\"",
"test-e2e": "lerna exec yarn test-e2e --ignore \"'@packages/{desktop-gui,driver,root,static,web-config}'\"",
"test-integration": "lerna exec yarn test-integration --ignore \"'@packages/{desktop-gui,driver,root,static,web-config}'\"",
"test-jscodeshift": "jest ./scripts/decaff",
"test-mocha": "mocha --reporter spec scripts/spec.js",
"test-mocha-snapshot": "mocha scripts/mocha-snapshot-spec.js",
"test-s3-api": "node -r ./packages/coffee/register -r ./packages/ts/register scripts/binary/s3-api-demo.ts",
"test-scripts": "mocha -r packages/coffee/register -r packages/ts/register --reporter spec 'scripts/unit/**/*spec.js'",
"test-scripts-watch": "yarn test-scripts --watch --watch-extensions 'ts,js,coffee'",
"test-s3-api": "node -r ./packages/ts/register scripts/binary/s3-api-demo.ts",
"test-scripts": "mocha -r packages/ts/register --reporter spec 'scripts/unit/**/*spec.js'",
"test-scripts-watch": "yarn test-scripts --watch --watch-extensions 'ts,js'",
"pretest-unit": "yarn ensure-deps",
"test-unit": "lerna exec yarn test-unit --ignore \"'@packages/{coffee,desktop-gui,driver,root,static,web-config}'\"",
"test-unit": "lerna exec yarn test-unit --ignore \"'@packages/{desktop-gui,driver,root,static,web-config}'\"",
"pretest-watch": "yarn ensure-deps",
"test-watch": "lerna exec yarn test-watch --ignore \"'@packages/{coffee,desktop-gui,driver,root,static,web-config}'\"",
"test-watch": "lerna exec yarn test-watch --ignore \"'@packages/{desktop-gui,driver,root,static,web-config}'\"",
"type-check": "node scripts/type_check",
"verify:mocha:results": "node ./scripts/verify_mocha_results",
"prewatch": "yarn ensure-deps",
Expand Down Expand Up @@ -138,7 +138,6 @@
"globby": "10.0.1",
"gulp": "4.0.2",
"gulp-awspublish": "4.0.0",
"gulp-coffee": "3.0.3",
"gulp-debug": "4.0.0",
"gulp-rename": "1.4.0",
"hasha": "5.0.0",
Expand Down Expand Up @@ -186,7 +185,6 @@
"through": "2.3.8",
"ts-node": "8.3.0",
"typescript": "3.7.4",
"vinyl-paths": "2.1.0",
"wait-on": "3.3.0"
},
"engines": {
Expand Down
1 change: 0 additions & 1 deletion packages/coffee/index.js

This file was deleted.

15 changes: 0 additions & 15 deletions packages/coffee/package.json

This file was deleted.

20 changes: 0 additions & 20 deletions packages/coffee/register.js

This file was deleted.

1 change: 0 additions & 1 deletion packages/driver/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
"@cypress/sinon-chai": "1.1.0",
"@cypress/unique-selector": "0.4.2",
"@cypress/what-is-circular": "1.0.1",
"@packages/coffee": "*",
"@packages/network": "*",
"@packages/runner": "*",
"@packages/ts": "*",
Expand Down
5 changes: 0 additions & 5 deletions packages/driver/ts/import-coffee.d.ts

This file was deleted.

2 changes: 0 additions & 2 deletions packages/extension/gulpfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ const pkg = require('./package.json')
const gulp = require('gulp')
const rimraf = require('rimraf')
const source = require('vinyl-source-stream')
const coffeeify = require('coffeeify')
const browserify = require('browserify')
const cypressIcons = require('@cypress/icons')

Expand Down Expand Up @@ -33,7 +32,6 @@ const manifest = (done) => {
const background = () => {
return browserify({
entries: 'app/init.js',
transform: coffeeify,
})
.bundle()
.pipe(source('background.js'))
Expand Down
2 changes: 0 additions & 2 deletions packages/extension/index.js
Original file line number Diff line number Diff line change
@@ -1,3 +1 @@
require('@packages/coffee/register')

module.exports = require('./lib/extension')
2 changes: 0 additions & 2 deletions packages/extension/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,9 @@
},
"devDependencies": {
"@cypress/icons": "0.7.0",
"@packages/coffee": "*",
"@packages/socket": "*",
"browserify": "16.3.0",
"chai": "3.5.0",
"coffeeify": "3.0.1",
"coffeescript": "1.12.7",
"cross-env": "6.0.3",
"eol": "0.9.1",
Expand Down
1 change: 0 additions & 1 deletion packages/extension/test/mocha.opts
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
test/unit
test/integration
--reporter spec
--compilers coffee:@packages/coffee/register
--recursive
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ CA_PATH=../../../ca
rm -rf $CA_PATH

# ensure regular root CA exists
node -r "@packages/coffee/register" -e "require('@packages/https-proxy/lib/ca').create('$CA_PATH')"
node -e "require('@packages/https-proxy/lib/ca').create('$CA_PATH')"

echo "remove and relink test CA pems"
for f in ca client server
Expand Down
1 change: 0 additions & 1 deletion packages/launcher/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
"ramda": "0.24.1"
},
"devDependencies": {
"@packages/coffee": "*",
"@packages/ts": "*",
"chai": "3.5.0",
"chai-as-promised": "7.1.1",
Expand Down
2 changes: 1 addition & 1 deletion packages/launcher/test/mocha.opts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
test/unit
--compilers ts:@packages/ts/register,coffee:@packages/coffee/register
--compilers ts:@packages/ts/register
--timeout 10000
--recursive
2 changes: 1 addition & 1 deletion packages/proxy/test/mocha.opts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
test/unit
--compilers ts:@packages/ts/register,coffee:@packages/coffee/register
--compilers ts:@packages/ts/register
--timeout 10000
--recursive
--require test/pretest.ts
2 changes: 1 addition & 1 deletion packages/reporter/test/mocha.opts
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
--require @packages/web-config/node-register
--watch-extensions ts,jsx,tsx,coffee,js
--watch-extensions ts,jsx,tsx,js
--file test/helper
2 changes: 1 addition & 1 deletion packages/runner/test/.mocharc.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"file": "test/helper.js",
"require": "../web-config/node-register",
"extension": "ts,jsx,tsx,coffee,js"
"extension": "ts,jsx,tsx,js"
}
5 changes: 0 additions & 5 deletions packages/runner/ts/import-coffee.d.ts

This file was deleted.

12 changes: 6 additions & 6 deletions packages/server/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ When running e2e tests, some test projects output verbose logs. To see them run

```bashtest-kitchensink
yarn test <path/to/test>
yarn test test/unit/api_spec.coffee
yarn test test/unit/api_spec.js
## or
yarn test-unit api_spec ## shorthand, uses globbing to find spec
```
Expand All @@ -62,7 +62,7 @@ yarn test-unit api_spec ## shorthand, uses globbing to find spec

```bash
yarn test <path/to/test>
yarn test test/integration/cli_spec.coffee
yarn test test/integration/cli_spec.js
## or
yarn test-integration cli_spec ## shorthand, uses globbing to find spec
```
Expand All @@ -71,22 +71,22 @@ yarn test-integration cli_spec ## shorthand, uses globbing to find spec

```bash
yarn test <path/to/test>
yarn test test/e2e/1_async_timeouts_spec.coffee
yarn test test/e2e/1_async_timeouts_spec.js
## or
yarn test-e2e 1_async ## shorthand, uses globbing to find spec
```

To keep the browser open after a spec run (for easier debugging and iterating on specs), you can pass the `--no-exit` flag to the e2e test command. Live reloading due to spec changes should also work:
```sh
yarn test test/e2e/2_go_spec.coffee --browser chrome --no-exit
yarn test test/e2e/2_go_spec.js --browser chrome --no-exit
```

### Updating snaphots

Prepend `SNAPSHOT_UPDATE=1` to any test command. See [`snap-shot-it` instructions](https://github.com/bahmutov/snap-shot-it#advanced-use) for more info.

```bash
SNAPSHOT_UPDATE=1 yarn test test/unit/api_spec.coffee
SNAPSHOT_UPDATE=1 yarn test test/integration/cli_spec.coffee
SNAPSHOT_UPDATE=1 yarn test test/unit/api_spec.js
SNAPSHOT_UPDATE=1 yarn test test/integration/cli_spec.js
SNAPSHOT_UPDATE=1 yarn test-e2e 1_async
```
6 changes: 3 additions & 3 deletions packages/server/__snapshots__/3_plugins_spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -119,14 +119,14 @@ exports['e2e plugins can modify config from plugins 1'] = `
`

exports['e2e plugins catches invalid browsers list returned from plugins 1'] = `
An invalid configuration value returned from the plugins file: \`cypress/plugins/index.coffee\`
An invalid configuration value returned from the plugins file: \`cypress/plugins/index.js\`
Expected at least one browser
`

exports['e2e plugins catches invalid browser returned from plugins 1'] = `
An invalid configuration value returned from the plugins file: \`cypress/plugins/index.coffee\`
An invalid configuration value returned from the plugins file: \`cypress/plugins/index.js\`
Found an error while validating the \`browsers\` list. Expected \`displayName\` to be a non-empty string. Instead the value was: \`{"name":"browser name","family":"chromium"}\`
Expand Down Expand Up @@ -348,7 +348,7 @@ exports['e2e plugins calls after:screenshot for cy.screenshot() and failure scre
`

exports['e2e plugins catches invalid viewportWidth returned from plugins 1'] = `
An invalid configuration value returned from the plugins file: \`cypress/plugins/index.coffee\`
An invalid configuration value returned from the plugins file: \`cypress/plugins/index.js\`
Expected \`viewportWidth\` to be a number. Instead the value was: \`"foo"\`
Expand Down
Loading

0 comments on commit 0e63263

Please sign in to comment.