You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by Aki Yoshida <ay...@apache.org> on 2012/02/24 22:14:28 UTC
Re: svn commit: r1293413 - in /cxf/trunk/systests/rs-security:
pom.xml src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java
Hi Sergey,
I was having a problem in building 2.5.x and trunk in a restricted
fresh environment, then I noticed that systests/rs-security was still
referring to cxf-rt-rs-security-oauth-test which no longer existed in
2.5.x and trunk. I saw the corresponding classes now exist in
systests/rs-security's test package.
However, because of this reference, the test itself was currently
picking up some older snapshot version of those classes from the
apache repos. I removed this reference so that the test uses those
corresponding classes in the same test package. Unfortunately, there
seems to have been some change between the older version and the one
in the test package that is causing one of the tests to fail. So, I
temporarily disabled this test. Could you take a look into it?
Thanks.
Regards, aki
2012/2/24 <ay...@apache.org>:
> Author: ay
> Date: Fri Feb 24 20:56:37 2012
> New Revision: 1293413
>
> URL: http://svn.apache.org/viewvc?rev=1293413&view=rev
> Log:
> remove unexistent cxf-rt-rs-security-oauth-test ref from systests/rs-security
>
> Modified:
> cxf/trunk/systests/rs-security/pom.xml
> cxf/trunk/systests/rs-security/src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java
>
> Modified: cxf/trunk/systests/rs-security/pom.xml
> URL: http://svn.apache.org/viewvc/cxf/trunk/systests/rs-security/pom.xml?rev=1293413&r1=1293412&r2=1293413&view=diff
> ==============================================================================
> --- cxf/trunk/systests/rs-security/pom.xml (original)
> +++ cxf/trunk/systests/rs-security/pom.xml Fri Feb 24 20:56:37 2012
> @@ -84,12 +84,6 @@
> </dependency>
> <dependency>
> <groupId>org.apache.cxf</groupId>
> - <artifactId>cxf-rt-rs-security-oauth-test</artifactId>
> - <version>${project.version}</version>
> - <scope>test</scope>
> - </dependency>
> - <dependency>
> - <groupId>org.apache.cxf</groupId>
> <artifactId>cxf-rt-transports-http</artifactId>
> <version>${project.version}</version>
> </dependency>
>
> Modified: cxf/trunk/systests/rs-security/src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java
> URL: http://svn.apache.org/viewvc/cxf/trunk/systests/rs-security/src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java?rev=1293413&r1=1293412&r2=1293413&view=diff
> ==============================================================================
> --- cxf/trunk/systests/rs-security/src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java (original)
> +++ cxf/trunk/systests/rs-security/src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java Fri Feb 24 20:56:37 2012
> @@ -34,13 +34,13 @@ import net.oauth.ParameterStyle;
>
> import org.apache.cxf.common.logging.LogUtils;
> import org.apache.cxf.common.util.StringUtils;
> -import org.apache.cxf.rs.security.oauth.test.OAuthTestUtils;
> import org.apache.cxf.testutil.common.AbstractBusClientServerTestBase;
>
> import org.eclipse.jetty.http.HttpHeaders;
>
> import org.junit.Assert;
> import org.junit.BeforeClass;
> +import org.junit.Ignore;
> import org.junit.Test;
>
> public class TemporaryCredentialServiceTest extends AbstractBusClientServerTestBase {
> @@ -56,6 +56,7 @@ public class TemporaryCredentialServiceT
> launchServer(OAuthServer.class, true));
> }
>
> + @Ignore
> @Test
> public void testGetTemporaryCredentialsURIQuery() throws Exception {
> Map<String, String> parameters = new HashMap<String, String>();
>
>
Re: svn commit: r1293413 - in /cxf/trunk/systests/rs-security: pom.xml
src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java
Posted by Sergey Beryozkin <sb...@gmail.com>.
Hi Aki
On 24/02/12 21:14, Aki Yoshida wrote:
> Hi Sergey,
>
> I was having a problem in building 2.5.x and trunk in a restricted
> fresh environment, then I noticed that systests/rs-security was still
> referring to cxf-rt-rs-security-oauth-test which no longer existed in
> 2.5.x and trunk. I saw the corresponding classes now exist in
> systests/rs-security's test package.
>
> However, because of this reference, the test itself was currently
> picking up some older snapshot version of those classes from the
> apache repos. I removed this reference so that the test uses those
> corresponding classes in the same test package. Unfortunately, there
> seems to have been some change between the older version and the one
> in the test package that is causing one of the tests to fail. So, I
> temporarily disabled this test. Could you take a look into it?
>
I'll take a look, should be something minor
thanks, Sergey
> Thanks.
>
> Regards, aki
>
> 2012/2/24<ay...@apache.org>:
>> Author: ay
>> Date: Fri Feb 24 20:56:37 2012
>> New Revision: 1293413
>>
>> URL: http://svn.apache.org/viewvc?rev=1293413&view=rev
>> Log:
>> remove unexistent cxf-rt-rs-security-oauth-test ref from systests/rs-security
>>
>> Modified:
>> cxf/trunk/systests/rs-security/pom.xml
>> cxf/trunk/systests/rs-security/src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java
>>
>> Modified: cxf/trunk/systests/rs-security/pom.xml
>> URL: http://svn.apache.org/viewvc/cxf/trunk/systests/rs-security/pom.xml?rev=1293413&r1=1293412&r2=1293413&view=diff
>> ==============================================================================
>> --- cxf/trunk/systests/rs-security/pom.xml (original)
>> +++ cxf/trunk/systests/rs-security/pom.xml Fri Feb 24 20:56:37 2012
>> @@ -84,12 +84,6 @@
>> </dependency>
>> <dependency>
>> <groupId>org.apache.cxf</groupId>
>> -<artifactId>cxf-rt-rs-security-oauth-test</artifactId>
>> -<version>${project.version}</version>
>> -<scope>test</scope>
>> -</dependency>
>> -<dependency>
>> -<groupId>org.apache.cxf</groupId>
>> <artifactId>cxf-rt-transports-http</artifactId>
>> <version>${project.version}</version>
>> </dependency>
>>
>> Modified: cxf/trunk/systests/rs-security/src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java
>> URL: http://svn.apache.org/viewvc/cxf/trunk/systests/rs-security/src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java?rev=1293413&r1=1293412&r2=1293413&view=diff
>> ==============================================================================
>> --- cxf/trunk/systests/rs-security/src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java (original)
>> +++ cxf/trunk/systests/rs-security/src/test/java/org/apache/cxf/systest/jaxrs/security/oauth/TemporaryCredentialServiceTest.java Fri Feb 24 20:56:37 2012
>> @@ -34,13 +34,13 @@ import net.oauth.ParameterStyle;
>>
>> import org.apache.cxf.common.logging.LogUtils;
>> import org.apache.cxf.common.util.StringUtils;
>> -import org.apache.cxf.rs.security.oauth.test.OAuthTestUtils;
>> import org.apache.cxf.testutil.common.AbstractBusClientServerTestBase;
>>
>> import org.eclipse.jetty.http.HttpHeaders;
>>
>> import org.junit.Assert;
>> import org.junit.BeforeClass;
>> +import org.junit.Ignore;
>> import org.junit.Test;
>>
>> public class TemporaryCredentialServiceTest extends AbstractBusClientServerTestBase {
>> @@ -56,6 +56,7 @@ public class TemporaryCredentialServiceT
>> launchServer(OAuthServer.class, true));
>> }
>>
>> + @Ignore
>> @Test
>> public void testGetTemporaryCredentialsURIQuery() throws Exception {
>> Map<String, String> parameters = new HashMap<String, String>();
>>
>>