ffc-pay-schemes 1.0.1

package/app/get-account-code-map/delinked-map.js

@@ -0,0 +1,31 @@
+const { SOS210, SOS471, SOS472, SOS473, SOS474 } = require('../constants/account-codes/ap')
+const { SOS471: SOS471Ad, SOS472: SOS472Ad, SOS473: SOS473Ad, SOS474: SOS474Ad, SOS330 } = require('../constants/account-codes/ar-admin')
+const { SOS471: SOS471Irr, SOS472: SOS472Irr, SOS473: SOS473Irr, SOS474: SOS474Irr, SOS310 } = require('../constants/account-codes/ar-irregular')
+const { G00, X01, X02, X03, X04 } = require('../constants/description-codes')
+
+module.exports = [{
+  lineCode: G00,
+  ap: SOS210,
+  arAdmin: SOS330,
+  arIrregular: SOS310
+}, {
+  lineCode: X01,
+  ap: SOS471,
+  arAdmin: SOS471Ad,
+  arIrregular: SOS471Irr
+}, {
+  lineCode: X02,
+  ap: SOS472,
+  arAdmin: SOS472Ad,
+  arIrregular: SOS472Irr
+}, {
+  lineCode: X03,
+  ap: SOS473,
+  arAdmin: SOS473Ad,
+  arIrregular: SOS473Irr
+}, {
+  lineCode: X04,
+  ap: SOS474,
+  arAdmin: SOS474Ad,
+  arIrregular: SOS474Irr
+}]

package/app/get-account-code-map/index.js

@@ -0,0 +1,57 @@
+const {
+  SFI,
+  SFI_PILOT,
+  LUMP_SUMS,
+  VET_VISITS,
+  CS,
+  BPS,
+  SFI23,
+  DELINKED,
+  SFI_EXPANDED,
+  COHT_REVENUE,
+  COHT_CAPITAL,
+  FPTT,
+  WMP
+} = require('../constants/scheme-ids')
+
+const defaultMap = require('./default-map')
+const sfiFrpsMap = require('./sfi-frps-map')
+const lumpSumsMap = require('./lump-sums-map')
+const vetVisitsMap = require('./vet-visits-map')
+const csMap = require('./cs-map')
+const bpsMap = require('./bps-map')
+const delinkedMap = require('./delinked-map')
+const cohtcMap = require('./cohtc-map')
+
+const schemeMap = new Map([
+  [SFI, sfiFrpsMap],
+  [SFI_PILOT, sfiFrpsMap],
+  [SFI23, sfiFrpsMap],
+  [SFI_EXPANDED, sfiFrpsMap],
+  [LUMP_SUMS, lumpSumsMap],
+  [VET_VISITS, vetVisitsMap],
+  [CS, csMap],
+  [BPS, bpsMap],
+  [DELINKED, delinkedMap],
+  [COHT_REVENUE, defaultMap],
+  [COHT_CAPITAL, cohtcMap],
+  [FPTT, sfiFrpsMap],
+  [WMP, sfiFrpsMap]
+])
+
+const getAccountCodeMap = (schemeId) => {
+  if (!schemeId) {
+    throw new Error('schemeId is required')
+  }
+
+  const mapForScheme = schemeMap.get(schemeId)
+  if (mapForScheme) {
+    return mapForScheme
+  }
+
+  throw new Error(`No account codes found for scheme ${schemeId}`)
+}
+
+module.exports = {
+  getAccountCodeMap
+}

package/app/get-account-code-map/lump-sums-map.js

@@ -0,0 +1,11 @@
+const { SOS210 } = require('../constants/account-codes/ap')
+const { SOS330 } = require('../constants/account-codes/ar-admin')
+const { SOS310 } = require('../constants/account-codes/ar-irregular')
+const { G00 } = require('../constants/description-codes')
+
+module.exports = [{
+  lineCode: G00,
+  ap: SOS210,
+  arAdmin: SOS330,
+  arIrregular: SOS310
+}]

package/app/get-account-code-map/sfi-frps-map.js

@@ -0,0 +1,15 @@
+const defaultMap = require('./default-map')
+
+const { SOS920 } = require('../constants/account-codes/ap')
+const { SOS940 } = require('../constants/account-codes/ar-admin')
+const { SOS960 } = require('../constants/account-codes/ar-irregular')
+const { P02 } = require('../constants/description-codes')
+
+const p02Entry = {
+  lineCode: P02,
+  ap: SOS920,
+  arAdmin: SOS940,
+  arIrregular: SOS960
+}
+
+module.exports = [defaultMap[0], p02Entry, ...defaultMap.slice(1)]

