They were Rails' default and are unnecessarily overridden.
Signed-off-by: NDavid Wagner <david@marvid.fr>

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4093#note_12563922

For clarification and consistency

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4093/diffs#note_12501400

Feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4641#note_12444891

Feedback from:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4093#note_12413950

Feedback from:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4093#note_12321267

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4093#note_12321240

This also moves tests from build_spec.rb to runner_spec.rb

Without that, Rails would complain:

> bind message supplies 5 parameters, but prepared statement "a4" requires 4

However without this workaround it would still work in Rails console,
so I still think this is a Rails bug somewhere.

for a particular project.

This reverts commit 3e991230.

# Conflicts:
#	app/models/project.rb

Favor `unless` over `if` for negative conditions
(or control flow ||).

See #17478

In 8278b763 the default behaviour of annotation
has changes, which was causing a lot of noise in diffs. We decided in #17382
that it is better to get rid of the whole annotate gem, and instead let people
look at schema.rb for the columns in a table.

Fixes: #17382

We can just use "arel_table" in these cases instead of
"SomeClass.arel_table".