This PR resolves #1676 by adding the applicationId parameter to relevant test methods. This is a breaking change, both in source and binary, however:
I was not able to find any bot that would actually suffer from the source incompatibility (they all use named arguments because of the already high argument count)
the binary incompatibility has little impact as these methods are only used in test pipelines.
Therefore, although it would be possible to add deprecated overrides to eliminate the incompatibility, I considered the effort to be unnecessary.
This PR also adds documentation where relevant to make the link between applicationId and connectorId more explicit. Please correct me if said link is unintended.
This PR resolves #1676 by adding the
applicationId
parameter to relevant test methods. This is a breaking change, both in source and binary, however:Therefore, although it would be possible to add deprecated overrides to eliminate the incompatibility, I considered the effort to be unnecessary.
This PR also adds documentation where relevant to make the link between applicationId and connectorId more explicit. Please correct me if said link is unintended.