Skip to content

Commit

Permalink
Fix method visibility issues
Browse files Browse the repository at this point in the history
All of these were caught by the spec_plugin_vis rake task.
  • Loading branch information
jeremyevans committed Dec 29, 2023
1 parent 6fe5617 commit 436e109
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 5 deletions.
4 changes: 2 additions & 2 deletions lib/sequel/extensions/auto_cast_date_and_time.rb
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,6 @@
#
module Sequel
module AutoCastDateAndTime
private

# :nocov:

# Mark the datasets as requiring sql standard date times. This is only needed
Expand All @@ -57,6 +55,8 @@ def requires_sql_standard_datetimes?
end
# :nocov:

private

# Explicitly cast SQLTime time objects to TIME.
def literal_sqltime_append(sql, v)
sql << "TIME "
Expand Down
3 changes: 2 additions & 1 deletion spec/extensions/pg_array_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,14 @@
before do
@db = Sequel.connect('mock://postgres')
@db.extend_datasets(Module.new do
def quote_identifiers?; false end
private
def default_time_format
"'%H:%M:%S'"
end
def default_timestamp_format
"'%Y-%m-%d %H:%M:%S'"
end
def quote_identifiers?; false end
end)
@db.extension(:pg_array)
@m = Sequel::Postgres
Expand Down
3 changes: 2 additions & 1 deletion spec/extensions/pg_multirange_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,14 @@ def @db.server_version(*) 140000 end
@R = Sequel::Postgres::PGRange
@MR = Sequel::Postgres::PGMultiRange
@db.extend_datasets do
def quote_identifiers?; false end
private
def default_time_format
"'%H:%M:%S'"
end
def default_timestamp_format
"'%Y-%m-%d %H:%M:%S'"
end
def quote_identifiers?; false end
end
@db.extension(:pg_array, :pg_multirange)
end
Expand Down
3 changes: 2 additions & 1 deletion spec/extensions/pg_range_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,14 @@
@db = Sequel.connect('mock://postgres')
@R = Sequel::Postgres::PGRange
@db.extend_datasets do
def quote_identifiers?; false end
private
def default_time_format
"'%H:%M:%S'"
end
def default_timestamp_format
"'%Y-%m-%d %H:%M:%S'"
end
def quote_identifiers?; false end
end
@db.extension(:pg_array, :pg_range)
end
Expand Down

0 comments on commit 436e109

Please sign in to comment.