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

[GSOC][RFC] microproject: use test_path_is_* functions in test scripts

From
Vincenzo MEZZELA <vincenzo.mezzela@gmail.com>
Date
Feb 13, 2024, 15:44 UTC
Message-ID
<cbb450b3-b333-4391-ac83-66c2daf0ae4d@gmail.com>
Hello everyone,

I'm Vincenzo, a Master's degree student in Computer Engineering and Cybersecurity.

As I'm approaching the end of my academic journey, I'm eager to contribute to the Git project and the GSoC represents a good opportunity to do so. Upon exploring the online documentation of git for the application to the GSoC I'm keen to begin the required microproject.

Among the microproject proposed here https://git.github.io/SoC-2024-Microprojects/ , I would like to work on replacing 'test -(e|f|g|...)' with test_path_is* .

Can you confirm if this task has already been taken by someone else?
Approach:

As far as I understood, The work consists in replacing the shell 'test' command in the test script under 't/' directory with the ones present in the t/test_lib_functions.sh as follows:

- test -f --> test_path_is_file
- test -d --> test_path_is_dir
- test -e --> test_path_exists

To approximately measure the number of required replacement, I run the following commands from the 't/' directory (branch master):

> # Files  that requires a replacement
 > git grep -r 'test -[efdx]' 2>/dev/null| awk '{print $1}' | uniq -c | 
sort -n -r
Show 11 quoted lines
>>    190 t7301-clean-interactive.sh:
>>     147 t7300-clean.sh:
>>      21 t2004-checkout-cache-temp.sh:
>>      17 t2401-worktree-prune.sh:
>>      16 t2003-checkout-cache-mkdir.sh:
>>      14 t0601-reffiles-pack-refs.sh:
>>      13 t4200-rerere.sh:
>>      12 t9146-git-svn-empty-dirs.sh:
>>      12 t7603-merge-reduce-heads.sh:
>>      12 lib-submodule-update.sh:
>>    ...
 >>
 > # Number of replacements
 > git grep -r 'test -[efdx]' 2>/dev/null| awk '{print $1}' | uniq -c | 
sort -n -r | awk '{sum += $1} END {print sum}'
 >> 853
 > # Number of files that requires a patch
 > git grep -r 'test -[efdx]' 2>/dev/null| awk '{print $1}' | uniq -c | 
wc -l
 >> 169

Although the replacement work might not be difficult, it spans over many different test files. Do you want me to submit a patch for each of them as part of the microproject? If not, How many patches do you expect me to submit?

Thanks, Vincenzo

Next: Christian Couder
Message 1 of 5 in “[GSOC][RFC] microproject: use test_path_is_* functions in test scripts”
  1. Vincenzo MEZZELAFeb 13, 2024
  2. Christian CouderFeb 13, 2024
  3. Eric SunshineFeb 13, 2024
  4. Junio C HamanoFeb 13, 2024
  5. vincenzo mezzelaFeb 14, 2024

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.