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

flow test configuration and default behaviour #1378

Closed
Tracked by #1390
wfalcon0x opened this issue Feb 3, 2024 · 1 comment · Fixed by #1550
Closed
Tracked by #1390

flow test configuration and default behaviour #1378

wfalcon0x opened this issue Feb 3, 2024 · 1 comment · Fixed by #1550
Assignees
Labels
Feature A new user feature or a new package API Feedback

Comments

@wfalcon0x
Copy link

wfalcon0x commented Feb 3, 2024

Instructions

Issue To Be Solved

Currently we require to execute a command line command with many parameters each time we want to run cadence tests with flow cli, e.g. flow test --cover --covercode="contracts" cadence/test/*

(Optional): Suggest A Solution

To increase developer efficiency while working with flow test I suggest to be able to run flow test with sensible defaults without parameters, and provide with an option to define a test configuration file, where defaults can be overridden.

The expected developer experience is to by running flow test tests are executed with sensible behaviour as long as they are placed in the directory defined by conventions the community agrees on.
Developers should also be able to configure in a test configuration file any custom location and other configurations if their project has different requirements.

Sensible conventions should be determined for each parameter, e.g. in the above example flow test could execute tests in the cadence/tests/* directory by default.

(Optional): Context

Testing frameworks in other language ecosystems (e.g. jest) adopted philosophy to make the tool work great by default, but give a way to configure if developers prefer to use it in a different way. This can positively effect developer experience.

@chasefleming
Copy link
Member

This seems like a great idea @wfalcon0x. I think we can work in during some upcoming changes. Thanks for creating this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature A new user feature or a new package API Feedback
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants