Skip to content

Commit 6a0d9bb

Browse files
committed
Added tests for merge request notes (#140).
1 parent 54e58dc commit 6a0d9bb

File tree

1 file changed

+39
-3
lines changed

1 file changed

+39
-3
lines changed

src/test/java/org/gitlab4j/api/TestNotesApi.java

Lines changed: 39 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
11
package org.gitlab4j.api;
22

3+
import org.gitlab4j.api.Constants.SortOrder;
34
import org.gitlab4j.api.GitLabApi.ApiVersion;
45
import org.gitlab4j.api.models.Issue;
6+
import org.gitlab4j.api.models.MergeRequest;
57
import org.gitlab4j.api.models.Note;
68
import org.gitlab4j.api.models.Project;
79
import org.junit.Before;
@@ -79,16 +81,50 @@ public void beforeMethod() {
7981
}
8082

8183
@Test
82-
public void testNotes() throws GitLabApiException {
84+
public void testIssueNotes() throws GitLabApiException {
8385

8486
Project project = gitLabApi.getProjectApi().getProject(TEST_NAMESPACE, TEST_PROJECT_NAME);
8587
assertNotNull(project);
8688

8789
for (Issue issue : gitLabApi.getIssuesApi().getIssues(project.getId())) {
8890
List<Note> notes = gitLabApi.getNotesApi().getIssueNotes(project.getId(), issue.getIid());
8991
assertNotNull(notes);
90-
// This requires some issues in the project
91-
// assertTrue(0 < notes.size());
92+
}
93+
}
94+
95+
@Test
96+
public void testIssueNotesPager() throws GitLabApiException {
97+
98+
Project project = gitLabApi.getProjectApi().getProject(TEST_NAMESPACE, TEST_PROJECT_NAME);
99+
assertNotNull(project);
100+
101+
for (Issue issue : gitLabApi.getIssuesApi().getIssues(project.getId())) {
102+
Pager<Note> pager = gitLabApi.getNotesApi().getIssueNotes(project.getId(), issue.getIid(), 10);
103+
assertNotNull(pager);
104+
}
105+
}
106+
107+
@Test
108+
public void testMergeRequestNotes() throws GitLabApiException {
109+
110+
Project project = gitLabApi.getProjectApi().getProject(TEST_NAMESPACE, TEST_PROJECT_NAME);
111+
assertNotNull(project);
112+
113+
for (MergeRequest mr : gitLabApi.getMergeRequestApi().getMergeRequests(project.getId())) {
114+
List<Note> notes = gitLabApi.getNotesApi().getMergeRequestNotes(project.getId(), mr.getIid());
115+
assertNotNull(notes);
116+
}
117+
}
118+
119+
@Test
120+
public void testMergeRequestNotesPager() throws GitLabApiException {
121+
122+
Project project = gitLabApi.getProjectApi().getProject(TEST_NAMESPACE, TEST_PROJECT_NAME);
123+
assertNotNull(project);
124+
125+
for (MergeRequest mr : gitLabApi.getMergeRequestApi().getMergeRequests(project.getId())) {
126+
Pager<Note> pager = gitLabApi.getNotesApi().getMergeRequestNotes(project.getId(), mr.getIid(), SortOrder.DESC, Note.OrderBy.CREATED_AT, 10);
127+
assertNotNull(pager);
92128
}
93129
}
94130
}

0 commit comments

Comments
 (0)