Skip to content

Fix trace sequence - #35

Open
ejlegarda wants to merge 3 commits into
glenselle:masterfrom
ejlegarda:fixTraceSequence
Open

ejlegarda wants to merge 3 commits into
glenselle:masterfrom
ejlegarda:fixTraceSequence

Conversation

@ejlegarda

Copy link
Copy Markdown

fixed missing sequence number when trace number is assigned a value in detail records

@glenselle

glenselle commented Aug 13, 2025

Copy link
Copy Markdown
Owner

When I make this change on the v1 branch the snapshot test fails and shows the diff. It looks like we extend one char beyond the length of the other lines which looks interesting.

 FAIL  lib/file/index.test.js
  ● File integration › creates a File with no options, adds a single batch, and generates the file output (snapshot)

    expect(received).toMatchInlineSnapshot(snapshot)

    Snapshot name: `File integration creates a File with no options, adds a single batch, and generates the file output (snapshot) 1`

    - Snapshot  - 2
    + Received  + 2

    @@ -1,8 +1,8 @@
     "101 081000032 1234567892508122131A094101Some Bank              Your Company Inc       #A000001
      5220Test Company                        123456789 WEBPAYROLL   202508250812   1011000010000000
    - 622011000015123456789        0001000000ID123          John Doe              A11000000001      
    + 622011000015123456789        0001000000ID123          John Doe              A110000000010000000
      705Invoice 12345                                                                   01000000001
      82200000020001100001000000000000000001000000123456789                          011000010000000
      9000001000001000000010001100001000000000000000001000000                                       
      9999999999999999999999999999999999999999999999999999999999999999999999999999999999999999999999
      9999999999999999999999999999999999999999999999999999999999999999999999999999999999999999999999

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants