X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=dgit.git;a=blobdiff_plain;f=tests%2Ftests%2Fquilt-gbp;h=cf148f613ee6f3b6ac150828cefa86154f579e69;hp=d3624e9ee6f5183350b10a586c3beab2724513b1;hb=0e41808f2b25117c153c55d3913d85ce21b735fd;hpb=44fa80787dee7a78629768a78041ded8c309addc

diff --git a/tests/tests/quilt-gbp b/tests/tests/quilt-gbp
index d3624e9e..cf148f61 100755
--- a/tests/tests/quilt-gbp
+++ b/tests/tests/quilt-gbp
@@ -2,45 +2,76 @@
 set -e
 . tests/lib
 
+# `dgit build-source` cleans before applying patches.  However, the
+# clean targets of 3.0 (quilt) packages are allowed to assume that
+# patches are applied.  If they are not, the clean targets can
+# misbehave in basically two ways:
+#
+# - fail to clean everything
+# - error out and stop the build
+#
+# In this test, what we want to see is whether dgit detects that quilt
+# fixup cannot be linear because the user has failed to pass --gbp.
+# So we need to ensure the package's clean target does not error out,
+# because that blocks dgit attempting quilt linearisation.  This
+# environment variable tells the example package's clean target not to
+# error out if it notices that patches have not yet been applied.
+export DGIT_TEST_TOLERATE_UNPATCHED_CLEAN=true
+
+t-tstunt-parsechangelog
+
 t-gbp-example-prep
 
 t-expect-fail 'quilt fixup cannot be linear' \
   t-dgit build-source
 
-t-dgit --quilt=gbp build-source
+t-git-config dgit-distro.test-dummy.dgit-tag-format new
+t-expect-fail 'requires split view so server needs to support' \
+t-dgit -wgf --quilt=gbp build-source
+t-newtag
+
+t-dgit --quilt=gbp --dgit-view-save=split.b1 build-source
+git rev-parse split.b1
 
-t-dgit --quilt=gbp --gbp=no-such-command-gbp build-source
+t-dgit --quilt=gbp --gbp-pq=no-such-command-gbp build-source
 
 echo spong >debian/pointless-for-dgit-test
 git add debian/pointless-for-dgit-test
 git commit -m Pointless
 
 t-expect-fail no-such-command-gbp \
-t-dgit --quilt=gbp --clean=git --gbp=no-such-command-gbp build-source
+t-dgit --quilt=gbp --clean=git --gbp-pq=no-such-command-gbp build-source
 
-t-dgit --quilt=gbp --clean=git build-source
+test-push-1 () {
+	t-refs-same-start
+	t-ref-head
+}
 
-t-expect-fail 'specifies a different tree to your HEAD commit' \
-  t-dgit push
+test-push-2 () {
+	t-dgit --quilt=gbp --dgit-view-save=split.p push
+
+	t-gbp-pushed-good
+}
 
-exit 0
+test-push-1
 
-===
+t-dgit --quilt=gbp --clean=git --dgit-view-save=split.b build-source
 
-want to
+t-expect-fail "HEAD specifies a different tree to $p" \
+  t-dgit push
+
+test-push-2
+
+echo wombat >>debian/pointless-for-dgit-test
+git add debian/pointless-for-dgit-test
+git commit -m 'Pointless 2'
 
- do push
- t-pushed-good
- check that
-   - we didn't update caller's branch
+t-commit 'Check pseudomerge' 1.0-3
 
- do fetch
- check that this didn't update dgit remote ref
+test-push-1
 
- check that Dgit hash in dsc refers to tree identical
- to dpkg-source -x
+t-dgit --quilt=gbp --clean=git --dgit-view-save=split.b build-source
 
- check that that tree
-  - has patches applied (reuse rules build magic)
-  - has .gitignore edited
+test-push-2
 
+t-ok