chore(deps): update dependency @sap/cds-compiler to v6.8.0

[renovate]

This PR contains the following updates:

Package	Change	Age	Confidence
@sap/cds-compiler	6.7.3 → 6.8.0

---

Warning

Some dependencies could not be looked up. Check the Dependency Dashboard for more information.

---

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR was generated by Mend Renovate. View the repository job log.

[renovate]

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

[zongqichen]

Hi @stewsk, we found that there is new content in csn introduced in new version:

FAIL __tests__/unit/ord.e2e.test.js
  ● Tests for Data Product definition › Check interop CSN content

    expect(received).toMatchSnapshot()

    Snapshot name: `Tests for Data Product definition Check interop CSN content 1`

    - Snapshot  - 0
    + Received  + 1

    @@ -1102,10 +1102,11 @@
          },
        },
        "meta": {
          "__name": "DPBooks",
          "__namespace": "sap.capdpprod",
    +     "compilerCsnFlavor": "inferred",
          "document": {
            "version": "1.0.0",
          },
          "flavor": "effective",
        },

      295 |         expect(interopCsn.i18n.de.Stock).toBe("Bestand");
      296 |
    > 297 |         expect(interopCsn).toMatchSnapshot();
          |                            ^
      298 |     });
      299 |
      300 |     it("Check interop CSN annotation mapping through ORD", async () => {

      at Object.toMatchSnapshot (__tests__/unit/ord.e2e.test.js:297:28)

Should we update our snapshot? Or it is something not on purpose?

Best,
Zongqi

[stewsk]

Hi @zongqichen , the change is on purpose, so yes, please update the snapshot.
Alternatively you could ignore all the properties inside the meta section that you are not interested in.

[stewsk]

Hi @zongqichen , as discussed in the meeting with GreenLedger, I'm going to remove the `compilerCsnFlavor" from the meta. Is it ok if I do it in this PR?

[zongqichen]

Hi @zongqichen , as discussed in the meeting with GreenLedger, I'm going to remove the `compilerCsnFlavor" from the meta. Is it ok if I do it in this PR?

Hi @stewsk, I would suggest create a new pr for better tracking. You can link this error in the new one

[stewsk]

ok. With this other change (-> #387) it is then of course no longer necessary to adapt the snapshot

[stewsk]

failing test is addressed in #387