[stable10][DONOTMERGE] permutate all litmus iterations with php and databases#32639
[stable10][DONOTMERGE] permutate all litmus iterations with php and databases#32639patrickjahns wants to merge 1 commit into
Conversation
ac46daa to
fdb1bb8
Compare
fdb1bb8 to
aa68076
Compare
|
Rebased since owncloud/testing#23 was merged so we get new Oracle results. |
|
this is from the Litmus log on a local Drone run for the failed PROPPATCH: |
|
and the PROPPATCH failure in owncloud.log, that should help already: |
|
In the Drone env, I tried to directly So maybe the exception is a side effect. |
|
while debugging the method above I found this: #32660 |
|
Regarding Oracle, I suspect it could be a "empty string vs null" issue. |
|
Regarding severity of the Oracle + PROPPATCH issue: the error only happens when setting custom properties on regular files. This is not a common situation in OC itself so other PROPPATCH should run fine on system tags, comments, etc and also on files with non-custom properties. |
|
added some log statements and it seems that at some point Litmus is sending complex values (sub-tags): Since the value cannot be converted to string, it might trip up with Oracle. |
|
It turns out that I pasted the debug.log of the wrong Litmus test. It was actually "propvalnspace" that failed: |
|
Now reproducible without Litmus: Create proppatch-oracle.xml: Then: |
|
I've now raised #32670 for the PROPATCH case. There's a fundamental problem how we deal with complex properties and it doesn't behave correctly with MySQL either. |
|
closing at this was a one-time shot of running the tests |
Description
Testing that litmus test works as expected for all permutations on stable10
How Has This Been Tested?
🤖
Screenshots (if appropriate):
Types of changes
Checklist:
Open tasks: