starknet_transaction_prover: add ServiceConfig validation tests

[avi-starkware]

Adds unit tests for ServiceConfig::from_args: missing rpc_url, zero
max_concurrent_requests / max_connections, mutual exclusion of --no-cors
and --cors-allow-origin, --no-cors clearing config-file origins,
--skip-fee-field-validation reaching ProverConfig, config-file values used
when CLI flags are absent, and CLI flags overriding config-file values.

The CORS cases here cover only the wiring — that from_args routes origins
through normalize_cors_allow_origins and propagates its rejection — plus
wildcard_ignores_invalid, which has no counterpart at the other layer.
cors_test.rs (#14048) owns the normalization matrix itself; duplicating it here
meant two places to update for one behaviour.

tls_config_validation loses a case column that was None in every case, and
gains the shape it was evidently there for: a config file supplying
tls_cert_file with no CLI key, which the CLI-only cases cannot express.

[reviewable-StarkWare]

This change is 

[avi-starkware]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• starknet_transaction_prover: cover the CORS layer attachment in start_server #15036
• starknet_transaction_prover: unit-test both overload-rejection paths #15035
• starknet_transaction_prover: unit-test the JSON-RPC error mapping #15034
• starknet_transaction_prover: add deployment smoke script and guide #14052
• starknet_transaction_prover: add manual testing guide #14404
• starknet_transaction_prover: drop stale dead_code allow on RpcVirtualBlockExecutor #14740
• starknet_transaction_prover: extract shared server test utils #14739
• starknet_transaction_prover: make RPC error spec test exhaustive via enum #14051
• starknet_transaction_prover: add TLS integration tests #14050
• starknet_transaction_prover: add ServiceConfig validation tests #14049 👈 (View in Graphite)
• starknet_transaction_prover: add CORS validation unit tests #14048
• main-v0.14.4

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

PR Summary

Low Risk
Changes are limited to unit tests; they document existing validation behavior without modifying server config logic.

Overview
Test-only expansion and cleanup around ServiceConfig::from_args in config_test.rs—no production behavior changes.

New coverage pins required rpc_node_url, rejection of zero limits (max_concurrent_requests, max_connections, max_request_body_size), invalid parses (IP, fee token address, blocking check URL), mutual exclusion of --no-cors and --cors-allow-origin, --skip-fee-field-validation affecting ProverConfig, and config-file vs CLI precedence (including --no-cors clearing file origins). TLS tests drop a redundant rstest column and add config_file_tls_cert_without_key_is_rejected for partial TLS from JSON only.

CORS wiring tests are slimmed down so cors_test.rs holds the full normalization matrix; here they only assert that from_args routes through normalization and propagates rejections. Assertions increasingly use assert_matches for clearer error expectations.

^{Reviewed by Cursor Bugbot for commit 6445447. Bugbot is set up for automated code reviews on this repo. Configure here.}

[github-actions]

There hasn't been any activity on this pull request recently, and in order to prioritize active work, it has been marked as stale.
This PR will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!