New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add CI Testing For Slack Notifications #505

Open

pnadolny13 opened this issue Dec 14, 2022 · 0 comments

Assignees

Contributor

pnadolny13 commented Dec 14, 2022

The slack notifications job doesnt get tested in CI because we dont have a dev slack channel to push to. Options:

Full integration test to dev slack channel
- create new dev slack channel
- create new slack integration to get independent web hook. I believe its 1-1 with a channel
- configure the dev slack uri in the CI environment
- add slack job to CI pipeline. Requires setting up matrixes in the publish CI step
  
  squared/.github/workflows/test.yml
  
  Line 131 in d36e570
  
  reverse_el_tests:
  
  .
Integration test to a local notifier like how I test locally with ["macosx://"]
- find a notifier that works in CI
- configure the dev slack uri in the CI environment
- add slack job to CI pipeline. Requires setting up matrixes in the publish CI step
  
  squared/.github/workflows/test.yml
  
  Line 131 in d36e570
  
  reverse_el_tests:
  
  .

pnadolny13 added this to Data Team

pnadolny13 self-assigned this

pnadolny13 moved this to Backlog in Data Team

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment