Skip to content

Commit

Permalink
add explicit check for CI in tests
Browse files Browse the repository at this point in the history
  • Loading branch information
EdwardX29 committed Jan 9, 2025
1 parent d9a2e96 commit 10f183e
Showing 1 changed file with 13 additions and 3 deletions.
16 changes: 13 additions & 3 deletions sstable/rowblk/rowblk_iter_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import (
"bytes"
"fmt"
"math"
"os"
"runtime"
"strconv"
"strings"
Expand Down Expand Up @@ -467,7 +468,10 @@ func TestBlockSyntheticSuffix(t *testing.T) {

func TestSingularKVBlockRestartsOverflow(t *testing.T) {

t.Skip("Skipping test: requires too much memory for CI now.")
_, isCI := os.LookupEnv("CI")
if isCI {
t.Skip("Skipping test: requires too much memory for CI now.")
}

// Test that SeekGE() and SeekLT() function correctly
// with a singular large KV > 2GB.
Expand Down Expand Up @@ -511,7 +515,10 @@ func TestSingularKVBlockRestartsOverflow(t *testing.T) {

func TestBufferExceeding256MBShouldPanic(t *testing.T) {

t.Skip("Skipping test: requires too much memory for CI now.")
_, isCI := os.LookupEnv("CI")
if isCI {
t.Skip("Skipping test: requires too much memory for CI now.")
}

// Test that writing to a block that is already >= 256MiB
// causes a panic to occur.
Expand Down Expand Up @@ -559,7 +566,10 @@ func TestBufferExceeding256MBShouldPanic(t *testing.T) {

func TestMultipleKVBlockRestartsOverflow(t *testing.T) {

t.Skip("Skipping test: requires too much memory for CI now.")
_, isCI := os.LookupEnv("CI")
if isCI {
t.Skip("Skipping test: requires too much memory for CI now.")
}

// Tests that SeekGE() works when iter.restarts is
// greater than math.MaxUint32 for multiple KVs.
Expand Down

0 comments on commit 10f183e

Please sign in to comment.