Coverage Run unit tests first before docker containers are set up

[Minfante377]

This PR implements:

• Run unit tests as part of the CI
• Add a missing integration mark to one of the integration tests

Should close #1051

[sungwy]

Thank you @Minfante377 for this PR, and @youssef-itanii as well!

[kevinjqliu]

Thanks for the contribution! Added a comment about testing this method and some nits

[kevinjqliu]

nit: to match the other command, also added -v to match test-integration

Suggested change

	poetry run coverage run --data-file=.coverage.integration --source=pyiceberg/ -m pytest tests/ -m integration ${PYTEST_ARGS}
	poetry run coverage run --source=pyiceberg/ --data-file=.coverage.integration -m pytest tests/ -v -m integration ${PYTEST_ARGS}

[Minfante377]

Done

[kevinjqliu]

Does CI fail if we don't add this line? It's what we're testing for in #1051

[Minfante377]

It fails indeed. If I don't add that marker the test-coverage-unit command will try to run that test without initializing any docker container

[kevinjqliu]

LGTM!

Verified that unit test fails
#1060
https://github.com/apache/iceberg-python/actions/runs/10380009878/job/28739139875?pr=1060

=========================== short test summary info ============================
FAILED tests/integration/test_reads.py::test_table_scan_default_to_large_types[session_catalog_hive] - thrift.transport.TTransport.TTransportException: Could not connect to any of [('::1', 9083, 0, 0), ('127.0.0.1', 9083)]
ERROR tests/integration/test_reads.py::test_table_scan_default_to_large_types[session_catalog] - requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=8181): Max retries exceeded with url: /v1/config (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f2b1351e0d0>: Failed to establish a new connection: [Errno 111] Connection refused'))
=== 1 failed, 2781 passed, 834 deselected, 1113 warnings, 1 error in 54.83s ====