dev@glassfish.java.net

Re: [ACTION] Please review these admin console pom changes

From: Jason Lee <jasondlee_at_sun.com>
Date: Fri, 13 Mar 2009 20:31:31 -0500

Jane, if I update the adminconsole pom to testng-5.6, all tests pass.
I'm not real picky about the version, so this is works for me. :)

On Mar 13, 2009, at 8:19 PM, Jane Young wrote:

> Thanks for doing this test.
>
> Since admin console is using testng plugin version 5.8 and the rest
> of the quicklook tests are using 5.6 there may be issues when
> running the tests in the same Maven reactor. Please see Plugin-
> Tracking Design Flaws:
> http://docs.codehaus.org/display/MAVEN/Maven+2.1+Plugin+and+Extension+Loading+Design
>
> Since all the QL tests passed, it may be okay but I'm wondering if
> it's possible to change all the testng plugin to version 5.8 and use
> pluginManagement in the top level pom.xml to declare the version.
> Ming, is this possible?
>
> What happens if you admin console tests use testng version 5.6?
>
> Thanks,
> Jane
>
>
> Jason Lee wrote:
>>
>> I copied this POM from an existing test, so I can't really tell you
>> why it's in there twice (though the first dep is on a plugin). I
>> did, though, remove the dep on plugin, leaving only the main dep,
>> and all QL tests continued to pass, so it doesn't seem to have been
>> important/necessary. With that change, are you comfortable with me
>> committing the change to this pom?
>>
>> FWIW, in case you're tracking this type of changes, I modified the
>> console tests to remove the tests that checked for the frameset, as
>> the console is no longer frame-based.
>>
>> On Mar 13, 2009, at 6:38 PM, Jane Young wrote:
>>
>>> Hi Jason,
>>>
>>> The admingui/war/pom.xml looks fine.
>>>
>>> In tests/quicklook/adminconsole/pom.xml, there are two places
>>> declaring the testng dependency. Maybe one of the dependency
>>> should be removed? Can someone from QA take a look at this?
>>>
>>> Thanks,
>>> Jane
>>>
>>>
>>> Jason Lee wrote:
>>>>
>>>> Below are changes to two console-related pom files. The first
>>>> updates the Scales dep to the current product version, and the
>>>> second modifies the QL testng dep to drop the jdk5 classifier:
>>>>
>>>> Index: admingui/war/pom.xml
>>>> ===================================================================
>>>> --- admingui/war/pom.xml (revision 25379)
>>>> +++ admingui/war/pom.xml (working copy)
>>>> @@ -75,7 +75,7 @@
>>>> <dependency>
>>>> <groupId>com.sun.mojarra</groupId>
>>>> <artifactId>mojarra-scales</artifactId>
>>>> - <version>1.1</version>
>>>> + <version>1.2</version>
>>>> <scope>compile</scope>
>>>> </dependency>
>>>> <dependency>
>>>> Index: tests/quicklook/adminconsole/pom.xml
>>>> ===================================================================
>>>> --- tests/quicklook/adminconsole/pom.xml (revision 25379)
>>>> +++ tests/quicklook/adminconsole/pom.xml (working copy)
>>>> @@ -97,8 +97,7 @@
>>>> <dependency>^M
>>>> <groupId>org.testng</groupId>^M
>>>> <artifactId>testng</artifactId>^M
>>>> - <version>5.6</version>^M
>>>> - <classifier>jdk15</classifier>^M
>>>> + <version>5.8</version>^M
>>>> </dependency>^M
>>>> <dependency>^M
>>>> <groupId>com.sun.jsftemplating</groupId>^M
>>>> @@ -114,7 +113,6 @@
>>>> <groupId>org.testng</groupId>^M
>>>> <artifactId>testng</artifactId>^M
>>>> <version>5.8</version>^M
>>>> - <classifier>jdk15</classifier>^M
>>>> </dependency>^M
>>>> <dependency>^M
>>>> <groupId>commons-httpclient</groupId>^M
>>>>
>>>> Jason Lee
>>>> Senior Java Developer
>>>> GlassFish Administration Console
>>>>
>>>> Sun Microsystems, Inc.
>>>> Phone x31197/+1 405-343-1964
>>>> Email jasondlee_at_sun.com
>>>> Blog http://blogs.sun.com/jasondlee
>>>> Blog http://blogs.steeplesoft.com
>>>
>>
>> Jason Lee
>> Senior Java Developer
>> GlassFish Administration Console
>>
>> Sun Microsystems, Inc.
>> Phone x31197/+1 405-343-1964
>> Email jasondlee_at_sun.com
>> Blog http://blogs.sun.com/jasondlee
>> Blog http://blogs.steeplesoft.com
>

         Jason Lee
Senior Java Developer
GlassFish Administration Console

Sun Microsystems, Inc.
Phone x31197/+1 405-343-1964
Email jasondlee_at_sun.com
Blog http://blogs.sun.com/jasondlee
Blog http://blogs.steeplesoft.com