feat(to): table also for xlsx and ods

[jqnatividad]

for #3568

[Copilot]

Pull request overview

Extends the qsv to --table option so it can also control the worksheet name when converting to Excel (xlsx) and OpenDocument Spreadsheet (ods), in addition to existing database targets, addressing the workflow described in #3568.

Changes:

• Allow --table for to xlsx and to ods, using it as the sheet name (with basic validation).
• Add/adjust integration tests for happy paths and sheet-name validation errors for xlsx/ods.
• Update help text/docs to describe --table behavior for xlsx/ods and refine the datapackage restriction message.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
src/cmd/to.rs	Adds xlsx/ods-specific --table validation and applies --table renaming for xlsx/ods conversions.
tests/test_to.rs	Adds new tests covering --table behavior for xlsx/ods and updates datapackage error expectation.
docs/help/to.md	Documents --table usage/examples for xlsx and ods.
docs/help/TableOfContents.md	Updates the Polars badge/hash reference in the help TOC.

Comments suppressed due to low confidence (2)

tests/test_to.rs:555

• This test uses wrk.output(&mut cmd) but never asserts the command succeeded (exit status). If qsv to xlsx fails, the test may still pass/fail on stdout/file checks instead of clearly reporting the command failure. Prefer wrk.assert_success(&mut cmd) (or at least assert output.status.success()) before inspecting stdout/file existence.

    let output = wrk.output(&mut cmd);
    let stdout = String::from_utf8_lossy(&output.stdout);
    assert!(
        stdout.contains("My Sheet"),
        "Expected sheet name 'My Sheet' in output, got: {stdout}"
    );

tests/test_to.rs:647

• This test inspects stdout from wrk.output(&mut cmd) without asserting the process exit status. Add an explicit success assertion (e.g., wrk.assert_success(&mut cmd) or assert!(output.status.success())) so failures in the to ods invocation are caught directly and produce clearer diagnostics.

    let output = wrk.output(&mut cmd);
    let stdout = String::from_utf8_lossy(&output.stdout);
    assert!(
        stdout.contains("My Sheet"),
        "Expected sheet name 'My Sheet' in output, got: {stdout}"
    );