Issue 110

[MatrixFrog]

Fix for issue 110 with test case

[dsaff]

This is linked from issue 110, so closing it here.

[dsaff]

Linking here to issue #110

[dsaff]

Is there any way to do this that doesn't involve throwing an exception?

[dsaff]

@MatrixFrog, do you have further time to spend on this? Thanks.

[dsaff]

Closing for now. Feel free to re-open!

[djh82]

Could you not doe something like this?

Modify addArrayValues to be:

private void addArrayValues(String name, List list, Object array, ParameterSignature sig) {

if (Collection.class.isAssignableFrom(array.getClass())) {

Collection<?> c = Collection.class.cast(array);

array = c.toArray((Object[]) Array.newInstance(sig.getType(), c.size()));

}

for (int i= 0; i < Array.getLength(array); i++)

list.add(PotentialAssignment.forValue(name + "[" + i + "]", Array.get(array, i)));

}

and addFields to be:

 private void addFields(ParameterSignature sig, List list) {

for (final Field field : fClass.getJavaClass().getFields()) {

if (Modifier.isStatic(field.getModifiers())) {

Class type= field.getType();
if ((sig.canAcceptArrayType(type) ||
Collection.class.isAssignableFrom(type)
&& sig.canAcceptType((Class) (((ParameterizedType) field.getGenericType()).getActualTypeArguments()[0])))

&& field.getAnnotation(DataPoints.class) != null) {

addArrayValues(field.getName(), list, getStaticFieldValue(field), sig);

} else if (sig.canAcceptType(type)

&& field.getAnnotation(DataPoint.class) != null) {

list.add(PotentialAssignment

.forValue(field.getName(), getStaticFieldValue(field)));

}

}

}

}

[Stephan202]

Just for reference: this pull request has been superseded by #658.