2
50%
master: %

Ran 10 Nov 2017 03:02PM UTC

Files 79

Run time 2s

Badge

Embed ▾

Committed 10 Nov 2017 02:59PM UTC coverage: 46.056% (+0.1%) from 45.922%

Job # LATEST_GO=true

Build Type

push

travis-ci

Committed by

buger

Commit Message

loader: set up exactly one sub-router per host

Otherwise, multiple sub-routers per host (such as one per API
definition, what we were doing) can keep some of the listen paths from
being reachable. See the added comment for details.

Without the fix:

	--- FAIL: TestApiLoaderLongestPathFirst (0.15s)
		api_test.go:767: {host1 ab}: wanted /ab/ab, got /a/ab
		api_test.go:767: {host1 a/b/c}: wanted /a/b/c/a/b/c, got /a/a/b/c
		api_test.go:767: {host3 a/b/c}: wanted /a/b/c/a/b/c, got /a/a/b/c
		api_test.go:767: {host3 abc}: wanted /abc/abc, got /ab/abc
		api_test.go:767: {host1 abc}: wanted /abc/abc, got /a/abc
		api_test.go:767: {host2 abc}: wanted /abc/abc, got /a/abc
		api_test.go:767: {host2 ab}: wanted /ab/ab, got /a/ab
		api_test.go:767: {host1 ab/c}: wanted /ab/c/ab/c, got /a/ab/c
		api_test.go:767: {host3 ab/c}: wanted /ab/c/ab/c, got /ab/ab/c

Fixes #1271.

Run Details

5944 of 12906 relevant lines covered (46.06%)

0.5 hits per line

TykTechnologies / tyk / 3393 / 2
50%
master: %

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3393.2 (LATEST_GO=true)

TykTechnologies / tyk / 3393 / 2 50% master: %

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3393.2 (LATEST_GO=true)

TykTechnologies / tyk / 3393 / 2
50%
master: %

README BADGES
x