ti2-tourplan 1.0.128 → 1.0.129

package/.github/workflows/nodejs.yml

@@ -22,12 +22,17 @@ jobs:
   publish-npm:
     if: startsWith(github.ref, 'refs/tags/v')
     needs: build_and_test
+    permissions:
+      contents: read
+      id-token: write
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v1
+      - uses: actions/checkout@v6
+      - uses: actions/setup-node@v6
         with:
-          node-version: 12
+          node-version: '24'
+          registry-url: 'https://registry.npmjs.org'
+          package-manager-cache: false
       - name: Validate release tag
         id: release_meta
         env:
@@ -55,9 +60,5 @@ jobs:
 
           echo "tag_version=${TAG_VERSION}" >> "$GITHUB_OUTPUT"
           echo "dist_tag=${DIST_TAG}" >> "$GITHUB_OUTPUT"
-      - name: Configure npm auth
-        run: echo "//registry.npmjs.org/:_authToken=${NPM_TOKEN}" > ~/.npmrc
-        env:
-          NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
       - name: Publish package
         run: npm publish --tag "${{ steps.release_meta.outputs.dist_tag }}"

package/index.test.js

@@ -5,10 +5,11 @@ const path = require('path');
 const xml2js = require('xml2js');
 const R = require('ramda');
 const hash = require('object-hash');
+const js2xmlparser = require('js2xmlparser');
 const { XMLParser } = require('fast-xml-parser');
 const { typeDefs: itineraryProductTypeDefs, query: itineraryProductQuery } = require('./node_modules/ti2/controllers/graphql-schemas/itinerary-product');
 const { typeDefs: itineraryBookingTypeDefs, query: itineraryBookingQuery } = require('./node_modules/ti2/controllers/graphql-schemas/itinerary-booking');
