You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by rg...@apache.org on 2010/03/03 19:16:44 UTC
svn commit: r918611 - in /struts/sandbox/trunk/struts2-cdi-plugin: pom.xml
src/test/java/org/apache/struts2/cdi/CdiObjectFactoryTest.java
src/test/java/org/apache/struts2/cdi/FooConsumer.java
src/test/java/org/apache/struts2/cdi/FooService.java
Author: rgielen
Date: Wed Mar 3 18:16:43 2010
New Revision: 918611
URL: http://svn.apache.org/viewvc?rev=918611&view=rev
Log:
- adjusted dependencies to Weld 1.0.1 Final
- removed bootstrap build for Weld-SE pre final
- fixed failing tests
- minor tests enhancements
Modified:
struts/sandbox/trunk/struts2-cdi-plugin/pom.xml
struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/CdiObjectFactoryTest.java
struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/FooConsumer.java
struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/FooService.java
Modified: struts/sandbox/trunk/struts2-cdi-plugin/pom.xml
URL: http://svn.apache.org/viewvc/struts/sandbox/trunk/struts2-cdi-plugin/pom.xml?rev=918611&r1=918610&r2=918611&view=diff
==============================================================================
--- struts/sandbox/trunk/struts2-cdi-plugin/pom.xml (original)
+++ struts/sandbox/trunk/struts2-cdi-plugin/pom.xml Wed Mar 3 18:16:43 2010
@@ -10,10 +10,22 @@
<artifactId>struts2-cdi-plugin</artifactId>
<repositories>
+ <!-- The home of the Weld artifacts -->
<repository>
- <id>java.net</id>
- <name>java.net Repository</name>
- <url>http://download.java.net/maven/2</url>
+ <id>jboss.oss.repositories</id>
+ <name>JBoss Repositories group at Sontatype OSS Nexus</name>
+ <url>http://oss.sonatype.org/content/groups/jboss/</url>
+ </repository>
+ <!-- The home of jboss-interceptor -->
+ <repository>
+ <id>jboss</id>
+ <url>http://repository.jboss.com/maven2</url>
+ <releases>
+ <enabled>true</enabled>
+ </releases>
+ <snapshots>
+ <enabled>false</enabled>
+ </snapshots>
</repository>
</repositories>
@@ -22,21 +34,29 @@
<dependency>
<groupId>javax.enterprise</groupId>
<artifactId>cdi-api</artifactId>
- <version>1.0</version>
+ <version>1.0-SP1</version>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>org.jboss.weld</groupId>
<artifactId>weld-core</artifactId>
- <version>1.0.0</version>
+ <version>1.0.1-Final</version>
<scope>provided</scope>
+<!--
+ <exclusions>
+ <exclusion>
+ <groupId>org.jboss.interceptor</groupId>
+ <artifactId>jboss-interceptor</artifactId>
+ </exclusion>
+ </exclusions>
+-->
</dependency>
<dependency>
<groupId>org.jboss.weld</groupId>
<artifactId>weld-se</artifactId>
- <version>1.0.0-SNAPSHOT</version>
+ <version>1.0.1-Final</version>
<scope>test</scope>
</dependency>
@@ -63,33 +83,4 @@
</dependencies>
- <profiles>
- <profile>
- <id>bootstrap</id>
- <build>
- <plugins>
- <plugin>
- <groupId>org.apache.maven.plugins</groupId>
- <artifactId>maven-scm-plugin</artifactId>
- <version>1.0</version>
- <executions>
- <execution>
- <id>resolve-weld-se</id>
- <phase>initialize</phase>
- <goals>
- <goal>bootstrap</goal>
- </goals>
- <configuration>
- <goals>install</goals>
- <connectionUrl>scm:svn:http://anonsvn.jboss.org/repos/weld/java-se/trunk
- </connectionUrl>
- </configuration>
- </execution>
- </executions>
- </plugin>
- </plugins>
- </build>
- </profile>
- </profiles>
-
</project>
\ No newline at end of file
Modified: struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/CdiObjectFactoryTest.java
URL: http://svn.apache.org/viewvc/struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/CdiObjectFactoryTest.java?rev=918611&r1=918610&r2=918611&view=diff
==============================================================================
--- struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/CdiObjectFactoryTest.java (original)
+++ struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/CdiObjectFactoryTest.java Wed Mar 3 18:16:43 2010
@@ -2,6 +2,8 @@
import org.jboss.weld.environment.se.StartMain;
import static org.junit.Assert.*;
+
+import org.jboss.weld.environment.se.WeldContainer;
import org.junit.Before;
import org.junit.Test;
import org.springframework.mock.jndi.SimpleNamingContextBuilder;
@@ -10,8 +12,6 @@
/**
* CdiObjectFactoryTest.
- *
- * @author Rene Gielen
*/
public class CdiObjectFactoryTest {
@@ -21,7 +21,8 @@
builder.activate();
StartMain sm = new StartMain(new String[0]);
- builder.bind(CdiObjectFactory.CDI_JNDIKEY_BEANMANAGER_COMP, sm.go());
+ WeldContainer weldContainer = sm.go();
+ builder.bind(CdiObjectFactory.CDI_JNDIKEY_BEANMANAGER_COMP, weldContainer.getBeanManager());
}
@Test
@@ -34,6 +35,7 @@
final CdiObjectFactory cdiObjectFactory = new CdiObjectFactory();
FooConsumer fooConsumer = (FooConsumer) cdiObjectFactory.buildBean(FooConsumer.class.getCanonicalName(), null, false);
assertNotNull(fooConsumer);
+ assertNotNull(fooConsumer.fooService);
}
@Test public void testGetInjectionTarget() throws Exception {
Modified: struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/FooConsumer.java
URL: http://svn.apache.org/viewvc/struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/FooConsumer.java?rev=918611&r1=918610&r2=918611&view=diff
==============================================================================
--- struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/FooConsumer.java (original)
+++ struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/FooConsumer.java Wed Mar 3 18:16:43 2010
@@ -4,14 +4,12 @@
/**
* FooConsumer.
- *
- * @author Rene Gielen
*/
public class FooConsumer {
@Inject
- private FooService fooService;
-
+ FooService fooService;
+
public void foo() {
System.out.println(fooService.getHello());
}
Modified: struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/FooService.java
URL: http://svn.apache.org/viewvc/struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/FooService.java?rev=918611&r1=918610&r2=918611&view=diff
==============================================================================
--- struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/FooService.java (original)
+++ struts/sandbox/trunk/struts2-cdi-plugin/src/test/java/org/apache/struts2/cdi/FooService.java Wed Mar 3 18:16:43 2010
@@ -2,8 +2,6 @@
/**
* FooService.
- *
- * @author Rene Gielen
*/
public class FooService {