Configurable message source name delimiter in tests

I’m in the process of creating a binder implementation for Amazon SQS which unfortunately doesn’t allow the period (.) character in queue names. From http://docs.aws.amazon.com/cli/latest/reference/sqs/create-queue.html:

The name of the new queue. The following limits apply to this name:

• A queue name can have up to 80 characters.
• Valid values: alphanumeric characters, hyphens (- ), and underscores (_ ).
• A FIFO queue name must end with the .fifo suffix.

Queue names are case-sensitive.

The abstract test classes use hardcoded names such as foo.0 which result in 400 errors from the SQS API. I’d like to provide a hook to be able to override the message source/target name delimiter so such a name could be foo-0 instead. I’d be happy to submit a PR but want to make sure this would be a welcomed change.

[garyrussell]

Thanks; I was just trying to point out that the origin of the names in the tests was from XD (SCDF’s predecessor), so just “fixing” the tests isn’t a total solution.

[olegz]

@JacobASeverson I wonder if a simpler approach would be to change the queue names in Abstract test classes to something commonly accepted (i.e., underscores foo_0) In any event your contribution is definitely welcomed and we can debate some of the details later when you have a PR.