pg_range extension with fixture_dependencies

[adam12]

Hi, Jeremy et all,

I've had to create this workaround for creating a fixture (via fixture_dependencies) that creates a model record which has a tsrange field.

I'm wondering if this is something that should be done by the pg_range extension when creating the SQL expression? I know this is a special case, so perhaps this is the best I'm going to get with YAML as the base.

Thoughts?

# fixture
one:
  date_range: ["2022-06-01 00:00:00", "2022-06-30 23:59:59"]

DB.extension :pg_range

class MyModel < Sequel::Model
  # date_range is a `tsrange` column type
  def date_range=(value)
    # Cast to Range here, since the Fixture provides an Array
    value = Range.new(*value) if value.is_a?(Array)
    super(value)
  end
end

[jeremyevans]

You can provide a string in the same format that PostgreSQL uses, since pg_range already handles that. If that works, I think it's better than adding support for arrays. Can you try that?

[adam12]

Indeed, the string format that PostgreSQL accepts worked fine!

This updated the fixture to look like this :

# fixture
one:
  date_range: "[2022-06-01 00:00:00, 2022-06-30 23:59:59]"

Thanks!

EDIT: The brackets have a significance.

[rgalanakis]

We have a plugin for PG ranges that we use in all our project, feel free to use it for inspiration: https://github.com/lithictech/sequel-tstzrange-fields/blob/main/spec/sequel/plugins/tstzrange_fields_spec.rb

You can do stuff like model.date_range_begin = x, model.date_range = 'empty', model.date_range = Float::INFINITY, model.date_range = {'begin' => x, end: y}, stuff like that. We don't handle tuples, but definitely should, and would be easy to add (https://github.com/lithictech/sequel-tstzrange-fields/blob/main/lib/sequel/plugins/tstzrange_fields.rb#L82).

I'm not sure if it's worth making that plugin work for non-tstz ranges, but feel free to use the code and modify it for tsrange, or fork it and add support for other range types, whatever you want, if you think it's useful.

[adam12]

I'll check it out, thanks :)