Skip to content

Commit 52dfab1

Browse files
authored
fix: remove unnecessary env vars (#1886)
<!-- Follow semantic-release guidelines for the PR title, which is used in the changelog. Title should follow the format `<type>(<scope>): <subject>`, where - Type is one of: build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test|BREAKING CHANGE - Scope (optional) describes the place of the change (eg a particular milestone) and is usually omitted - subject should be a non-capitalized one-line description in present imperative tense and not ending with a period See https://github.com/angular/angular.js/blob/master/DEVELOPERS.md#-git-commit-guidelines for more details. --> ## Description <!-- Short description of the pull request --> Remove variables that are not needed in tests. ## Motivation <!-- Background on use case, changes needed --> This will allow to raise error when connection to rabbitmq fails from #1872 ## Fixes <!-- Please provide a list of the issues fixed by this PR --> * Remove variables that are not needed in tests. * ## Changes: <!-- Please provide a list of the changes implemented by this PR --> * changes made ## Tests included - [ ] Included for each change/fix? - [ ] Passing? <!-- Merge will not be approved unless tests pass --> ## Documentation - [ ] swagger documentation updated (required for API changes) - [ ] official documentation updated ### official documentation info <!-- If you have updated the official documentation, please provide PR # and URL of the updated pages -->
1 parent aa6d05a commit 52dfab1

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

.github/workflows/test.yml

-5
Original file line numberDiff line numberDiff line change
@@ -173,11 +173,6 @@ jobs:
173173
CLUSTER_NAME: es-cluster
174174
MEM_LIMIT: 4G
175175
JOB_CONFIGURATION_FILE: test/config/jobconfig.yaml
176-
RABBITMQ_ENABLED: "yes"
177-
RABBITMQ_HOSTNAME: "rabbitmq"
178-
RABBITMQ_PORT: 5672
179-
RABBITMQ_USERNAME: "guest"
180-
RABBITMQ_PASSWORD: "guest"
181176

182177
# Start mongo container and app before running api tests
183178
run: |

0 commit comments

Comments
 (0)