Commit 603ceea2 authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Add tests and improve logic

Signed-off-by: default avatarDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
parent b28714b6
...@@ -4,7 +4,8 @@ module API ...@@ -4,7 +4,8 @@ module API
helpers do helpers do
def current_settings def current_settings
@current_setting ||= ApplicationSetting.current @current_setting ||=
(ApplicationSetting.current || ApplicationSetting.create_from_defaults)
end end
end end
......
...@@ -7,11 +7,23 @@ describe API::API, 'Settings', api: true do ...@@ -7,11 +7,23 @@ describe API::API, 'Settings', api: true do
let(:admin) { create(:admin) } let(:admin) { create(:admin) }
describe "GET /applicaiton/settings" do describe "GET /application/settings" do
# TODO: Implement it "should return application settings" do
get api("/application/settings", admin)
expect(response.status).to eq(200)
expect(json_response).to be_an Hash
expect(json_response['default_projects_limit']).to eq(42)
expect(json_response['signin_enabled']).to be_truthy
end
end end
describe "PUT /applicaiton/settings" do describe "PUT /application/settings" do
# TODO: Implement it "should update application settings" do
put api("/application/settings", admin),
default_projects_limit: 3, signin_enabled: false
expect(response.status).to eq(200)
expect(json_response['default_projects_limit']).to eq(3)
expect(json_response['signin_enabled']).to be_falsey
end
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment