Fix unreasonable update tests

[danjoa]

The tests fixed for cds9 in this PR did some rather weird things like:

UPDATE(Foo).with({ ID:111, bar:12 }) // would change all rows in Foo to have the same ID: 111

... which should rather be like that:

UPDATE(Foo,{ID:111}).with({ bar:12 }) // updates Foo #111 only 

[nkaputnik]

@Sv7enNowitzki can you look into this?

[Sv7enNowitzki]

Hi @danjoa and @nkaputnik ,

I don't understand. In the statement UPDATE(Foo).with({ ID:111, bar:12 }), will there be duplicate ID: 111? Since it's targeting the same Foo entity, theoretically, there should only be a unique value ID: 111, right?

Best Regards,
Wenjun

[danjoa]

There was no where clause at all, hence the update would apply to all rows.

[danjoa]

With the latest change from @johannes-vogel tests are green now, for both cds8 (current latest) as well as upcoming cds9 (tested internally).