Fixed bug on CLI output IONTEXT to file

[lziq]

Issue #473

Problem Description

As show in https://github.com/partiql/partiql-lang-kotlin/blob/main/cli/src/org/partiql/cli/Cli.kt#L52-53, the use block in L52 invokes close() function on output. In case of FileOutputStream, it indeed closes the output stream. Thus, the write() function in L53 throws an exception.

Description of changes:

1. Move L53 into the use block.
2. Create a test case to cover file output.

[codecov-commenter]

Codecov Report

Merging #479 (4ff861e) into main (36e20d1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               main     #479   +/-   ##
=========================================
  Coverage     82.38%   82.38%           
  Complexity     1406     1406           
=========================================
  Files           171      171           
  Lines         10816    10817    +1     
  Branches       1781     1781           
=========================================
+ Hits           8911     8912    +1     
  Misses         1361     1361           
  Partials        544      544           

Flag	Coverage Δ
CLI	19.33% <100.00%> (+0.26%)	⬆️
EXAMPLES	74.85% <ø> (ø)
LANG	84.85% <ø> (ø)
PTS	∅ <ø> (∅)
TEST_SCRIPT	79.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
cli/src/org/partiql/cli/Cli.kt	80.00% <100.00%> (+1.05%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36e20d1...4ff861e. Read the comment docs.

[abhikuhikar]

Change looks reasonable to me. However, after executing partiql -q "1" --output-format ION_TEXT -o temp.ion, it still waits to read from stdin but does not throw a stream closed error after providing input to stdin as it used to. However, I believe this should not be the case. The command should not wait for the input from stdin.

[lziq]

Change looks reasonable to me. However, after executing partiql -q "1" --output-format ION_TEXT -o temp.ion, it still waits to read from stdin but does not throw a stream closed error after providing input to stdin as it used to. However, I believe this should not be the case. The command should not wait for the input from stdin.

I think this issue is resolved in another PR, #478