Merge pull request #106 from rafiss/activerecord-19.1

Skip Activerecord tests for <=v19.1

These are failing currently because of missing pg_catalog tables.

We'll skip the test for versions <=19.1 until we can debug the underlying issue.
(Note, we may decide to leave the skipped test permanently, as our tooling
support page specifies that we support ActiveRecord for CockroachDB 20.1+.)

Author: rafiss

testing/main_test.go

@@ -157,10 +157,14 @@ func testORM(
 	db, dbURL, version, stopDB := initTestDatabase(t, app)
 	defer stopDB()
 
-	if orm == "django"  && (strings.HasPrefix(version, "v2.0") || strings.HasPrefix(version, "v2.1")) {
+	if orm == "django" && (strings.HasPrefix(version, "v2.0") || strings.HasPrefix(version, "v2.1")) {
 		t.Skip("TestDjango fails on CRDB <=v2.1 due to missing foreign key support.")
 	}
 
+	if orm == "activerecord" && (strings.HasPrefix(version, "v2.") || strings.HasPrefix(version, "v19.1")) {
+		t.Skip("TestActiveRecord fails on CRDB <=v19.1 due to missing pg_catalog support.")
+	}
+
 	td := testDriver{
 		db:          db,
 		dbName:      app.dbName(),