Skip to content

feat: add ability to use assertions#17

Merged
mayt merged 6 commits intomayt:masterfrom
BaseInfinity:assertions--add-ability-to-assert-in-natural-language
Dec 17, 2023
Merged

feat: add ability to use assertions#17
mayt merged 6 commits intomayt:masterfrom
BaseInfinity:assertions--add-ability-to-assert-in-natural-language

Conversation

@BaseInfinity
Copy link
Copy Markdown
Contributor

@BaseInfinity BaseInfinity commented Dec 15, 2023

I have a pull request open that outputs the generated code, I think what would make this useful in the testing space if it had the ability to assert things as well.

Let me know if this is starting to deviate too much from your original intentions.

I forked with intentions of making this an E2E testing tool for Playwright, whereas your intention seems to be more general purpose and you may not want all of my changes so feel free to reject changes that don't match eye to eye with your goals =)

@BaseInfinity BaseInfinity force-pushed the assertions--add-ability-to-assert-in-natural-language branch from e025fba to be3126f Compare December 15, 2023 00:26
@BaseInfinity
Copy link
Copy Markdown
Contributor Author

proof if it working =)

Screen Shot 2023-12-14 at 4 27 02 PM

@BaseInfinity
Copy link
Copy Markdown
Contributor Author

Thanks for the CR! I'll most likely get to it later tonight or tomorrow morning =)

@BaseInfinity
Copy link
Copy Markdown
Contributor Author

Thanks for the CR, this is ready for review again

@BaseInfinity
Copy link
Copy Markdown
Contributor Author

This is ready for more CR =)

@mayt mayt merged commit d7400a6 into mayt:master Dec 17, 2023
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