thepaperpilot/forgejo
1
0
Fork 0
Code Issues Pull requests Releases Activity Actions

Remove unnecessary ".Link" usages (#29909) (part 2)

Browse source 
5143ebb507 Add rel="nofollow" to issue filter links

has a test that fails because it assumes the link starts with the link
where it now starts with a ?
This commit is contained in:
Earl Warren 2024-03-25 07:13:11 +01:00
parent c758c5612e
commit 4061cb32f7
No known key found for this signature in database
GPG key ID: 0579CB2928A78A00
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
tests/integration/issue_test.go
View file

@ -807,7 +807,7 @@ func TestIssueFilterNoFollow(t *testing.T) {
htmlDoc := NewHTMLParser(t, resp.Body) htmlDoc := NewHTMLParser(t, resp.Body)
// Check that every link in the filter list has rel="nofollow". // Check that every link in the filter list has rel="nofollow".
filterLinks := htmlDoc.Find(".issue-list-toolbar-right a[href*=\"/issues?q=\"]") filterLinks := htmlDoc.Find(".issue-list-toolbar-right a[href*=\"?q=\"]")
assert.True(t, filterLinks.Length() > 0) assert.True(t, filterLinks.Length() > 0)
filterLinks.Each(func(i int, link *goquery.Selection) { filterLinks.Each(func(i int, link *goquery.Selection) {
rel, has := link.Attr("rel") rel, has := link.Attr("rel")