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

Improve development: fake #194

Merged
merged 2 commits into from
Sep 30, 2019
Merged

Improve development: fake #194

merged 2 commits into from
Sep 30, 2019

Conversation

extsoft
Copy link
Contributor

@extsoft extsoft commented Sep 30, 2019

The proposed changes appertain to #155.

The contribution:

  • updates all affected documentation
  • provides commits messages which comply with the CONTRIBUTING.md > Committing the changes rules
  • updates the completion scripts if requires
  • complies with all requirements from README.md > Hands-on development notes

The change simplifies the analysis of faild tests as the first log lines
always will be a fake executable creation, and other lines will be about
creation of fake commands.
All `fake` functions have a new usage format now:
- `fake <full command> <exit> <stdout> <stderr>`
- `fake-pass <full command> <stdout>`
- `fake-fail <full command> <stderr>`

Also, clean up function is now available as `fake-clean`.

#155
@extsoft extsoft merged commit 4b8e27b into master Sep 30, 2019
@extsoft extsoft deleted the 155 branch September 30, 2019 10:57
extsoft added a commit that referenced this pull request Sep 30, 2019
We have to keep `_` and `-` symbols when fake commands as they will
provide better differentiation between commands or their arguments.

#194
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.

None yet

1 participant