Skip to content

Commit b548437

Browse files
committed
Add novos testes #52
1 parent 07a3470 commit b548437

File tree

2 files changed

+16
-4
lines changed

2 files changed

+16
-4
lines changed

src/main/java/br/ufrn/dct/apf/service/MemberService.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,12 @@ public Member save(Member member) throws BusinessRuleException {
4040
return memberDB;
4141
}
4242

43-
public Member getMember(Long projectId, Long memberId) {
44-
return repository.findByProjectIdAndUserId(projectId, memberId);
43+
public Member getMember(Long projectId, Long userId) {
44+
return repository.findByProjectIdAndUserId(projectId, userId);
4545
}
4646

47-
public Boolean isMember(Long projectId, Long memberId) {
48-
return getMember(projectId, memberId) != null;
47+
public Boolean isMember(Long projectId, Long userId) {
48+
return getMember(projectId, userId) != null;
4949
}
5050

5151
public List<Member> getMembersByProject(Long projectId) {

src/test/java/br/ufrn/dct/apf/service/TeamServiceTest.java

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -219,6 +219,13 @@ public void save() throws BusinessRuleException {
219219
softAssert.assertAll();
220220
}
221221

222+
@Test(expectedExceptions = BusinessRuleException.class,
223+
expectedExceptionsMessageRegExp = "error.member.service.member.is.null")
224+
public void saveMemberNull() throws BusinessRuleException {
225+
Member isNull = null;
226+
memberService.save(isNull);
227+
}
228+
222229
@Test
223230
public void saveMember() throws BusinessRuleException {
224231

@@ -275,6 +282,8 @@ public void saveMember() throws BusinessRuleException {
275282
softAssert.assertTrue(team2.contains(t02), "T05.4 - True:");
276283
softAssert.assertTrue(team2.contains(m1), "T05.5 - True:");
277284
softAssert.assertTrue(team2.contains(m2), "T05.6 - True:");
285+
softAssert.assertTrue(memberService.isMember(p1.getId(), m1.getUser().getId()), "T05.7 - True:");
286+
softAssert.assertTrue(memberService.isMember(p1.getId(), m2.getUser().getId()), "T05.8 - True:");
278287

279288
List<Project> projectByDev = projectService.findByUserId(desenvolvedor.getId());
280289
softAssert.assertEquals(projectByDev.size(), 1, "T06 - Equals:");
@@ -286,6 +295,9 @@ public void saveMember() throws BusinessRuleException {
286295
Project project = projectByDev.get(0);
287296
softAssert.assertEquals(project.getId(), p1.getId(), "T07 - Equals:");
288297

298+
List<Member> listMembers = memberService.getMembersByProject(p1.getId());
299+
softAssert.assertEquals(listMembers.size(), 2, "T08 - Equals:");
300+
289301
memberService.delete(m1.getId());
290302
memberService.delete(m2.getId());
291303

0 commit comments

Comments
 (0)