Skip to content

Commit a987ded

Browse files
committed
test(node): Reference dataCollection instead of removed sendDefaultPii in OpenAI assertions
sendDefaultPii was removed in v11 in favour of dataCollection. The disabled scenarios now opt out via dataCollection.genAI, so the comment describing the absence assertions is updated to match.
1 parent bf90934 commit a987ded

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • dev-packages/node-integration-tests/suites/tracing/openai

dev-packages/node-integration-tests/suites/tracing/openai/test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -323,7 +323,7 @@ describe('OpenAI integration', () => {
323323
value: 'auto.ai.openai',
324324
});
325325

326-
// PII attributes must never be recorded when `sendDefaultPii` is disabled.
326+
// GenAI inputs and outputs must never be recorded when `dataCollection.genAI` disables them.
327327
// Asserting over every span in the envelope catches attributes leaking onto
328328
// spans which are not individually inspected above.
329329
for (const span of container.items) {
@@ -856,7 +856,7 @@ describe('OpenAI integration', () => {
856856
value: 10,
857857
});
858858

859-
// PII attributes must never be recorded when `sendDefaultPii` is disabled.
859+
// GenAI inputs and outputs must never be recorded when `dataCollection.genAI` disables them.
860860
// Asserting over every span in the envelope catches attributes leaking onto
861861
// spans which are not individually inspected above.
862862
for (const span of container.items) {

0 commit comments

Comments
 (0)