From a51e89fd84863eceac3f8a7225785452b87393ac Mon Sep 17 00:00:00 2001 From: gutmet Date: Sun, 20 Nov 2022 21:51:36 +0100 Subject: [PATCH] cleanup --- CONTRIBUTING.mkd | 10 --- examples_test.go | 76 --------------------- main_test.go | 173 ----------------------------------------------- script/bootstrap | 6 -- script/lint | 6 -- script/profile | 7 -- script/test | 3 - 7 files changed, 281 deletions(-) delete mode 100644 CONTRIBUTING.mkd delete mode 100644 examples_test.go delete mode 100644 main_test.go delete mode 100755 script/bootstrap delete mode 100755 script/lint delete mode 100755 script/profile delete mode 100755 script/test diff --git a/CONTRIBUTING.mkd b/CONTRIBUTING.mkd deleted file mode 100644 index 0339bec..0000000 --- a/CONTRIBUTING.mkd +++ /dev/null @@ -1,10 +0,0 @@ -# Contributing - -* Raise an issue if appropriate -* Fork the repo -* Bootstrap the dev dependencies (run `./script/bootstrap`) -* Make your changes -* Use [gofmt](https://golang.org/cmd/gofmt/) -* Make sure the tests pass (run `./script/test`) -* Make sure the linters pass (run `./script/lint`) -* Issue a pull request diff --git a/examples_test.go b/examples_test.go deleted file mode 100644 index 2dc48f4..0000000 --- a/examples_test.go +++ /dev/null @@ -1,76 +0,0 @@ -package linkheader_test - -import ( - "fmt" - - "github.com/tomnomnom/linkheader" -) - -func ExampleParse() { - header := "; rel=\"next\"," + - "; rel=\"last\"" - links := linkheader.Parse(header) - - for _, link := range links { - fmt.Printf("URL: %s; Rel: %s\n", link.URL, link.Rel) - } - - // Output: - // URL: https://api.github.com/user/58276/repos?page=2; Rel: next - // URL: https://api.github.com/user/58276/repos?page=2; Rel: last -} - -func ExampleParseMultiple() { - headers := []string{ - "; rel=\"next\"", - "; rel=\"last\"", - } - links := linkheader.ParseMultiple(headers) - - for _, link := range links { - fmt.Printf("URL: %s; Rel: %s\n", link.URL, link.Rel) - } - - // Output: - // URL: https://api.github.com/user/58276/repos?page=2; Rel: next - // URL: https://api.github.com/user/58276/repos?page=2; Rel: last -} - -func ExampleLinks_FilterByRel() { - header := "; rel=\"next\"," + - "; rel=\"last\"" - links := linkheader.Parse(header) - - for _, link := range links.FilterByRel("last") { - fmt.Printf("URL: %s; Rel: %s\n", link.URL, link.Rel) - } - - // Output: - // URL: https://api.github.com/user/58276/repos?page=2; Rel: last - -} - -func ExampleLink_String() { - link := linkheader.Link{ - URL: "http://example.com/page/2", - Rel: "next", - } - - fmt.Printf("Link: %s\n", link.String()) - - // Output: - // Link: ; rel="next" -} - -func ExampleLinks_String() { - - links := linkheader.Links{ - {URL: "http://example.com/page/3", Rel: "next"}, - {URL: "http://example.com/page/1", Rel: "last"}, - } - - fmt.Printf("Link: %s\n", links.String()) - - // Output: - // Link: ; rel="next", ; rel="last" -} diff --git a/main_test.go b/main_test.go deleted file mode 100644 index d07af61..0000000 --- a/main_test.go +++ /dev/null @@ -1,173 +0,0 @@ -package linkheader - -import "testing" - -func TestSimple(t *testing.T) { - // Test case stolen from https://github.com/thlorenz/parse-link-header :) - header := "; rel=\"next\", " + - "; rel=\"prev\"; pet=\"cat\", " + - "; rel=\"last\"" - - links := Parse(header) - - if len(links) != 3 { - t.Errorf("Should have been 3 links returned, got %d", len(links)) - } - - if links[0].URL != "https://api.github.com/user/9287/repos?page=3&per_page=100" { - t.Errorf("First link should have URL 'https://api.github.com/user/9287/repos?page=3&per_page=100'") - } - - if links[0].Rel != "next" { - t.Errorf("First link should have rel=\"next\"") - } - - if len(links[0].Params) != 0 { - t.Errorf("First link should have exactly 0 params, but has %d", len(links[0].Params)) - } - - if len(links[1].Params) != 1 { - t.Errorf("Second link should have exactly 1 params, but has %d", len(links[1].Params)) - } - - if links[1].Params["pet"] != "cat" { - t.Errorf("Second link's 'pet' param should be 'cat', but was %s", links[1].Params["pet"]) - } - -} - -func TestEmpty(t *testing.T) { - links := Parse("") - if links != nil { - t.Errorf("Return value should be nil, but was %d", len(links)) - } -} - -// Although not often seen in the wild, the grammar in RFC 5988 suggests that it's -// valid for a link header to have nothing but a URL. -func TestNoRel(t *testing.T) { - links := Parse("") - - if len(links) != 1 { - t.Fatalf("Length of links should be 1, but was %d", len(links)) - } - - if links[0].URL != "http://example.com" { - t.Errorf("URL should be http://example.com, but was %s", links[0].URL) - } -} - -func TestLinkMethods(t *testing.T) { - header := "; rel=\"prev\"; pet=\"cat\"" - links := Parse(header) - link := links[0] - - if link.HasParam("foo") { - t.Errorf("Link should not have param 'foo'") - } - - val := link.Param("pet") - if val != "cat" { - t.Errorf("Link should have param pet=\"cat\"") - } - - val = link.Param("foo") - if val != "" { - t.Errorf("Link should not have value for param 'foo'") - } - -} - -func TestLinksMethods(t *testing.T) { - header := "; rel=\"next\", " + - "; rel=\"stylesheet\"; pet=\"cat\", " + - "; rel=\"stylesheet\"" - - links := Parse(header) - - filtered := links.FilterByRel("next") - - if filtered[0].URL != "https://api.github.com/user/9287/repos?page=3&per_page=100" { - t.Errorf("URL did not match expected") - } - - filtered = links.FilterByRel("stylesheet") - if len(filtered) != 2 { - t.Errorf("Filter for stylesheet should yield 2 results but got %d", len(filtered)) - } - - filtered = links.FilterByRel("notarel") - if len(filtered) != 0 { - t.Errorf("Filter by non-existant rel should yeild no results") - } - -} - -func TestParseMultiple(t *testing.T) { - headers := []string{ - "; rel=\"next\"", - "; rel=\"last\"", - } - - links := ParseMultiple(headers) - - if len(links) != 2 { - t.Errorf("Should have returned 2 links") - } -} - -func TestLinkToString(t *testing.T) { - l := Link{ - URL: "http://example.com/page/2", - Rel: "next", - Params: map[string]string{ - "foo": "bar", - }, - } - - have := l.String() - - parsed := Parse(have) - - if len(parsed) != 1 { - t.Errorf("Expected only 1 link") - } - - if parsed[0].URL != l.URL { - t.Errorf("Re-parsed link header should have matching URL, but has `%s`", parsed[0].URL) - } - - if parsed[0].Rel != l.Rel { - t.Errorf("Re-parsed link header should have matching rel, but has `%s`", parsed[0].Rel) - } - - if parsed[0].Param("foo") != "bar" { - t.Errorf("Re-parsed link header should have foo=\"bar\" but doesn't") - } -} - -func TestLinksToString(t *testing.T) { - ls := Links{ - {URL: "http://example.com/page/3", Rel: "next"}, - {URL: "http://example.com/page/1", Rel: "last"}, - } - - have := ls.String() - - want := "; rel=\"next\", ; rel=\"last\"" - - if have != want { - t.Errorf("Want `%s`, have `%s`", want, have) - } -} - -func BenchmarkParse(b *testing.B) { - - header := "; rel=\"next\", " + - "; rel=\"prev\"; pet=\"cat\", " + - "; rel=\"last\"" - - for i := 0; i < b.N; i++ { - _ = Parse(header) - } -} diff --git a/script/bootstrap b/script/bootstrap deleted file mode 100755 index 1fff31f..0000000 --- a/script/bootstrap +++ /dev/null @@ -1,6 +0,0 @@ -#!/bin/sh -PROJDIR=$(cd `dirname $0`/.. && pwd) - -echo "Installing gometalinter and linters..." -go get github.com/alecthomas/gometalinter -gometalinter --install diff --git a/script/lint b/script/lint deleted file mode 100755 index dfed93a..0000000 --- a/script/lint +++ /dev/null @@ -1,6 +0,0 @@ -#!/bin/sh -PROJDIR=$(cd `dirname $0`/.. && pwd) - -cd ${PROJDIR} -go get -gometalinter diff --git a/script/profile b/script/profile deleted file mode 100755 index f5b92ce..0000000 --- a/script/profile +++ /dev/null @@ -1,7 +0,0 @@ -#!/bin/sh -set -e -PROJDIR=$(cd `dirname $0`/.. && pwd) -cd ${PROJDIR} - -go test -bench . -benchmem -cpuprofile cpu.out -go tool pprof linkheader.test cpu.out diff --git a/script/test b/script/test deleted file mode 100755 index 01b91fd..0000000 --- a/script/test +++ /dev/null @@ -1,3 +0,0 @@ -#!/bin/sh -PROJDIR=$(cd `dirname $0`/.. && pwd) -cd $PROJDIR && go test