seattlerb/zentest — pull request #15 by bhenderson (2012-03-03 18:46:21+00:00)
Title: Changes when run_command hook is run and also passes it cmd.

the run_tests method has this code:

hook :run_command

new_mtime = self.find_files_to_test
return unless new_mtime
self.last_mtime = new_mtime

cmd = self.make_test_cmd self.files_to_test
return if cmd.empty?

If I have for example the autotest/restart plugin activated and I edit my .autotest file, the run_command hook will fire, but cmd.empty? will return true.

Use case for changing this is autotest/growl which will clear the screen on hook run_command, but I don't want it to clear the screen if a new command doesn't run.

I think it can be remedied by just putting the hook below that last line, but I'm not sure how it will affect other plugins. Passing it cmd is cool, but I know that will definitely affect lots of plugins so I'm cool if that doesn't get pulled in.

If I'm way off just let me know and I can deal.

thanks,