Remove redundant tests method.
correcting method documentation [ci skip]
Merge pull request #15063 from cbeer/correct-attribute_present-docs

correcting method documentation [ci skip]
Update attribute_methods.rb

Improve documentation
Merge pull request #15060 from tgxworld/remove_unnecessary_tests_method

Remove redundant tests method.
Modify assert conditions not to be affected
by each database bind variable naming conventions
Merge pull request #14999 from yahonda/modify_bind_variable_name

Modify assert conditions not to be affected
by each database bind variable naming conventions

Closes #14999.
Handle other pk types in PostgreSQL gracefully.

In #10410 it was noted that you can no longer create PK's with the
type of bigserial in PostgreSQL in 4.0.0.rc1. This is mostly
because the newer adapter is checking for column type with the
id column instead of just letting it pass through like it did
before.

Side effects:
You may just create a PK column of a type that you really don't
want to be your PK. As far as I can tell this was allowed in 3.2.X
and perhaps an exception should be raised if you try and do
something extremely dumb.
Merge pull request #10505 from patricksrobertson/bigserial_id_not_identifying_pk

Handle other pk types in PostgreSQL gracefully.

Closes #10505.

- rebased
- test slightly modified
pg, fix Infinity and NaN values conversion.

Before this patch `Infinity`, `-Infinity` and `Nan` were read as `0`.
Merge pull request #13334 from gregolsen/postgresql_infinity

pg, fix Infinity and NaN values conversion.

Closes #13334.

* rebased
* removed guard
* inlined guard into case statement
flip conditional to use if/else instead of unless/else

Use if/else instead of unless/else so conditional reads better.
raise error if around callback has no yield

Around requires a yield. If an around_destroy callback is called
but there is no yield the callback will fire but the record
will not be destroyed and throw a record not destroyed error.

This isn't clear that a yield is required and this raise provides
that clarity.
add new class for failed around terminator

I did this because to get the around to fail in other tests
would cause the others to fail as well because they would all
raise and not save.
add test to test that ArgumentError is raised if no block

This test tests that an error is raised because this around
callback will fail. It also ensures that the message returned
is "no block given (yield)" to the user. It also tests that the
object is not saved.