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

[PATCH 07/11] Add more tests for git hash-object

From
Adam Roben <aroben@apple.com>
Date
Apr 23, 2008, 19:17 UTC
Message-ID
<1208978273-98146-8-git-send-email-aroben@apple.com>
In-Reply-To
<1208978273-98146-7-git-send-email-aroben@apple.com>
Signed-off-by: Adam Roben <aroben@apple.com>
---
Junio C Hamano <gitster@pobox.com> wrote:
Show 24 quoted lines
> > +test_expect_success \
> > +    'hash a file' \
> > +    "test $hello_sha1 = \$(git hash-object hello)"
> > +
> > +test_expect_success \
> > +    'hash from stdin' \
> > +    "test $hello_sha1 = \$(echo '$hello_content' | git hash-object --stdin)"
> 
> Needs to make sure no object has been written to the object
> database at this point?
> 
> > +test_expect_success \
> > +    'hash a file and write to database' \
> > +    "test $hello_sha1 = \$(git hash-object -w hello)"
> 
> ... and make sure the objectis written here?
> 
> > +test_expect_success \
> > +    'hash from stdin and write to database' \
> > +    "test $hello_sha1 = \$(echo '$hello_content' | git hash-object -w --stdin)"
> > +
> > +test_done
> 
> ... and/or here?
Fixed.
 t/t1007-hash-object.sh |  120 ++++++++++++++++++++++++++++++++++++++----------
 1 files changed, 96 insertions(+), 24 deletions(-)
diff --git a/t/t1007-hash-object.sh b/t/t1007-hash-object.sh
index 543c078..eb54f1f 100755
--- a/t/t1007-hash-object.sh
+++ b/t/t1007-hash-object.sh
@@ -4,32 +4,104 @@ test_description=git-hash-object
 
 . ./test-lib.sh
 
-test_expect_success \
-    'git hash-object -w --stdin saves the object' \
-    'obname=$(echo foo | git hash-object -w --stdin) &&
-    obpath=$(echo $obname | sed -e "s/\(..\)/\1\//") &&
-    test -r .git/objects/"$obpath" &&
-    rm -f .git/objects/"$obpath"'
-    
-test_expect_success \
-    'git hash-object --stdin -w saves the object' \
-    'obname=$(echo foo | git hash-object --stdin -w) &&
-    obpath=$(echo $obname | sed -e "s/\(..\)/\1\//") &&
-    test -r .git/objects/"$obpath" &&
-    rm -f .git/objects/"$obpath"'    
+function echo_without_newline()
+{
+    echo "$@\c"
+}
 
-test_expect_success \
-    'git hash-object --stdin file1 <file0 first operates on file0, then file1' \
-    'echo foo > file1 &&
-    obname0=$(echo bar | git hash-object --stdin) &&
-    obname1=$(git hash-object file1) &&
-    obname0new=$(echo bar | git hash-object --stdin file1 | sed -n -e 1p) &&
-    obname1new=$(echo bar | git hash-object --stdin file1 | sed -n -e 2p) &&
-    test "$obname0" = "$obname0new" &&
-    test "$obname1" = "$obname1new"'
+function test_blob_does_not_exist()
+{
+    test_expect_success \
+        "blob does not exist in database" \
+        "test_must_fail git cat-file blob $1"
+}
+
+function test_blob_exists()
+{
+    test_expect_success \
+        "blob exists in database" \
+        "git cat-file blob $1"
+}
+
+hello_content="Hello World"
+hello_sha1=5e1c309dae7f45e0f39b1bf3ac3cd9db12e7d689
+
+example_content="This is an example"
+example_sha1=ddd3f836d3e3fbb7ae289aa9ae83536f76956399
+
+function setup_repo()
+{
+    echo_without_newline "$hello_content" > hello
+    echo_without_newline "$example_content" > example
+}
+
+test_repo=test
+function push_repo()
+{
+    test_create_repo $test_repo
+    cd $test_repo
+
+    setup_repo
+}
+
+function pop_repo()
+{
+    cd ..
+    rm -rf $test_repo
+}
+
+setup_repo
+
+# Argument checking
 
 test_expect_success \
-    'git hash-object refuses multiple --stdin arguments' \
-    '! git hash-object --stdin --stdin < file1'
+    "multiple '--stdin's are rejected" \
+    "test_must_fail git hash-object --stdin --stdin < example"
+
+# Behavior
+
+push_repo
+
+    test_expect_success \
+        "hash a file" \
+        "test $hello_sha1 = \$(git hash-object hello)"
+
+    test_blob_does_not_exist $hello_sha1
+
+    test_expect_success \
+        "hash from stdin" \
+        "test $example_sha1 = \$(git hash-object --stdin < example)"
+
+    test_blob_does_not_exist $example_sha1
+
+    test_expect_success \
+        "hash a file and write to database" \
+        "test $hello_sha1 = \$(git hash-object -w hello)"
+
+    test_blob_exists $hello_sha1
+
+    test_expect_success \
+        'git hash-object --stdin file1 <file0 first operates on file0, then file1' \
+        'echo foo > file1 &&
+        obname0=$(echo bar | git hash-object --stdin) &&
+        obname1=$(git hash-object file1) &&
+        obname0new=$(echo bar | git hash-object --stdin file1 | sed -n -e 1p) &&
+        obname1new=$(echo bar | git hash-object --stdin file1 | sed -n -e 2p) &&
+        test "$obname0" = "$obname0new" &&
+        test "$obname1" = "$obname1new"'
+
+pop_repo
+
+for args in "-w --stdin" "--stdin -w"; do
+    push_repo
+
+        test_expect_success \
+            "hash from stdin and write to database ($args)" \
+            "test $example_sha1 = \$(git hash-object $args < example)"
+
+        test_blob_exists $example_sha1
+
+    pop_repo
+done
 
 test_done
-- 
1.5.5.1.152.g9aeb7
Previous: Adam RobenNext: Adam Roben
Message 8 of 19 in “Speed up git-svn fetch”
  1. Adam RobenApr 23, 2008
  2. 01/11 Add tests for git cat-fileAdam Roben, Apr 23, 2008
  3. 02/11 git-cat-file: Small refactor of cmd_cat_fileAdam Roben, Apr 23, 2008
  4. 03/11 git-cat-file: Make option parsing a little more flexibleAdam Roben, Apr 23, 2008
  5. 04/11 git-cat-file: Add --batch-check optionAdam Roben, Apr 23, 2008
  6. 05/11 git-cat-file: Add --batch optionAdam Roben, Apr 23, 2008
  7. 06/11 Move git-hash-object tests from t5303 to t1007Adam Roben, Apr 23, 2008
  8. 07/11 Add more tests for git hash-objectAdam Roben, Apr 23, 2008
  9. 08/11 git-hash-object: Add --stdin-paths optionAdam Roben, Apr 23, 2008
  10. 09/11 Git.pm: Add command_bidi_pipe and command_close_bidi_pipeAdam Roben, Apr 23, 2008
  11. 10/11 Git.pm: Add hash_and_insert_object and cat_blobAdam Roben, Apr 23, 2008
  12. 11/11 git-svn: Speed up fetchAdam Roben, Apr 23, 2008
  13. Junio C HamanoApr 25, 2008
  14. Eric WongApr 25, 2008
  15. Junio C HamanoApr 25, 2008
  16. Junio C HamanoApr 25, 2008
  17. Junio C HamanoMay 6, 2008
  18. Adam RobenApr 23, 2008
  19. Eric WongApr 25, 2008

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.