Rename xctest executable to avoid inferring it as module name

[ankitspd]

Fixes https://bugs.swift.org/browse/SR-1243

[ankitspd]

or should it be called TestSuite.xctest ?

[czechboy0]

I'm having the issue that this fixes here: https://github.com/czechboy0/Redbird
Just clone, run swift build & test on Linux and you should see it. Would really appreciate this fix making it into the next toolchain!

[ankitspd]

Now I am wondering if its better to pass explicit -module-name flag instead as we try to find this executable via swift-test, will need to append name there too.

[rolivieri]

Is there an ETA for when this fix will be merged? This is a critical issue as described in https://bugs.swift.org/browse/SR-1243.

[ankitspd]

@rolivieri I need a confirmation from @mxcl or @ddunbar which way is better here -module-name or renaming the executable, then CI and then we can merge it.

[ddunbar]

I think that we should do both:

• Pass -module-name explicitly.
• Rename the built bundle. I think the name should be <name>Tests (note plural) for consistency with existing Xcode behavior.

We should also:

• Have a follow on bug for a verification step in SwiftPM that there is no module name collision across the entire package graph. This is probably even a StarterBug.
• Figure out why our CI didn't catch this.

[czechboy0]

@ddunbar I think there are two reasons why CI didn't catch this.

1. This fails under special circumstances, namely (from the SR bug description)

A helper function is defined inside a class that extends XCTestCase and receives as a parameter a structure defined in the Swift module that is being tested. If the helper function is commented out or the parameter type of the function is changed to, say String, then 'swift test' executes with no errors.

I am fortunate enough to have this issue in Redbird.
And
2. There still isn't a way to run swift test on a fixture in tests, something that #266 would help with. This is also not the first time that swift build succeeded on a fixture, but swift test failed. I think we should have a way to run swift test on a fixture to prevent bugs like these from sneaking in.

[ddunbar]

Agreed, can you open an SR on the missing coverage for swift test?

[ankitspd]

@ddunbar Updated the PR, will file a bug for collision check

[ddunbar]

@swift-ci please test

[czechboy0]

@ddunbar Here's a SR for running swift-test on fixtures: https://bugs.swift.org/browse/SR-1351