cds-dbs icon indicating copy to clipboard operation
cds-dbs copied to clipboard

test: for unresolved entities (fails on Postgres and HANA, works on SQLite)

Open nils opened this issue 1 year ago • 3 comments

This is only a PR for providing tests. I do not know enough about the internals of cds-dbs to fix the issue (at least without an expert providing help).

The tests fail only for Postgres (cd postgres && npm run test) [PS: And HANA, apparently?]

I think the issue only appeared after #466 was merged. That PR changed when query.SELECT.expand = 'root' is added, it is not added any more for unresolved entities.

For simple queries (without json) Postgres does not return columns in the case they were written in the query. There is a mismatch (BookId is in the result, but as bookid) which causes issues in CAP (e.g. when the data is serialized to OData responses, the correct case of the properties is required).

nils avatar Mar 05 '24 12:03 nils

Hi Nils,

thanks for the PR. We are already aware and working on a fix for it.

johannes-vogel avatar Mar 06 '24 14:03 johannes-vogel

@johannes-vogel we still have the issue... (see last pipeline run)

  1. Is there any timeline/progress you can share on it?
  2. Maybe if the team capacity does not allow to fix it on short notice, are there any hints/preferred solutions you can share for somebody else to fix it as a PR?

nils avatar Apr 10 '24 08:04 nils

@johannes-vogel we still have the issue... (see last pipeline run)

  1. Is there any timeline/progress you can share on it?
  2. Maybe if the team capacity does not allow to fix it on short notice, are there any hints/preferred solutions you can share for somebody else to fix it as a PR?

It should work already with cds 7.8. I'm about to update the deps with #573 and will update this PR afterwards for proving that it works.

johannes-vogel avatar Apr 10 '24 09:04 johannes-vogel