From f20b2a08c41adb73bf4d8c1a66f87284f1f1d290 Mon Sep 17 00:00:00 2001
From: David Dworken <david@daviddworken.com>
Date: Sat, 10 Feb 2024 23:02:29 -0800
Subject: [PATCH] Fix logic error that broke test sharding

---
 client/client_test.go | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/client/client_test.go b/client/client_test.go
index 70be5d2..430167d 100644
--- a/client/client_test.go
+++ b/client/client_test.go
@@ -61,7 +61,7 @@ var shellTesters []shellTester = []shellTester{bashTester{}, zshTester{}}
 func numTestShards() int {
 	numTestShardsStr := os.Getenv("NUM_TEST_SHARDS")
 	if numTestShardsStr == "" {
-		return 0
+		return -1
 	}
 	numTestShards, err := strconv.Atoi(numTestShardsStr)
 	if err != nil {
@@ -73,7 +73,7 @@ func numTestShards() int {
 func currentShardNumber() int {
 	currentShardNumberStr := os.Getenv("CURRENT_SHARD_NUM")
 	if currentShardNumberStr == "" {
-		return 0
+		return -1
 	}
 	currentShardNumber, err := strconv.Atoi(currentShardNumberStr)
 	if err != nil {
@@ -83,13 +83,13 @@ func currentShardNumber() int {
 }
 
 func isShardedTestRun() bool {
-	return numTestShards() != 0 && currentShardNumber() != 0
+	return numTestShards() != -1 && currentShardNumber() != -1
 }
 
 func markTestForSharding(t *testing.T, testShardNumber int) {
 	fmt.Printf("DDWORKENDEBUG: markTestForSharding: isShardedTestRun()=%#v testShardNumber=%#v numTestShards()=%#v currentShardNumber()=%#v", isShardedTestRun(), testShardNumber, numTestShards(), currentShardNumber())
 	if isShardedTestRun() {
-		if testShardNumber%numTestShards() == currentShardNumber() {
+		if testShardNumber%numTestShards() != currentShardNumber() {
 			t.Skip("Skipping sharded test")
 		}
 	}