7263da542b
Replaced by `#lease_connection` to better reflect what it does. `ActiveRecord::Base#connection` is deprecated in the same way but without a removal timeline nor a deprecation warning. Inside the Active Record test suite, we do remove `Base.connection` to ensure it's not used internally. Some callsites have been converted to use `with_connection`, some other have been more simply migrated to `lease_connection` and will serve as a list of callsites to convert for https://github.com/rails/rails/pull/50793
52 lines
1.3 KiB
Ruby
52 lines
1.3 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require "abstract_unit"
|
|
require "action_dispatch"
|
|
require "active_record"
|
|
|
|
class JsonParamsParsingTest < ActionDispatch::IntegrationTest
|
|
def test_prevent_null_query
|
|
# Make sure we have data to find
|
|
klass = Class.new(ActiveRecord::Base) do
|
|
def self.name; "Foo"; end
|
|
establish_connection adapter: "sqlite3", database: ":memory:"
|
|
connection.create_table "foos" do |t|
|
|
t.string :title
|
|
t.timestamps null: false
|
|
end
|
|
end
|
|
klass.create
|
|
assert klass.first
|
|
|
|
app = ->(env) {
|
|
request = ActionDispatch::Request.new env
|
|
params = ActionController::Parameters.new request.parameters
|
|
if params[:t]
|
|
klass.find_by_title(params[:t])
|
|
else
|
|
nil
|
|
end
|
|
}
|
|
|
|
assert_nil app.call(make_env("t" => nil))
|
|
assert_nil app.call(make_env("t" => [nil]))
|
|
|
|
[[[nil]], [[[nil]]]].each do |data|
|
|
assert_nil app.call(make_env("t" => data))
|
|
end
|
|
ensure
|
|
klass.lease_connection.drop_table("foos")
|
|
end
|
|
|
|
private
|
|
def make_env(json)
|
|
data = JSON.dump json
|
|
content_length = data.length
|
|
{
|
|
"CONTENT_LENGTH" => content_length,
|
|
"CONTENT_TYPE" => "application/json",
|
|
"rack.input" => StringIO.new(data)
|
|
}
|
|
end
|
|
end
|