package/app/get-account-code-map/vet-visits-map.js

@@ -0,0 +1,11 @@
+const { SOS210 } = require('../constants/account-codes/ap')
+const { SOS330 } = require('../constants/account-codes/ar-admin')
+const { SOS310 } = require('../constants/account-codes/ar-irregular')
+const { G00 } = require('../constants/description-codes')
+
+module.exports = [{
+  lineCode: G00,
+  ap: SOS210,
+  arAdmin: SOS330,
+  arIrregular: SOS310
+}]

package/app/get-scheme-batch-properties.js

@@ -0,0 +1,9 @@
+const schemeBatchProperties = require('./constants/scheme-batch-properties')
+
+const getSchemeBatchProperties = (schemeId) => {
+  return schemeBatchProperties.find(scheme => scheme.schemeId === schemeId)
+}
+
+module.exports = {
+  getSchemeBatchProperties
+}

package/app/get-scheme-properties.js

@@ -0,0 +1,31 @@
+const schemes = require('./constants/schemes')
+const { MANUAL } = require('./constants/scheme-ids')
+
+const getSchemeProperties = (schemeId, sourceSystem, pillar) => {
+  let scheme = schemeId
+    ? schemes.find(item => item.schemeId === schemeId)
+    : schemes.find(item => item.sourceSystem === sourceSystem)
+
+  if (!scheme) {
+    throw new Error(`Scheme with ID ${schemeId}, sourceSystem ${sourceSystem} not found`)
+  }
+
+  if (scheme?.schemeId === MANUAL && pillar) {
+    const pillarScheme = schemes.find(item => item.pillar === pillar)
+
+    if (pillarScheme) {
+      scheme = {
+        ...scheme,
+        pillar,
+        deliveryBody: pillarScheme.deliveryBody,
+        fundCode: pillarScheme.fundCode
+      }
+    }
+  }
+
+  return scheme
+}
+
+module.exports = {
+  getSchemeProperties
+}

package/app/get-schemes.js

@@ -0,0 +1,12 @@
+const schemeIds = require('./constants/scheme-ids')
+const schemeNames = require('./constants/scheme-names')
+
+const getSchemes = () =>
+  Object.keys(schemeIds).map((schemeKey) => ({
+    schemeId: schemeIds[schemeKey],
+    schemeName: schemeNames[schemeKey]
+  }))
+
+module.exports = {
+  getSchemes
+}

package/app/scheme-does-not-require-ppas.js

@@ -0,0 +1,9 @@
+const schemesNotRequiringPPAs = require('./constants/schemes-not-requiring-ppas')
+
+const schemeDoesNotRequirePPAs = (schemeId) => {
+  return schemesNotRequiringPPAs.includes(schemeId)
+}
+
+module.exports = {
+  schemeDoesNotRequirePPAs
+}

package/app/scheme-provides-accounting-values.js

@@ -0,0 +1,9 @@
+const schemesProvidingAccountingValues = require('./constants/schemes-providing-account-values')
+
+const schemeProvidesAccountingValues = (schemeId) => {
+  return schemesProvidingAccountingValues.includes(schemeId)
+}
+
+module.exports = {
+  schemeProvidesAccountingValues
+}

package/azure-pipelines.yml

