chore(ci): switch to github actions

When this project was first started, Github Actions were not an
option. Now that they are, and that teamcity is less accessible
(lesser used, needs connection, ci link is not up to date), we
are switching to gh actions.

Fixes #279

Author: BuonOmo

.github/workflows/ci.yml

@@ -0,0 +1,86 @@
+# Inspired from:
+# - https://github.com/cockroachdb/sqlalchemy-cockroachdb/blob/master/.github/workflows/ci.yml
+# - https://github.com/rgeo/activerecord-postgis-adapter/blob/master/.github/workflows/tests.yml
+name: Test
+
+on:
+  # Triggers the workflow on push or pull request events.
+  push:
+    # This should disable running the workflow on tags, according to the
+    # on.<push|pull_request>.<branches|tags> GitHub Actions docs.
+    branches:
+      - "*"
+  pull_request:
+    types: [opened, reopened, synchronize]
+
+  # Allows you to run this workflow manually from the Actions tab
+  workflow_dispatch:
+
+# This allows a subsequently queued workflow run to interrupt previous runs.
+concurrency:
+  group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}'
+  cancel-in-progress: true
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        crdb: [v23.1.5]
+        ruby: [ruby-head]
+    name: Test (crdb=${{ matrix.crdb }} ruby=${{ matrix.ruby }})
+    steps:
+      - name: Set Up Actions
+        uses: actions/checkout@v3
+      - name: Install GEOS
+        run: sudo apt-get install libgeos-dev
+      - name: Set Up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+           ruby-version: ${{ matrix.ruby }}
+           bundler-cache: true
+      - name: Install and Start Cockroachdb
+        run: |
+          # Download CockroachDB
+          wget -qO- https://binaries.cockroachdb.com/cockroach-${{ matrix.crdb }}.linux-amd64.tgz | tar xvz
+
+          export PATH=./cockroach-${{ matrix.crdb }}.linux-amd64/:$PATH
+          readonly urlfile=cockroach-url
+
+          # Start a CockroachDB server and wait for it to become ready.
+          rm -f "$urlfile"
+          rm -rf cockroach-data
+          # Start CockroachDB.
+          cockroach start-single-node --max-sql-memory=25% --cache=25% --insecure --host=localhost --spatial-libs=./cockroach-${{ matrix.crdb }}.linux-amd64/lib --listening-url-file="$urlfile" >/dev/null 2>&1 &
+          # Ensure CockroachDB is stopped on script exit.
+          # Wait until CockroachDB has started.
+          for i in {0..3}; do
+            [[ -f "$urlfile" ]] && break
+            backoff=$((2 ** i))
+            echo "server not yet available; sleeping for $backoff seconds"
+            sleep $backoff
+          done
+          cockroach sql --insecure -e "
+            CREATE DATABASE activerecord_unittest;
+            CREATE DATABASE activerecord_unittest2;
+            SET CLUSTER SETTING sql.stats.automatic_collection.enabled = false;
+            SET CLUSTER SETTING sql.stats.histogram_collection.enabled = false;
+            SET CLUSTER SETTING jobs.retention_time = '180s';
+            SET CLUSTER SETTING sql.defaults.experimental_alter_column_type.enabled = 'true';
+
+            ALTER RANGE default CONFIGURE ZONE USING num_replicas = 1, gc.ttlseconds = 30;
+            ALTER TABLE system.public.jobs CONFIGURE ZONE USING num_replicas = 1, gc.ttlseconds = 30;
+            ALTER RANGE meta CONFIGURE ZONE USING num_replicas = 1, gc.ttlseconds = 30;
+            ALTER RANGE system CONFIGURE ZONE USING num_replicas = 1, gc.ttlseconds = 30;
+            ALTER RANGE liveness CONFIGURE ZONE USING num_replicas = 1, gc.ttlseconds = 30;
+
+            SET CLUSTER SETTING kv.range_merge.queue_interval = '50ms';
+            SET CLUSTER SETTING kv.raft_log.disable_synchronization_unsafe = 'true';
+            SET CLUSTER SETTING jobs.registry.interval.cancel = '180s';
+            SET CLUSTER SETTING jobs.registry.interval.gc = '30s';
+            SET CLUSTER SETTING kv.range_split.by_load_merge_delay = '5s';
+
+            SET CLUSTER SETTING sql.defaults.experimental_temporary_tables.enabled = 'true';
+          "
+      - name: Test
+        run: bundle exec rake test

