Great. 

For ValueRecordsAggregate, the following test fails at the moment. Is this the 
functionality that is wanted? I dont really understand the use case for this 
class .. so if tal and others confirm that this is what the behaviour should 
be, i will make the changes. (a patch would be nice .. :)

package org.apache.poi.hssf.record.aggregates;

import org.apache.poi.hssf.record.*;

public class TestValueRecordsAggregate extends junit.framework.TestCase {
    public TestValueRecordsAggregate(String name) {
        super (name);
    }
    
    public void testRecordGet() {
        ValueRecordsAggregate vra = new ValueRecordsAggregate();
        NumberRecord n1 = new NumberRecord();
        n1.setRow((short)1);n1.setColumn((short)1);n1.setValue(1.5);
        NumberRecord n2 = new NumberRecord();
        n1.setRow((short)2);n1.setColumn((short)2);n1.setValue(2.5);
        
        NumberRecord n3 = new NumberRecord();
        n1.setRow((short)2);n1.setColumn((short)2);n1.setValue(3.5);
        
       vra.insertCell(n1);vra.insertCell(n2);
       //removing different object but same row/col 
       vra.removeCell(n3);
       
       java.util.Iterator i = vra.getIterator();
       assertTrue("list is not empty",i.hasNext());
       while (i.hasNext()) {
           CellValueRecordInterface c  = (CellValueRecordInterface) i.next();
           System.out.println("Row = " +c.getRow() + " col = " +c.getColumn() );
            assertTrue("Cell is: Row = " +c.getRow() + " col = " +c.getColumn
(), !(c.getRow() == 2 && c.getColumn() == 2)); 
       }
       
    }
    
     public static void main(String [] args) {
        System.out
        .println("Testing 
org.apache.poi.hssf.record.aggregates.VallueRecordAggregate");
        junit.textui.TestRunner.run(TestValueRecordsAggregate.class);
    }
}




Quoting Tal Moshaiov <[EMAIL PROTECTED]>:

> Yepp, it seems to work just fine.
> 10x
> 
> Tal
> 

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to