Commit dc7b714e by John Doe

changed data requests to POST because they can update the view redux

parent a1988d29
...@@ -12,6 +12,7 @@ default: &default ...@@ -12,6 +12,7 @@ default: &default
development: development:
<<: *default <<: *default
database: db/development.sqlite3 database: db/development.sqlite3
timeout: 10000
# Warning: The database defined as "test" will be erased and # Warning: The database defined as "test" will be erased and
# re-generated from your development database when you run "rake". # re-generated from your development database when you run "rake".
......
...@@ -19,7 +19,7 @@ Rails.application.routes.draw do ...@@ -19,7 +19,7 @@ Rails.application.routes.draw do
end end
resources :db_elements, only: [:index] do resources :db_elements, only: [:index] do
collection do collection do
get 'data' post 'data'
end end
end end
......
...@@ -4,7 +4,7 @@ require 'rails_helper' ...@@ -4,7 +4,7 @@ require 'rails_helper'
RSpec.describe DbElementsController, type: :request do RSpec.describe DbElementsController, type: :request do
let(:user1) { create(:user, first_name: 'John') } let(:user1) { create(:user, first_name: 'John') }
let(:user2) { create(:user, first_name: 'Sam') } let(:user2) { create(:user, first_name: 'Sam') }
describe 'GET #data' do describe 'POST #data' do
# retrieve data for elements listed by array of ids # retrieve data for elements listed by array of ids
context 'with authenticated user' do context 'with authenticated user' do
before do before do
...@@ -31,7 +31,7 @@ RSpec.describe DbElementsController, type: :request do ...@@ -31,7 +31,7 @@ RSpec.describe DbElementsController, type: :request do
allow(LoadElementData).to receive(:new).and_return(@mock_service) allow(LoadElementData).to receive(:new).and_return(@mock_service)
@auth_headers = user1.create_new_auth_token @auth_headers = user1.create_new_auth_token
get '/db_elements/data.json', post '/db_elements/data.json',
params: { elements: [@elem1.id, @elem2.id].to_json, params: { elements: [@elem1.id, @elem2.id].to_json,
start_time: 0, end_time: 100, redux_json: 'new home view' }, start_time: 0, end_time: 100, redux_json: 'new home view' },
headers: @auth_headers headers: @auth_headers
...@@ -44,7 +44,7 @@ RSpec.describe DbElementsController, type: :request do ...@@ -44,7 +44,7 @@ RSpec.describe DbElementsController, type: :request do
end end
it 'returns error if time bounds are invalid' do it 'returns error if time bounds are invalid' do
@auth_headers = user1.create_new_auth_token @auth_headers = user1.create_new_auth_token
get '/db_elements/data.json', post '/db_elements/data.json',
params: { elements: [@elem1.id, @elem2.id].to_json, params: { elements: [@elem1.id, @elem2.id].to_json,
start_time: 100, end_time: 0 }, start_time: 100, end_time: 0 },
headers: @auth_headers headers: @auth_headers
...@@ -59,7 +59,7 @@ RSpec.describe DbElementsController, type: :request do ...@@ -59,7 +59,7 @@ RSpec.describe DbElementsController, type: :request do
stream2.db_elements << @elem3 stream2.db_elements << @elem3
@auth_headers = user1.create_new_auth_token @auth_headers = user1.create_new_auth_token
get '/db_elements/data.json', post '/db_elements/data.json',
params: { elements: [@elem1.id, @elem3.id].to_json, params: { elements: [@elem1.id, @elem3.id].to_json,
start_time: 100, end_time: 0 }, start_time: 100, end_time: 0 },
headers: @auth_headers headers: @auth_headers
...@@ -68,7 +68,7 @@ RSpec.describe DbElementsController, type: :request do ...@@ -68,7 +68,7 @@ RSpec.describe DbElementsController, type: :request do
end end
context 'without sign-in' do context 'without sign-in' do
it 'returns unauthorized' do it 'returns unauthorized' do
get '/db_elements/data.json' post '/db_elements/data.json'
expect(response).to have_http_status(:unauthorized) expect(response).to have_http_status(:unauthorized)
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 sign in to comment