Skip to content

Commit 17ce609

Browse files
author
Derek Wasinger
committed
test cleanup
1 parent 77f200f commit 17ce609

22 files changed

+361
-473
lines changed

src/main/java/com/redhat/labs/omp/models/gitlab/Namespace.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,12 @@
33
import javax.json.bind.annotation.JsonbProperty;
44

55
import lombok.AllArgsConstructor;
6+
import lombok.Builder;
67
import lombok.Data;
78
import lombok.NoArgsConstructor;
89

910
@Data
11+
@Builder
1012
@NoArgsConstructor
1113
@AllArgsConstructor
1214
public class Namespace {

src/main/java/com/redhat/labs/omp/models/gitlab/ProjectSearchResults.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,12 @@
33
import javax.json.bind.annotation.JsonbProperty;
44

55
import lombok.AllArgsConstructor;
6+
import lombok.Builder;
67
import lombok.Data;
78
import lombok.NoArgsConstructor;
89

910
@Data
11+
@Builder
1012
@NoArgsConstructor
1113
@AllArgsConstructor
1214
public class ProjectSearchResults {

src/main/java/com/redhat/labs/omp/resources/CacheResource.java renamed to src/main/java/com/redhat/labs/omp/resource/CacheResource.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.redhat.labs.omp.resources;
1+
package com.redhat.labs.omp.resource;
22

33
import java.util.Optional;
44

@@ -17,7 +17,7 @@
1717
import com.redhat.labs.cache.cacheStore.EngagementDataCache;
1818
import com.redhat.labs.exception.FileNotFoundException;
1919
import com.redhat.labs.omp.models.gitlab.File;
20-
import com.redhat.labs.omp.resources.filters.Logged;
20+
import com.redhat.labs.omp.resources.filter.Logged;
2121
import com.redhat.labs.omp.service.FileService;
2222

2323
@Path("/api/cache")

src/main/java/com/redhat/labs/omp/resources/EngagementResource.java renamed to src/main/java/com/redhat/labs/omp/resource/EngagementResource.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.redhat.labs.omp.resources;
1+
package com.redhat.labs.omp.resource;
22

33
import javax.inject.Inject;
44
import javax.ws.rs.Consumes;
@@ -37,7 +37,7 @@ public class EngagementResource {
3737
public Response createEngagement(Engagement engagement, @Context UriInfo uriInfo) {
3838

3939
Project project = engagementService.createEngagement(engagement);
40-
40+
4141
UriBuilder builder = uriInfo.getAbsolutePathBuilder();
4242
builder.path(Integer.toString(project.getId()));
4343
return Response.created(builder.build()).build();

src/main/java/com/redhat/labs/omp/resources/FileResource.java renamed to src/main/java/com/redhat/labs/omp/resource/FileResource.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.redhat.labs.omp.resources;
1+
package com.redhat.labs.omp.resource;
22

33
import java.util.Optional;
44

src/main/java/com/redhat/labs/omp/resources/GroupResource.java renamed to src/main/java/com/redhat/labs/omp/resource/GroupResource.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.redhat.labs.omp.resources;
1+
package com.redhat.labs.omp.resource;
22

33
import java.util.Optional;
44

Lines changed: 36 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,36 +1,56 @@
1-
package com.redhat.labs.omp.resources;
1+
package com.redhat.labs.omp.resource;
22

33
import java.util.Optional;
44

55
import javax.inject.Inject;
66
import javax.ws.rs.Consumes;
77
import javax.ws.rs.GET;
8+
import javax.ws.rs.POST;
89
import javax.ws.rs.Path;
910
import javax.ws.rs.Produces;
1011
import javax.ws.rs.QueryParam;
12+
import javax.ws.rs.core.Context;
1113
import javax.ws.rs.core.MediaType;
1214
import javax.ws.rs.core.Response;
15+
import javax.ws.rs.core.UriBuilder;
16+
import javax.ws.rs.core.UriInfo;
1317

1418
import org.apache.http.HttpStatus;
1519
import org.eclipse.microprofile.config.inject.ConfigProperty;
1620

21+
import com.redhat.labs.omp.models.Engagement;
1722
import com.redhat.labs.omp.models.gitlab.File;
18-
import com.redhat.labs.omp.resources.filters.Logged;
23+
import com.redhat.labs.omp.models.gitlab.Project;
24+
import com.redhat.labs.omp.resources.filter.Logged;
25+
import com.redhat.labs.omp.service.EngagementService;
1926
import com.redhat.labs.omp.service.FileService;
2027

21-
@Path("/api/file")
28+
@Path("/api")
2229
@Produces(MediaType.APPLICATION_JSON)
2330
@Consumes(MediaType.APPLICATION_JSON)
24-
public class LegacyFileResource {
31+
public class LegacyResource {
32+
33+
/*
34+
*
35+
*
36+
* NOTE: This resource should be removed once the OMP Backend API
37+
* has been updated to point to the /api/v1/* endpoints
38+
*
39+
*
40+
*/
2541

2642
@Inject
2743
FileService fileService;
2844

45+
@Inject
46+
EngagementService engagementService;
47+
2948
@ConfigProperty(name = "file_branch", defaultValue = "master")
3049
protected String defaultBranch;
3150

3251
@GET
3352
@Logged
53+
@Path("/file")
3454
public Response getFileFromGitByName(@QueryParam("name") String fileName, @QueryParam("repo_id") Integer repoId,
3555
@QueryParam("branch") String branch) {
3656
Optional<File> optional = fileService.getFile(repoId, fileName, (null == branch) ? defaultBranch : branch);
@@ -42,4 +62,16 @@ public Response getFileFromGitByName(@QueryParam("name") String fileName, @Query
4262

4363
}
4464

65+
@POST
66+
@Path("/residencies")
67+
public Response createEngagement(Engagement engagement, @Context UriInfo uriInfo) {
68+
69+
Project project = engagementService.createEngagement(engagement);
70+
71+
UriBuilder builder = uriInfo.getAbsolutePathBuilder();
72+
builder.path(Integer.toString(project.getId()));
73+
return Response.created(builder.build()).build();
74+
75+
}
76+
4577
}

src/main/java/com/redhat/labs/omp/resources/ProjectResource.java renamed to src/main/java/com/redhat/labs/omp/resource/ProjectResource.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.redhat.labs.omp.resources;
1+
package com.redhat.labs.omp.resource;
22

33
import java.util.Optional;
44

src/main/java/com/redhat/labs/omp/resources/filters/Logged.java renamed to src/main/java/com/redhat/labs/omp/resources/filter/Logged.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.redhat.labs.omp.resources.filters;
1+
package com.redhat.labs.omp.resources.filter;
22

33
import javax.ws.rs.NameBinding;
44
import java.lang.annotation.Retention;

src/main/java/com/redhat/labs/omp/resources/filters/RequestLoggingFilter.java renamed to src/main/java/com/redhat/labs/omp/resources/filter/RequestLoggingFilter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.redhat.labs.omp.resources.filters;
1+
package com.redhat.labs.omp.resources.filter;
22

33
import org.slf4j.Logger;
44
import org.slf4j.LoggerFactory;

0 commit comments

Comments
 (0)