Gemfile

@@ -14,7 +14,7 @@ module RailsTag
 
     def gemspec_requirement
       File
-        .foreach("activerecord-cockroachdb-adapter.gemspec", chomp: true)
+        .foreach(File.expand_path("activerecord-cockroachdb-adapter.gemspec", __dir__), chomp: true)
         .find { _1[/add_dependency\s.activerecord.,\s.(.*)./] }
 
       Gem::Requirement.new(Regexp.last_match(1))

README.md

@@ -321,7 +321,7 @@ p modified_fac.parse_wkt(wkt)
 #=>  #<RGeo::Geographic::SphericalPolygonImpl>
 ```
 
-Be careful when performing calculations on potentially invalid geometries, as the results might be nonsensical. For example, the area returned of an hourglass made of 2 equivalent triangles with a self-intersection in the middle is 0. 
+Be careful when performing calculations on potentially invalid geometries, as the results might be nonsensical. For example, the area returned of an hourglass made of 2 equivalent triangles with a self-intersection in the middle is 0.
 
 Note that when using the `spherical_factory`, there is a chance that valid geometries will be interpreted as invalid due to floating point issues with small geometries.
 

lib/active_record/connection_adapters/cockroachdb/schema_statements.rb

@@ -112,7 +112,7 @@ def new_column_from_field(table_name, field)
         # since type alone is not enough to format the column.
         # Ex. type_to_sql(:geography, limit: "Point,4326")
         # => "geography(Point,4326)"
-        # 
+        #
         def type_to_sql(type, limit: nil, precision: nil, scale: nil, array: nil, **) # :nodoc:
           sql = \
             case type.to_s

test/cases/adapters/postgresql/active_schema_test.rb

@@ -64,8 +64,8 @@ def test_add_index
     end
 
     private
-      def method_missing(method_symbol, *arguments)
-        ActiveRecord::Base.connection.send(method_symbol, *arguments)
+      def method_missing(...)
+        ActiveRecord::Base.connection.send(...)
       end
   end
 end

test/cases/adapters/postgresql/postgis_test.rb

@@ -148,8 +148,10 @@ def test_spatial_factory_retrieval
   def test_point_to_json
     obj = klass.new
     assert_match(/"latlon":null/, obj.to_json)
-    obj.latlon = factory.point(1.0, 2.0)
-    assert_match(/"latlon":"POINT\s\(1\.0\s2\.0\)"/, obj.to_json)
+    obj.latlon = factory.point(1.1, 2.0)
+    # NOTE: rgeo opiniated in trimming numbers (e.g 1.0 would be trimmed to 1).
+    #   We follow this convention here.
+    assert_match(/"latlon":"POINT\s\(1.1\s2\)"/, obj.to_json)
   end
 
   def test_custom_column

test/cases/adapters/postgresql/quoting_test.rb

@@ -0,0 +1,30 @@
+require "cases/helper_cockroachdb"
+
+# Load dependencies from ActiveRecord test suite
+require "support/schema_dumping_helper"
+
+module CockroachDB
+  class PostgresqlQuotingTest < ActiveRecord::PostgreSQLTestCase
+    def setup
+      @conn = ActiveRecord::Base.connection
+      @raise_int_wider_than_64bit = ActiveRecord.raise_int_wider_than_64bit
+    end
+
+    # Replace the original test since numbers are quoted.
+    def test_do_not_raise_when_int_is_not_wider_than_64bit
+      value = 9223372036854775807
+      assert_equal "'9223372036854775807'", @conn.quote(value)
+
+      value = -9223372036854775808
+      assert_equal "'-9223372036854775808'", @conn.quote(value)
+    end
+
+    # Replace the original test since numbers are quoted.
+    def test_do_not_raise_when_raise_int_wider_than_64bit_is_false
+      ActiveRecord.raise_int_wider_than_64bit = false
+      value = 9223372036854775807 + 1
+      assert_equal "'9223372036854775808'", @conn.quote(value)
+      ActiveRecord.raise_int_wider_than_64bit = @raise_int_wider_than_64bit
+    end
+  end
+end

test/cases/connection_adapters/schema_cache_test.rb

@@ -15,7 +15,7 @@ def setup
       # See test/excludes/ActiveRecord/ConnectionAdapters/SchemaCacheTest.rb
       def test_yaml_loads_5_1_dump
         body = File.open(schema_dump_path).read
-        cache = YAML.load(body)
+        cache = YAML.unsafe_load(body)
 
         assert_no_queries do
           assert_equal 11, cache.columns("posts").size
@@ -32,7 +32,7 @@ def test_yaml_loads_5_1_dump
       # See test/excludes/ActiveRecord/ConnectionAdapters/SchemaCacheTest.rb
       def test_yaml_loads_5_1_dump_without_indexes_still_queries_for_indexes
         body = File.open(schema_dump_path).read
-        @cache = YAML.load(body)
+        @cache = YAML.unsafe_load(body)
 
         # Simulate assignment in railtie after loading the cache.
         old_cache, @connection.schema_cache = @connection.schema_cache, @cache
@@ -51,7 +51,7 @@ def test_yaml_loads_5_1_dump_without_indexes_still_queries_for_indexes
       # See test/excludes/ActiveRecord/ConnectionAdapters/SchemaCacheTest.rb
       def test_yaml_loads_5_1_dump_without_database_version_still_queries_for_database_version
         body = File.open(schema_dump_path).read
-        @cache = YAML.load(body)
+        @cache = YAML.unsafe_load(body)
 
         # Simulate assignment in railtie after loading the cache.
         old_cache, @connection.schema_cache = @connection.schema_cache, @cache

test/cases/fixtures_test.rb

@@ -313,12 +313,13 @@ def before_setup
           firm_id bigint,
           firm_name character varying,
           credit_limit integer,
+          status string,
           #{'a' * max_identifier_length} integer
         )
       ")
 
       Company.connection.drop_table :companies, if_exists: true
-      Company.connection.exec_query("CREATE SEQUENCE companies_nonstd_seq")
+      Company.connection.exec_query("CREATE SEQUENCE IF NOT EXISTS companies_nonstd_seq")
       Company.connection.exec_query("
         CREATE TABLE companies (
           id BIGINT PRIMARY KEY DEFAULT nextval('companies_nonstd_seq'),
@@ -329,7 +330,8 @@ def before_setup
           client_of bigint,
           rating bigint,
           account_id integer,
-          description character varying
+          description character varying,
+          status integer
         )
       ")
 
@@ -361,6 +363,7 @@ def teardown
         t.references :firm, index: false
         t.string  :firm_name
         t.integer :credit_limit
+        t.string :status
         t.integer "a" * max_identifier_length
       end
 
@@ -375,6 +378,7 @@ def teardown
         t.bigint :rating, default: 1
         t.integer :account_id
         t.string :description, default: ""
+        t.integer :status, default: 0
         t.index [:name, :rating], order: :desc
         t.index [:name, :description], length: 10
         t.index [:firm_id, :type, :rating], name: "company_index", length: { type: 10 }, order: { rating: :desc }

test/cases/helper_cockroachdb.rb

@@ -1,5 +1,7 @@
-require 'bundler/setup'
-Bundler.require :development
+require 'bundler'
+Bundler.setup
+
+require "minitest/excludes"
 
 # Turn on debugging for the test environment
 ENV['DEBUG_COCKROACHDB_ADAPTER'] = "1"