Re: FYI: JScrollBar constructors - new tests

[David Gilbert <david dot gilbert at object-refinery dot com>]

Mark Wielaard wrote:

> Hi David,
>
> On Wed, 2005-11-23 at 17:04 +0000, David Gilbert wrote:
>  
>
> > +    // use a known look and feel
> > +    try
> > +    {
> > +      UIManager.setLookAndFeel(new MetalLookAndFeel());
> > +    }
> > +    catch (Exception e)
> > +    {
> > +      e.printStackTrace();
> > +    }
> >    
>
> Better to use harness.debug(e) for this.
>
> Cheers,
>
> Mark
>  
You'd think I'd know better by now!  Fixed by this patch:

2005-11-24  David Gilbert  <david.gilbert@object-refinery.com>

   * gnu/testlet/javax/swing/JScrollBar/constructors.java
   (constructor1): Use harness.debug() to log exception,
   (constructor2): Likewise.

Regards,

Dave
Index: gnu/testlet/javax/swing/JScrollBar/constructors.java
===================================================================
RCS file: /cvs/mauve/mauve/gnu/testlet/javax/swing/JScrollBar/constructors.java,v
retrieving revision 1.1
diff -u -r1.1 constructors.java
--- gnu/testlet/javax/swing/JScrollBar/constructors.java	23 Nov 2005 16:57:02 -0000	1.1
+++ gnu/testlet/javax/swing/JScrollBar/constructors.java	24 Nov 2005 21:05:11 -0000
@@ -55,7 +55,7 @@
     }
     catch (Exception e)
     {
-      e.printStackTrace();
+      harness.debug(e);
     }
 
     JScrollBar bar = new JScrollBar();
@@ -77,7 +77,7 @@
     }
     catch (Exception e)
     {
-      e.printStackTrace();
+      harness.debug(e);
     }
 
     JScrollBar bar = new JScrollBar(JScrollBar.HORIZONTAL);