stgit and rm a file

Jon Smirl· Aug 1, 2008, 02:02 UTC · lore

-- 
Jon Smirl
jonsmirl@gmail.com

Karl Hasselström· Aug 4, 2008, 13:41 UTC · re: Jon Smirl · lore

[StGit PATCH] Add some tests of refreshing removed files

Signed-off-by: Karl Hasselström <kha@treskal.com>

---

 t/t2702-refresh-rm.sh |  101 +++++++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 101 insertions(+), 0 deletions(-)
 create mode 100755 t/t2702-refresh-rm.sh

diff --git a/t/t2702-refresh-rm.sh b/t/t2702-refresh-rm.sh
new file mode 100755
index 0000000..896ebf3
--- /dev/null
+++ b/t/t2702-refresh-rm.sh
@@ -0,0 +1,101 @@
+#!/bin/sh
+
+test_description='"stg refresh" with removed files'
+
+. ./test-lib.sh
+
+# Ignore our own temp files.
+cat >> .git/info/exclude <<EOF
+expected*.txt
+files*.txt
+status*.txt
+EOF
+
+reset () {
+    stg pop -a > /dev/null
+    git reset --hard > /dev/null
+}
+
+test_expect_success 'Initialize StGit stack' '
+    stg init &&
+    echo x > x.txt &&
+    echo y > y.txt &&
+    git add x.txt y.txt &&
+    git commit -m "Add some files"
+'
+
+cat > expected0.txt <<EOF
+D y.txt
+EOF
+printf '' > expected1.txt
+test_expect_success 'git-rm a file' '
+    stg new -m p0 &&
+    git rm y.txt &&
+    stg status > status0.txt &&
+    test_cmp expected0.txt status0.txt &&
+    stg refresh &&
+    stg status > status1.txt &&
+    test_cmp expected1.txt status1.txt &&
+    stg files | sort > files.txt &&
+    test_cmp expected0.txt files.txt
+'
+
+reset
+
+cat > expected0.txt <<EOF
+D y.txt
+M x.txt
+EOF
+printf '' > expected1.txt
+test_expect_success 'git-rm a file together with other changes' '
+    stg new -m p1 &&
+    echo x2 >> x.txt &&
+    git rm y.txt &&
+    stg status > status0.txt &&
+    test_cmp expected0.txt status0.txt &&
+    stg refresh &&
+    stg status > status1.txt &&
+    test_cmp expected1.txt status1.txt &&
+    stg files | sort > files.txt &&
+    test_cmp expected0.txt files.txt
+'
+
+reset
+
+cat > expected0.txt <<EOF
+D y.txt
+EOF
+printf '' > expected1.txt
+test_expect_success 'rm a file' '
+    stg new -m p2 &&
+    rm y.txt &&
+    stg status > status0.txt &&
+    test_cmp expected0.txt status0.txt &&
+    stg refresh &&
+    stg status > status1.txt &&
+    test_cmp expected1.txt status1.txt &&
+    stg files | sort > files.txt &&
+    test_cmp expected0.txt files.txt
+'
+
+reset
+
+cat > expected0.txt <<EOF
+D y.txt
+M x.txt
+EOF
+printf '' > expected1.txt
+test_expect_success 'rm a file together with other changes' '
+    stg new -m p3 &&
+    echo x2 >> x.txt &&
+    rm y.txt &&
+    stg status > status0.txt &&
+    test_cmp expected0.txt status0.txt &&
+    stg refresh &&
+    stg status > status1.txt &&
+    test_cmp expected1.txt status1.txt &&
+    stg files | sort > files.txt &&
+    test_cmp expected0.txt files.txt
+'
+
+test_done

Jon Smirl· Aug 4, 2008, 14:07 UTC · re: Karl Hasselström · lore

Re: [StGit PATCH] Add some tests of refreshing removed files

On 8/4/08, Karl Hasselström <kha@treskal.com> wrote:

> Signed-off-by: Karl Hasselström <kha@treskal.com>
>
>  ---
>
>  These four tests all pass on Catalin's current master. Do they fail
>  for you, or did I miss something in your problem description?

-- 
Jon Smirl
jonsmirl@gmail.com

Jon Smirl· Aug 4, 2008, 14:08 UTC · re: Jon Smirl · lore

Re: [StGit PATCH] Add some tests of refreshing removed files

On 8/4/08, Jon Smirl <jonsmirl@gmail.com> wrote:

> On 8/4/08, Karl Hasselström <kha@treskal.com> wrote:
>  > Signed-off-by: Karl Hasselström <kha@treskal.com>
>  >
>  >  ---
>  >
>  >  These four tests all pass on Catalin's current master. Do they fail
>  >  for you, or did I miss something in your problem description?
>
>
> I just updated to current git and stg and retested. Tracking a remove
>  is working now for me. Tools were about three months old before
>  updating.
>
>  Not sure what changed. But I definitely couldn't get stgit to do it
>  and had to use git commands to generate the patch. I would remove the
>  file, refresh and the patch was empty.
>  stg status would show the file in D state.  As soon as I touched
>  something else and refreshed the rm would appear in the patch.

The file I was trying to rm was a hidden file, .mailmap

>
>  Maybe there is more to the triggering sequence than a simple rm. I'll
>  keep a watch out and see if I can figure out how I got into the state.
>
>
>  --
>  Jon Smirl
>  jonsmirl@gmail.com
>

-- 
Jon Smirl
jonsmirl@gmail.com

Karl Hasselström· Aug 4, 2008, 14:43 UTC · re: Jon Smirl · lore

Re: [StGit PATCH] Add some tests of refreshing removed files

On 2008-08-04 10:07:45 -0400, Jon Smirl wrote:

> I just updated to current git and stg and retested. Tracking a
> remove is working now for me.

Well, that's good.

> Tools were about three months old before updating.

> Not sure what changed. But I definitely couldn't get stgit to do it
> and had to use git commands to generate the patch. I would remove
> the file, refresh and the patch was empty. stg status would show the
> file in D state. As soon as I touched something else and refreshed
> the rm would appear in the patch.

> Maybe there is more to the triggering sequence than a simple rm.
> I'll keep a watch out and see if I can figure out how I got into the
> state.

Thanks.

-- 
Karl Hasselström, kha@treskal.com
      www.treskal.com/kalle