Skip to content
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

[WFLY-4922] Add smoke test for <deny-uncovered-http-methods/> tag #7748

Merged
merged 1 commit into from Jul 14, 2015

Conversation

jtymel
Copy link

@jtymel jtymel commented Jul 14, 2015

With Java Servlet 3.1 specification comes possibility to use tag in web.xml file. This new feature should be tested by a basic smoke test.

https://issues.jboss.org/browse/WFLY-4922

@wildfly-ci
Copy link

Linux with security manager Build 1882 is now running using a merge of 7ea925a

@wildfly-ci
Copy link

Windows Build 2059 is now running using a merge of 7ea925a

@wildfly-ci
Copy link

Linux Build 6998 is now running using a merge of 7ea925a

@wildfly-ci
Copy link

Linux with security manager Build 1882 outcome was SUCCESS using a merge of 7ea925a
Summary: Tests passed: 871, ignored: 280 Build time: 0:06:15

@wildfly-ci
Copy link

Windows Build 2059 outcome was FAILURE using a merge of 7ea925a
Summary: Tests failed: 1 (1 new), passed: 2921, ignored: 342 Build time: 0:48:31

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.clustering.cluster.ejb.remote.RemoteFailoverTestCase(SYNC-tcp).testStatelessFailover: <no details avaliable>

@wildfly-ci
Copy link

Linux Build 6998 outcome was SUCCESS using a merge of 7ea925a
Summary: Tests passed: 2921, ignored: 342, muted: 1 Build time: 0:50:59


private HttpResponse getHttpResponse(HttpUriRequest request, String user, String password) throws IOException {
String auth = user + ":" + password;
byte[] encodedAuth = Base64.encodeBase64(auth.getBytes(Charset.forName("US-ASCII")));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use JDK8's java.util.Base64 instead of external lib

@jtymel
Copy link
Author

jtymel commented Jul 14, 2015

Changes made. Could anyone review the code once again, please?

@wildfly-ci
Copy link

Linux Build 7001 is now running using a merge of 9f23014

@wildfly-ci
Copy link

Windows Build 2061 is now running using a merge of 9f23014

@wildfly-ci
Copy link

Linux with security manager Build 1884 is now running using a merge of 9f23014

@wildfly-ci
Copy link

Linux with security manager Build 1884 outcome was SUCCESS using a merge of 9f23014
Summary: Tests passed: 871, ignored: 280 Build time: 0:06:49

import java.io.IOException;
import java.net.URL;
import java.nio.charset.Charset;
import java.util.Base64;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't used anymore after auth was removed

@wildfly-ci
Copy link

Windows Build 2061 outcome was SUCCESS using a merge of 9f23014
Summary: Tests passed: 2922, ignored: 342 Build time: 0:50:03

@wildfly-ci
Copy link

Linux Build 7001 outcome was FAILURE using a merge of 9f23014
Summary: Tests failed: 1 (1 new), passed: 2919, ignored: 342, muted: 2 Build time: 0:50:21

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptTestCase.encryptedAndSignedRequest: <no details avaliable>

@wildfly-ci
Copy link

Linux with security manager Build 1885 is now running using a merge of 5c5e52a

@wildfly-ci
Copy link

Linux Build 7005 is now running using a merge of 5c5e52a

@wildfly-ci
Copy link

Windows Build 2063 is now running using a merge of 5c5e52a

@wildfly-ci
Copy link

Linux with security manager Build 1885 outcome was SUCCESS using a merge of 5c5e52a
Summary: Tests passed: 871, ignored: 280 Build time: 0:06:21

@wildfly-ci
Copy link

Linux Build 7005 outcome was SUCCESS using a merge of 5c5e52a
Summary: Tests passed: 2920, ignored: 342, muted: 3 Build time: 0:50:17

@wildfly-ci
Copy link

Windows Build 2063 outcome was SUCCESS using a merge of 5c5e52a
Summary: Tests passed: 2923, ignored: 342 Build time: 0:50:29

@bstansberry bstansberry added the ready-for-merge Only for use by those with merge permissions! label Jul 14, 2015
bstansberry added a commit that referenced this pull request Jul 14, 2015
[WFLY-4922] Add smoke test for <deny-uncovered-http-methods/> tag
@bstansberry bstansberry merged commit 71624dd into wildfly:master Jul 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge Only for use by those with merge permissions!
Projects
None yet
4 participants