include io-blksize parameter

[ghost]

Closes #2864

[tertsdiepraam]

Nice! Thanks for adding tests for this too!

[ghost]

Is there something I need to do to close this pull request?

[tertsdiepraam]

No, I should've merged this 8 days ago :)

[alextibbles]

FYI I noticed this test failure today:

failures:

---- test_split::test_split_number_with_io_blksize stdout ----
current_directory_resolved: 
run: /home/alex/code/rust/coreutils/target/debug/coreutils split -n 5 asciilowercase.txt ---io-blksize 1024
open: /tmp/user/1000/.tmpM2oq8D/xaa
open: /tmp/user/1000/.tmpM2oq8D/xab
open: /tmp/user/1000/.tmpM2oq8D/xac
open: /tmp/user/1000/.tmpM2oq8D/xad
open: /tmp/user/1000/.tmpM2oq8D/xae
thread 'test_split::test_split_number_with_io_blksize' panicked at 'assertion failed: `(left == right)`
  left: `"uvwxyz\n"`,
 right: `"uvwxyz"`', /home/alex/code/rust/coreutils/tests/by-util/test_split.rs:466:5

Running my system split manually it looks like the behavior is correct, so the expectation should be updated to include the \n.

[ghost]

@alextibbles Good catch. I have to update a newly introduced test case to incorporate some changes between the commit and the merge.

[ghost]

I created a pull request #3140. Please take a look.