-const { CUSTOM_RATE_ID_NAME } = require('./utils');
+const { CUSTOM_RATE_ID_NAME, hostConnectXmlOptions } = require('./utils');
 const {
   getRatesObjectArray,
   getImmediateLastDateRange,
@@ -169,6 +170,69 @@ describe('search tests', () => {
         expect(newBookingName).toBe(`${'Ae'.repeat(29)}A`);
       });
 
+      it('serializes valid DOB instead of age and falls back to age for invalid DOB', async () => {
+        mockCallTourplan
+          .mockImplementationOnce(async () => ({
+            AddServiceReply: { BookingId: 'valid-dob' },
+          }))
+          .mockImplementationOnce(async () => ({
+            AddServiceReply: { BookingId: 'invalid-dob' },
+          }));
+        const payloadFor = passenger => ({
+          quoteName: 'DOB serialization test',
+          optionId: 'ABC123',
+          startDate: '2026-07-03',
+          reference: 'TESTREF',
+          paxConfigs: [{
+            roomType: 'Double',
+            adults: 1,
+            passengers: [passenger],
+          }],
+          notes: '',
+        });
+
+        await app.addServiceToItinerary({
+          axios,
+          token,
+          payload: payloadFor({
+            firstName: 'Ada',
+            lastName: 'Lovelace',
+            passengerType: 'Adult',
+            dob: '1990-02-28',
+            age: 36,
+          }),
+        });
+        await app.addServiceToItinerary({
+          axios,
+          token,
+          payload: payloadFor({
+            firstName: 'Grace',
+            lastName: 'Hopper',
+            passengerType: 'Adult',
+            dob: '1990-02-29',
+            age: 36,
+          }),
+        });
+
+        const validModel = mockCallTourplan.mock.calls[0][0].model;
+        const invalidModel = mockCallTourplan.mock.calls[1][0].model;
+        const validPax = validModel.AddServiceRequest.RoomConfigs.RoomConfig[0]
+          .PaxList.PaxDetails[0];
+        const invalidPax = invalidModel.AddServiceRequest.RoomConfigs.RoomConfig[0]
+          .PaxList.PaxDetails[0];
+        const validXml = js2xmlparser.parse('Request', validModel, hostConnectXmlOptions);
+        const invalidXml = js2xmlparser.parse('Request', invalidModel, hostConnectXmlOptions);
+
+        expect(validPax).toMatchObject({ DateOfBirth: '1990-02-28' });
+        expect(validPax).not.toHaveProperty('Age');
+        expect(validXml).toContain('<DateOfBirth>1990-02-28</DateOfBirth>');
+        expect(validXml).not.toContain('<Age>');
+        expect(invalidPax).toMatchObject({ Age: 36 });
+        expect(invalidPax).not.toHaveProperty('DateOfBirth');
+        expect(invalidXml).toContain('<Age>36</Age>');
+        expect(invalidXml).not.toContain('<DateOfBirth>');
+      });
+
       it('limits new booking names supplied by directHeaderPayload after escaping XML characters', async () => {
         mockCallTourplan.mockImplementationOnce(async () => ({
           AddServiceReply: {
@@ -799,66 +863,287 @@ describe('search tests', () => {
     expect(retVal).toMatchSnapshot();
   });
 
-  it('searchProductsForItinerary skips AC in full catalog when skipAccommodation is set', async () => {
-    const debugSpy = jest.spyOn(console, 'debug').mockImplementation(() => {});
-    const bdOption = {
-      Opt: 'TESTBDOPTION00001',
-      OptGeneral: {
-        SupplierId: '1001',
-        SupplierName: 'Test Supplier',
-        Description: 'Test BD Product',
-        SType: 'N',
-        AdultsAllowed: 'Y',
-        Adult_From: 16,
-        Adult_To: 999,
-        ButtonName: 'Sightseeing',
+  describe('searchProductsForItinerary omitServiceCodes', () => {
+    const optionGeneral = {
+      SupplierId: '1001',
+      SupplierName: 'Test Supplier',
+      Description: 'Test Product',
+      SType: 'N',
+      AdultsAllowed: 'Y',
+      Adult_From: 16,
+      Adult_To: 999,
+      ButtonName: 'Sightseeing',
+    };
+    const acOption = { Opt: 'TESTACOPTION00001', OptGeneral: optionGeneral };
+    const bdOption = { Opt: 'TESTBDOPTION00001', OptGeneral: optionGeneral };
+    const trOption = { Opt: 'TESTTROPTION00001', OptGeneral: optionGeneral };
+    const getServicesReply = {
+      GetServicesReply: {
+        TPLServices: {
+          TPLService: [
+            { Code: 'AC', Name: 'Accommodation' },
+            { Code: 'BD', Name: 'Bundle' },
+            { Code: 'TR', Name: 'Transfer' },
+          ],
+        },
       },
     };
+    const optionsByOpt = {
+      '???AC????????????': acOption,
+      '???BD????????????': bdOption,
+      '???TR????????????': trOption,
+    };
+    const defaultCallTourplan = mockCallTourplan.getMockImplementation();
+    const getOptionInfoCalls = () => mockCallTourplan.mock.calls
+      .map(([call]) => call.model.OptionInfoRequest)
+      .filter(Boolean)
+      .map(req => req.Opt);
 
-    mockCallTourplan
-      .mockImplementationOnce(async () => ({
-        GetServicesReply: {
-          TPLServices: {
-            TPLService: [
-              { Code: 'AC', Name: 'Accommodation' },
-              { Code: 'BD', Name: 'Bundle' },
-            ],
-          },
-        },
-      }))
-      .mockImplementationOnce(async ({ model }) => {
-        const opt = model.OptionInfoRequest.Opt;
-        if (opt === '???AC????????????') {
-          throw new Error('AC OptionInfo should have been skipped');
+    afterEach(() => {
+      mockCallTourplan.mockImplementation(defaultCallTourplan);
+    });
+
+    const mockFullCatalog = ({ throwOnOpt } = {}) => {
+      mockCallTourplan.mockImplementation(async ({ model }) => {
+        if (model.GetServicesRequest) return getServicesReply;
+        const opt = model.OptionInfoRequest && model.OptionInfoRequest.Opt;
+        if (throwOnOpt && throwOnOpt.includes(opt)) {
+          throw new Error(`${opt} OptionInfo should have been skipped`);
         }
-        if (opt === '???BD????????????') {
-          return { OptionInfoReply: { Option: bdOption } };
+        if (optionsByOpt[opt]) {
+          return { OptionInfoReply: { Option: optionsByOpt[opt] } };
         }
         throw new Error(`Unexpected OptionInfo Opt: ${opt}`);
       });
+    };
 
-    const retVal = await app.searchProductsForItinerary({
-      axios,
-      token,
-      typeDefsAndQueries,
-      payload: {
-        searchInput: '',
-        skipAccommodation: true,
-      },
+    it('fetches all service codes when omitServiceCodes is not provided', async () => {
+      const debugSpy = jest.spyOn(console, 'debug').mockImplementation(() => {});
+      mockFullCatalog();
+
+      const retVal = await app.searchProductsForItinerary({
+        axios,
+        token,
+        typeDefsAndQueries,
+        payload: { searchInput: '' },
+      });
+
+      expect(getOptionInfoCalls()).toEqual([
+        '???AC????????????',
+        '???BD????????????',
+        '???TR????????????',
+      ]);
+      expect(debugSpy).not.toHaveBeenCalled();
+      expect(retVal.products).toHaveLength(1);
+      expect(retVal.products[0].options.map(o => o.optionId)).toEqual([
+        'TESTACOPTION00001',
+        'TESTBDOPTION00001',
+        'TESTTROPTION00001',
+      ]);
+      debugSpy.mockRestore();
     });
 
-    const optionInfoCalls = mockCallTourplan.mock.calls
-      .map(([call]) => call.model.OptionInfoRequest)
-      .filter(Boolean);
-    expect(optionInfoCalls).toHaveLength(1);
-    expect(optionInfoCalls[0].Opt).toBe('???BD????????????');
-    expect(debugSpy).toHaveBeenCalledWith(
-      '[tourplan] Accommodation (AC) service code will be skipped in full catalog product search',
-    );
-    expect(retVal.products).toHaveLength(1);
-    expect(retVal.products[0].options[0].optionId).toBe('TESTBDOPTION00001');
+    it('fetches all service codes when omitServiceCodes is empty', async () => {
+      const debugSpy = jest.spyOn(console, 'debug').mockImplementation(() => {});
+      mockFullCatalog();
 
-    debugSpy.mockRestore();
+      const retVal = await app.searchProductsForItinerary({
+        axios,
+        token,
+        typeDefsAndQueries,
+        payload: {
+          searchInput: '',
+          omitServiceCodes: [],
+        },
+      });
+
+      expect(getOptionInfoCalls()).toEqual([
+        '???AC????????????',
+        '???BD????????????',
+        '???TR????????????',
+      ]);
+      expect(debugSpy).not.toHaveBeenCalled();
+      expect(retVal.products[0].options).toHaveLength(3);
+      debugSpy.mockRestore();
+    });
+
+    it('omits a single listed service code', async () => {
+      const debugSpy = jest.spyOn(console, 'debug').mockImplementation(() => {});
+      mockFullCatalog({ throwOnOpt: ['???AC????????????'] });
+
+      const retVal = await app.searchProductsForItinerary({
+        axios,
+        token,
+        typeDefsAndQueries,
+        payload: {
+          searchInput: '',
+          omitServiceCodes: ['AC'],
+        },
+      });
+
+      expect(getOptionInfoCalls()).toEqual([
+        '???BD????????????',
+        '???TR????????????',
+      ]);
+      expect(debugSpy).toHaveBeenCalledWith(
+        '[tourplan] Omitting service code(s) from full catalog product search: AC',
+      );
+      expect(retVal.products[0].options.map(o => o.optionId)).toEqual([
+        'TESTBDOPTION00001',
+        'TESTTROPTION00001',
+      ]);
+      debugSpy.mockRestore();
+    });
+
+    it('omits multiple listed service codes', async () => {
+      const debugSpy = jest.spyOn(console, 'debug').mockImplementation(() => {});
+      mockFullCatalog({ throwOnOpt: ['???AC????????????', '???TR????????????'] });
+
+      const retVal = await app.searchProductsForItinerary({
+        axios,
+        token,
+        typeDefsAndQueries,
+        payload: {
+          searchInput: '',
+          omitServiceCodes: ['AC', 'TR'],
+        },
+      });
+
+      expect(getOptionInfoCalls()).toEqual(['???BD????????????']);
+      expect(debugSpy).toHaveBeenCalledWith(
+        '[tourplan] Omitting service code(s) from full catalog product search: AC, TR',
+      );
+      expect(retVal.products[0].options.map(o => o.optionId)).toEqual([
+        'TESTBDOPTION00001',
+      ]);
+      debugSpy.mockRestore();
+    });
+
+    it('coerces a single string omitServiceCodes value to an array', async () => {
+      const debugSpy = jest.spyOn(console, 'debug').mockImplementation(() => {});
+      mockFullCatalog({ throwOnOpt: ['???AC????????????'] });
+
+      const retVal = await app.searchProductsForItinerary({
+        axios,
+        token,
+        typeDefsAndQueries,
+        payload: {
+          searchInput: '',
+          omitServiceCodes: 'AC',
+        },
+      });
+
+      expect(getOptionInfoCalls()).toEqual([
+        '???BD????????????',
+        '???TR????????????',
+      ]);
+      expect(debugSpy).toHaveBeenCalledWith(
+        '[tourplan] Omitting service code(s) from full catalog product search: AC',
+      );
+      expect(retVal.products[0].options.map(o => o.optionId)).toEqual([
+        'TESTBDOPTION00001',
+        'TESTTROPTION00001',
+      ]);
+      debugSpy.mockRestore();
+    });
+
+    it('normalizes case and whitespace in omitServiceCodes', async () => {
+      const debugSpy = jest.spyOn(console, 'debug').mockImplementation(() => {});
+      mockFullCatalog({ throwOnOpt: ['???AC????????????'] });
+
+      const retVal = await app.searchProductsForItinerary({
+        axios,
+        token,
+        typeDefsAndQueries,
+        payload: {
+          searchInput: '',
+          omitServiceCodes: [' ac '],
+        },
+      });
+
+      expect(getOptionInfoCalls()).toEqual([
+        '???BD????????????',
+        '???TR????????????',
+      ]);
+      expect(debugSpy).toHaveBeenCalledWith(
+        '[tourplan] Omitting service code(s) from full catalog product search: AC',
+      );
+      expect(retVal.products[0].options.map(o => o.optionId)).toEqual([
+        'TESTBDOPTION00001',
+        'TESTTROPTION00001',
+      ]);
+      debugSpy.mockRestore();
+    });
+
+    it('splits a comma-separated omitServiceCodes string', async () => {
+      const debugSpy = jest.spyOn(console, 'debug').mockImplementation(() => {});
+      mockFullCatalog({ throwOnOpt: ['???AC????????????', '???TR????????????'] });
+
+      const retVal = await app.searchProductsForItinerary({
+        axios,
+        token,
+        typeDefsAndQueries,
+        payload: {
+          searchInput: '',
+          omitServiceCodes: 'AC, TR',
+        },
+      });
+
+      expect(getOptionInfoCalls()).toEqual(['???BD????????????']);
+      expect(debugSpy).toHaveBeenCalledWith(
+        '[tourplan] Omitting service code(s) from full catalog product search: AC, TR',
+      );
+      expect(retVal.products[0].options.map(o => o.optionId)).toEqual([
+        'TESTBDOPTION00001',
+      ]);
+      debugSpy.mockRestore();
+    });
+
+    it('splits comma-separated entries inside an omitServiceCodes array', async () => {
+      const debugSpy = jest.spyOn(console, 'debug').mockImplementation(() => {});
+      mockFullCatalog({ throwOnOpt: ['???AC????????????', '???TR????????????'] });
+
+      const retVal = await app.searchProductsForItinerary({
+        axios,
+        token,
+        typeDefsAndQueries,
+        payload: {
+          searchInput: '',
+          omitServiceCodes: ['AC,SM', ' tr '],
+        },
+      });
+
+      expect(getOptionInfoCalls()).toEqual(['???BD????????????']);
+      expect(debugSpy).toHaveBeenCalledWith(
+        '[tourplan] Omitting service code(s) from full catalog product search: AC, SM, TR',
+      );
+      expect(retVal.products[0].options.map(o => o.optionId)).toEqual([
+        'TESTBDOPTION00001',
+      ]);
+      debugSpy.mockRestore();
+    });
+
+    it('throws No products found when every service code is omitted', async () => {
+      mockFullCatalog({
+        throwOnOpt: [
+          '???AC????????????',
+          '???BD????????????',
+          '???TR????????????',
+        ],
+      });
+
+      await expect(app.searchProductsForItinerary({
+        axios,
+        token,
+        typeDefsAndQueries,
+        payload: {
+          searchInput: '',
+          omitServiceCodes: ['AC', 'BD', 'TR'],
+        },
+      })).rejects.toThrow('No products found');
+
+      expect(getOptionInfoCalls()).toEqual([]);
+    });
   });
 
   describe('availability tests', () => {

package/itinerary-products-search.js

@@ -22,8 +22,9 @@ const searchProductsForItinerary = async ({
     // lastUpdatedFrom is used to get options that were updated after a certain date in Tourplan
     // example: lastUpdatedFrom: '2024-04-22 05:17:57.427Z'
     lastUpdatedFrom,
-    // skip Accommodation (AC) service code in full catalog search
-    skipAccommodation,
+    // service codes to omit from full catalog search
+    // (e.g. ['AC'], 'AC', or 'AC,SM'); when empty/absent, get all
+    omitServiceCodes,
   },
   callTourplan,
 }) => {
@@ -60,7 +61,7 @@ const searchProductsForItinerary = async ({
     /*
       Full catalog path:
       1. getServiceCodes -> [AC, BD]
-      2. for each serviceCode getoptions (optionally skip AC when skipAccommodation is set)
+      2. for each serviceCode getoptions (omitting all service codes in omitServiceCodes)
       3. convert them to ti2 products structure
       4. merge all products from all serviceCodes
     */
@@ -79,12 +80,26 @@ const searchProductsForItinerary = async ({
     });
     let serviceCodes = R.pathOr([], ['GetServicesReply', 'TPLServices', 'TPLService'], getServicesReply);
     if (!Array.isArray(serviceCodes)) serviceCodes = [serviceCodes];
-    if (skipAccommodation) {
-      console.debug('[tourplan] Accommodation (AC) service code will be skipped in full catalog product search');
+    let rawOmitCodes = [];
+    if (Array.isArray(omitServiceCodes)) {
+      rawOmitCodes = omitServiceCodes;
+    } else if (omitServiceCodes) {
+      rawOmitCodes = [omitServiceCodes];
+    }
+    const omitSet = new Set(
+      rawOmitCodes
+        .flatMap(code => String(code == null ? '' : code).split(','))
+        .map(code => code.trim().toUpperCase())
+        .filter(Boolean),
+    );
+    if (omitSet.size > 0) {
+      console.debug(
+        `[tourplan] Omitting service code(s) from full catalog product search: ${[...omitSet].join(', ')}`,
+      );
     }
     serviceCodes = serviceCodes
       .map(s => s.Code)
-      .filter(code => !(skipAccommodation && code === 'AC'));
+      .filter(code => !omitSet.has(String(code).trim().toUpperCase()));
     await Promise.each(serviceCodes, async serviceCode => {
       const getOptionsModel = {
         OptionInfoRequest: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ti2-tourplan",
-  "version": "1.0.128",
+  "version": "1.0.129",
   "description": "Tourplan's TI2 Plugin",
   "main": "index.js",
   "scripts": {

package/utils.js

@@ -71,6 +71,26 @@ const escapeInvalidXmlChars = str => {
     .replace(BAD_XML_CHARS, '');
 };
 
+const getValidDateOfBirth = dob => {
+  if (typeof dob !== 'string') return undefined;
+  const match = /^(\d{4})-(\d{2})-(\d{2})$/.exec(dob);
+  if (!match) return undefined;
+
+  const year = Number(match[1]);
+  const month = Number(match[2]);
+  const day = Number(match[3]);
+  if (year === 0) return undefined;
+
+  // setUTCFullYear avoids Date.UTC's 0-99 year remapping.
+  const date = new Date(0);
+  date.setUTCFullYear(year, month - 1, day);
+  if (date.getUTCFullYear() !== year ||
+    date.getUTCMonth() !== month - 1 ||
+    date.getUTCDate() !== day) return undefined;
+
+  return dob;
+};
+
 const getRoomConfigs = (paxConfigs, noPaxList) => {
   // There should be only 1 RoomConfigs for AddServiceRequest
   const RoomConfigs = {};
@@ -139,12 +159,13 @@ const getRoomConfigs = (paxConfigs, noPaxList) => {
           }[p.passengerType] || 'A',
         };
         if (p.salutation) EachPaxDetails.Title = escapeInvalidXmlChars(p.salutation);
-        if (p.dob) EachPaxDetails.DateOfBirth = p.dob;
+        const validDateOfBirth = getValidDateOfBirth(p.dob);
+        if (validDateOfBirth) EachPaxDetails.DateOfBirth = validDateOfBirth;
         // NOTE: TourPlan API doesn't accept age as empty string, i.e. empty XML tag <Age/>
         // and trhows and error like - "1000 SCN System.InvalidOperationException: There is an
         // error in XML document (29, 8). (Input string was not in a correct format.)"
         // The solution is to NOT send the Age tag if it's empty
-        if (!R.isNil(p.age) && !Number.isNaN(p.age) && p.age) {
+        if (!validDateOfBirth && !R.isNil(p.age) && !Number.isNaN(p.age) && p.age) {
           if (!(p.passengerType === passengerTypeMap.Adult && p.age === 0)) {
             EachPaxDetails.Age = p.age;
           }

package/utils.test.js

@@ -1,4 +1,105 @@
-const { escapeInvalidXmlChars } = require('./utils');
+const { escapeInvalidXmlChars, getRoomConfigs } = require('./utils');
+
+describe('getRoomConfigs', () => {
+  const getPaxDetails = passenger => getRoomConfigs([{
+    roomType: 'Double',
+    passengers: [passenger],
+  }]).RoomConfig[0].PaxList.PaxDetails[0];
+
+  it('uses a valid DOB instead of age', () => {
+    const withDobAndAge = getPaxDetails({
+      firstName: 'Ada',
+      lastName: 'Lovelace',
+      passengerType: 'Adult',
+      dob: '1990-02-28',
+      age: 36,
+    });
+    const withDobOnly = getPaxDetails({
+      passengerType: 'Child',
+      dob: '2000-02-29',
+    });
+    const withPre100Dob = getPaxDetails({
+      passengerType: 'Adult',
+      dob: '0099-12-31',
+      age: 99,
+    });
+
+    expect(withDobAndAge.DateOfBirth).toBe('1990-02-28');
+    expect(withDobAndAge).not.toHaveProperty('Age');
+    expect(withDobOnly.DateOfBirth).toBe('2000-02-29');
+    expect(withDobOnly).not.toHaveProperty('Age');
+    expect(withPre100Dob.DateOfBirth).toBe('0099-12-31');
+    expect(withPre100Dob).not.toHaveProperty('Age');
+  });
+
+  it('uses age when DOB is absent', () => {
+    const paxDetails = getPaxDetails({
+      passengerType: 'Adult',
+      age: 36,
+    });
+
+    expect(paxDetails.Age).toBe(36);
+    expect(paxDetails).not.toHaveProperty('DateOfBirth');
+  });
+
+  it.each([
+    ['blank', ''],
+    ['malformed', '1990/02/28'],
+    ['unpadded', '1990-2-28'],
+    ['date-time', '1990-02-28T00:00:00Z'],
+    ['boxed string', Object('1990-02-28')],
+    ['Date object', new Date('1990-02-28T00:00:00Z')],
+    ['number', 19900228],
+  ])('falls back to age for a %s DOB', (description, dob) => {
+    const paxDetails = getPaxDetails({
+      passengerType: 'Adult',
+      dob,
+      age: 36,
+    });
+
+    expect(paxDetails.Age).toBe(36);
+    expect(paxDetails).not.toHaveProperty('DateOfBirth');
+  });
+
+  it.each([
+    ['year zero', '0000-01-01'],
+    ['month zero', '1990-00-01'],
+    ['day zero', '1990-01-00'],
+    ['day rollover', '1990-04-31'],
+    ['non-leap day', '2025-02-29'],
+    ['non-leap century day', '1900-02-29'],
+  ])('falls back to age for an impossible %s DOB', (description, dob) => {
+    const paxDetails = getPaxDetails({
+      passengerType: 'Adult',
+      dob,
+      age: 36,
+    });
+
+    expect(paxDetails.Age).toBe(36);
+    expect(paxDetails).not.toHaveProperty('DateOfBirth');
+  });
+
+  it('preserves the PersonId early return', () => {
+    expect(getPaxDetails({
+      personId: 'person-123',
+      passengerType: 'Adult',
+      dob: '1990-02-28',
+      age: 36,
+    })).toEqual({ PersonId: 'person-123' });
+  });
+
+  it('preserves zero, blank, and NaN age behavior without a valid DOB', () => {
+    const adultZero = getPaxDetails({ passengerType: 'Adult', age: 0 });
+    const childZero = getPaxDetails({ passengerType: 'Child', age: 0 });
+    const blank = getPaxDetails({ passengerType: 'Adult', age: '' });
+    const nan = getPaxDetails({ passengerType: 'Adult', age: NaN });
+
+    [adultZero, childZero, blank, nan].forEach(paxDetails => {
+      expect(paxDetails).not.toHaveProperty('Age');
+      expect(paxDetails).not.toHaveProperty('DateOfBirth');
+    });
+  });
+});
 
 describe('escapeInvalidXmlChars', () => {
   it('returns empty string for falsy input', () => {