GOCBC-1818: Fix panic when running test suite with -short flag

We have a few places in the test suite that assume a globalCluster is configured. This is not the case when running the tests with the -short flag, which only runs unit tests.

Change-Id: I324b19f1916715a14cd458a7c59b647da979d129
Reviewed-on: https://review.couchbase.org/c/gocb/+/244631
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Charles Dixon <chvckd@gmail.com>

Author: DemetrisChr

Makefile

@@ -4,6 +4,7 @@ devsetup:
 
 test:
 	go test ./
+
 fasttest:
 	go test -short ./
 

scope_searchindexes_test.go

@@ -269,15 +269,18 @@ func (suite *UnitTestSuite) TestScopeSearchIndexesAnalyzeDocumentCore() {
 		Body:       io.NopCloser(bytes.NewReader(analyzeResp)),
 	}
 
-	indexName := "searchy"
+	cli := new(mockConnectionManager)
+	bucket := suite.bucket("mock", cli)
+	scope := suite.newScope(bucket, "test")
+	indexName := "test-index"
 
 	mockProvider := new(mockMgmtProvider)
 	mockProvider.
 		On("executeMgmtRequest", nil, mock.AnythingOfType("mgmtRequest")).
 		Run(func(args mock.Arguments) {
 			req := args.Get(1).(mgmtRequest)
 
-			expectedPath := fmt.Sprintf("/api/bucket/%s/scope/%s/index/%s/analyzeDoc", globalScope.bucket.bucketName, globalScope.scopeName, indexName)
+			expectedPath := fmt.Sprintf("/api/bucket/%s/scope/%s/index/%s/analyzeDoc", bucket.Name(), scope.Name(), indexName)
 
 			suite.Assert().Equal(expectedPath, req.Path)
 			suite.Assert().Equal(ServiceTypeSearch, req.Service)
@@ -299,11 +302,10 @@ func (suite *UnitTestSuite) TestScopeSearchIndexesAnalyzeDocumentCore() {
 		tracer:            newTracerWrapper(&NoopTracer{}, ObservabilityConfig{}),
 	}
 
-	res, err := mgr.AnalyzeDocument(globalScope, indexName, struct{}{}, &AnalyzeDocumentOptions{
+	res, err := mgr.AnalyzeDocument(scope, indexName, struct{}{}, &AnalyzeDocumentOptions{
 		Timeout: 1 * time.Second,
 	})
-	suite.Require().Nil(err, err)
-
+	suite.Require().NoError(err)
 	suite.Require().NotNil(res)
 }
 

testmain_test.go

@@ -138,7 +138,7 @@ func TestMain(m *testing.M) {
 		setupCluster()
 	}
 
-	if !globalCluster.IsProtostellar() {
+	if globalCluster != nil && !globalCluster.IsProtostellar() {
 		gocbcore.EnableHttpResponseTracking()
 	}
 
@@ -177,7 +177,7 @@ func TestMain(m *testing.M) {
 		log.Printf("No goroutines appear to have leaked (%d before == %d after)", initialGoroutineCount, finalGoroutineCount)
 	}
 
-	if !globalCluster.IsProtostellar() && !gocbcore.ReportLeakedHttpResponses() {
+	if globalCluster != nil && !globalCluster.IsProtostellar() && !gocbcore.ReportLeakedHttpResponses() {
 		result = 1
 	}