Jeff King <peff@xxxxxxxx> writes: > If we do, I think it would be OK to just drop the test_must_be_empty > line. Now that we are capturing only stdout in "output", checking that > the usage message is there is probably sufficient. Yes. > In the meantime, I think I prefer keeping the slightly more strict form > above. I am also in favor of keeping it, if only to learn why we do not want the overly strict check when the time comes. That would mean we would be loosening with a concrete reason why we would want to loose it at that time, instead of guessing that it might be overly strict without seeing the fallout before seeing any. Thanks.
