try enabling cassandra

signalfx · Apr 21, 2020 · e8d5e72 · e8d5e72
1 parent 02fc10e
commit e8d5e72
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -245,20 +245,20 @@ workflows:
   build_test:
     jobs:
       - lint_vet
-      # - go_test:
-      #    name: test_cassandra
-      #    executor: go_cassandra
-      #    packages: "`go list ./contrib/... | grep 'gocql' | xargs`"
-      #    after_deps:
-      #      - dockerize_wait:
-      #          address: tcp://localhost:9042
-      #          timeout: 2m
       - go_test:
           # test all remaining contrib tests not included in the other jobs
           name: test_contrib
           # disable checkptr until this issue gets resolved: https://github.com/tidwall/buntdb/issues/62
           options: -v -race -gcflags=all=-d=checkptr=0
           packages: "`go list ./contrib/... | grep -v -P 'gocql|elastic|kafka|memcache|globalsign|mongo|redis|redigo|sql|gorm' | sed 's|github.com/signalfx/signalfx-go-tracing|.|g' | xargs`"
+      - go_test:
+          name: test_cassandra
+          executor: go_cassandra
+          packages: "`go list ./contrib/... | grep 'gocql' | xargs`"
+          after_deps:
+            - dockerize_wait:
+                address: tcp://localhost:9042
+                timeout: 2m
       - go_test:
           name: test_ddtrace
           packages: "./ddtrace/..."

diff --git a/contrib/gocql/gocql/gocql_test.go b/contrib/gocql/gocql/gocql_test.go
@@ -115,9 +115,9 @@ func TestChildWrapperSpan(t *testing.T) {
 	assert.Equal(childSpan.Tag(ext.ResourceName), "SELECT * from trace.person")
 	assert.Equal(childSpan.Tag(ext.CassandraKeyspace), "trace")
 	if iter.Host() != nil {
+		assert.Equal(childSpan.Tag(ext.CassandraCluster), "datacenter1")
 		assert.Equal(childSpan.Tag(ext.TargetPort), "9042")
 		assert.Equal(childSpan.Tag(ext.TargetHost), "127.0.0.1")
-		assert.Equal(childSpan.Tag(ext.CassandraCluster), "datacenter1")
 	}
 }