@tellescope/sdk 1.237.3 → 1.237.5

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

@@ -580,6 +580,87 @@ export const inbox_thread_assignment_updates_tests = async ({ sdk, sdkNonAdmin }
       sdk.api.inbox_threads.deleteOne(sortThread3.id),
     ])
 
+    // ========== InboxStatus Preservation Tests ==========
+    // These tests verify that outbound messages do NOT reset inboxStatus
+
+    // Test 27: Outbound SMS should NOT reset inboxStatus
+    console.log("Testing outbound SMS should NOT reset inboxStatus...")
+
+    const statusTestSMS1 = await sdk.api.sms_messages.createOne({
+      message: "Inbound test message for status test",
+      enduserId: testEnduser.id,
+      inbound: true,
+      phoneNumber: "+15555559999",
+      enduserPhoneNumber: "+15555559876",
+      logOnly: true,
+    })
+
+    // Build threads using reset_threads + build_threads pattern
+    const statusTestFrom = new Date(Date.now() - 60000)
+    await sdk.api.inbox_threads.reset_threads()
+    await sdk.api.inbox_threads.build_threads({ from: statusTestFrom, to: new Date() })
+
+    const statusTestThreads = await sdk.api.inbox_threads.load_threads({})
+    const statusTestThread = statusTestThreads.threads.find(t =>
+      t.type === 'SMS' && t.enduserIds.includes(testEnduser.id) && t.phoneNumber === "+15555559999"
+    )
+    assert(!!statusTestThread, "Status test SMS thread should be created")
+    assert(statusTestThread!.inboxStatus === 'New', `Initial status should be 'New', got '${statusTestThread!.inboxStatus}'`)
+
+    // Update thread status to "Resolved"
+    await sdk.api.inbox_threads.updateOne(statusTestThread!.id, { inboxStatus: "Resolved" })
+
+    // Create outbound SMS (should NOT reset status)
+    const statusTestSMS2 = await sdk.api.sms_messages.createOne({
+      message: "Outbound reply - should not reset status",
+      enduserId: testEnduser.id,
+      inbound: false,
+      phoneNumber: "+15555559999",
+      enduserPhoneNumber: "+15555559876",
+      logOnly: true,
+    })
+
+    // Rebuild threads - status should remain "Resolved"
+    await sdk.api.inbox_threads.build_threads({ from: statusTestFrom, to: new Date() })
+
+    const threadAfterOutbound = (await sdk.api.inbox_threads.load_threads({ ids: [statusTestThread!.id] })).threads[0]
+    assert(threadAfterOutbound.inboxStatus === 'Resolved', `Status should remain 'Resolved' after outbound message, got '${threadAfterOutbound.inboxStatus}'`)
+    assert(!!threadAfterOutbound.outboundTimestamp, "outboundTimestamp should be set after outbound message")
+
+    console.log("✅ Outbound SMS does NOT reset inboxStatus test passed")
+
+    // Test 28: New inbound SMS SHOULD update inboxStatus
+    console.log("Testing new inbound SMS SHOULD update inboxStatus...")
+
+    // Wait to ensure ObjectId timestamps are in different seconds (MongoDB ObjectIds have second-level precision)
+    await new Promise(resolve => setTimeout(resolve, 1100))
+
+    const statusTestSMS3 = await sdk.api.sms_messages.createOne({
+      message: "New inbound - should update status",
+      enduserId: testEnduser.id,
+      inbound: true,
+      phoneNumber: "+15555559999",
+      enduserPhoneNumber: "+15555559876",
+      inboxStatus: "New",
+      logOnly: true,
+    })
+
+    // Rebuild threads - status SHOULD be updated from new inbound
+    await sdk.api.inbox_threads.build_threads({ from: statusTestFrom, to: new Date() })
+
+    const threadAfterNewInbound = (await sdk.api.inbox_threads.load_threads({ ids: [statusTestThread!.id] })).threads[0]
+    assert(threadAfterNewInbound.inboxStatus === 'New', `Status SHOULD be 'New' after new inbound message, got '${threadAfterNewInbound.inboxStatus}'`)
+
+    console.log("✅ New inbound SMS DOES update inboxStatus test passed")
+
+    // Cleanup status preservation test resources
+    await Promise.all([
+      sdk.api.sms_messages.deleteOne(statusTestSMS1.id),
+      sdk.api.sms_messages.deleteOne(statusTestSMS2.id),
+      sdk.api.sms_messages.deleteOne(statusTestSMS3.id),
+      sdk.api.inbox_threads.deleteOne(statusTestThread!.id),
+    ])
+
     console.log("🎉 All InboxThread assignment update tests passed!")
 
   } finally {

package/src/tests/tests.ts

@@ -12368,6 +12368,25 @@ const replace_enduser_template_values_tests = async () => {
   const d = Date.now()
   assert(replace_enduser_template_values(d as any, enduser) === d as any, 'fail non-string', 'non-string')
 
+  // Test height/weight subfields with values present
+  const enduserWithVitals = await sdk.api.endusers.createOne({
+    fname: "Vitals",
+    height: { value: 72, unit: 'inches' },
+    weight: { value: 180, unit: 'lbs' }
+  })
+
+  assert(replace_enduser_template_values('{{enduser.height.value}}', enduserWithVitals) === '72', 'fail height.value', 'height.value')
+  assert(replace_enduser_template_values('{{enduser.height.unit}}', enduserWithVitals) === 'inches', 'fail height.unit', 'height.unit')
+  assert(replace_enduser_template_values('{{enduser.weight.value}}', enduserWithVitals) === '180', 'fail weight.value', 'weight.value')
+  assert(replace_enduser_template_values('{{enduser.weight.unit}}', enduserWithVitals) === 'lbs', 'fail weight.unit', 'weight.unit')
+
+  // Test undefined safeguards - enduser without height/weight should return empty string
+  assert(replace_enduser_template_values('{{enduser.height.value}}', enduser) === '', 'fail undefined height.value', 'undefined height.value')
+  assert(replace_enduser_template_values('{{enduser.height.unit}}', enduser) === '', 'fail undefined height.unit', 'undefined height.unit')
+  assert(replace_enduser_template_values('{{enduser.weight.value}}', enduser) === '', 'fail undefined weight.value', 'undefined weight.value')
+  assert(replace_enduser_template_values('{{enduser.weight.unit}}', enduser) === '', 'fail undefined weight.unit', 'undefined weight.unit')
+
+  await sdk.api.endusers.deleteOne(enduserWithVitals.id)
   await sdk.api.endusers.deleteOne(enduser.id)
 }