Normalize temp dir in Given a directory step

[swissspidy]

No description provided.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/Context/GivenStepDefinitions.php	0.00%	3 Missing ⚠️
src/Context/FeatureContext.php	0.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[gemini-code-assist]

Code Review

This pull request updates the temporary directory handling in GivenStepDefinitions.php to resolve macOS path normalization issues by incorporating realpath() and stripping the /private/var/ prefix. The review feedback recommends extending the path normalization to include /private/tmp/ to ensure consistency and refactoring the directory assignment logic for better readability.

[gemini-code-assist]

On macOS, /tmp is also a symbolic link to /private/tmp. Since realpath() is now used, it might return /private/tmp if the system temp directory is set to /tmp (or if TMPDIR is set to /tmp). To avoid potential regressions in the security check on line 64, the regex should also handle the /private/tmp/ prefix, similar to how it handles /private/var/.

		$temp_dir = preg_replace( '|^/private/(var|tmp)/|', '/$1/', $temp_dir );

[gemini-code-assist]

The assignment of $temp_dir can be simplified by using the Elvis operator and calling Path::normalize() once, which improves readability and slightly reduces redundancy.

		$temp_dir = Path::normalize( realpath( sys_get_temp_dir() ) ?: sys_get_temp_dir() );