m-czernek / acrobot

A simple acronym bot for Google Chat
Apache License 2.0
32 stars 7 forks source link

Provide the user information about unsuccessful operations (exceptions) #8

Closed m-czernek closed 4 years ago

m-czernek commented 5 years ago

This resolves issue #7. Requires testing on OpenShift, though automated tests pass.

Also, we should add an automated test to verify whether exception info is returned to the user.

m-czernek commented 5 years ago

The original implementation did not work due to Google's middleware limit of 4096 characters for each message. Consequently, the implementation has changed, and we now report exception causes rather than stack traces (possibly more user friendly..?).

This is now tested on OpenShift, automated tests are passing. We're still missing an automated test for this feature though.

m-czernek commented 4 years ago

Tests added. Build is passing. Removing JDK 8 due to:

Merging