feat(license): wrap POST /license/import-csv endpoint

[Valyrian-Code]

Refs #52.

Wraps the POST /license/import-csv endpoint as LicenseEndpoint.import_licenses_csv().

Performs a multipart upload of a CSV file with optional delimiter (default ",") and enclosure (default "\""), returning the server's multi-line summary message indicating inserted and already-existing rows.

The implementation matches the live contract verified against Fossology 4.4.0:

• Endpoint path is /license/import-csv (the OpenAPI spec was renamed from the original /license/import in feat(API): import csv-license file fossology#2292; the current spec lives alongside /license/export-csv, /license/import-json, /license/export-json)

• Required field: file_input; optional delimiter, enclosure

• 200 success → Info schema with multi-line message

• 400 → validation error

Tests cover the live happy path by round-tripping a small CSV against the container (the server is idempotent on duplicate shortnames, so repeated test runs do not pollute state), as well as a mocked 400 Bad Request error path.

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a new import_licenses_csv method to the Fossology license API wrapper, enabling bulk import of licenses from a CSV file via POST /license/import-csv, along with tests for success and error scenarios.

Changes:

• New import_licenses_csv method on the license client with delimiter/enclosure options.
• Integration-style test for successful CSV import.
• Mocked test for error handling on import failure.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
fossology/license.py	Implements the CSV license import API method.
tests/test_license.py	Adds success and error tests for the new method.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Valyrian-Code]

Reviewed each suggestion against the rest of the codebase and applied only the one that is consistent with the existing project style.

Applied — payload-asserting mocked test (230fbe4)

Added test_import_licenses_csv_request_payload using @responses.activate. The test verifies that the multipart request contains:

• The correct file_input field name
• The configured delimiter value
• The configured enclosure value
• The expected CSV content

This addressed a genuine coverage gap.

Not applied — str | Path parameter type

Checked existing file-upload APIs for consistency. upload_file (uploads.py:136) uses file: str | None, and pathlib.Path appears only once within the fossology/ package (foss_cli.py:404), not in any library-facing API.

Keeping csv_file: str preserves the established convention used throughout the library.

Not applied — FileNotFoundError documentation and HTTP error wrapping

Verified that the sibling add_license implementation follows the same error-handling pattern: it checks expected status codes (201, 409) and raises FossologyApiError for other responses. import_licenses_csv follows that same approach.

Additionally, none of the existing file-based methods (upload_file, add_license) explicitly document FileNotFoundError. Introducing that documentation for a single method would be inconsistent with the current API surface.

If the project wants to adopt either of these patterns, it would be better handled as a separate, repository-wide change affecting the existing methods as well.

All 16 tests in test_license.py pass locally against a live Fossology 4.4.0 container.

[Valyrian-Code]

cc @deveaud-m

Could you please review this when you have a moment?
Thank you.