[LD-142] Fix slack notifications

[nowakweronika]

Pull-Request

Description

Why?

We want to be aware of any failed job on the develop and main branches.

What?

Added relevant condition to the Include Slack Notification step.

Links to related issues

• Fixes LD-142

Demo

How to test

• Step 1: Add "exit 1" to any step in one of the job which is mentioned in needs context in test-results job. Example:

name: Run tests on Firebase Test Lab
  run: |-
    gcloud firebase test android run \
      --app="app/build/outputs/apk/debug/app-debug.apk" \
      --test="app/build/outputs/apk/androidTest/debug/app-debug-androidTest.apk" \
      --device="model=Pixel2.arm,version=33,locale=en,orientation=portrait" \
      --type=instrumentation \
      --use-orchestrator \
      --test-runner-class="com.appunite.loudius.util.InstrumentationTestRunner" \
      --timeout="20m" \
      --results-dir="${{ steps.generate-dir.outputs.results_dir }}" \
      --results-bucket="${{ steps.generate-dir.outputs.bucket }}" \
      --environment-variables "clearPackageData=true,coverage=true,coverageFilePath=/sdcard/Download/" \
      --directories-to-pull /sdcard/Download
+    exit 1

• Expected: After test-results job finished, you should be able to see relevant message on loudius-internal slack channel.

Documentation

• Link to github actions docs: https://docs.github.com/en/actions/learn-github-actions/contexts#needs-context

Checklist

- [ ] Functionality is covered by unit tests
- [ ] Functionality is covered by integration tests

• I've updated PR description with relevant information
• I've done self code review
• I've manually tested if the code and app works

[github-actions]

Overall Project	73.2%	❌

There is no coverage information present for the Files changed

[github-actions]

Overall Project	73.2%	❌

There is no coverage information present for the Files changed

[github-actions]

Overall Project	73.2%	❌

There is no coverage information present for the Files changed

[jacek-marchwicki]

LGTM, but please do squash instead of merge, because there is a lot of useless commits ;p