Fix derived branch name default.

[bosschaert]

Description

The branch name is used by the playwright tests to derive the hostname to use. While the current approach works for PRs, it doesn't work when running the playwright tests on the main branch.

The previous default didn't work if the variable had an empty String as value, in which case it should default to main. There was a default in the code, but this doesn't work in this scenario.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have signed the Adobe Open Source CLA.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[aem-code-sync]

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

• Re-run PSI checks
• Re-sync branch

Commits

• 895f8e7 ✅ (latest)
• 47dae2a ✅