diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2015-10-22 10:56:41 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2015-10-22 10:56:41 +0200 |
| commit | daa6fe4173ba9215a7044887dc28cc8ee4408985 (patch) | |
| tree | b45ac7d2c72f44cce9c217d3d0009a95d5076068 /qadevOOo/runner | |
| parent | 8bde421ccec9c10fe1382ad68485852889dd4c74 (diff) | |
Why check that resValue != oldValue
...if all we want to verify is that resValue == newValue (and it is the test
code's responsibility to come up with a newValue != oldValue, but which might
sometimes not be possible)?
Change-Id: Ibfce322b50bb0074ca76c33ad0c83541a8d4dc41
Diffstat (limited to 'qadevOOo/runner')
| -rw-r--r-- | qadevOOo/runner/lib/MultiPropertyTest.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/qadevOOo/runner/lib/MultiPropertyTest.java b/qadevOOo/runner/lib/MultiPropertyTest.java index 9946710bd7d7..cbf8f5db93ea 100644 --- a/qadevOOo/runner/lib/MultiPropertyTest.java +++ b/qadevOOo/runner/lib/MultiPropertyTest.java @@ -386,7 +386,7 @@ public class MultiPropertyTest extends MultiMethodTest { // if no exception thrown // check that the new value is set - if ((!compare(resValue, newValue)) || (compare(resValue, oldValue))) + if (!compare(resValue, newValue)) { log.println("Value for '" + propName + "' hasn't changed as expected"); try |