NEXT-35502 - Fix migration saleschannel test

[akf-bw]

1. Why is this change necessary?

• Run the Unit / PHPUnit for migration,migration-tests, sometimes the tests will fail at the testItDoesNotBreakIfNoHeadlessSalesChannelIsPresent function, because the sales channel can be blocked by other tests (e.g. order tests that refer to the saleschannel) running at the same time

2. What does this change do, exactly?

• Changed Migration1620820321AddDefaultDomainForHeadlessSaleschannelTest to catch ForeignKeyConstraintViolationException

3. Describe each step to reproduce the issue or behaviour.

• Run the Unit / PHPUnit for migration,migration-tests, sometimes the tests will fail at the testItDoesNotBreakIfNoHeadlessSalesChannelIsPresent function, because the sales channel can be blocked by other tests (e.g. order tests that refer to the saleschannel) running at the same time

4. Please link to the relevant issues (if any).

• /

5. Checklist

• I have rebased my changes to remove merge conflicts
• I have written tests and verified that they fail without my change
• I have created a changelog file with all necessary information about my changes
• I have written or adjusted the documentation according to my changes
• This change has comments for package types, values, functions, and non-obvious lines of code
• I have read the contribution requirements and fulfil them.

[shopware-github-importer]

Hello,

thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-35502

Please use this issue to track the state of your pull request.

[akf-bw]

@OliverSkroblin Is there something holding this PR back from being merged?

[OliverSkroblin]

@OliverSkroblin Is there something holding this PR back from being merged?

Not really, just internal pipelines which are failling right now. Nothing regarding your code change

[akf-bw]

@OliverSkroblin It looks like recent pipeline changes that reorder test execution time unintentionally fixed the bug from the pipeline. However, just to be safe, it would be nice to merge this so we can be sure that the pipeline doesn't fail due to an invalid error.