682d637d93
Fixes #18 If any pathConfig is the shortest prefix of a query, find and return it.
316 lines
8.3 KiB
Go
316 lines
8.3 KiB
Go
// Copyright 2017 Google Inc. All Rights Reserved.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
package main
|
|
|
|
import (
|
|
"bytes"
|
|
"io/ioutil"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"sort"
|
|
"testing"
|
|
)
|
|
|
|
func TestHandler(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
config string
|
|
path string
|
|
|
|
goImport string
|
|
goSource string
|
|
}{
|
|
{
|
|
name: "explicit display",
|
|
config: "host: example.com\n" +
|
|
"paths:\n" +
|
|
" /portmidi:\n" +
|
|
" repo: https://github.com/rakyll/portmidi\n" +
|
|
" display: https://github.com/rakyll/portmidi _ _\n",
|
|
path: "/portmidi",
|
|
goImport: "example.com/portmidi git https://github.com/rakyll/portmidi",
|
|
goSource: "example.com/portmidi https://github.com/rakyll/portmidi _ _",
|
|
},
|
|
{
|
|
name: "display GitHub inference",
|
|
config: "host: example.com\n" +
|
|
"paths:\n" +
|
|
" /portmidi:\n" +
|
|
" repo: https://github.com/rakyll/portmidi\n",
|
|
path: "/portmidi",
|
|
goImport: "example.com/portmidi git https://github.com/rakyll/portmidi",
|
|
goSource: "example.com/portmidi https://github.com/rakyll/portmidi https://github.com/rakyll/portmidi/tree/master{/dir} https://github.com/rakyll/portmidi/blob/master{/dir}/{file}#L{line}",
|
|
},
|
|
{
|
|
name: "Bitbucket Mercurial",
|
|
config: "host: example.com\n" +
|
|
"paths:\n" +
|
|
" /gopdf:\n" +
|
|
" repo: https://bitbucket.org/zombiezen/gopdf\n" +
|
|
" vcs: hg\n",
|
|
path: "/gopdf",
|
|
goImport: "example.com/gopdf hg https://bitbucket.org/zombiezen/gopdf",
|
|
goSource: "example.com/gopdf https://bitbucket.org/zombiezen/gopdf https://bitbucket.org/zombiezen/gopdf/src/default{/dir} https://bitbucket.org/zombiezen/gopdf/src/default{/dir}/{file}#{file}-{line}",
|
|
},
|
|
{
|
|
name: "Bitbucket Git",
|
|
config: "host: example.com\n" +
|
|
"paths:\n" +
|
|
" /mygit:\n" +
|
|
" repo: https://bitbucket.org/zombiezen/mygit\n" +
|
|
" vcs: git\n",
|
|
path: "/mygit",
|
|
goImport: "example.com/mygit git https://bitbucket.org/zombiezen/mygit",
|
|
goSource: "example.com/mygit https://bitbucket.org/zombiezen/mygit https://bitbucket.org/zombiezen/mygit/src/default{/dir} https://bitbucket.org/zombiezen/mygit/src/default{/dir}/{file}#{file}-{line}",
|
|
},
|
|
{
|
|
name: "subpath",
|
|
config: "host: example.com\n" +
|
|
"paths:\n" +
|
|
" /portmidi:\n" +
|
|
" repo: https://github.com/rakyll/portmidi\n" +
|
|
" display: https://github.com/rakyll/portmidi _ _\n",
|
|
path: "/portmidi/foo",
|
|
goImport: "example.com/portmidi git https://github.com/rakyll/portmidi",
|
|
goSource: "example.com/portmidi https://github.com/rakyll/portmidi _ _",
|
|
},
|
|
{
|
|
name: "subpath with trailing config slash",
|
|
config: "host: example.com\n" +
|
|
"paths:\n" +
|
|
" /portmidi/:\n" +
|
|
" repo: https://github.com/rakyll/portmidi\n" +
|
|
" display: https://github.com/rakyll/portmidi _ _\n",
|
|
path: "/portmidi/foo",
|
|
goImport: "example.com/portmidi git https://github.com/rakyll/portmidi",
|
|
goSource: "example.com/portmidi https://github.com/rakyll/portmidi _ _",
|
|
},
|
|
}
|
|
for _, test := range tests {
|
|
h, err := newHandler([]byte(test.config))
|
|
if err != nil {
|
|
t.Errorf("%s: newHandler: %v", test.name, err)
|
|
continue
|
|
}
|
|
s := httptest.NewServer(h)
|
|
resp, err := http.Get(s.URL + test.path)
|
|
if err != nil {
|
|
s.Close()
|
|
t.Errorf("%s: http.Get: %v", test.name, err)
|
|
continue
|
|
}
|
|
data, err := ioutil.ReadAll(resp.Body)
|
|
resp.Body.Close()
|
|
s.Close()
|
|
if resp.StatusCode != http.StatusOK {
|
|
t.Errorf("%s: status code = %s; want 200 OK", test.name, resp.Status)
|
|
}
|
|
if err != nil {
|
|
t.Errorf("%s: ioutil.ReadAll: %v", test.name, err)
|
|
continue
|
|
}
|
|
if got := findMeta(data, "go-import"); got != test.goImport {
|
|
t.Errorf("%s: meta go-import = %q; want %q", test.name, got, test.goImport)
|
|
}
|
|
if got := findMeta(data, "go-source"); got != test.goSource {
|
|
t.Errorf("%s: meta go-source = %q; want %q", test.name, got, test.goSource)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestBadConfigs(t *testing.T) {
|
|
badConfigs := []string{
|
|
"paths:\n" +
|
|
" /missingvcs:\n" +
|
|
" repo: https://bitbucket.org/zombiezen/gopdf\n",
|
|
"paths:\n" +
|
|
" /unknownvcs:\n" +
|
|
" repo: https://bitbucket.org/zombiezen/gopdf\n" +
|
|
" vcs: xyzzy\n",
|
|
"cache_max_age: -1\n" +
|
|
"paths:\n" +
|
|
" /portmidi:\n" +
|
|
" repo: https://github.com/rakyll/portmidi\n",
|
|
}
|
|
for _, config := range badConfigs {
|
|
_, err := newHandler([]byte(config))
|
|
if err == nil {
|
|
t.Errorf("expected config to produce an error, but did not:\n%s", config)
|
|
}
|
|
}
|
|
}
|
|
|
|
func findMeta(data []byte, name string) string {
|
|
var sep []byte
|
|
sep = append(sep, `<meta name="`...)
|
|
sep = append(sep, name...)
|
|
sep = append(sep, `" content="`...)
|
|
i := bytes.Index(data, sep)
|
|
if i == -1 {
|
|
return ""
|
|
}
|
|
content := data[i+len(sep):]
|
|
j := bytes.IndexByte(content, '"')
|
|
if j == -1 {
|
|
return ""
|
|
}
|
|
return string(content[:j])
|
|
}
|
|
|
|
func TestPathConfigSetFind(t *testing.T) {
|
|
tests := []struct {
|
|
paths []string
|
|
query string
|
|
want string
|
|
subpath string
|
|
}{
|
|
{
|
|
paths: []string{"/portmidi"},
|
|
query: "/portmidi",
|
|
want: "/portmidi",
|
|
},
|
|
{
|
|
paths: []string{"/portmidi"},
|
|
query: "/portmidi/",
|
|
want: "/portmidi",
|
|
},
|
|
{
|
|
paths: []string{"/portmidi"},
|
|
query: "/foo",
|
|
want: "",
|
|
},
|
|
{
|
|
paths: []string{"/portmidi"},
|
|
query: "/zzz",
|
|
want: "",
|
|
},
|
|
{
|
|
paths: []string{"/abc", "/portmidi", "/xyz"},
|
|
query: "/portmidi",
|
|
want: "/portmidi",
|
|
},
|
|
{
|
|
paths: []string{"/abc", "/portmidi", "/xyz"},
|
|
query: "/portmidi/foo",
|
|
want: "/portmidi",
|
|
subpath: "foo",
|
|
},
|
|
{
|
|
paths: []string{"/example/helloworld", "/", "/y", "/foo"},
|
|
query: "/x",
|
|
want: "/",
|
|
subpath: "x",
|
|
},
|
|
{
|
|
paths: []string{"/example/helloworld", "/", "/y", "/foo"},
|
|
query: "/",
|
|
want: "/",
|
|
subpath: "",
|
|
},
|
|
{
|
|
paths: []string{"/example/helloworld", "/", "/y", "/foo"},
|
|
query: "/example",
|
|
want: "/",
|
|
subpath: "example",
|
|
},
|
|
{
|
|
paths: []string{"/example/helloworld", "/", "/y", "/foo"},
|
|
query: "/example/foo",
|
|
want: "/",
|
|
subpath: "example/foo",
|
|
},
|
|
{
|
|
paths: []string{"/example/helloworld", "/", "/y", "/foo"},
|
|
query: "/y",
|
|
want: "/y",
|
|
},
|
|
{
|
|
paths: []string{"/example/helloworld", "/", "/y", "/foo"},
|
|
query: "/x/y/",
|
|
want: "/",
|
|
subpath: "x/y/",
|
|
},
|
|
{
|
|
paths: []string{"/example/helloworld", "/y", "/foo"},
|
|
query: "/x",
|
|
want: "",
|
|
},
|
|
}
|
|
emptyToNil := func(s string) string {
|
|
if s == "" {
|
|
return "<nil>"
|
|
}
|
|
return s
|
|
}
|
|
for _, test := range tests {
|
|
pset := make(pathConfigSet, len(test.paths))
|
|
for i := range test.paths {
|
|
pset[i].path = test.paths[i]
|
|
}
|
|
sort.Sort(pset)
|
|
pc, subpath := pset.find(test.query)
|
|
var got string
|
|
if pc != nil {
|
|
got = pc.path
|
|
}
|
|
if got != test.want || subpath != test.subpath {
|
|
t.Errorf("pathConfigSet(%v).find(%q) = %v, %v; want %v, %v",
|
|
test.paths, test.query, emptyToNil(got), subpath, emptyToNil(test.want), test.subpath)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestCacheHeader(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
config string
|
|
cacheControl string
|
|
}{
|
|
{
|
|
name: "default",
|
|
cacheControl: "public, max-age=86400",
|
|
},
|
|
{
|
|
name: "specify time",
|
|
config: "cache_max_age: 60\n",
|
|
cacheControl: "public, max-age=60",
|
|
},
|
|
{
|
|
name: "zero",
|
|
config: "cache_max_age: 0\n",
|
|
cacheControl: "public, max-age=0",
|
|
},
|
|
}
|
|
for _, test := range tests {
|
|
h, err := newHandler([]byte("paths:\n /portmidi:\n repo: https://github.com/rakyll/portmidi\n" +
|
|
test.config))
|
|
if err != nil {
|
|
t.Errorf("%s: newHandler: %v", test.name, err)
|
|
continue
|
|
}
|
|
s := httptest.NewServer(h)
|
|
resp, err := http.Get(s.URL + "/portmidi")
|
|
if err != nil {
|
|
t.Errorf("%s: http.Get: %v", test.name, err)
|
|
continue
|
|
}
|
|
resp.Body.Close()
|
|
got := resp.Header.Get("Cache-Control")
|
|
if got != test.cacheControl {
|
|
t.Errorf("%s: Cache-Control header = %q; want %q", test.name, got, test.cacheControl)
|
|
}
|
|
}
|
|
}
|