@@ -0,0 +1,45 @@
+# Pipeline name convention
+name: '$(Build.SourceBranchName)$(System.PullRequest.SourceBranch)$(Rev:.r)'
+
+# Trigger configuration
+trigger:
+  - main
+  - feature/*
+  - release/*
+
+pr:
+  - main
+  - releases/*
+
+# Parameters
+parameters:
+  - name: prDeployment
+    type: boolean
+    default: true
+  - name: branchDeployment
+    type: boolean
+    default: true
+  - name: sndDeployment
+    type: boolean
+    default: true
+
+# Resources
+resources:
+  repositories:
+    - repository: templates
+      endpoint: DEFRA
+      type: github
+      name: defra/fcp-pipeline-common
+      # ref: pl-709   # Optional: build specific pipeline branch
+
+# Pipeline template
+extends:
+  template: fcp-common-pipeline.yaml@templates
+  parameters:
+    appFrameworkType: nodejs   # Optional: nodejs (default), dotnet, java
+    Type: npm           # Optional: kubernetes (default), npm, functions, nuget
+    npmLegacyPeerDeps: false   # Optional: default is false
+    failOnThreshold: moderate  # Optional: moderate (default), low, high, critical
+    prDeployment: ${{ parameters.prDeployment }}
+    branchDeployment: ${{ parameters.branchDeployment }}
+    sndDeployment: ${{ parameters.sndDeployment }}

package/eslint.config.js

@@ -0,0 +1,3 @@
+'use strict'
+
+module.exports = require('neostandard')({ globals: ['describe', 'beforeEach', 'expect', 'test', 'afterEach', 'jest', 'beforeAll', 'afterAll'] })

package/index.js

@@ -0,0 +1,21 @@
+const { createInvoiceNumber } = require('./app/create-invoice-number')
+const { createSplitInvoiceNumber } = require('./app/create-split-invoice-number')
+const { getAccountCodeMap } = require('./app/get-account-code-map')
+const { getSchemeBatchProperties } = require('./app/get-scheme-batch-properties')
+const { getSchemeProperties } = require('./app/get-scheme-properties')
+const { getSchemes } = require('./app/get-schemes')
+const { schemeDoesNotRequirePPAs } = require('./app/scheme-does-not-require-ppas')
+const { schemeProvidesAccountingValues } = require('./app/scheme-provides-accounting-values')
+const schemeIds = require('./app/constants/scheme-ids')
+
+module.exports = {
+  createInvoiceNumber,
+  createSplitInvoiceNumber,
+  getAccountCodeMap,
+  getSchemeBatchProperties,
+  getSchemeProperties,
+  getSchemes,
+  schemeDoesNotRequirePPAs,
+  schemeProvidesAccountingValues,
+  schemeIds
+}

package/jest.config.js

@@ -0,0 +1,40 @@
+module.exports = {
+  collectCoverage: true,
+  collectCoverageFrom: [
+    '**/*.js',
+    '!**/*.test.js'
+  ],
+  coverageDirectory: 'test-output',
+  coverageReporters: [
+    'text-summary',
+    'lcov',
+    'cobertura'
+  ],
+  coveragePathIgnorePatterns: [
+    '<rootDir>/node_modules/',
+    '<rootDir>/test-output/',
+    '<rootDir>/test/',
+    '<rootDir>/jest.config.js',
+    '<rootDir>/app/config',
+    '<rootDir>/jest.setup.js',
+    '<rootDir>/eslint.config.js'
+  ],
+  modulePathIgnorePatterns: [
+    'node_modules'
+  ],
+  reporters: [
+    'default',
+    [
+      'jest-junit',
+      {
+        suiteName: 'jest tests',
+        outputDirectory: 'test-output',
+        outputName: 'junit.xml'
+      }
+    ]
+  ],
+  testEnvironment: 'node',
+  testPathIgnorePatterns: [],
+  setupFilesAfterEnv: ['./jest.setup.js'],
+  verbose: true
+}

package/jest.setup.js

@@ -0,0 +1,2 @@
+const TIMEOUT = 30000
+jest.setTimeout(TIMEOUT)

package/package.json

@@ -0,0 +1,34 @@
+{
+  "name": "ffc-pay-schemes",
+  "version": "1.0.1",
+  "description": "Module for defining and providing details for the payment schemes supported by the Payment Hub.",
+  "main": "index.js",
+  "scripts": {
+    "test": "jest --runInBand --forceExit",
+    "test:unit": "PORT=0 node --experimental-vm-modules node_modules/jest/bin/jest.js --testPathPatterns=test/unit",
+    "test:watch": "jest --coverage=false --onlyChanged --watch --runInBand",
+    "test:lint": "eslint",
+    "test:lint:fix": "eslint --fix",
+    "prepare": "node --eval \"const fs=require('fs'),cp=require('child_process');if(fs.existsSync('.git')&&!process.env.CI&&!process.env.TF_BUILD)cp.execSync('husky',{stdio:'inherit'})\""
+  },
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/DEFRA/ffc-pay-schemes.git"
+  },
+  "author": "Defra DDTS",
+  "contributors": [
+    "Sam Plackett samuel.plackett@atos.ai"
+  ],
+  "license": "OGL-UK-3.0",
+  "bugs": {
+    "url": "https://github.com/DEFRA/ffc-pay-schemes/issues"
+  },
+  "homepage": "https://github.com/DEFRA/ffc-pay-schemes#readme",
+  "devDependencies": {
+    "eslint": "9.39.5",
+    "husky": "9.1.7",
+    "jest": "30.4.2",
+    "jest-junit": "17.0.0",
+    "neostandard": "0.13.0"
+  }
+}

