-
Notifications
You must be signed in to change notification settings - Fork 735
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support getting repository from project card #1260
Support getting repository from project card #1260
Conversation
Blocked by #1261 |
@@ -55,6 +55,8 @@ public void testCreateCardFromIssue() throws IOException { | |||
GHIssue issue = repo.createIssue("new-issue").body("With body").create(); | |||
GHProjectCard card = column.createCard(issue); | |||
assertThat(card.getContentUrl(), equalTo(issue.getUrl())); | |||
assertThat(card.getContent(), notNullValue()); | |||
assertThat(card.getContent().getRepository(), notNullValue()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to assert some meaningful information, such as Id and/or description. Just notNullValue()
is not a sufficient test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, this was just to get the right API calls being made.
Signed-off-by: Jonathan Leitschuh <[email protected]>
19e9a95
to
1cdbc17
Compare
Codecov Report
@@ Coverage Diff @@
## main #1260 +/- ##
============================================
+ Coverage 77.47% 77.50% +0.02%
- Complexity 1901 1905 +4
============================================
Files 189 189
Lines 5985 6000 +15
Branches 327 328 +1
============================================
+ Hits 4637 4650 +13
Misses 1150 1150
- Partials 198 200 +2
Continue to review full report at Codecov.
|
Co-authored-by: Liam Newman <[email protected]>
8ad4369
to
0c71293
Compare
Hey @bitwiseman, I'm currently participating in Hacktoberfest. If you think this PR was useful, would you be so kind as to add the |
Closes #1097
Before submitting a PR:
We love getting PRs, but we hate asking people for the same basic changes every time.
main
. Create your PR from that branch.mvn -D enable-ci clean install site
locally. If this command doesn't succeed, your change will not pass CI.When creating a PR: