[Jython-checkins] SF.net SVN: jython:[6138] trunk/jython/tests/java/org/python/tests/props

[<fwi...@us...>]

Revision: 6138
          http://jython.svn.sourceforge.net/jython/?rev=6138&view=rev
Author:   fwierzbicki
Date:     2009-03-31 16:35:51 +0000 (Tue, 31 Mar 2009)

Log Message:
-----------
Added comment to test and removed dead code from Readonly.java

Modified Paths:
--------------
    trunk/jython/tests/java/org/python/tests/props/Readonly.java
    trunk/jython/tests/java/org/python/tests/props/ReadonlyTest.java

Modified: trunk/jython/tests/java/org/python/tests/props/Readonly.java
===================================================================
--- trunk/jython/tests/java/org/python/tests/props/Readonly.java	2009-03-31 16:31:06 UTC (rev 6137)
+++ trunk/jython/tests/java/org/python/tests/props/Readonly.java	2009-03-31 16:35:51 UTC (rev 6138)
@@ -3,6 +3,5 @@
 public class Readonly {
     private String a;
     public void setA(String a) { this.a = a; }
-    //public String getA() { return a; }
 }
 

Modified: trunk/jython/tests/java/org/python/tests/props/ReadonlyTest.java
===================================================================
--- trunk/jython/tests/java/org/python/tests/props/ReadonlyTest.java	2009-03-31 16:31:06 UTC (rev 6137)
+++ trunk/jython/tests/java/org/python/tests/props/ReadonlyTest.java	2009-03-31 16:35:51 UTC (rev 6138)
@@ -16,6 +16,7 @@
     }
 
     public void testReadonly() {
+        //This used to cause an NPE see http://bugs.jython.org/issue1295 
         interp.exec("from org.python.tests.props import Readonly;Readonly().a = 'test'");
     }
 }


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.