Age	Commit message (Collapse)	Author	Lines
45 hours	Keep a followed diff to the path it names	Bryce Kwon	-0/+27
	A followed diff kept every path that started with the requested one, so following small.c took small.cpp in as well. A path now has to be the requested one or lie below it, as the pathspec of an unfollowed diff matches.
45 hours	Check the feed's elements, limits and path	Bryce Kwon	-32/+102
	No entry content was checked, so a wrong link, id or date passed, and the path control looked at the status alone. A max-atom-items of 0 or below, all=1 and the epoch an empty feed takes had no test. The xmllint checks read a feed an earlier test left behind, which was another feed whenever that test failed early.
45 hours	Test serving a snapshot signature	Bryce Kwon	-23/+61
	No test fetched a signature within the size limit, so the status, the headers that keep a browser from sniffing it and the body went unchecked, as did the answer for a ref with none and the plain tar signature a compressed tar falls back to.
45 hours	Request every snapshot name back to its ref	Bryce Kwon	-69/+197
	The names on the links were checked but never requested back, and the tests shared one repository whose refs each test added to, so a run of a subset broke them. On a case-insensitive filesystem a loose V1.0 also answered lookups for v1.0, so the case tests checked different tags from one filesystem to another. The slash ref sat at master's tip, where the commit compare could not tell it from the default branch.
45 hours	Compare each snapshot format with git archive	Bryce Kwon	-58/+47
	Only the entry count and one file were checked, never the archive itself. git's own writer at the same commit gives the same bytes, so each format is held to it, which also proves the commit served. That leaves unzip -t unused, which busybox may not have.
45 hours	Compare more of the patch page with git	Bryce Kwon	-33/+82
	The root patch was checked for its signature alone, so one that lost its diff passed. The headers, the note a path adds to the stat and a merge in a range had no test, and a renamed file's patch was grepped rather than compared. The version is read in the test that needs it, so a missing build/VERSION fails there instead of surfacing later as a mismatch.
45 hours	Check which commits max-patch-count keeps	Bryce Kwon	-9/+21
	Only the number of patches was counted, so keeping the oldest two of a range would have passed. The default of 50 and the 0 that lifts the bound had no test.
45 hours	Check the rawdiff is served as plain text	Bryce Kwon	-24/+17
	strip_headers dropped the headers before every compare, so a rawdiff served under another type passed. Each test now makes its own request, and the ids it needs, rather than reading what an earlier one left.
45 hours	Test the diff options a request or config sets	Bryce Kwon	-0/+94
	None of dt=2, ignorews, context and its cap, side-by-side-diffs or rename-limit had a test anywhere in the suite, nor the 0 that the MANUAL says lifts the rename limit.
45 hours	Check what each side-by-side cell holds	Bryce Kwon	-18/+106
	The span checks compared page totals, so a span on the wrong side or over the wrong characters passed while the totals agreed. A fill that read cells a longer pair left behind passed as well, since only a literal pair after a longer one shows it. Nothing checked the cell classes the forced-colours border keys on, or the rows for unpaired, overlong, tabbed, binary and cut short lines.
45 hours	Test enable-plain-email=0 where it changes output	Bryce Kwon	-0/+28
	No test set it, though it switches the patch to a format of cgit's own and drops the address from the feed and the trailer idents.
45 hours	Check the stats page counts and orders authors	Bryce Kwon	-10/+60
	No test asserted a count, and the fixture had a single author, so a page that stopped counting or sorting passed. The commit graph came in for a language breakdown that is gone. It stays because the walk still releases each commit as it goes, which a commit read from the graph has to survive.
45 hours	Check the feed escapes what XML cannot carry	Bryce Kwon	-0/+45
	Every fixture subject was plain ASCII, so xmllint never met a byte the escaping exists for. git commit rewrites stray bytes as Latin-1 and the commit parser replaces any that are left, so the commit is written by hand and the description carries a stray byte to the feed's own check.
45 hours	Name the followed path in the diffstat header	Bryce Kwon	-0/+22
	With follow set the diff runs over the whole commit and show_filepair applies the path, so the header read (limited to '') whether or not a path was given.
45 hours	Check a renamed file's diff leaves the rest out	Bryce Kwon	-8/+15
	A new path is diffed across the whole commit so the rename can be found, and a filter then keeps that path alone. The rename commit touched one file, so a diff that dropped the filter passed.
45 hours	Check each side's line numbers in a later hunk	Bryce Kwon	-6/+31
	The test for a hunk that carries a length ran on foo's diff, whose only numbered side is +1 with no length, so it repeated the comma-less test. Every fixture hunk started at line 1, so a parser that always answered 1 passed. A line 0 never gets a link, so the absent 0 link could not show either way.
45 hours	Test stray bytes in a tag's tagger and message	Bryce Kwon	-0/+21
45 hours	Test the link from a tag to the tag it tags	Bryce Kwon	-0/+14
	A robustness test asked for the inner tag's page by id, but nothing checked that the outer tag's page links there.
45 hours	Test the tag page for a name that is no tag	Bryce Kwon	-0/+6
45 hours	Hold the tag tests to a working page	Bryce Kwon	-14/+16
	The tag requests passed on any answer short of a crash, and the title test read a page the cache kept from an earlier test. The id pattern also matched an empty id, the date row was checked only for its opening tag, and the lightweight tag was never shown to lack a date row. Two tests read a page an earlier test wrote.
45 hours	Test the download row on the commit page	Bryce Kwon	-0/+10
45 hours	Test a note on the commit page	Bryce Kwon	-0/+8
45 hours	Test subject links on the commit page	Bryce Kwon	-0/+7
45 hours	Test a stray byte in a message with no encoding	Bryce Kwon	-1/+9
	The comment names a message with no encoding header as one case, but only a header naming an unknown charset was tested. Without a header the text is taken as UTF-8 and never converted, so only the clean-up stands between its stray bytes and the page.
46 hours	Check the octopus merge draws no diff	Bryce Kwon	-7/+19
	The octopus took its first parent's tree, so its diff would have been empty whether drawn or not, and only the missing options were checked, with nothing to show the right page rendered. A merge of two parents, the other side of the limit, had no test.
46 hours	Hold the commit tests to a working page	Bryce Kwon	-18/+14
	The commit requests passed on any answer short of a crash, and the root commit checks read a page another test wrote. The tree row was found by its href alone, never by the tree id it names.
46 hours	Test the blob page by object id	Bryce Kwon	-1/+29
	Only a robustness test touched the blob page, so a request by bare id, the binary type, the untrusted headers and the refusal of a malformed id had no check.
46 hours	Test plain on a symlink and a submodule entry	Bryce Kwon	-0/+24
46 hours	Test the PDF exception to the unsafe plain types	Bryce Kwon	-0/+18
46 hours	Test the untrusted headers on a plain file	Bryce Kwon	-0/+16
	The content security policy had no test anywhere, and nosniff was only checked on a scanned repository.
46 hours	Test a plain listing below the root	Bryce Kwon	-0/+23
46 hours	Hold the plain tests to a working page	Bryce Kwon	-16/+12
	The plain requests passed on any answer short of a crash, and the listing test never looked at where its links led. The types config was a copy of the shared one with its page cache still on, which cgitrc_copy exists to switch off.
46 hours	Read the stylesheet rules by selector	Bryce Kwon	-7/+51
	Both tests cut one block out of cgit.css by its opening line, so rewriting that selector list broke them while a later rule overriding the declaration passed. The helper gathers a property's declarations from every rule naming the selector, so the last one is the one a browser would apply. A rule under some other selector can still win, which only a browser shows.
46 hours	Test the blame tooltip with and without addresses	Bryce Kwon	-0/+13
46 hours	Test a link target that climbs above the root	Bryce Kwon	-2/+6
	Only an absolute target was tested, so ignoring a failed normalisation passed, and the mutation run left that change alive.
46 hours	Follow the blame page's crumb and previous links	Bryce Kwon	-2/+9
	Both fixes were for links that led to a 404, and the tests read the href without asking for the page it names.
46 hours	Test what blame says for paths it cannot blame	Bryce Kwon	-0/+12
46 hours	Test that blame is refused while disabled	Bryce Kwon	-0/+6
46 hours	Test the blame link in the file view's header	Bryce Kwon	-0/+8
46 hours	Prove the rows the blame link check looks past	Bryce Kwon	-6/+10
	The checks that blob.bin, big.bin and large.txt get no blame link never showed those rows were in the listing, so a listing that dropped them passed.
46 hours	Hold the tree tests to a working page	Bryce Kwon	-36/+50
	The tree requests passed on any answer short of a crash. Several tests read a page or a row an earlier test wrote, so run alone they passed or failed by accident, and two leant on a repository from the test before without saying so. The file-50 check also matched its href, and the blame check skipped the form feed and NEL its title covers.
46 hours	Check the tag message line by line	Bryce Kwon	-2/+3
	The check removed every newline before matching, so a body that kept the blank line after the subject read the same, and the page would have shown that line under white-space pre-wrap.
46 hours	Check the commit page's parent row	Bryce Kwon	-1/+4
	The pattern also matched the commit row's link to itself, so the test passed with no parent row at all.
46 hours	Pin the listing's blame link for a late NUL	Bryce Kwon	-0/+23
	The listing judges binary by the first 8000 bytes, as git does, while the blame page and the file view look for a NUL anywhere, since one past that window would end a Lua filter's output early. The listing keeps the cheap sniff, because the whole-blob check would inflate every blob it names, so a file whose NUL lies past the window gets a blame link that ends on "Blob is binary". The test pins that trade, so a change to either side has to face it.
46 hours	Test a long subject in the expanded log	Bryce Kwon	-0/+13
46 hours	Test the lowest message limit that cuts	Bryce Kwon	-0/+10
46 hours	Test the log search by case and by committer	Bryce Kwon	-0/+19
46 hours	Test the log orders commit-sort selects	Bryce Kwon	-0/+41
	commit-sort had no test anywhere. The history here is one where the plain walk, date order and topo order all differ, which the test checks first so a fixture that stopped telling them apart would fail rather than pass.
46 hours	Test the file and line counts in the log	Bryce Kwon	-0/+17
	The shared config turns both columns on, but no test read a count, so a wrong or swapped one passed.
46 hours	Test the branch and tag labels in the log	Bryce Kwon	-0/+15
	No test anywhere drew a label, so the peel that tells an annotated tag from a lightweight one had no check.