Skip to content

Commit

Permalink
fix some deprecation warnings, 0.7 compatibility and test on mac
Browse files Browse the repository at this point in the history
  • Loading branch information
KristofferC committed Apr 16, 2018
1 parent 1db899c commit 87b11a2
Show file tree
Hide file tree
Showing 10 changed files with 37 additions and 32 deletions.
3 changes: 3 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,7 @@
language: julia
os:
- linux
- osx
julia:
- 0.6
- nightly
Expand Down
2 changes: 1 addition & 1 deletion REQUIRE
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
julia 0.6

Compat 0.43
Compat 0.62
JSON
MbedTLS
HTTP 0.6.3
Expand Down
4 changes: 2 additions & 2 deletions src/activity/events.jl
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ sig_header(request::HTTP.Request) = HTTP.header(request, "X-Hub-Signature")

function has_valid_secret(request::HTTP.Request, secret)
if has_sig_header(request)
secret_sha = "sha1="*bytes2hex(MbedTLS.digest(MbedTLS.MD_SHA1, HTTP.payload(request, String), secret))
secret_sha = "sha1="*bytes2hex(MbedTLS.digest(MbedTLS.MD_SHA1, HTTP.payload(request), secret))
return sig_header(request) == secret_sha
end
return false
Expand Down Expand Up @@ -97,7 +97,7 @@ function handle_event_request(request, handle;
return HTTP.Response(204, "event ignored")
end

event = event_from_payload!(event_header(request), JSON.parse(HTTP.payload(request, String)))
event = event_from_payload!(event_header(request), JSON.parse(IOBuffer(HTTP.payload(request))))

if !(isa(repos, Nothing)) && !(from_valid_repo(event, repos))
return HTTP.Response(400, "invalid repo")
Expand Down
4 changes: 2 additions & 2 deletions src/git/reference.jl
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,12 @@ name(ref::Reference) = String(split(get(ref.ref), "refs/")[2])

@api_default function reference(api::GitHubAPI, repo, ref_obj; options...)
result = gh_get_json(api, "/repos/$(name(repo))/git/refs/$(name(ref_obj))"; options...)
return Reference.(result)
return Reference(result)
end

@api_default function references(api::GitHubAPI, repo; options...)
results, page_data = gh_get_paged_json(api, "/repos/$(name(repo))/git/refs"; options...)
return Reference.(results), page_data
return Reference.((results,)), page_data
end

@api_default function create_reference(api::GitHubAPI, repo; options...)
Expand Down
42 changes: 21 additions & 21 deletions src/utils/GitHubType.jl
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ end
@generated function json2github(::Type{G}, data::Dict) where {G<:GitHubType}
types = G.types
fields = fieldnames(G)
args = Vector{Expr}(length(fields))
args = Vector{Expr}(undef, length(fields))
for i in eachindex(fields)
field, T = fields[i], first(types[i].parameters)
key = field == :typ ? "type" : string(field)
Expand Down Expand Up @@ -125,27 +125,27 @@ end
###################

function Base.show(io::IO, g::GitHubType)
print(io, "$(typeof(g)) (all fields are Nullable):")
for field in fieldnames(typeof(g))
val = getfield(g, field)
if !(isnull(val))
gotval = get(val)
println(io)
print(io, " $field: ")
if isa(gotval, Vector)
print(io, typeof(gotval))
else
showcompact(io, gotval)
end
if get(io, :compact, false)
uri_id = namefield(g)
if isnull(uri_id)
print(io, typeof(g), "(…)")
else
print(io, typeof(g), "($(repr(get(uri_id))))")
end
end
end

function Base.showcompact(io::IO, g::GitHubType)
uri_id = namefield(g)
if isnull(uri_id)
print(io, typeof(g), "(…)")
else
print(io, typeof(g), "($(repr(get(uri_id))))")
print(io, "$(typeof(g)) (all fields are Nullable):")
for field in fieldnames(typeof(g))
val = getfield(g, field)
if !(isnull(val))
gotval = get(val)
println(io)
print(io, " $field: ")
if isa(gotval, Vector)
print(io, typeof(gotval))
else
show(IOContext(io, :compact => true), gotval)
end
end
end
end
end
2 changes: 1 addition & 1 deletion src/utils/requests.jl
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ get_page_links(r) = split(HTTP.header(r, "Link",), ",")
function find_page_link(links, rel)
relstr = "rel=\"$(rel)\""
for i in 1:length(links)
if contains(links[i], relstr)
if occursin(relstr, links[i])
return i
end
end
Expand Down
2 changes: 2 additions & 0 deletions test/commit_comment.jl
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
using Compat.Base64

function create_event()
data_base64 = "eyJhY3Rpb24iOiJjcmVhdGVkIiwiY29tbWVudCI6eyJ1cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvY29tbWVudHMvMTQwNDcwNTgiLCJodG1sX3VybCI6Imh0dHBzOi8vZ2l0aHViLmNvbS9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2NvbW1pdC8zMmQzNWYyODU3NzdiMDc3ZDhiNmEyNTIxMzA5ZDFhYjY0NmQyMzc5I2NvbW1pdGNvbW1lbnQtMTQwNDcwNTgiLCJpZCI6MTQwNDcwNTgsInVzZXIiOnsibG9naW4iOiJqcmV2ZWxzIiwiaWQiOjMyNzc0NDMsImF2YXRhcl91cmwiOiJodHRwczovL2F2YXRhcnMuZ2l0aHVidXNlcmNvbnRlbnQuY29tL3UvMzI3NzQ0Mz92PTMiLCJncmF2YXRhcl9pZCI6IiIsInVybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscyIsImh0bWxfdXJsIjoiaHR0cHM6Ly9naXRodWIuY29tL2pyZXZlbHMiLCJmb2xsb3dlcnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL2ZvbGxvd2VycyIsImZvbGxvd2luZ191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMvZm9sbG93aW5ney9vdGhlcl91c2VyfSIsImdpc3RzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9naXN0c3svZ2lzdF9pZH0iLCJzdGFycmVkX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9zdGFycmVkey9vd25lcn17L3JlcG99Iiwic3Vic2NyaXB0aW9uc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMvc3Vic2NyaXB0aW9ucyIsIm9yZ2FuaXphdGlvbnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL29yZ3MiLCJyZXBvc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMvcmVwb3MiLCJldmVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL2V2ZW50c3svcHJpdmFjeX0iLCJyZWNlaXZlZF9ldmVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL3JlY2VpdmVkX2V2ZW50cyIsInR5cGUiOiJVc2VyIiwic2l0ZV9hZG1pbiI6ZmFsc2V9LCJwb3NpdGlvbiI6bnVsbCwibGluZSI6bnVsbCwicGF0aCI6IiIsImNvbW1pdF9pZCI6IjMyZDM1ZjI4NTc3N2IwNzdkOGI2YTI1MjEzMDlkMWFiNjQ2ZDIzNzkiLCJjcmVhdGVkX2F0IjoiMjAxNS0xMC0yOFQxNzo1Njo0OVoiLCJ1cGRhdGVkX2F0IjoiMjAxNS0xMC0yOFQxNzo1Njo0OVoiLCJib2R5IjoiYFJ1bkJlbmNobWFya3MoXCJiaW5hcnlcIiwgXCJ1bmFyeVwiKWAifSwicmVwb3NpdG9yeSI6eyJpZCI6NDQxODkxMDcsIm5hbWUiOiJCZW5jaG1hcmtUcmFja2Vycy5qbCIsImZ1bGxfbmFtZSI6Ikp1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwiLCJvd25lciI6eyJsb2dpbiI6Ikp1bGlhQ0kiLCJpZCI6OTk1NzYwNCwiYXZhdGFyX3VybCI6Imh0dHBzOi8vYXZhdGFycy5naXRodWJ1c2VyY29udGVudC5jb20vdS85OTU3NjA0P3Y9MyIsImdyYXZhdGFyX2lkIjoiIiwidXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9KdWxpYUNJIiwiaHRtbF91cmwiOiJodHRwczovL2dpdGh1Yi5jb20vSnVsaWFDSSIsImZvbGxvd2Vyc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL0p1bGlhQ0kvZm9sbG93ZXJzIiwiZm9sbG93aW5nX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvSnVsaWFDSS9mb2xsb3dpbmd7L290aGVyX3VzZXJ9IiwiZ2lzdHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9KdWxpYUNJL2dpc3Rzey9naXN0X2lkfSIsInN0YXJyZWRfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9KdWxpYUNJL3N0YXJyZWR7L293bmVyfXsvcmVwb30iLCJzdWJzY3JpcHRpb25zX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvSnVsaWFDSS9zdWJzY3JpcHRpb25zIiwib3JnYW5pemF0aW9uc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL0p1bGlhQ0kvb3JncyIsInJlcG9zX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvSnVsaWFDSS9yZXBvcyIsImV2ZW50c191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL0p1bGlhQ0kvZXZlbnRzey9wcml2YWN5fSIsInJlY2VpdmVkX2V2ZW50c191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL0p1bGlhQ0kvcmVjZWl2ZWRfZXZlbnRzIiwidHlwZSI6Ik9yZ2FuaXphdGlvbiIsInNpdGVfYWRtaW4iOmZhbHNlfSwicHJpdmF0ZSI6ZmFsc2UsImh0bWxfdXJsIjoiaHR0cHM6Ly9naXRodWIuY29tL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwiLCJkZXNjcmlwdGlvbiI6IlRyYWNrIGEgSnVsaWEgcGFja2FnZSdzIGJlbmNobWFyayByZXN1bHRzIGFzIHBhcnQgb2YgQ0kiLCJmb3JrIjpmYWxzZSwidXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsIiwiZm9ya3NfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2ZvcmtzIiwia2V5c191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwva2V5c3sva2V5X2lkfSIsImNvbGxhYm9yYXRvcnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2NvbGxhYm9yYXRvcnN7L2NvbGxhYm9yYXRvcn0iLCJ0ZWFtc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvdGVhbXMiLCJob29rc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvaG9va3MiLCJpc3N1ZV9ldmVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2lzc3Vlcy9ldmVudHN7L251bWJlcn0iLCJldmVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2V2ZW50cyIsImFzc2lnbmVlc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvYXNzaWduZWVzey91c2VyfSIsImJyYW5jaGVzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9icmFuY2hlc3svYnJhbmNofSIsInRhZ3NfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL3RhZ3MiLCJibG9ic191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvZ2l0L2Jsb2Jzey9zaGF9IiwiZ2l0X3RhZ3NfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2dpdC90YWdzey9zaGF9IiwiZ2l0X3JlZnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2dpdC9yZWZzey9zaGF9IiwidHJlZXNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2dpdC90cmVlc3svc2hhfSIsInN0YXR1c2VzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9zdGF0dXNlcy97c2hhfSIsImxhbmd1YWdlc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvbGFuZ3VhZ2VzIiwic3RhcmdhemVyc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvc3RhcmdhemVycyIsImNvbnRyaWJ1dG9yc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvY29udHJpYnV0b3JzIiwic3Vic2NyaWJlcnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL3N1YnNjcmliZXJzIiwic3Vic2NyaXB0aW9uX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9zdWJzY3JpcHRpb24iLCJjb21taXRzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9jb21taXRzey9zaGF9IiwiZ2l0X2NvbW1pdHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2dpdC9jb21taXRzey9zaGF9IiwiY29tbWVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2NvbW1lbnRzey9udW1iZXJ9IiwiaXNzdWVfY29tbWVudF91cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvaXNzdWVzL2NvbW1lbnRzey9udW1iZXJ9IiwiY29udGVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2NvbnRlbnRzL3srcGF0aH0iLCJjb21wYXJlX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9jb21wYXJlL3tiYXNlfS4uLntoZWFkfSIsIm1lcmdlc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvbWVyZ2VzIiwiYXJjaGl2ZV91cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwve2FyY2hpdmVfZm9ybWF0fXsvcmVmfSIsImRvd25sb2Fkc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvZG93bmxvYWRzIiwiaXNzdWVzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9pc3N1ZXN7L251bWJlcn0iLCJwdWxsc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvcHVsbHN7L251bWJlcn0iLCJtaWxlc3RvbmVzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9taWxlc3RvbmVzey9udW1iZXJ9Iiwibm90aWZpY2F0aW9uc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvbm90aWZpY2F0aW9uc3s/c2luY2UsYWxsLHBhcnRpY2lwYXRpbmd9IiwibGFiZWxzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9sYWJlbHN7L25hbWV9IiwicmVsZWFzZXNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL3JlbGVhc2Vzey9pZH0iLCJjcmVhdGVkX2F0IjoiMjAxNS0xMC0xM1QxNjoxNzowNFoiLCJ1cGRhdGVkX2F0IjoiMjAxNS0xMC0yMlQxNDo0NTowNFoiLCJwdXNoZWRfYXQiOiIyMDE1LTEwLTI4VDE3OjUyOjM2WiIsImdpdF91cmwiOiJnaXQ6Ly9naXRodWIuY29tL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwuZ2l0Iiwic3NoX3VybCI6ImdpdEBnaXRodWIuY29tOkp1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwuZ2l0IiwiY2xvbmVfdXJsIjoiaHR0cHM6Ly9naXRodWIuY29tL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwuZ2l0Iiwic3ZuX3VybCI6Imh0dHBzOi8vZ2l0aHViLmNvbS9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsIiwiaG9tZXBhZ2UiOm51bGwsInNpemUiOjM5Nywic3RhcmdhemVyc19jb3VudCI6MSwid2F0Y2hlcnNfY291bnQiOjEsImxhbmd1YWdlIjoiSnVsaWEiLCJoYXNfaXNzdWVzIjp0cnVlLCJoYXNfZG93bmxvYWRzIjp0cnVlLCJoYXNfd2lraSI6dHJ1ZSwiaGFzX3BhZ2VzIjpmYWxzZSwiZm9ya3NfY291bnQiOjEsIm1pcnJvcl91cmwiOm51bGwsIm9wZW5faXNzdWVzX2NvdW50IjoxLCJmb3JrcyI6MSwib3Blbl9pc3N1ZXMiOjEsIndhdGNoZXJzIjoxLCJkZWZhdWx0X2JyYW5jaCI6Im1hc3RlciJ9LCJvcmdhbml6YXRpb24iOnsibG9naW4iOiJKdWxpYUNJIiwiaWQiOjk5NTc2MDQsInVybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vb3Jncy9KdWxpYUNJIiwicmVwb3NfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9vcmdzL0p1bGlhQ0kvcmVwb3MiLCJldmVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9vcmdzL0p1bGlhQ0kvZXZlbnRzIiwibWVtYmVyc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL29yZ3MvSnVsaWFDSS9tZW1iZXJzey9tZW1iZXJ9IiwicHVibGljX21lbWJlcnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9vcmdzL0p1bGlhQ0kvcHVibGljX21lbWJlcnN7L21lbWJlcn0iLCJhdmF0YXJfdXJsIjoiaHR0cHM6Ly9hdmF0YXJzLmdpdGh1YnVzZXJjb250ZW50LmNvbS91Lzk5NTc2MDQ/dj0zIiwiZGVzY3JpcHRpb24iOiJDb250aW5vdXMgSW50ZWdyYXRpb24gKENJKSBTdXBwb3J0IGZvciB0aGUgSnVsaWEgTGFuZ3VhZ2UifSwic2VuZGVyIjp7ImxvZ2luIjoianJldmVscyIsImlkIjozMjc3NDQzLCJhdmF0YXJfdXJsIjoiaHR0cHM6Ly9hdmF0YXJzLmdpdGh1YnVzZXJjb250ZW50LmNvbS91LzMyNzc0NDM/dj0zIiwiZ3JhdmF0YXJfaWQiOiIiLCJ1cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMiLCJodG1sX3VybCI6Imh0dHBzOi8vZ2l0aHViLmNvbS9qcmV2ZWxzIiwiZm9sbG93ZXJzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9mb2xsb3dlcnMiLCJmb2xsb3dpbmdfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL2ZvbGxvd2luZ3svb3RoZXJfdXNlcn0iLCJnaXN0c191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMvZ2lzdHN7L2dpc3RfaWR9Iiwic3RhcnJlZF91cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMvc3RhcnJlZHsvb3duZXJ9ey9yZXBvfSIsInN1YnNjcmlwdGlvbnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL3N1YnNjcmlwdGlvbnMiLCJvcmdhbml6YXRpb25zX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9vcmdzIiwicmVwb3NfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL3JlcG9zIiwiZXZlbnRzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9ldmVudHN7L3ByaXZhY3l9IiwicmVjZWl2ZWRfZXZlbnRzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9yZWNlaXZlZF9ldmVudHMiLCJ0eXBlIjoiVXNlciIsInNpdGVfYWRtaW4iOmZhbHNlfX0="

Expand Down
2 changes: 1 addition & 1 deletion test/event_tests.jl
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
include("commit_comment.jl")
event_request = create_event()
event_json = JSON.parse(HTTP.payload(event_request, String))
event_json = JSON.parse(IOBuffer(HTTP.payload(event_request)))
event = GitHub.event_from_payload!("commit_comment", event_json)

@testset "WebhookEvent" begin
Expand Down
6 changes: 3 additions & 3 deletions test/ghtype_tests.jl
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ function test_show(g::GitHub.GitHubType)
@test repr(g) == String(take!(tmpio))

tmpio = IOBuffer()
showcompact(tmpio, g)
show(IOContext(tmpio, :compact => true), g)

@test "$(typeof(g))($(repr(name(g))))" == String(take!(tmpio))
end
Expand Down Expand Up @@ -540,7 +540,7 @@ end

test_show(hook_result)
end

@testset "Gist" begin
gist_json = JSON.parse("""
{
Expand Down Expand Up @@ -734,7 +734,7 @@ end
"updated_at": "2017-07-08T16:18:44"
}
""")

app_result = App(app_json)
@test name(app_result) == Int(app_json["id"])
end
Expand Down
2 changes: 1 addition & 1 deletion test/read_only_api_tests.jl
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@ end
@test entry["type"] == "blob"

b = blob(github_jl, entry["sha"]; auth=auth)
@test contains(String(base64decode(replace(get(b.content),"\n" => ""))), "GitHub.jl")
@test occursin("GitHub.jl", String(base64decode(replace(get(b.content),"\n" => ""))))

break
end
Expand Down

0 comments on commit 87b11a2

Please sign in to comment.