index : cgit

#!/bin/sh

# The diff page, unified or side-by-side under dt=1. The first checks read
# the markup for one added file in the fixture repository. The rest build
# repositories of their own, shaped for the side-by-side renderer's two
# trickiest cases, a hunk covering a single line and a file where every
# line changed.

test_description='Check content on diff page'
. ./setup.sh

test_expect_success 'generate foo/diff' 'cgit_url "foo/diff" >tmp'
test_expect_success 'find diff header' 'grep "a/file-5 b/file-5" tmp'
test_expect_success 'find blob link' 'grep "<a href=./foo/tree/file-5?id=" tmp'
test_expect_success 'find added file' 'grep "new file mode 100644" tmp'

test_expect_success 'find hunk header' '
	grep "<div class=.hunk.>@@ -0,0 +1 @@</div>" tmp
'

test_expect_success 'find added line' '
	grep "<div class=.add.>+5</div>" tmp
'

# The unified diff sits in a single cell, which holds the files together
# rather than tabulating them, so a screen reader is told it is layout.
test_expect_success 'the unified diff table is read as layout' '
	grep "<table class=.diff. role=.presentation.>" tmp
'

# A revision that names no commit is quoted the way the request wrote it, as
# the commit and patch pages quote it, on either side of the diff.
test_expect_success 'a tree on either side is named as the request gave it' '
	tree=$(git -C repos/foo rev-parse --short HEAD^{tree}) &&
	cgit_url "foo/diff/&id=$tree" >tmp &&
	grep "<div class=.error.>Not a commit: $tree</div>" tmp &&
	cgit_url "foo/diff/&id2=$tree" >tmp &&
	grep "<div class=.error.>Not a commit: $tree</div>" tmp
'

# git writes a hunk length only when the hunk spans more than one line, so the
# header for a single line reads "@@ -1 +1 @@" with no comma in it. Requiring
# that comma left those hunks numbered from zero, which dropped the line
# number link altogether.
test_expect_success 'set up a repo with single-line hunks' '
	mkrepo repos/hunk1 1 &&
	(
		cd repos/hunk1 &&
		printf "only\n" >one.txt &&
		git add one.txt &&
		git commit -m "add one.txt" &&
		printf "ONLY\n" >one.txt &&
		git commit -am "change one.txt"
	) &&
	cgitrc_repo hunk1rc hunk1 &&
	git -C repos/hunk1 diff HEAD~1 HEAD >raw.diff &&
	grep "^@@ -1 +1 @@" raw.diff
'

test_expect_success 'ssdiff numbers a comma-less hunk from its real line' '
	old=$(git -C repos/hunk1 rev-parse HEAD~1) &&
	new=$(git -C repos/hunk1 rev-parse HEAD) &&
	cgit_ok hunk1rc "url=hunk1/diff/&dt=1" >tmp &&
	grep "<a href=./hunk1/tree/one.txt?id=$old#n1.>1</a>" tmp &&
	grep "<a href=./hunk1/tree/one.txt?id=$new#n1.>1</a>" tmp
'

# A line dropped near the top leaves the two sides of the later hunk starting
# at different lines, so each side has to be numbered from its own half of the
# header.
test_expect_success 'set up a repo whose later hunk starts apart on each side' '
	mkrepo repos/later 1 &&
	(
		cd repos/later &&
		test_seq 1 20 >twenty.txt &&
		git add twenty.txt &&
		git commit -m "add twenty.txt" &&
		test_seq 1 20 | sed -e 2d -e s/^15$/fifteen/ >twenty.txt &&
		git commit -am "change twenty.txt"
	) &&
	cgitrc_repo laterrc later &&
	git -C repos/later diff HEAD~1 HEAD >raw.diff &&
	grep "^@@ -12,7 +11,7 @@" raw.diff
'

test_expect_success 'ssdiff numbers each side of a hunk from its own start' '
	old=$(git -C repos/later rev-parse HEAD~1) &&
	new=$(git -C repos/later rev-parse HEAD) &&
	cgit_ok laterrc "url=later/diff/&dt=1" >tmp &&
	grep -x "<td class=.lineno.><a href=./later/tree/twenty.txt?id=$old#n12.>12</a></td><td class=.ctx.>12</td>" tmp &&
	grep -x "<td class=.lineno.><a href=./later/tree/twenty.txt?id=$new#n11.>11</a></td><td class=.ctx.>12</td></tr>" tmp &&
	grep "<a href=./later/tree/twenty.txt?id=$old#n15.>15</a></td><td class=.changed.>" tmp &&
	grep "<a href=./later/tree/twenty.txt?id=$new#n14.>14</a></td><td class=.changed.>" tmp
'

