Skip to content

Latest commit

 

History

History
510 lines (333 loc) · 31.2 KB

CONTRIBUTING.md

File metadata and controls

510 lines (333 loc) · 31.2 KB
Raw

Contributing to Cypress

Thanks for taking the time to contribute! 😄

Once you learn how to use Cypress, you can contribute in many ways:

  • Join the Cypress Gitter chat and answer questions. Teaching others how to use Cypress is a great way to learn more about how it works.
  • Blog about Cypress. We display blogs featuring Cypress on our Examples page. If you'd like your blog featured, open a PR to add it to our docs.
  • Write some documentation or improve our existing docs. Know another language? You can help us translate them. See our guide to contributing to our docs.
  • Give a talk about Cypress. Contact us ahead of time and we'll send you some swag. 👕

Want to dive deeper into how Cypress works? There are several ways you can help with the development of Cypress:

Table of Contents

CI status

Build status Description
CircleCI cypress-test-node-versions
CircleCI cypress-test-ci-environments
CircleCI cypress-test-module-api
CircleCI cypress-test-nested-projects
CircleCI cypress-on
CircleCI cypress-test-example-repos
CircleCI docsearch-scraper
Docker Build Status cypress-docker-images
Build status Windows CI

Code of Conduct

All contributors are expecting to abide by our Code of Conduct.

Opening Issues

The most important things to do are:

Finally, if you are up to date, supported, have collected information about the problem, and have the best reproduction instructions you can give, you are ready to open an issue.

Update Cypress

Before filing a bug, make sure you are up to date. Your issue may have already been fixed. Even if you do not see the issue described as resolved in a newer version, a newer version may help in the process of debugging your issue by giving more helpful error messages.

See our document on installing cypress

Getting more information

For some issues, there are places you can check for more information. This may help you resolve the issue yourself. Even if it does not, this information can help us figure out and resolve an issue.

  • For issues in the web browser, check the JavaScript console and your Network tab in your DevTools.
  • Click on any command in the Command Log where the failure occurred, this will log more information about the error to the JavaScript console.
  • Use Cypress debug or pause commands to step through your commands.
  • Ask other Cypress users for help in our chat.
  • Try more advanced troubleshooting from troubleshooting Cypress doc.

Fill out our Issue Template

When opening an issue, there is a provided issue template. Fill out the information according to the template. This is information needed for Cypress to continue forward with your problem. Any issues that do not follow the issue template will be closed.

Describe Problems

When you file a feature request or bug, we need you to describe the problem you are facing first, not just your desired solution.

Often, your problem may have a lot in common with other similar problems. If we understand your use case, we can compare it to other use cases and sometimes find a more powerful or more general solution which solves several problems at once. Understanding the root issue can let us merge and contextualize things. Sometimes there's already a way to solve your problem that might just not be obvious.

Also, your proposed solution may not be compatible with the direction we want to take the product, but we may be able to come up with another solution which has approximately the same effect and does fit into the product direction.

Reproducibility

It is nearly impossible for us to resolve many issues if we can not reproduce them. Your best chance of getting a bug looked at quickly is to provide a repository with a reproducible bug that can be cloned and run.

Triaging Issues

When an issue is opened in cypress, we need to evaluate the issue to determine what steps should be taken next. So, when approaching new issues, there are some steps that should be taken.

Is this a question?

Some opened issue are questions, not bug reports or feature requests. Issues are reserved for potential bugs or feature requests only. If this is the case, you should:

  • Explain that issues in our GitHub repo are reserved for potential bugs or feature requests and that the issue will be closed since it appears to be neither a bug nor a feature request.
  • Guide them to existing resources where their questions can be asked like our community chat, our documentation, or Stack Overflow.
  • Cypress offers support via email when signing up for any of our our paid plans, so remind them that this is an option. Cypress also offers screen sharing and workshops with our premium support options if they would like something higher-touch.
  • Add the type: question label to the issue.
  • Close the issue.

Does this issue belong in this repository?

Other open source repos

