Skip to content

Commit

Permalink
cleaning code
Browse files Browse the repository at this point in the history
  • Loading branch information
ofekshenawa committed Dec 16, 2024
1 parent 6763990 commit 545e950
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 8 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,11 +72,11 @@ jobs:
# Set up Docker Compose environment
- name: Set up Docker Compose environment
run: |
docker compose --profile all up -d --build
docker compose --profile all up -d
- name: Run tests
env:
USE_PREBUILT_REDIS: "true"
USE_CONTAINERIZED_REDIS: "true"
RE_CLUSTER: "true"
run: |
go test \
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-redis-enterprise.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ jobs:
- name: Test
env:
RE_CLUSTER: "1"
USE_PREBUILT_REDIS: "1"
USE_CONTAINERIZED_REDIS: "1"
run: |
go test \
--ginkgo.skip-file="ring_test.go" \
Expand Down
2 changes: 1 addition & 1 deletion docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

services:

redis:
redis-stanalone:
image: redislabs/client-libs-test:8.0-M02
container_name: redis-standalone
environment:
Expand Down
7 changes: 3 additions & 4 deletions main_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ var cluster = &clusterScenario{
}

var RECluster = false
var USE_PREBUILT_REDIS = false
var USE_CONTAINERIZED_REDIS = false

func registerProcess(port string, p *redisProcess) {
if processes == nil {
Expand All @@ -83,9 +83,8 @@ var _ = BeforeSuite(func() {
}
var err error
RECluster, _ = strconv.ParseBool(os.Getenv("RE_CLUSTER"))
USE_PREBUILT_REDIS, _ = strconv.ParseBool(os.Getenv("USE_PREBUILT_REDIS"))
// panic("RECluster: " + strconv.FormatBool(RECluster) + " USE_PREBUILT_REDIS: " + strconv.FormatBool(USE_PREBUILT_REDIS))
if !RECluster || !USE_PREBUILT_REDIS {
USE_CONTAINERIZED_REDIS, _ = strconv.ParseBool(os.Getenv("USE_CONTAINERIZED_REDIS"))
if !RECluster || !USE_CONTAINERIZED_REDIS {

redisMain, err = startRedis(redisPort)
Expect(err).NotTo(HaveOccurred())
Expand Down

0 comments on commit 545e950

Please sign in to comment.