craigcitro / apitools / 77

89%

push

travis-ci

<a href="https://github.com/craigcitro/apitools/commit/<a class=hub.com/craigcitro/apitools/commit/4ad7475ebc1f0289b57041a575cc3fb3d38b72ab">4ad7475eb<a href="https://github.com/craigcitro/apitools/commit/4ad7475ebc1f0289b57041a575cc3fb3d38b72ab">&quot;&gt;Fix : before / in paths for apitools.

Currently, if we have a top-level methodPath of the form `path:methodName`,
we&amp;#39;ll pass this to `urlparse.urljoin`, where this is interpreted as
`scheme:netloc`. In cases other than media upload, this is **not** what we
want.

This fixes the issue by adding our own custom `urljoin`, which detects and
handles this case. We also add a test.

(This is basically a copy of
https://www.google.com/url?sa=D&amp;amp;q=https%3A%2F%2Fgithub.com%2Fgoogle%2Fgoogle-api-python-client%2Fcommit%2F</a><a class="double-link" href="https://github.com/craigcitro/apitools/commit/<a class="double-link" href="https://github.com/craigcitro/apitools/commit/7ee535d14007b3b05ca9cab1a35e93fb01657537">7ee535d14</a>">7ee535d14</a><a href="https://github.com/craigcitro/apitools/commit/4ad7475ebc1f0289b57041a575cc3fb3d38b72ab">
tweaked for apitools.)

(reviewed internally)

3409 of 5846 relevant lines covered (58.31%)

0.58 hits per line