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

documentation for @agoric/synpress #1103

Merged
merged 17 commits into from
Jun 4, 2024
Merged

Conversation

rabi-siddique
Copy link
Contributor

@rabi-siddique rabi-siddique commented May 22, 2024

Documenting the agoric/synpress testing framework

@rabi-siddique rabi-siddique changed the title documentation for agoric-synpress documentation for @agoric/synpress May 22, 2024
Copy link

cloudflare-workers-and-pages bot commented May 22, 2024

Deploying documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: 975de7c
Status: ✅  Deploy successful!
Preview URL: https://faba3a90.documentation-7tp.pages.dev
Branch Preview URL: https://rs-agoric-synpress-doc.documentation-7tp.pages.dev

View logs

Copy link

github-actions bot commented May 22, 2024

Cloudflare deployment logs are available here

@rabi-siddique rabi-siddique marked this pull request as ready for review May 22, 2024 08:28
@rabi-siddique rabi-siddique requested a review from toliaqat May 22, 2024 08:47
main/.vitepress/themeConfig/nav.js Outdated Show resolved Hide resolved

## Overview

[`@agoric/synpress`](https://github.com/agoric-labs/synpress) is an end-to-end (e2e) testing framework specifically designed for Decentralized Applications (DApps). Built on [Cypress.io](https://www.cypress.io/), it provides a fully-featured testing environment. Using `@agoric/synpress`, developers can automate and test interactions with the Keplr wallet.
Copy link
Contributor

Choose a reason for hiding this comment

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

Replace @agoric/synpress with Synpress to start the sentence with a capitalized word.

Copy link
Contributor

Choose a reason for hiding this comment

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

should we differentiate our project from the actual Synpress project? to avoid any branding confusion

_redirects Outdated Show resolved Hide resolved
main/.vitepress/config.mjs Outdated Show resolved Hide resolved
main/testing/end-to-end.md Outdated Show resolved Hide resolved
Copy link
Contributor

@toliaqat toliaqat left a comment

Choose a reason for hiding this comment

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

Left a few suggestions. @frazarshad @rabi-siddique, Please have a look

main/.vitepress/themeConfig/nav.js Outdated Show resolved Hide resolved
main/.vitepress/config.mjs Outdated Show resolved Hide resolved

[`@agoric/synpress`](https://github.com/agoric-labs/synpress) is an end-to-end (e2e) testing framework specifically designed for DApps. Built on [Cypress.io](https://www.cypress.io/), it provides a fully-featured testing environment. Using `@agoric/synpress`, developers can automate and test interactions with the Keplr wallet.

`@agoric/synpress` is a fork of the [`synthetixio/synpress`](https://github.com/Synthetixio/synpress) framework, which is a e2e testing framework written for the Metamask wallet. `@agoric/synpress` heavily modifies it in order to provide a robust testing framework for the Keplr wallet and Cosmos SDK.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
`@agoric/synpress` is a fork of the [`synthetixio/synpress`](https://github.com/Synthetixio/synpress) framework, which is a e2e testing framework written for the Metamask wallet. `@agoric/synpress` heavily modifies it in order to provide a robust testing framework for the Keplr wallet and Cosmos SDK.
### Background
`@agoric/synpress` is a modified fork of [`synthetixio/synpress`], which was originally designed for the Metamask wallet. These modifications make it a robust framework tailored for the Keplr wallet and Cosmos SDK.

## Getting Started

### Installation:
`@agoric/synpress` can be installed as a dev dependancy to any node project using the following command
Copy link
Contributor

Choose a reason for hiding this comment

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

No need to add multiple commands. One is enough and developers can find their relevant commands themselves.

Suggested change
`@agoric/synpress` can be installed as a dev dependancy to any node project using the following command
`@agoric/synpress` can be installed using the following command.

main/testing/end-to-end-testing.md Outdated Show resolved Hide resolved

```

3. You're done! you can now trigger the tests by running:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
3. You're done! you can now trigger the tests by running:
3. You can now trigger the tests by running this command:

Comment on lines 77 to 82
Optionally, you can create a custom config file for your synpress setup. Since `@agoric/synpress` is based on Cypress, it follows the same config file format.
`@agoric/synpress` aleardy has some
configurations set up in this
[file](https://github.com/agoric-labs/synpress/blob/master/synpress.config.js).
To override this and add your custom config, you can create your own config
file `synpress.config.js` in `/project_dir/tests/e2e`
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Optionally, you can create a custom config file for your synpress setup. Since `@agoric/synpress` is based on Cypress, it follows the same config file format.
`@agoric/synpress` aleardy has some
configurations set up in this
[file](https://github.com/agoric-labs/synpress/blob/master/synpress.config.js).
To override this and add your custom config, you can create your own config
file `synpress.config.js` in `/project_dir/tests/e2e`
You can optionally create a custom configuration file for your Synpress setup. Since `@agoric/synpress` is based on Cypress, it follows the same configuration [file](https://github.com/agoric-labs/synpress/blob/master/synpress.config.js) format. To override the default settings and add your custom configurations, create a file named synpress.config.js in /project_dir/tests/e2e with the following content:

});
```

use this config by passing the `--configFile` flag to synpress
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
use this config by passing the `--configFile` flag to synpress
Use this configuration by passing the `--configFile` flag to Synpress.


### Projects Using `@agoric/synpress`

Here are some examples of projects utilizing `@agoric/synpress` for their e2e tests, showcasing the framework in action and demonstrating its capabilities:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Here are some examples of projects utilizing `@agoric/synpress` for their e2e tests, showcasing the framework in action and demonstrating its capabilities:
Here are some examples of projects utilizing `@agoric/synpress` for their e2e tests. These examples showcase the framework in action and demonstrate its capabilities:

main/testing/end-to-end-testing.md Outdated Show resolved Hide resolved
@frazarshad frazarshad requested a review from toliaqat May 27, 2024 09:17
@rabi-siddique rabi-siddique force-pushed the rs-agoric-synpress-doc branch 10 times, most recently from 9d8a82c to c964f05 Compare June 4, 2024 15:46
Copy link
Contributor

@toliaqat toliaqat left a comment

Choose a reason for hiding this comment

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

Looks mostly good. Left couple of comments to fix.

main/e2e-testing.md Outdated Show resolved Hide resolved
main/e2e-testing.md Outdated Show resolved Hide resolved
@rabi-siddique rabi-siddique force-pushed the rs-agoric-synpress-doc branch 3 times, most recently from 641dad8 to 3de8926 Compare June 4, 2024 16:39
@rabi-siddique rabi-siddique force-pushed the rs-agoric-synpress-doc branch from 3de8926 to 147b845 Compare June 4, 2024 16:42
@rabi-siddique rabi-siddique force-pushed the rs-agoric-synpress-doc branch from 147b845 to 975de7c Compare June 4, 2024 16:44
@rabi-siddique rabi-siddique merged commit c75deff into main Jun 4, 2024
5 checks passed
@rabi-siddique rabi-siddique deleted the rs-agoric-synpress-doc branch June 4, 2024 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants