Age	Commit message (Collapse)	Author	Lines
13 hours	Correct the test comments that went stale	Bryce Kwon	-10/+10
13 hours	Trim the tidy check down to what it does	Bryce Kwon	-17/+3
	Nothing sets NO_TIDY_WARNINGS, and a warning never fails the check, so the knob only quieted the output of a verbose run.
13 hours	Drop the exit lines that follow test_done	Bryce Kwon	-6/+0
	test_done ends the script on every path, skip_all included.
13 hours	Share one lock-file helper between the caches	Bryce Kwon	-0/+25
14 hours	Ask for every path below the about page in one test	Bryce Kwon	-22/+12
	All of them stop at the same early return in about_fn, so each now gets the status, the html error page and a clean log.
14 hours	Name the missing home without searching for one	Bryce Kwon	-10/+1
	strace logs a call that names the path whether or not it succeeds, so a home that existed would still be caught, and cgit unsets HOME before git runs anyway.
14 hours	Work out each gitlink's target once	Bryce Kwon	-11/+2
	Test bodies run in the main shell, so the target named in the fixture reaches every check that greps for it.
14 hours	Serve both readme escaping checks from one config	Bryce Kwon	-16/+9
14 hours	Ask each page that sorts odd branches in one loop	Bryce Kwon	-23/+8
14 hours	Write the test filter scripts with write_script	Bryce Kwon	-9/+3
14 hours	Derive the API test's config from the loud one	Bryce Kwon	-6/+1
14 hours	Check the filter flush on the pages already fetched	Bryce Kwon	-16/+7
	The about check already finds the div right before the filtered text, so its copy went, and the tree page is flattened where it was first read.
14 hours	Cut test comments that repeat what is said nearby	Bryce Kwon	-13/+4
14 hours	Correct two test comments that went stale	Bryce Kwon	-3/+3
14 hours	Drop test lines that add nothing	Bryce Kwon	-4/+0
	The trash directory starts empty and setup.sh already made the cache, the clamp check already fails on a short query, and the commit the setup named is still there to pin.
14 hours	Leave two log checks to the log tests	Bryce Kwon	-12/+0
	t0103 already renders foo/log and finds commit 5, and its search for a bracket fails the same way this literal search did if --fixed-strings ever goes.
14 hours	Leave the blob size refusals to the limits tests	Bryce Kwon	-22/+5
	t0204 asks the tree, plain, blob and blame pages for one oversized blob and checks the exact status and message, so the looser copy here went.
14 hours	Look up a request's commit in one place	Bryce Kwon	-0/+10
	The diff page now names a revision that is no commit the way the request wrote it, as the other pages already did, rather than as the full id it resolved to. The helpers sit beside the error page in ui-shared.c, since shared.c prints nothing.
14 hours	Reuse the ids t0109 has already looked up	Bryce Kwon	-2/+0
14 hours	Drop the patch greps the comparisons already make	Bryce Kwon	-20/+0
	The From, Date, Subject and signature lines are part of what the next test compares in full against git format-patch, for the single commit and for the range alike.
14 hours	Put the valgrind comment above the loop it explains	Bryce Kwon	-3/+3
14 hours	Drop two setup.sh lines that change nothing	Bryce Kwon	-5/+3
	The library has read TEST_RESULTS_SAN_DIR for the last time by the time it returns, and the trash directory it makes is empty, so there is no cache to remove.
14 hours	Tighten the checks that match stray text	Bryce Kwon	-8/+8
	Every page holds foo in its footer element, the foo+bar row holds bar, the branch switcher holds master, and commit 1 and file-1 also match commits 10 to 19 and file-10 to file-19. A search page repeats its term in the search box. So these checks passed on pages that lacked what they look for, and each now matches the link itself.
14 hours	Require a whole page when a filter exits early	Bryce Kwon	-2/+3
	When a filter cannot run, cgit dies mid-page with its Status line already out, so the check passed with no filter at all. That is the case wherever true lives only in /bin, as on busybox. true now comes from the PATH, and the page has to reach its end.
14 hours	Make the main cgitrc filter check run the filter	Bryce Kwon	-3/+6
	The check asked for the summary page, which never runs an about filter, so it passed with exec:/bin/true on macOS, which has no such file, and with a filter that does not exist at all. Its first line read the stderr the previous test left behind. execvp searches the PATH, so a plain true runs the same on macOS, Debian and busybox.
14 hours	Let go of a slot lock that cannot take its key	Bryce Kwon	-0/+15
	A failed truncate or key write returned with the descriptor still open, so the lock was held while the page rendered uncached and the file was left behind. It is removed before its last close, as every holder does.
14 hours	Keep lock contention out of the error log	Bryce Kwon	-2/+20
	process_slot logged every failure to lock, so each request that found its slot being filled left a line, while refresh_slot logged none, so an expired slot in a directory the server cannot write was served stale for good without a word. lock_slot now logs the step that failed, as the scan does, and leaves out only another request at work.
14 hours	Lock the cached repolist with flock	Bryce Kwon	-0/+38
	The slot lock has to be an flock, since an exec filter drops a record lock when it closes the stdout a fill writes through. The scan only got by with fcntl because nothing reopens its file while it runs. One kind of lock lets the two share their code, and on Linux, where the kinds never see each other, an flock on the scan's file now holds a scan off like any other.
14 hours	Open the lock files close-on-exec	Bryce Kwon	-0/+27
	Each exec filter and snapshot compressor a fill runs inherited the slot's lock descriptor beside the stdout it writes into. An flock lasts until its last copy closes and nothing waits for a process a filter leaves behind, so a fill killed before its rename could leave the slot locked, and served stale or uncached, for as long as such a process lived. The scan holds its lock across no exec and opens the same way so that the two stay alike.
14 hours	Tidy the cache test's helpers and outputs	Bryce Kwon	-24/+18
	The header's last sentence named two of the many later subjects, so it goes too. The second request's stderr is still kept, for the lock tests that come next.
14 hours	Use test_dir_is_empty for the empty cache checks	Bryce Kwon	-16/+8
	The library's check also lists whatever was left behind when it fails.
15 hours	Write a session's expiry in whole seconds	Bryce Kwon	-0/+11
	On Lua 5.3 and later a length worked out with a fraction, such as 1.5 * 24 * 60 * 60, is a float, so the signed expiry came out as 1793609024.0, which validate_value refuses, and every login failed. Max-Age took the same .0, which makes a browser ignore it.
15 hours	Drop an auth prerequisite no test asks for	Bryce Kwon	-4/+2
15 hours	Set the Lua prerequisite once for every test	Bryce Kwon	-87/+27
	setup.sh already asks the binary, so the seven scripts that each turned the answer into a prerequisite now share one. test-lib names the missing prerequisite beside every test it skips, so the notes that cgit lacks Lua go, while the ones announcing a fallback stay. The list of standalone interpreters moves into lib.sh and matches the versions the extensions claim directly.
15 hours	Move the highlighter fallback check into t0502	Bryce Kwon	-24/+12
	t0502 owns the shipped highlighter and already writes a cgitrc naming it, so the check needs no config of its own there. t0204's header still called it the last case long after it stopped being one.
15 hours	Share the request helpers in the auth checks	Bryce Kwon	-77/+49
	The two identical posts for an unknown user become one, and no two checks share a name, so a failure report points at a single check.
15 hours	Let an unmirrored sink fail the way Lua would	Bryce Kwon	-12/+0
15 hours	Drop the auth stubs no check reaches	Bryce Kwon	-20/+6
	Every check replaces get_secret, the only caller of posix.sys.stat, link and unlink, and nothing outside the harness calls preload.
15 hours	Drop auth guards for input cgit never sends	Bryce Kwon	-3/+4
	cgit names no repository with an empty string, never nil, so the check that the index stays up proved nothing, and auth-file in fact refuses the index too while its repos file cannot be read. A session_seconds of 0 signs a session that has expired by the time it comes back, so it needs no branch of its own in set_cookie.
15 hours	Strip control characters from the location alone	Bryce Kwon	-2/+0
	set_cookie only ever writes an empty value or a token secure_value built from url-encoded text, digits and hex, so stripping it changed nothing.
15 hours	Take a signed cookie apart with one match	Bryce Kwon	-0/+1
	The loop let a valid cookie through with a field appended, a doubled separator or a zero in front of its expiry. The payload now carries the expiry as written, so only the spelling secure_value issues verifies.
15 hours	Drop a highlighter check that tested the harness	Bryce Kwon	-6/+0
	The filter hands a NUL through untouched, and only the harness's copy of cgit's sink cut the text there, so the check would have failed the day the filter replaced NULs the way about-render does.
15 hours	Drop the highlighter's extension fallback	Bryce Kwon	-14/+5
	The table of extensions and the second lookup served only a Scintillua with no detect(), which now leaves every file uncoloured but escaped. The fake detect() names a lexer after the file's suffix, so every test lexer is reached through it.
15 hours	Drop a stray blank line from the about-render tests	Bryce Kwon	-1/+0
15 hours	Drop the readme hooks that nothing reads	Bryce Kwon	-6/+5
	Nothing in assets/ or in README's list of hooks reads a fenced block's data-lang or the manpage class on a man page's wrapper, so the header now names the two wrappers the stylesheet does style.
15 hours	Trim a snapshot name without reading out of bounds	Bryce Kwon	-0/+30
	For a url of .git, or an empty one, repo_basename stepped its index to the byte before its buffer, read it, and would have cleared it had it held a slash. Git's strbuf trims take the same steps safely, and they lift the 1024-byte cap that failed the page for a longer url.
15 hours	Open the empty page's clone table on its first url	Bryce Kwon	-0/+29
	The page guessed from the two settings cgit_add_clone_urls reads, so a clone-url that expanded to nothing, or an empty one turning the global off, still drew a Clone heading over no rows. The summary already waits for a first url, and both now draw the heading the same way.
15 hours	Tidy up after a directory chain however it ends	Bryce Kwon	-0/+27
	A row whose first entry is itself a directory leaked that entry's name. A level git could not read as a tree returned early instead, which also left the row's log, stats and plain links on the deeper path.
15 hours	Free the log's tip when its walk fails	Bryce Kwon	-0/+19
	Every exit now goes through one label, which also clears the walk arguments that the normal end used to leave allocated.
15 hours	Refuse pathspec magic on the stats page	Bryce Kwon	-0/+9