Adding more cockroach versions to tests (#719)

* Adding more cockroach versions to tests

* Do not recreate constraint on down migration

* Run cockroach 20 in sigle node mode

.circleci/config.yml

@@ -169,6 +169,32 @@ jobs:
       - run:
           name: test
           command: npm run migrate up -- -m test/cockroach --no-lock && npm run migrate down 0 -- -m test/cockroach --no-lock --timestamps
+  test-cockroach-19:
+    docker:
+      - <<: *node-image
+        environment:
+          - DATABASE_URL=postgresql://root@localhost:26257/circle_test
+      - image: cockroachdb/cockroach:v19.2.11
+        command: ['start', '--insecure', '--host=localhost']
+    steps:
+      - <<: *restore
+      - <<: *cockroach-wait
+      - run:
+          name: test
+          command: npm run migrate up -- -m test/cockroach --no-lock && npm run migrate down 0 -- -m test/cockroach --no-lock --timestamps
+  test-cockroach-20:
+    docker:
+      - <<: *node-image
+        environment:
+          - DATABASE_URL=postgresql://root@localhost:26257/circle_test
+      - image: cockroachdb/cockroach:v20.2.1
+        command: ['start-single-node', '--insecure', '--host=localhost']
+    steps:
+      - <<: *restore
+      - <<: *cockroach-wait
+      - run:
+          name: test
+          command: npm run migrate up -- -m test/cockroach --no-lock && npm run migrate down 0 -- -m test/cockroach --no-lock --timestamps
   test-node-10:
     docker:
       - image: circleci/node:10
@@ -454,6 +480,12 @@ workflows:
       - test-cockroach-2:
           requires:
             - install
+      - test-cockroach-19:
+          requires:
+            - install
+      - test-cockroach-20:
+          requires:
+            - install
       - test-node-10:
           requires:
             - install

test/cockroach/014_add_constraint.js

@@ -5,3 +5,7 @@ exports.up = (pgm) => {
     check: 'id < 30',
   })
 }
+
+exports.down = () => {
+  // do nothing
+}

test/cockroach/017_drop_constraint.js

@@ -4,8 +4,6 @@ exports.up = (pgm) => {
   pgm.dropConstraint('t1', constraint.constraint)
 }
 
-exports.down = (pgm) => {
-  pgm.addConstraint('t1', constraint.constraint, {
-    check: 'true',
-  })
+exports.down = () => {
+  // do nothing
 }