Issues may be opened about wanting changes to our documentation, our example-kitchensink app, or another repository. In this case you should:

  • Thank them for their contribution.
  • Explain that this repo is only for bugs or feature requests of the Cypress product.
  • If you have permission to 'Transfer the issue', do so. If not, explain that they can open an issue in our other repository and link to the repository.
  • Close the issue (if not already transferred).

Our Dashboard Service

Issues may be opened about wanting features in our Dashboard Service. In this case you should:

  • Thank them for expressing interest in a new feature.
  • Refer them to the Dashboard ProductBoard: "You can express interest and see progress for this feature on our Roadmap from our Dashboard's product board here: https://portal.productboard.com/cypress-io/1-cypress-dashboard All related work for the Dashboard features is handled in that ProductBoard and will be handled by the Dashboard team directly when you comment there."
  • Close the issue
  • Close the issue to comments

Is this already an open issue?

Search all issues for keywords from the issue to ensure there isn't already an issue open for this. GitHub has some search tips that may help you better find the relevant issue.

If an issue already exists you should:

  • Thank them for their contribution.
  • Explain that this issue if a duplicate of another issue, linking to the relevant issue (#1234).
  • Add the type: duplicate label to the issue.
  • Close the issue.

Does the issue provide all the information from our issue template?

When opening an issue, there is a provided issue template. If the opened issue does not provide enough information asked from the issue template you should:

  • Explain that we require new issues follow our provided issue template and that issues that are opened without this information are automatically closed per our contributing guidelines.
  • Close the issue.

Are they running the current version of Cypress?

If they listed an older version of Cypress in their issue. We don't want to spend the time to set up a reproducible project (which can be time consuming) only to find that bumping the Cypress version fixes it. You should:

  • Ask them to update to the newest version of Cypress and comment about the results.
  • Add the stage: awaiting response label to the issue.

Is the fix or feature within our vision for Cypress?

There will inevitably be suggestions that will not fit within the scope of Cypress's vision for our product. If an issue or pull request falls under this category you should:

  • Thank them for their contribution.
  • Explain why it doesn’t fit into the scope of Cypress, and offer clear suggestions for improvement, if you’re able. Be kind, but firm.
  • Link to relevant documentation, if there is any. If you notice repeated requests for things you don’t want to accept, add them into the documentation to avoid repeating yourself.
  • Add the stage: wontfix label to the issue.
  • Close the issue/pull request

Is what they are describing actually happening?

The best way to determine the validity of a bug is to recreate it yourself. Follow the directions or information provided to recreate the bug that is described. Did they provide a repository that demonstrates the bug? Great - fork it and run the project and steps required. If they did not provide a repository, the best way to reproduce the issue is to have a 'sandbox' project up and running locally for Cypress. This is just a simple project with Cypress installed where you can freely edit the application under test and the tests themselves to recreate the problem.

Attempting to recreate the bug will lead to a few scenarios:

1. You can not recreate the bug

If you cannot recreate the situation happening you should:

  • Thank them for their contribution.
  • Explain that there is not enough information to reproduce the bug. Provide information on how you went about recreating the scenario, if you’re able. Note your OS, Browser, Cypress version and any other information.
  • Link them to our contributing guideline for opening issues.
  • Note that if no reproducible example is provided, we will unfortunately have to close the issue.
  • Add the stage: needs information label to the issue.

2. You can recreate the bug

If you can recreate the bug you should:

  • Thank them for their contribution.
  • Explain that you were able to recreate the bug. Provide the exact test code ran and the versions of Cypress, OS, and browser you used to recreate it.
  • If you know where the code is that could possibly fix this issue - link to the file or line of code from the cypress repo and remind the user that we are open source and that we gladly accept PRs, even if they are a work in progress.
  • Add the stage: ready for work label to the issue.

3. You can tell the problem is a user error

In recreating the issue, you may realize that they had a typo or used the Cypress API incorrectly, etc. In this case you should:

  • Leave a comment informing the user of their error.
  • Link to relevant documentation, if there is any. If you notice repeated user errors for the same situation, add them into the documentation to avoid repeating yourself.
  • Close the issue.

Has the issue gone stale?

Some issues are opened and sadly forgotten about by the person originally opening the issue.

Not enough information ever provided

Sometimes we request more information to be provided (label stage: needs information) for an open issue, but no one is able to provide a reproducible example or they simply never respond. This does not mean that we don't believe that there is a bug! We just, unfortunately, do not have a path forward to fix it without this information. In this case you should:

  • Add a comment reminding them or our request for more information and that the issue will be closed if it is not provided. Sometimes issues get forgotten about, and all the person needs is a gentle reminder.
  • If there is still no response after a weeks time, explain that you are closing the issue due to not enough information or inactivity and that they can comment in the issue with a reproducible example and we will reopen the issue.
  • Close the issue.

They already solved their issue

Some issues are resolved by the community, by giving some guidance or a workaround, but the original opener of the issue forgets to close the issue. In this case you should:

  • Explain that you are closing the issue as resolved and that they can comment if they are still having the issue and we will consider reopening it.
  • Close the issue.

Writing Documentation

Cypress documentation lives in a separate repository with its own dependencies and build tools. See Documentation Contributing Guideline.

Writing code

Working on your first Pull Request? You can learn how from this free series How to Contribute to an Open Source Project on GitHub

What you need to know before getting started

Cypress and Packages

Cypress is a large open source project. When you want to contribute to Cypress, you may be unsure which part of the project to work within.

This repository is made up of various packages. They are discrete modules with different responsibilities, but each is necessary for the Cypress app and is not necessarily useful outside of the Cypress app.

Here is a list of the core packages in this repository with a short description, located within the packages directory:

Folder Name Purpose
cli The command-line tool that is packaged as an npm module.
coffee A centralized version of CoffeeScript used for other packages.
desktop-gui The front-end code for the Cypress Desktop GUI.
driver The code that is used to drive the behavior of the API commands.
electron The Cypress implementation of Electron.
example Our example kitchen-sink application.
extension The Cypress Chrome browser extension
https-proxy This does https proxy for handling http certs and traffic.
launcher Finds and launches browsers installed on your system.
reporter The reporter shows the running results of the tests (The Command Log UI).
root Dummy package pointing at the root of the repository.
runner The runner is the minimal "chrome" around the user's application under test.
server The <3 of Cypress. This orchestrates everything. The backend node process.
socket A wrapper around socket.io to provide common libraries.
static Serves static assets used in the Cypress GUI.
ts A centralized version of typescript.

We try to tag all issues with a pkg/ tag describing the appropriate package the work is required in. For example, the pkg/driver label is tagged on issues that require work in the driver package.

Requirements

You must have node and npm installed to run the project. We use avn, a utility to switch to the right npm version, in each folder. Currently, Cypress should be developed using the version specified in root .node-version file.

Getting Started

Install all dependencies:

npm install

This will install this repo's direct dependencies as well as the dependencies for every individual package.

Then, build all the packages and start the app:

npm run build
npm start

If there are errors building the packages, prefix the commands with DEBUG=cypress:* to see more details.

This outputs a lot of debugging lines. To focus on an individual module, run with DEBUG=cypress:launcher for instance.

When running npm start this routes through the CLI and eventually calls npm run dev with the proper arguments. This enables Cypress day-to-day development to match the logic of the built binary + CLI integration.

If you want to bypass the CLI entirely, you can use the npm run dev task and pass arguments directly. For example to headlessly run a project in a given folder, while trying to record to the Dashboard

npm run dev -- --run-project /project/folder --record --key <key>

Tasks

Each package is responsible for building itself and testing itself and can do so using whatever tools are appropriate, but each conforms to a standard set of npm scripts so that building, watching, testing, etc. can be orchestrated from the root of this repo. Here are the npm scripts supported and what they mean:

Task Purpose
build Build the package
build-prod Build all assets for production (if makes sense)
start Run a server for serving files
watch Watch source files and build development assets when they are saved. This may also run a server for serving files and run tests related to a saved file.
clean Remove any assets created by build-dev or build-prod
clean-deps Remove any dependencies installed (usually by npm)
test Runs all tests once
test-watch Run all tests in watch mode

Not every package requires or makes use of every script, so it is simply omitted from that package's package.json and not run.

You can run npm run all <script name> from the root directory to run a script in every package that utilizes that script. Many times, you may only be working on one or two packages at a time, so it won't be necessary or desirable to run a script for every package. You can use the --packages option to specify in which package(s) to run the script.

You can even run npm run all install to install all npm dependencies for each package. Note that this is already done automatically for you when you run npm install.

npm run all watch-dev -- --packages core-desktop-gui

Separate the package names with commas to specify multiple packages:

npm run all watch-dev -- --packages core-desktop-gui,core-runner

By default, all tasks run in parallel. This is faster than running serially, but the output ends up mixed together and, if things go wrong, it can be difficult see where the error occurred. To run tasks serially, use the --serial flag:

npm run all build-prod -- --serial

build-prod will be run sequentially for every package, so the output for each package won't be jumbled with the output of the others.

It is not recommended to use --serial with any script that is long-running, like watch-dev or test, since they need to be parallel.

Debugging

Some packages use debug to log debug messages to the console. The naming scheme should be cypress:<package name>. For example to see launcher messages during unit tests start it using

cd packages/launcher
DEBUG=cypress:launcher npm test

If you want to see log messages from all Cypress projects use wild card

DEBUG=cypress:*

Or for an individual package:

DEBUG=cypress:cli
DEBUG=cypress:server
DEBUG=cypress:launcher

Coding Style

We use eslint to lint all JavaScript code and follow rules specified in @cypress/eslint-plugin-dev plugin.

When you edit files, you can quickly fix all changed files before you commit using

npm run lint-changed-fix

When committing files, we run a Git pre-commit hook to lint the staged JS files. See the lint-staged project. If this command fails, you may need to run npm run lint-changed-fix and commit those changes.

We DO NOT use Prettier to format code. You can find .prettierignore file that ignores all files in this repository. To ensure this file is loaded, please always open the root repository folder in your text editor, otherwise your code formatter might execute, reformatting lots of source files.

Tests

For most packages there are typically unit and some integration tests.

Our true e2e tests are in packages/server, which test the full stack all together.

Please refer to each packages' README.md which documents how to run tests. It is not feasible to try to run all of the tests together. We run our entire test fleet across over a dozen containers in CI.

If you're curious how we manage all of these tests in CI check out our circle.yml file found in the root cypress directory.

Docker

Sometimes tests pass locally, but fail in CI. Our CI environment is dockerized. In order to run the image used in CI locally:

  1. Install Docker and get it running on your machine.
  2. Run the following command from the root of the project:
npm run docker

There is a script scripts/run-docker-local.sh that runs the cypress image (see circle.yml for the current image name).

The image will start and will map the root of the repository to /cypress inside the image. Now you can modify the files using your favorite environment and rerun tests inside the docker environment.

Troubleshooting

Sometimes building inside the image has problems with node-sass library.

Error: Missing binding /cypress/packages/desktop-gui/node_modules/node-sass/vendor/linux-x64-48/binding.node
Node Sass could not find a binding for your current environment: Linux 64-bit with Node.js 6.x

Found bindings for the following environments:
  - OS X 64-bit with Node.js 6.x

This usually happens because your environment has changed since running `npm install`.
Run `npm rebuild node-sass` to build the binding for your current environment.

From the running container, go into that project and rebuild node-sass.

$ npm run docker
cd packages/desktop-gui
npm rebuild node-sass

Docker for built binary

You can also use Docker to simulate and debug the built binary. In a temporary folder (for example from the folder /tmp/test-folder/) start a Docker image:

$ docker run -it -w /app -v $PWD:/app cypress/base:8 /bin/bash

Point the installation at a specific binary and npm (if needed) and set local cache folder to unzip the downloaded binary into a subfolder.

$ export CYPRESS_INSTALL_BINARY=https://cdn.cypress.io/beta/.../cypress.zip
$ export CYPRESS_CACHE_FOLDER=./cypress-cache
$ npm i https://cdn.cypress.io/beta/npm/.../cypress.tgz

Note that unzipping the Linux binary inside a Docker container onto a mapped volume drive is slow. But once this is done you can modify the application resource folder in the local folder /tmp/test-folder/node_modules/cypress/cypress-cache/3.3.0/Cypress/resources/app to debug issues.

Packages

Generally when making contributions, you are typically making them to a small number of packages. Most of your local development work will be inside a single package at a time.

Each package documents how to best work with it, so consult the README.md of each package.

They will outline development and test procedures. When in doubt just look at the scripts of each package.json file. Everything we do at Cypress is contained there.

Committing Code

Branches

The repository is setup with two main (protected) branches.

  • master is the code already published in the last Cypress version.
  • develop is the current latest "pre-release" code. This branch is set as the default branch, and all pull requests should be made against this branch.

Pull Requests

  • When opening a PR for a specific issue already open, please name the branch you are working on using the convention issue-[issue number]. For example, if your PR fixes Issue #803, name your branch issue-803. If the PR is a larger issue, you can add more context like issue-803-new-scrollable-area If there is not an associated open issue, create an issue using our Issue Template.
  • PR's can be opened before all the work is finished. In fact we encourage this! Please write [WIP] in the title of your Pull Request if your PR is not ready for review - someone will review your PR as soon as the [WIP] is removed.
  • Fill out the Pull Request Tempalte completely within the body of the PR. If you feel some areas are not relevant add N/A as opposed to deleteing those sections. PR's will not be reviewed if this template is not filled in.
  • Please check the "Allow edits from maintainers" checkbox when submitting your PR. This will make it easier for the maintainers to make minor adjustments, to help with tests or any other changes we may need. Allow edits from maintainers checkbox

Pull Request Reviews

After a PR has been opened, our cypress-bot will comment on the PR detailing the guidelines to be used to review Pull Requests. Please read these guidelines carefully and make any updates where you see the PR may not be meeting the quality of these guidelines.

Some rules about Pull Requests Reviews:

  1. The contributor opening the pull request may not approve their own PR.
  2. The PR will not be merged if some reviewers have voted "Needs changes".

If any of the Pull Request Review guidelines can't be met, a comment will be left by the reviewer with 'Request changes'. Please make any updates as appropriate and we will rereview once those changes are addressed.

During a Pull Request Review, the following should be done:

  • Run the code and use it as the end user would. Double check issue and PR description to ensure it is meeting requirements.
  • Read through every line of changed code (Yes, we know this could be a LOT).
  • If you don’t understand why some piece of code is required, ask for clarification! Likely the contributor had a reason and can provide the answer quicker than investigating yourself.

Testing

This repository is exhaustively tested by CircleCI. Additionally we test the code by running it against various other example projects. See CI badges and links at the top of this document.

To run local tests, consult the README.md of each package.

Dependencies

We use RenovateBot to automatically upgrade our dependencies. The bot uses the settings in renovate.json to maintain our Update Dependencies issue and open PRs. You can manually select a package to open a PR from our Update Dependencies issue.

After a PR has been opened for a dependency update, our cypress-bot will comment on the PR detailing the guidelines to be used to review the dependency update. Please read these guidelines carefully and make any updates where you see the PR may not be meeting the quality of these guidelines.

Deployment

We will try to review and merge pull requests quickly. After merging we will try releasing a new version. If you want to know our build process or build your own Cypress binary, read DEPLOY.md

Known problems

ENFILE or EMFILE

If you get ENFILE: file table overflow, ENFILE: too many open files or any other ENFILE or EMFILE errors on Mac, that means you are doing synchronous file system operations. Cypress should NEVER do them. Instead we should use async file system operations and let graceful-fs retry them. Find the place where the synchronous fs operation is done from the stacktrace and make it async.