Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Added public UpdateProperty method to the MessagingService interface in pkg/solace/messaging_service.go. Also added a barren implementation of UpdateProperty to MessagingServiceImpl in internal/impl/messgaging_service_impl.go. This barren implementation will be populated by a proper implementation in a future commit, but was added as a part of this commit so that the API would build with the interface changes. * Fixed some typos in the previous commit and fixed some problems based on architecture changes. * SOL-81488: Fixed more typos in documentation for public interface. * SOL-81488: Changed doc string to reflect proper error return for native errors. * SOL-81489: Implemented updateProperty() and the related methods. This was built using artifacts from solclient_Linux26-x86_64_debug_100.0.42.17.tar.gz. These artifacts are not included as a part of this commit, nor will they be committed. * SOL-81478: Added test for updating the oauth tokens multiple times before Connect() is called on the service. * SOL-81487: Making an interim commit with an unofficial CCSMP load to unblock QA. An official release of CCSMP will be added in a future commit. * SOL-81491: Added mock method to the transport mock to pass go vet. Implementation of mock method is FFC, which seems to be consistent with many of the other mock methods in the file. * SOL-81491: Fixed typos * Swapped to test jenkins-pipeline branch * Fixed failing git actions (#8) * Updated test certificate to use sha256 signing algorithm instead of sha1 * Updated api-badserver.pem, reverted api-server.pem * Use Ubuntu 20.04 for GitHub Actions workflow SOL-84761 * Reverted pipeline branch back to main * SOL-81477: Added test for updating oauth tokens multiple times after the initial connection. * SOL-81483: Added test to verify that if a service updates its tokens to invalid token values and attempts to reconnect, the connection is refused. * SOL-81483: Fixed typo where this test was in the incorrect section. * SOL-81489: Fixed error where failed state check returned IllegalArgumentError instead of IllegalStateError. * SOL-81479: Added a test to verify that if the application tries to update either OAuth token type on a disconnected service, that an IllegalStateError error will be returned. * SOL-81480: Added a test to verify that if an invalid property is passed to the UpdateProperty method of the service, that an IllegalArgumentError error is returned. * SOL-81481: Added a test to verify that if a non-string value is used to update the value of a token, an InvalidDataTypeError error will be returned. * SOL-84763: Fixed documentation bug. The docs listed properties in SCREAMING_SNAKE_CASE instead of in PascalCase. * SOL-81487: Updated CCSMP binaries with official optimized CCSMP release, version 7.24.0.9. I diffed the old licenses.txt file and the licenses.txt file from the new CCSMP binary and found no differences, so it was not included as a part of this commit. * SOL-66078: Fixed API implementation and tests in response to PR feedback. * SOL-66078: Made changes in response to PR feedback. Removed type check on value passed to UpdateProperty(). * SOL-81483: Added test coverage for invalid value type and nil token value in addition to the previous case for only invalid string token value. * SOL-81483: Removed unnecessary 'close' calls on channels in tests. Co-authored-by: Eric Therrien <[email protected]> Co-authored-by: Michael Cardy <[email protected]>
- Loading branch information