Mark list-of-oneOf with an invalid item as unparsed

[ddadamhooper]

What does this PR do?

Correctly fails to parse a FormulaAndFunctionEventQueryDefinition when it contains an invalid group_by list entry.

The code generator was failing to propagate unparsed from a oneOf to its parent in this case.

(DataDog's APIs don't generally return invalid objects, so this shouldn't affect users. The value comes with an upcoming PR, where a oneOf item contains a list which contains another oneOf item. In that case, we need to detect invalid objects from the inner oneOf.)

Review checklist

Please check relevant items below:

• This PR includes all newly recorded cassettes for any modified tests.
• This PR does not rely on API client schema changes.
  • The CI should be fully passing.
• Or, this PR relies on API schema changes and this is a Draft PR that includes tests.
  • Note: CI shouldn't be run on this Draft PR, as its expected to fail without the corresponding schema changes.

[ddadamhooper]

@codex review

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 80fbe0bc2c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[jack-edmonds-dd]

approval for modelSimple.j2 (I'm assuming the rest of the changes are generated).

[ddadamhooper]

@codex review

[chatgpt-codex-connector]

💡 Codex Review

datadog-api-client-java/.generator/src/generator/templates/modelSimple.j2

Line 123 in 911009d

for ({{ itemsDataType }} item : {{ variableName }}) {

Guard list fluent setters against null

For optional array-of-model fields, this template now makes the generated fluent setter iterate the caller-provided value unconditionally. In generated methods such as FormulaAndFunctionUserJourneyQueryDefinition.groupBy(List<...>), passing null to clear/omit the optional property was previously accepted and is still accepted by the generated setGroupBy, but now throws a NullPointerException before returning; please mirror the setter's null guard before iterating.

---

datadog-api-client-java/src/test/java/com/datadog/api/World.java

Line 586 in 911009d

java.nio.file.Paths.get("/tmp/unparsed_final.log"),

Remove the diagnostic write from shared test execution

When a BDD scenario response contains an UnparsedObject, this shared helper now appends to /tmp/unparsed_final.log and swallows any failure. That makes normal local/CI test runs mutate global temp state and potentially race or accumulate stale output across parallel runs, while the test harness never consumes the file; please remove this leftover diagnostic or gate it behind an explicit debug flag.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[ddadamhooper]

@codex review

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9c67b6d59a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".