package/sonar-project.properties

@@ -0,0 +1,3 @@
+sonar.javascript.exclusions=**/jest.config.js,**/jest.setup.js,**/__mocks__/**,**/node_modules/**,**/test/**,**/test-output/**,**/eslint.config.js
+sonar.javascript.lcov.reportPaths=test-output/lcov.info
+sonar.exclusions=/test/**,**/*.test.js,*snyk_report.html,*snyk_report.css

package/test/unit/create-invoice-number/create-default-invoice-number.test.js

@@ -0,0 +1,15 @@
+const { createDefaultInvoiceNumber } = require('../../../app/create-invoice-number/create-default-invoice-number')
+
+describe('createDefaultInvoiceNumber', () => {
+  test('creates an invoice number with a padded payment request number', () => {
+    expect(createDefaultInvoiceNumber({
+      agreementNumber: 'AGR123',
+      paymentRequestNumber: 7
+    })).toBe('AGR123V007')
+  })
+
+  test('returns null when required values are missing', () => {
+    expect(createDefaultInvoiceNumber({ agreementNumber: 'AGR123' })).toBeNull()
+    expect(createDefaultInvoiceNumber({ paymentRequestNumber: 7 })).toBeNull()
+  })
+})

package/test/unit/create-invoice-number/create-es-invoice-number.test.js

@@ -0,0 +1,10 @@
+const { createESInvoiceNumber } = require('../../../app/create-invoice-number/create-es-invoice-number')
+
+describe('createESInvoiceNumber', () => {
+  test('creates an ES invoice number', () => {
+    expect(createESInvoiceNumber({
+      invoiceNumber: 'INV123',
+      contractNumber: 'CON456'
+    })).toBe('I(INV123)CON456')
+  })
+})

package/test/unit/create-invoice-number/create-imps-invoice-number.test.js

@@ -0,0 +1,27 @@
+const { createIMPSInvoiceNumber } = require('../../../app/create-invoice-number/create-imps-invoice-number')
+
+describe('createIMPSInvoiceNumber', () => {
+  test('adds the trader to an invoice number', () => {
+    expect(createIMPSInvoiceNumber({
+      invoiceNumber: 'INV/123',
+      trader: 'TRADER'
+    })).toBe('INV/TRADER123')
+  })
+
+  test('returns the invoice number when it already contains the trader', () => {
+    const paymentRequest = {
+      invoiceNumber: 'INV/TRADER123',
+      trader: 'TRADER'
+    }
+
+    expect(createIMPSInvoiceNumber(paymentRequest))
+      .toBe(paymentRequest.invoiceNumber)
+  })
+
+  test('returns null when the invoice number has no separator', () => {
+    expect(createIMPSInvoiceNumber({
+      invoiceNumber: 'INV123',
+      trader: 'TRADER'
+    })).toBeNull()
+  })
+})

package/test/unit/create-invoice-number/create-siti-agri-invoice-number.test.js

@@ -0,0 +1,26 @@
+const { createSitiAgriInvoiceNumber } = require('../../../app/create-invoice-number/create-siti-agri-invoice-number')
+
+describe('createSitiAgriInvoiceNumber', () => {
+  test('creates a SITI Agri invoice number', () => {
+    expect(createSitiAgriInvoiceNumber({
+      invoiceNumber: 'A1234567',
+      contractNumber: 'CON123',
+      paymentRequestNumber: 5
+    })).toBe('S1234567CON123V005')
+  })
+
+  test('returns null when the invoice number is too short', () => {
+    expect(createSitiAgriInvoiceNumber({
+      invoiceNumber: '123456',
+      contractNumber: 'CON123',
+      paymentRequestNumber: 5
+    })).toBeNull()
+  })
+
+  test('returns null when required values are missing', () => {
+    expect(createSitiAgriInvoiceNumber({
+      invoiceNumber: 'A1234567',
+      paymentRequestNumber: 5
+    })).toBeNull()
+  })
+})

package/test/unit/create-invoice-number/create-standard-scheme-invoice-number.test.js

