Skip to content

Commit

Permalink
Remove code related to CSV export for Profile V1
Browse files Browse the repository at this point in the history
+ Not important anymore because Profile V2 CSV export is done completely client-side
+ #211
  • Loading branch information
alexsoble committed Apr 5, 2016
1 parent 0595099 commit 6ba92b8
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 132 deletions.
111 changes: 0 additions & 111 deletions app/csv_exporters/student_profile_csv_exporter.rb

This file was deleted.

22 changes: 1 addition & 21 deletions spec/controllers/students_controller_spec.rb
Expand Up @@ -42,13 +42,6 @@ def make_request(options = { student_id: nil, format: :html })
expect(response).to redirect_to(new_educator_session_path)
end
end
context 'csv' do
it 'sends a 401 unauthorized' do
make_request({ student_id: student.id, format: :csv })
expect(response.status).to eq 401
expect(response.body).to eq "You need to sign in before continuing."
end
end
end

context 'when educator is logged in' do
Expand Down Expand Up @@ -78,19 +71,6 @@ def make_request(options = { student_id: nil, format: :html })

end

context 'csv' do

it 'is successful' do
make_request({ student_id: student.id, format: :csv })
expect(response).to be_success
end
it 'assigns the student correctly' do
make_request({ student_id: student.id, format: :csv })
expect(assigns(:student)).to eq student
end

end

context 'educator has grade level access' do
let(:educator) { FactoryGirl.create(:educator, grade_level_access: [student.grade] )}

Expand Down Expand Up @@ -292,7 +272,7 @@ def make_post_request(student, service_params = {})
let!(:educator) { FactoryGirl.create(:educator, :admin) }
let!(:provided_by_educator) { FactoryGirl.create(:educator) }
let!(:student) { FactoryGirl.create(:student) }

before do
sign_in(educator)
end
Expand Down

0 comments on commit 6ba92b8

Please sign in to comment.