Commit graph

9 commits

Author SHA1 Message Date
Phil Hughes
9c132216b7
spec fixes 2017-12-14 11:31:50 +00:00
Phil Hughes
3032989ad4
added specs 2017-12-12 09:11:01 +00:00
Phil Hughes
e001f4b772
fixed failing tests 2017-12-11 17:52:52 +00:00
Sean McGivern
5883ce95ef current_application_settings belongs on Gitlab::CurrentSettings
The initializers including this were doing so at the top level, so every object
loaded after them had a `current_application_settings` method. However, if
someone had rack-attack enabled (which was loaded before these initializers), it
would try to load the API, and fail, because `Gitlab::CurrentSettings` didn't
have that method.

To fix this:

1. Don't include `Gitlab::CurrentSettings` at the top level. We do not need
   `Object.new.current_application_settings` to work.
2. Make `Gitlab::CurrentSettings` explicitly `extend self`, as we already use it
   like that in several places.
3. Change the initializers to use that new form.
2017-08-31 13:38:33 +01:00
Douwe Maan
d3b24a7638 Show last commit for current tree on tree page 2017-05-17 20:59:26 +03:00
Douwe Maan
00fc9f2607 Load tree readme asynchronously 2017-05-10 08:21:58 -05:00
Robert Speicher
985a0c39ef Use Project :repository trait where necessary in view specs 2017-03-27 18:45:37 -04:00
Connor Shea
8062380f60 Upgrade Devise from 4.1.1 to 4.2.0.
This fixes an issue with Rails 5 and brings us up-to-date with the latest Devise release.

This also replaces the deprecated Devise::TestHelpers with Devise::Test::ControllerHelpers.

Changelog: https://github.com/plataformatec/devise/blob/v4.2.0/CHANGELOG.md#420---2016-07-01
2016-09-27 20:08:49 -06:00
winniehell
ae9f0ca818 Add failing test for #20462 2016-08-02 03:32:28 +02:00