@@ -0,0 +1,31 @@
+const { createStandardSchemeInvoiceNumber } = require('../../../app/create-invoice-number/create-standard-scheme-invoice-number')
+
+describe('createStandardSchemeInvoiceNumber', () => {
+  test('creates a standard scheme invoice number', () => {
+    expect(createStandardSchemeInvoiceNumber({
+      invoiceNumber: 'A1234567',
+      contractNumber: 'CON123',
+      paymentRequestNumber: 5
+    })).toBe('A1234567CON123V005')
+  })
+
+  test('returns null when the invoice number is too short', () => {
+    expect(createStandardSchemeInvoiceNumber({
+      invoiceNumber: '123456',
+      contractNumber: 'CON123',
+      paymentRequestNumber: 5
+    })).toBeNull()
+  })
+
+  test('returns null when required values are missing', () => {
+    expect(createStandardSchemeInvoiceNumber({
+      invoiceNumber: 'A1234567',
+      paymentRequestNumber: 5
+    })).toBeNull()
+
+    expect(createStandardSchemeInvoiceNumber({
+      invoiceNumber: 'A1234567',
+      contractNumber: 'CON123'
+    })).toBeNull()
+  })
+})

package/test/unit/create-invoice-number/index.test.js

@@ -0,0 +1,113 @@
+jest.mock('../../../app/create-invoice-number/create-standard-scheme-invoice-number', () => ({
+  createStandardSchemeInvoiceNumber: jest.fn()
+}))
+jest.mock('../../../app/create-invoice-number/create-siti-agri-invoice-number', () => ({
+  createSitiAgriInvoiceNumber: jest.fn()
+}))
+jest.mock('../../../app/create-invoice-number/create-es-invoice-number', () => ({
+  createESInvoiceNumber: jest.fn()
+}))
+jest.mock('../../../app/create-invoice-number/create-imps-invoice-number', () => ({
+  createIMPSInvoiceNumber: jest.fn()
+}))
+jest.mock('../../../app/create-invoice-number/create-default-invoice-number', () => ({
+  createDefaultInvoiceNumber: jest.fn()
+}))
+
+const { createInvoiceNumber } = require('../../../app/create-invoice-number')
+const { createStandardSchemeInvoiceNumber } = require('../../../app/create-invoice-number/create-standard-scheme-invoice-number')
+const { createSitiAgriInvoiceNumber } = require('../../../app/create-invoice-number/create-siti-agri-invoice-number')
+const { createESInvoiceNumber } = require('../../../app/create-invoice-number/create-es-invoice-number')
+const { createIMPSInvoiceNumber } = require('../../../app/create-invoice-number/create-imps-invoice-number')
+const { createDefaultInvoiceNumber } = require('../../../app/create-invoice-number/create-default-invoice-number')
+
+const schemeIds = require('../../../app/constants/scheme-ids')
+const sourceSystems = require('../../../app/constants/source-systems')
+
+describe('createInvoiceNumber', () => {
+  const paymentRequest = {
+    invoiceNumber: 'INV123',
+    schemeId: schemeIds.SFI,
+    sourceSystem: 'SOURCE',
+    contractNumber: 'CON123',
+    agreementNumber: 'AGR123',
+    paymentRequestNumber: 1,
+    trader: 'TRADER'
+  }
+
+  beforeEach(() => {
+    jest.clearAllMocks()
+
+    createStandardSchemeInvoiceNumber.mockReturnValue('standard-invoice')
+    createSitiAgriInvoiceNumber.mockReturnValue('siti-agri-invoice')
+    createESInvoiceNumber.mockReturnValue('es-invoice')
+    createIMPSInvoiceNumber.mockReturnValue('imps-invoice')
+    createDefaultInvoiceNumber.mockReturnValue('default-invoice')
+  })
+
+  test.each(['MANUAL', 'FC', 'FPTT', 'WMP'])(
+    'returns the existing invoice number for %s',
+    (schemeKey) => {
+      const request = {
+        ...paymentRequest,
+        schemeId: schemeIds[schemeKey]
+      }
+
+      expect(createInvoiceNumber(request)).toBe(request.invoiceNumber)
+    }
+  )
+
+  test('returns the existing invoice number for a manual source system', () => {
+    const request = {
+      ...paymentRequest,
+      sourceSystem: sourceSystems.MANUAL
+    }
+
+    expect(createInvoiceNumber(request)).toBe(request.invoiceNumber)
+  })
+
+  test.each(['DELINKED', 'SFI_EXPANDED', 'COHT_REVENUE', 'COHT_CAPITAL'])(
+    'uses the standard scheme invoice number creator for %s',
+    (schemeKey) => {
+      const request = { ...paymentRequest, schemeId: schemeIds[schemeKey] }
+
+      expect(createInvoiceNumber(request)).toBe('standard-invoice')
+      expect(createStandardSchemeInvoiceNumber).toHaveBeenCalledWith(request)
+    }
+  )
+
+  test.each(['SFI', 'SFI_PILOT', 'LUMP_SUMS', 'CS', 'BPS', 'SFI23'])(
+    'uses the SITI Agri invoice number creator for %s',
+    (schemeKey) => {
+      const request = { ...paymentRequest, schemeId: schemeIds[schemeKey] }
+
+      expect(createInvoiceNumber(request)).toBe('siti-agri-invoice')
+      expect(createSitiAgriInvoiceNumber).toHaveBeenCalledWith(request)
+    }
+  )
+
+  test('uses the ES invoice number creator', () => {
+    const request = { ...paymentRequest, schemeId: schemeIds.ES }
+
+    expect(createInvoiceNumber(request)).toBe('es-invoice')
+    expect(createESInvoiceNumber).toHaveBeenCalledWith(request)
+  })
+
+  test('uses the IMPS invoice number creator', () => {
+    const request = { ...paymentRequest, schemeId: schemeIds.IMPS }
+
+    expect(createInvoiceNumber(request)).toBe('imps-invoice')
+    expect(createIMPSInvoiceNumber).toHaveBeenCalledWith(request)
+  })
+
+  test('uses the default invoice number creator for an unmapped scheme', () => {
+    const request = { ...paymentRequest, schemeId: schemeIds.MANUAL + 100 }
+
+    expect(createInvoiceNumber(request)).toBe('default-invoice')
+    expect(createDefaultInvoiceNumber).toHaveBeenCalledWith(request)
+  })
+
+  test('returns null when the payment request is invalid', () => {
+    expect(createInvoiceNumber()).toBeNull()
+  })
+})

