Skip to content

Commit 5109a9c

Browse files
committed
Add logging and the first test cases
1 parent 4560be4 commit 5109a9c

2 files changed

Lines changed: 88 additions & 0 deletions

File tree

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
package com.dkaedv.glghproxy;
2+
3+
import java.io.IOException;
4+
5+
import javax.servlet.Filter;
6+
import javax.servlet.FilterChain;
7+
import javax.servlet.FilterConfig;
8+
import javax.servlet.ServletException;
9+
import javax.servlet.ServletRequest;
10+
import javax.servlet.ServletResponse;
11+
import javax.servlet.http.HttpServletRequest;
12+
import javax.servlet.http.HttpServletResponse;
13+
14+
import org.apache.commons.logging.Log;
15+
import org.apache.commons.logging.LogFactory;
16+
import org.springframework.stereotype.Component;
17+
18+
@Component
19+
public class LoggingFilter implements Filter {
20+
private final static Log LOG = LogFactory.getLog(LoggingFilter.class);
21+
22+
@Override
23+
public void destroy() {
24+
}
25+
26+
@Override
27+
public void doFilter(ServletRequest req, ServletResponse res, FilterChain chain) throws IOException, ServletException {
28+
HttpServletResponse response = (HttpServletResponse) res;
29+
HttpServletRequest request = (HttpServletRequest) req;
30+
31+
LOG.info("Request to " + request.getRequestURI() + "?" + request.getQueryString());
32+
chain.doFilter(req, res);
33+
34+
request.getPathInfo();
35+
}
36+
37+
@Override
38+
public void init(FilterConfig arg0) throws ServletException {
39+
}
40+
}
Lines changed: 48 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,48 @@
1+
package com.dkaedv.glghproxy.converter;
2+
3+
import org.eclipse.egit.github.core.PullRequest;
4+
import org.gitlab.api.models.GitlabMergeRequest;
5+
import org.gitlab.api.models.GitlabUser;
6+
import org.junit.Test;
7+
8+
import static org.junit.Assert.*;
9+
10+
public class GitlabToGithubConverterTest {
11+
12+
@Test
13+
public void shouldConvertPullRequest() {
14+
GitlabMergeRequest mergeRequest = new GitlabMergeRequest();
15+
GitlabUser user = new GitlabUser();
16+
user.setEmail("hanswurscht@test.com");
17+
user.setId(5);
18+
mergeRequest.setAssignee(user);
19+
mergeRequest.setAuthor(user);
20+
mergeRequest.setId(15);
21+
mergeRequest.setIid(3);
22+
mergeRequest.setState("merged");
23+
24+
PullRequest pull = GitlabToGithubConverter.convertMergeRequest(mergeRequest, "http://gitlab", "testns", "test");
25+
26+
assertEquals("hanswurscht@test.com", pull.getAssignee().getEmail());
27+
assertEquals("http://gitlab/testns/test/merge_requests/3", pull.getHtmlUrl());
28+
29+
}
30+
31+
@Test
32+
public void shouldConvertMergedPullRequestWithNullAssignee() {
33+
GitlabMergeRequest mergeRequest = new GitlabMergeRequest();
34+
GitlabUser user = new GitlabUser();
35+
user.setEmail("hanswurscht@test.com");
36+
user.setUsername("hanswurscht");
37+
user.setId(5);
38+
mergeRequest.setAuthor(user);
39+
mergeRequest.setId(15);
40+
mergeRequest.setIid(3);
41+
mergeRequest.setState("merged");
42+
43+
PullRequest pull = GitlabToGithubConverter.convertMergeRequest(mergeRequest, "http://gitlab", "testns", "test");
44+
45+
assertEquals("hanswurscht@test.com", pull.getMergedBy().getEmail());
46+
47+
}
48+
}

0 commit comments

Comments
 (0)