I think that for things like that, you should ask first.
geir
Nathan Beyer wrote:
To make the tests consistent with the majority (my observation) of the
other Harmony test cases without them.
Besides that, from a pragmatic view, it's never used. The caveat to
that might be that lone hacker doing everything from the command-line
and invoking individual tests, if they have main methods, via a "java
..." command.
-Nathan
On 11/28/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote:
I'm game - why should the main() be removed?
geir
[EMAIL PROTECTED] wrote:
> Author: ndbeyer
> Date: Mon Nov 27 21:31:26 2006
> New Revision: 479913
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=479913
> Log:
> Remove unnecessary and empty methods
>
> Modified:
>
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/AbstractUndoableEditTest.java
>
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/CannotRedoExceptionTest.java
>
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/CannotUndoExceptionTest.java
>
> Modified:
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/AbstractUndoableEditTest.java
> URL:
http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/AbstractUndoableEditTest.java?view=diff&rev=479913&r1=479912&r2=479913
>
==============================================================================
> ---
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/AbstractUndoableEditTest.java
(original)
> +++
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/AbstractUndoableEditTest.java
Mon Nov 27 21:31:26 2006
> @@ -35,10 +35,6 @@
>
> private String defaultRedoName;
>
> - public static void main(final String[] args) {
> - junit.textui.TestRunner.run(AbstractUndoableEditTest.class);
> - }
> -
> @Override
> protected void setUp() throws Exception {
> obj = new AbstractUndoableEdit();
>
> Modified:
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/CannotRedoExceptionTest.java
> URL:
http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/CannotRedoExceptionTest.java?view=diff&rev=479913&r1=479912&r2=479913
>
==============================================================================
> ---
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/CannotRedoExceptionTest.java
(original)
> +++
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/CannotRedoExceptionTest.java
Mon Nov 27 21:31:26 2006
> @@ -14,25 +14,12 @@
> * See the License for the specific language governing permissions
and
> * limitations under the License.
> */
> -/**
> - * @author Evgeniya G. Maenkova
> - * @version $Revision$
> - */
> +
> package javax.swing.undo;
>
> import junit.framework.TestCase;
>
> public class CannotRedoExceptionTest extends TestCase {
> - @Override
> - protected void setUp() throws Exception {
> - super.setUp();
> - }
> -
> - @Override
> - protected void tearDown() throws Exception {
> - super.tearDown();
> - }
> -
> public void testCannotRedoException() {
> CannotRedoException exception = new CannotRedoException();
> assertNull(exception.getMessage());
>
> Modified:
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/CannotUndoExceptionTest.java
> URL:
http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/CannotUndoExceptionTest.java?view=diff&rev=479913&r1=479912&r2=479913
>
==============================================================================
> ---
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/CannotUndoExceptionTest.java
(original)
> +++
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/undo/CannotUndoExceptionTest.java
Mon Nov 27 21:31:26 2006
> @@ -14,25 +14,12 @@
> * See the License for the specific language governing permissions
and
> * limitations under the License.
> */
> -/**
> - * @author Evgeniya G. Maenkova
> - * @version $Revision$
> - */
> +
> package javax.swing.undo;
>
> import junit.framework.TestCase;
>
> public class CannotUndoExceptionTest extends TestCase {
> - @Override
> - protected void setUp() throws Exception {
> - super.setUp();
> - }
> -
> - @Override
> - protected void tearDown() throws Exception {
> - super.tearDown();
> - }
> -
> public void testCannotRedoException() {
> CannotUndoException exception = new CannotUndoException();
> assertNull(exception.getMessage());
>
>