# The character level highlight reuses one table across every changed pair, so
# these lines get shorter as the file goes on and each comparison is smaller
# than the one before it. That is where a cell left over from the previous
# pair shows itself, as a del or an add span sitting over the wrong
# characters.
test_expect_success 'set up a repo with shrinking changed lines' '
	mkrepo repos/lcs 1 &&
	(
		cd repos/lcs &&
		awk "BEGIN{for(n=120;n>0;n-=3){s=\"\";
			for(i=0;i<n;i++)s=s sprintf(\"%c\",97+(i*7+n)%26); print s}}" >lines.txt &&
		git add lines.txt &&
		git commit -m "add lines.txt" &&
		awk "BEGIN{for(n=120;n>0;n-=3){s=\"\";
			for(i=0;i<n;i++){c=97+(i*7+n)%26; if(i%7==0)c=97+(c-97+5)%26;
			s=s sprintf(\"%c\",c)} print s}}" >lines.txt &&
		git commit -am "change every line"
	) &&
	cgitrc_repo lcsrc lcs
'

test_expect_success 'every changed pair gets both a del and an add span' '
	cgit_rc lcsrc "url=lcs/diff/&dt=1" >tmp &&
	deletions=$(grep -o "<span class=.del.>" tmp | wc -l) &&
	additions=$(grep -o "<span class=.add.>" tmp | wc -l) &&
	test "$deletions" -gt 0 &&
	test "$deletions" -eq "$additions"
'

test_expect_success 'stripping the highlight leaves every line intact' '
	sed -e "s/<[^>]*>//g" tmp >plain.out &&
	git -C repos/lcs show HEAD~1:lines.txt >lines &&
	git -C repos/lcs show HEAD:lines.txt >>lines &&
	while read -r line
	do
		grep -qF "$line" plain.out || {
			echo "missing line: $line"
			return 1
		}
	done <lines
'

# The diffstat links a renamed file by its new name, and so does the log of
# that file, but the old name lies outside a diff limited to the new one. That
# diff runs over the whole commit to find the rename, so the commit changes
# another file as well, which the limit still has to leave out.
test_expect_success 'set up a renamed file' '
	mkrepo repos/moved 1 &&
	(
		cd repos/moved &&
		test_seq 1 10 >old.txt &&
		git add old.txt &&
		git commit -m "add old.txt" &&
		git mv old.txt new.txt &&
		echo changed >file-1 &&
		git commit -am "rename old.txt"
	) &&
	cgitrc_repo movedrc moved
'

test_expect_success 'the diff of a renamed file shows the rename alone' '
	cgit_ok movedrc "url=moved/diff/new.txt" >tmp &&
	grep "diff --git a/old.txt b/new.txt" tmp &&
	grep "</a> (renamed from old.txt)</td>" tmp &&
	grep "<div class=.diffstat-summary.>1 file changed, 0 insertions, 0 deletions</div>" tmp &&
	! grep "new file mode" tmp &&
	! grep "a/file-1 b/file-1" tmp
'

test_expect_success 'so does the commit page limited to the new name' '
	cgit_ok movedrc "url=moved/commit/new.txt" >tmp &&
	grep "diff --git a/old.txt b/new.txt" tmp &&
	grep "<div class=.diffstat-summary.>1 file changed, 0 insertions, 0 deletions</div>" tmp &&
	! grep "new file mode" tmp &&
	! grep "a/file-1 b/file-1" tmp
'

# Following a file diffs the whole commit and keeps the pairs on that path,
# and the header still names the path rather than the empty limit the walk
# runs under.
test_expect_success 'a followed file is diffed as the rename alone' '
	cgitrc_repo followrc moved enable-follow-links=1 &&
	cgit_ok followrc "url=moved/diff/new.txt&follow=1" >tmp &&
	grep "diff --git a/old.txt b/new.txt" tmp &&
	grep "Diffstat</a> (limited to .new.txt.)</div>" tmp &&
	grep "<div class=.diffstat-summary.>1 file changed, 0 insertions, 0 deletions</div>" tmp &&
	! grep "a/file-1 b/file-1" tmp &&
	cgit_ok followrc "url=moved/commit/new.txt&follow=1" >tmp &&
	grep "diff --git a/old.txt b/new.txt" tmp &&
	grep "Diffstat</a> (limited to .new.txt.)</div>" tmp &&
	! grep "a/file-1 b/file-1" tmp
'

test_expect_success 'following with no path limits nothing' '
	cgit_ok followrc "url=moved/commit/&follow=1" >tmp &&
	grep "diff --git a/file-1 b/file-1" tmp &&
	grep "<div class=.diffstat-header.><a href=[^>]*>Diffstat</a></div>" tmp
'

test_expect_success 'a file added outright still shows as new' '
	added=$(git -C repos/moved rev-parse HEAD~1) &&
	cgit_ok movedrc "url=moved/diff/old.txt&id=$added" >tmp &&
	grep "new file mode 100644" tmp &&
	grep "<div class=.add.>+10</div>" tmp
'

test_done