@tellescope/sdk 2.0.0-beta.0 → 2.0.0-beta.2

package/src/tests/api_tests/mdi_case_offerings.test.ts

@@ -91,10 +91,41 @@ const resolve_displayed_options = (
   ) ? field.options.choices : undefined
 }
 
+// Local replica of the API's mdiIgnore handling (md_integrations.ts): the same field lookup
+// (_id first, externalId fallback — Formsort / form-ingestion answers carry no fieldId), with
+// patient-mapped fields exempt. Used to filter both case_questions and case file uploads.
+type IgnorableField = { _id: string, externalId?: string, mdiIgnore?: boolean }
+type IgnorableResponse = { fieldId?: string, externalId?: string, answer: { type?: string } }
+
+const is_mdi_ignored = (response: IgnorableResponse, formFields: IgnorableField[]): boolean => {
+  if (response.externalId && PATIENT_MAPPED_EXTERNAL_IDS.has(response.externalId)) return false
+  const field = (
+       formFields.find(f => f._id.toString() === response.fieldId)
+    || formFields.find(f => response.externalId && f.externalId === response.externalId)
+  )
+  return !!field?.mdiIgnore
+}
+
+const case_question_responses = (responses: IgnorableResponse[], formFields: IgnorableField[]) => (
+  responses.filter(r =>
+       r.answer.type !== 'file' && r.answer.type !== 'files'
+    && !(r.externalId && PATIENT_MAPPED_EXTERNAL_IDS.has(r.externalId))
+    && !is_mdi_ignored(r, formFields)
+  )
+)
+
+const uploadable_file_responses = (responses: IgnorableResponse[], formFields: IgnorableField[]) => (
+  responses.filter(r =>
+       (r.answer.type === 'file' || r.answer.type === 'files')
+    && !is_mdi_ignored(r, formFields)
+  )
+)
+
 export const mdi_case_offerings_tests = async ({ sdk, sdkNonAdmin }: { sdk: Session, sdkNonAdmin: Session }) => {
   log_header("MDI Case Offerings Tests")
 
   let testFormId: string | undefined
+  let testFieldId: string | undefined
 
   try {
     // --- Schema round-trip tests ---
@@ -385,7 +416,114 @@ export const mdi_case_offerings_tests = async ({ sdk, sdkNonAdmin }: { sdk: Sess
       },
       { expectedResult: true }
     )
+
+    // --- mdiIgnore tests ---
+
+    await async_test(
+      "Form field mdiIgnore round-trips through create and update",
+      async () => {
+        if (!testFormId) throw new Error("No test form")
+
+        const field = await sdk.api.form_fields.createOne({
+          formId: testFormId,
+          title: 'Offering IDs',
+          type: 'string',
+          externalId: 'offeringIds',
+          mdiIgnore: true,
+          previousFields: [{ type: 'root', info: {} }],
+        })
+        testFieldId = field.id
+        const created = await sdk.api.form_fields.getOne(field.id)
+
+        await sdk.api.form_fields.updateOne(field.id, { mdiIgnore: false })
+        const updated = await sdk.api.form_fields.getOne(field.id)
+
+        return created.mdiIgnore === true && updated.mdiIgnore === false
+      },
+      { expectedResult: true }
+    )
+
+    const IGNORE_FIELDS: IgnorableField[] = [
+      { _id: 'field-ignored', externalId: 'internal_note', mdiIgnore: true },
+      { _id: 'field-offerings', externalId: 'offeringIds', mdiIgnore: true },
+      { _id: 'field-kept', externalId: 'symptoms' },
+      { _id: 'field-kept-explicit', externalId: 'duration', mdiIgnore: false },
+      { _id: 'field-allergies', externalId: 'allergies', mdiIgnore: true }, // patient-mapped → exempt
+      { _id: 'field-ignored-upload', externalId: 'internal_upload', mdiIgnore: true },
+      { _id: 'field-kept-upload', externalId: 'mdi:lab-result' },
+    ]
+
+    await async_test(
+      "mdiIgnore drops native form answers (matched by fieldId) from case_questions",
+      async () => {
+        const kept = case_question_responses([
+          { fieldId: 'field-ignored', answer: { type: 'string' } },
+          { fieldId: 'field-kept', answer: { type: 'string' } },
+          { fieldId: 'field-kept-explicit', answer: { type: 'string' } },
+        ], IGNORE_FIELDS)
+        return kept.length === 2 && kept.every(r => r.fieldId !== 'field-ignored')
+      },
+      { expectedResult: true }
+    )
+
+    await async_test(
+      "mdiIgnore drops Formsort / form-ingestion answers (matched by externalId) from case_questions",
+      async () => {
+        const kept = case_question_responses([
+          { fieldId: '', externalId: 'offeringIds', answer: { type: 'string' } },
+          { fieldId: '', externalId: 'symptoms', answer: { type: 'string' } },
+          { fieldId: '', externalId: 'unconfigured_key', answer: { type: 'string' } }, // no field → kept
+        ], IGNORE_FIELDS)
+        return (
+          kept.length === 2
+          && !kept.some(r => r.externalId === 'offeringIds')
+          && kept.some(r => r.externalId === 'unconfigured_key')
+        )
+      },
+      { expectedResult: true }
+    )
+
+    await async_test(
+      "mdiIgnore does not affect patient-mapped fields",
+      async () => (
+        !is_mdi_ignored({ fieldId: 'field-allergies', externalId: 'allergies', answer: { type: 'string' } }, IGNORE_FIELDS)
+      ),
+      { expectedResult: true }
+    )
+
+    await async_test(
+      "mdiIgnore skips file uploads for ignored fields only",
+      async () => {
+        const uploadable = uploadable_file_responses([
+          { fieldId: 'field-ignored-upload', answer: { type: 'file' } },
+          { fieldId: '', externalId: 'internal_upload', answer: { type: 'files' } },
+          { fieldId: 'field-kept-upload', answer: { type: 'file' } },
+        ], IGNORE_FIELDS)
+        return uploadable.length === 1 && uploadable[0].fieldId === 'field-kept-upload'
+      },
+      { expectedResult: true }
+    )
+
+    await async_test(
+      "mdiIgnore does not stop offering conditions from reading the answer",
+      async () => {
+        const offerings: MDIOffering[] = [
+          { offering_id: "03b7", conditions: { condition: { offeringIds: { $contains: "03b7" } as any } } },
+        ]
+        const responses = [{ externalId: 'offeringIds', value: '1d15,03b7,03b7' }]
+        const resolved = resolve_case_offerings(offerings, responses)
+        const questions = case_question_responses(
+          [{ fieldId: '', externalId: 'offeringIds', answer: { type: 'string' } }],
+          IGNORE_FIELDS,
+        )
+        return resolved.length === 1 && questions.length === 0
+      },
+      { expectedResult: true }
+    )
   } finally {
+    if (testFieldId) {
+      await sdk.api.form_fields.deleteOne(testFieldId).catch(console.error)
+    }
     if (testFormId) {
       await sdk.api.forms.deleteOne(testFormId).catch(console.error)
     }

package/src/tests/api_tests/organization_signup.test.ts

@@ -10,6 +10,7 @@ import {
   set_email_verification_code,
   setup_isolated_tenant,
   test_support_request,
+  IsolatedSuite,
 } from "../setup_isolated"
 
 const host = process.env.API_URL || 'http://localhost:8080' as const
@@ -225,6 +226,9 @@ export const organization_signup_tests = async () => {
   assert_checks('Organization signup', await organization_signup_checks())
 }
 
+/** Enrolls this suite in the auto-discovered parallel pool — see run_discovered_isolated_suites. */
+export const ISOLATED_SUITE: IsolatedSuite = { name: 'Organization signup', run: organization_signup_checks }
+
 // Allow running this test file independently
 if (require.main === module) {
   console.log(`🌐 Using API URL: ${host}`)