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

docs: updating readme with guides for how to generate tests #116

Merged
merged 5 commits into from
Sep 18, 2024
Merged

Conversation

lenareed
Copy link
Contributor

Guides for how to iterate on prompt and how to write assertions

Also included explicit instructions on how to open the extension.

Description

@lenareed lenareed requested a review from a team as a code owner September 18, 2024 00:05
README.md Outdated

## How Do Assertions work?

Once you've started generating tests, you might want to more quickly determine if your test succeeded or failed, this is where the _Assert Inputs_ fields come in. Assert Inputs are descriptions of screens which are used to evaluate the test generated. When assertions are provided, the LLM will determine if the assertion matches what is on the screen for each assertion provided against every screen visited. These assertions can describe the entire screen (e.g. login screen) or describe an element on the screen (e.g. page with checkout button). These assertions are _not_ used to generate test steps, any intent needed to drive the test generation must be included in the _Test Goal_ field. Below you can see examples of assertions and an example output.

Choose a reason for hiding this comment

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

This is a single line with 758 characters 😅 Would it be possible to break it down into 80 character chunks? Last time I checked with the team, that seemed like that preferred line length, especially when you try to place windows side by side. Easier to view them inside editors that don't have word-wrap enabled. Also easier to review any changes made in the future!

PS: This is true for some of the other lines above as well 🙏

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes! I can do that!

lenareed and others added 4 commits September 18, 2024 11:11
Co-authored-by: Alex Plischke <alex.plischke@saucelabs.com>
Co-authored-by: Alex Plischke <alex.plischke@saucelabs.com>
Co-authored-by: Alex Plischke <alex.plischke@saucelabs.com>
@lenareed lenareed merged commit 81a2a88 into main Sep 18, 2024
1 check passed
@lenareed lenareed deleted the DEVX-3028 branch September 18, 2024 23: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.

2 participants