git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 0/4] tests: make more use of 'test_must_be_empty'

From
Matthew DeVore <matvore@google.com>
Date
Aug 22, 2018, 17:52 UTC
Message-ID
<20180822175201.42406-1-matvore@google.com>
In-Reply-To
<20180819215725.29001-1-szeder.dev@gmail.com>

Note that you can also trivially convert a lot more instances of empty checking to the "right" way: 'test_line_count = 0'

$ grep -Ir 'test_line_count = 0' t | wc -l 76

I think it would be nice to clean these up as well.
Previous: SZEDER Gábor
Message 6 of 6 in “tests: make more use of 'test_must_be_empty'”
  1. 0/4 tests: make more use of 'test_must_be_empty'SZEDER Gábor, Aug 19, 2018
  2. 2/4 tests: use 'test_must_be_empty' instead of 'test ! -s'SZEDER Gábor, Aug 19, 2018
  3. 1/4 tests: use 'test_must_be_empty' instead of '! test -s'SZEDER Gábor, Aug 19, 2018
  4. 3/4 tests: use 'test_must_be_empty' instead of 'test_cmp /dev/null <out>'SZEDER Gábor, Aug 19, 2018
  5. 4/4 tests: use 'test_must_be_empty' instead of 'test_cmp <empty> <out>'SZEDER Gábor, Aug 19, 2018
  6. Matthew DeVoreAug 22, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.