qadevOOo: _XPropertySet.java: fix unrealistic expectations:
We can't assume that when setting properties whose semantics we don't know to random values in an arbitrary order, a non-changing value indicates any kind of error. The only real error this test can detect is if setting a property crashes.
This commit is contained in:
@@ -319,11 +319,15 @@ public class _XPropertySet extends MultiMethodTest {
|
|||||||
"property '"+ propertyName+"'");
|
"property '"+ propertyName+"'");
|
||||||
e.printStackTrace(log);
|
e.printStackTrace(log);
|
||||||
} // end of try-catch
|
} // end of try-catch
|
||||||
|
/* this is stupid: we can't set properties whose semantics we
|
||||||
|
* don't know to random values in an arbitrary order and
|
||||||
|
* expect that to actually work.
|
||||||
if( gValue.equals(sValue) )
|
if( gValue.equals(sValue) )
|
||||||
{
|
{
|
||||||
log.println("setting property '"+ propertyName+"' failed");
|
log.println("setting property '"+ propertyName+"' failed");
|
||||||
error = true;
|
error = true;
|
||||||
}
|
}
|
||||||
|
*/
|
||||||
}
|
}
|
||||||
tRes.tested("setPropertyValue()",!error);
|
tRes.tested("setPropertyValue()",!error);
|
||||||
} //endif
|
} //endif
|
||||||
|
Reference in New Issue
Block a user