package/test/unit/create-invoice-number/use-invoice-number-as-is.test.js

@@ -0,0 +1,9 @@
+const { useInvoiceNumberAsIs } = require('../../../app/create-invoice-number/use-invoice-number-as-is')
+
+describe('useInvoiceNumberAsIs', () => {
+  test('returns the invoice number unchanged', () => {
+    const invoiceNumber = 'INV-12345'
+
+    expect(useInvoiceNumberAsIs(invoiceNumber)).toBe(invoiceNumber)
+  })
+})

package/test/unit/create-split-invoice-number/create-default-invoice-number.test.js

@@ -0,0 +1,15 @@
+const { createDefaultInvoiceNumber } = require('../../../app/create-split-invoice-number/create-default-invoice-number')
+
+describe('createDefaultInvoiceNumber', () => {
+  test.each([
+    ['INV1234', 'A', 'INVA134'],
+    ['123456789', 'B', '12345B689'],
+    ['ABC-1234', '9', 'ABC-9134']
+  ])(
+    'inserts splitId %s into invoice number %s',
+    (invoiceNumber, splitId, expectedInvoiceNumber) => {
+      expect(createDefaultInvoiceNumber(invoiceNumber, splitId))
+        .toBe(expectedInvoiceNumber)
+    }
+  )
+})

package/test/unit/create-split-invoice-number/create-siti-agri-invoice-number.test.js

@@ -0,0 +1,15 @@
+const { createSitiAgriInvoiceNumber } = require('../../../app/create-split-invoice-number/create-siti-agri-invoice-number')
+
+describe('createSitiAgriInvoiceNumber', () => {
+  test.each([
+    ['1234567890123', 'A', '12345678A9023'],
+    ['ABCDEFGH12345', '9', 'ABCDEFGH91245'],
+    ['SITIAGRI000123', 'X', 'SITIAGRIX00023']
+  ])(
+    'inserts splitId %s into invoice number %s',
+    (invoiceNumber, splitId, expectedInvoiceNumber) => {
+      expect(createSitiAgriInvoiceNumber(invoiceNumber, splitId))
+        .toBe(expectedInvoiceNumber)
+    }
+  )
+})