Backport(v1.16) test_cat: specify the reading size to read in binary …

…mode (#4730) (#4799) **Which issue(s) this PR fixes**: Backport #4730 **What this PR does / why we need it**: The binary files may not be read properly. This changes to read files in binary mode. **Docs Changes**: **Release Note**: Signed-off-by: Shizuo Fujita <[email protected]> Signed-off-by: Kentaro Hayashi <[email protected]> Co-authored-by: Shizuo Fujita <[email protected]>
fluent · Jan 29, 2025 · 9bea070 · 9bea070
1 parent ce38a8d
commit 9bea070
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/test/command/test_cat.rb b/test/command/test_cat.rb
@@ -87,7 +87,7 @@ def test_cat_secondary_file
       d = create_driver
       d.run(expect_records: 1) do
         Open3.pipeline_w("#{ServerEngine.ruby_bin_path} #{FLUENT_CAT_COMMAND} --port #{@port} --format msgpack secondary") do |stdin|
-          stdin.write(File.read(path))
+          stdin.write(File.read(path, File.size(path)))
           stdin.close
         end
       end