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

Testing strategy for git phlow #229

Closed
Sharor opened this issue Sep 5, 2017 · 4 comments
Closed

Testing strategy for git phlow #229

Sharor opened this issue Sep 5, 2017 · 4 comments

Comments

@Sharor
Copy link
Contributor

Sharor commented Sep 5, 2017

This issue stems from #228 and Praqma/Praqmatic-Automated-Changelog#100

Git phlow in its entirety is a command line tool, and as such has its API directly exposed to its users. For any given project, the testing pyramid is a good indicator of where to put testing efforts.

image

For Git Phlow, this means more concretely that we need to figure out how that fits together with a commandline driven tool.
The manual testing of the image, is of course exploratory testing which should only ever be done to write more tests in a Praqma scenario.

Phrasing a testing strategy for golang, and figuring out how to consistently execute on the various underlying OS is relevant.
A key effort here should also be to evaluate what are "happy paths" that are worth testing on higher levels, as these tests are expensive to maintain and write.

It's also worth to consider if system tests are relevant for 'ghi' as this is basically the only part returning GUI.

@Sharor Sharor added this to the Backlog milestone Sep 5, 2017
@Sharor Sharor self-assigned this Sep 5, 2017
@buep buep modified the milestones: Workable, Backlog Sep 5, 2017
@buep
Copy link
Contributor

buep commented Sep 6, 2017

I'm expecting a commit now soon, since you're supposed to deliver something within time and time is out in 25 minutes :-)

@buep
Copy link
Contributor

buep commented Sep 6, 2017

Or a google document... anything :-)

Sharor added a commit that referenced this issue Sep 6, 2017
@Sharor
Copy link
Contributor Author

Sharor commented Sep 6, 2017

Lunch is a thing 😛

@Sharor
Copy link
Contributor Author

Sharor commented Sep 11, 2017

This has been integrated with master, and is considered closed post discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants