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

Prevent tests running multiple times #98

Merged
merged 2 commits into from Jan 6, 2014
Merged

Prevent tests running multiple times #98

merged 2 commits into from Jan 6, 2014

Conversation

mgarriott
Copy link
Contributor

When saving files with Vim on linux, tests are being run twice.

As seen in several Guard plugins, run_on_changes is running multiple times when saving files with Vim. This can be easily remedied by using run_on_modifications instead.

See:

guard/guard#297
guard/guard#495

The run_on_additions method is also being called when saving files with Vim, causing multiple invocations of the test(s). Because run_on_modifications is executed when a new file is created anyway, it should be unnecessary to run the tests explicitly from run_on_additions.

Adding a file triggers run_on_modifications which runs the tests. It is
therefore unnecessary to run the tests from run_on_additions.
rymai added a commit that referenced this pull request Jan 6, 2014
…times

Prevent tests running multiple times
@rymai rymai merged commit 513d28f into guard:master Jan 6, 2014
@rymai
Copy link
Member

rymai commented Jan 6, 2014

Thank you!

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

2 participants