Add 'labels' field to PullRequest (#115)

Co-authored-by: Tim Besard <tim.besard@gmail.com>

Author: KristofferC

src/GitHub.jl

@@ -178,11 +178,11 @@ export # secrets.jl
 
 # include -------
 
+include("issues/labels.jl")
 include("issues/pull_requests.jl")
 include("issues/issues.jl")
 include("issues/comments.jl")
 include("issues/reviews.jl")
-include("issues/labels.jl")
 
 # export -------
 

src/issues/issues.jl

@@ -53,3 +53,31 @@ end
     result = gh_patch_json(api, "/repos/$(name(repo))/issues/$(name(issue))"; options...)
     return Issue(result)
 end
+
+# labels #
+#--------#
+
+@api_default function labels(api::GitHubAPI, repo, issue; options...)
+    result = gh_get_json(api, "/repos/$(name(repo))/issues/$(name(issue))/labels"; options...)
+    return Label.(result)
+end
+
+@api_default function add_labels(api::GitHubAPI, repo, issue, labels; options...)
+    result = gh_post_json(api, "/repos/$(name(repo))/issues/$(name(issue))/labels";
+                          params=Dict("labels" => name.(labels)), options...)
+    return Label.(result)
+end
+
+@api_default function set_labels(api::GitHubAPI, repo, issue, labels; options...)
+    result = gh_put_json(api, "/repos/$(name(repo))/issues/$(name(issue))/labels";
+                         params=Dict("labels" => name.(labels)), options...)
+    return Label.(result)
+end
+
+@api_default function remove_all_labels(api::GitHubAPI, repo, issue; options...)
+    return gh_delete(api, "/repos/$(name(repo))/issues/$(name(issue))/labels"; options...)
+end
+
+@api_default function remove_label(api::GitHubAPI, repo, issue, label; options...)
+    return gh_delete(api, "/repos/$(name(repo))/issues/$(name(issue))/labels/$(name(label))"; options...)
+end

src/issues/labels.jl

@@ -11,28 +11,3 @@ end
 namefield(label::Label) = label.name
 
 Label(name::AbstractString) = Label(Dict("name" => name))
-
-@api_default function labels(api::GitHubAPI, repo, issue; options...)
-    result = gh_get_json(api, "/repos/$(name(repo))/issues/$(name(issue))/labels"; options...)
-    return Label.(result)
-end
-
-@api_default function add_labels(api::GitHubAPI, repo, issue, labels; options...)
-    result = gh_post_json(api, "/repos/$(name(repo))/issues/$(name(issue))/labels";
-                          params=Dict("labels" => name.(labels)), options...)
-    return Label.(result)
-end
-
-@api_default function set_labels(api::GitHubAPI, repo, issue, labels; options...)
-    result = gh_put_json(api, "/repos/$(name(repo))/issues/$(name(issue))/labels";
-                         params=Dict("labels" => name.(labels)), options...)
-    return Label.(result)
-end
-
-@api_default function remove_all_labels(api::GitHubAPI, repo, issue; options...)
-    return gh_delete(api, "/repos/$(name(repo))/issues/$(name(issue))/labels"; options...)
-end
-
-@api_default function remove_label(api::GitHubAPI, repo, issue, label; options...)
-    return gh_delete(api, "/repos/$(name(repo))/issues/$(name(issue))/labels/$(name(label))"; options...)
-end

src/issues/pull_requests.jl

@@ -30,6 +30,7 @@
     mergeable::Union{Bool, Nothing}
     merged::Union{Bool, Nothing}
     locked::Union{Bool, Nothing}
+    labels::Union{Vector{Label}, Nothing}
 end
 
 PullRequest(number::Real) = PullRequest(Dict("number" => number))

test/ghtype_tests.jl

@@ -471,7 +471,8 @@ end
         nothing,
         nothing,
         nothing,
-        pr_json["locked"]
+        pr_json["locked"],
+        nothing
     )
 
     @test PullRequest(pr_json) == pr_result

test/read_only_api_tests.jl

@@ -110,7 +110,11 @@ end
     state_param = Dict("state" => "all")
 
     # test GitHub.pull_request, GitHub.pull_requests
-    @test pull_request(ghjl, 37; auth = auth).title == "Fix dep warnings"
+    let pr = pull_request(ghjl, 37; auth = auth)
+        @test pr.title == "Fix dep warnings"
+        @test length(pr.labels) == 1
+        @test pr.labels[1].name == "enhancement"
+    end
     @test hasghobj(37, first(pull_requests(ghjl; auth = auth, params = state_param)))
 
     # test GitHub.issue, GitHub.issues