You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ja...@apache.org on 2015/05/16 00:59:35 UTC
svn commit: r1679655 -
/pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/TestListBox.java
Author: jahewson
Date: Fri May 15 22:59:35 2015
New Revision: 1679655
URL: http://svn.apache.org/r1679655
Log:
PDFBOX-2459: Introduce type safety for setValue()
Modified:
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/TestListBox.java
Modified: pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/TestListBox.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/TestListBox.java?rev=1679655&r1=1679654&r2=1679655&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/TestListBox.java (original)
+++ pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/TestListBox.java Fri May 15 22:59:35 2015
@@ -140,7 +140,7 @@ public class TestListBox extends TestCas
// without multiselect setting multiple items shall fail
try
{
- choice.setValues(exportValues);
+ choice.setValue(exportValues);
fail( "Missing IllegalArgumentException" );
}
catch( IllegalArgumentException e )
@@ -151,7 +151,7 @@ public class TestListBox extends TestCas
// ensure that the choice field does allow multiple selections
choice.setMultiSelect(true);
// now this call must suceed
- choice.setValues(exportValues);
+ choice.setValue(exportValues);
// assert that the option values have been correctly set
COSArray valueItems = (COSArray) choice.getCOSObject().getItem(COSName.V);