From 3ddd936b039474259cff3793c767ecb7f74e89e0 Mon Sep 17 00:00:00 2001
From: Matt Jankowski <mjankowski@thoughtbot.com>
Date: Tue, 11 Apr 2017 16:00:43 -0400
Subject: Refactor exports controller (#1567)

* Add basic coverage for settings/exports controller

* Remove unused @account variable from settings/exports controller

* Add coverage for download export actions

* Remove deprecated `render :text` in favor of `send_data` for csv downloads

* Add model to handle exports

* Use Export class in settings/exports controller

* Simplify settings/exports controller methods

* Move settings/export to more restful routes
---
 .../exports/blocked_accounts_controller_spec.rb         | 17 +++++++++++++++++
 .../exports/following_accounts_controller_spec.rb       | 17 +++++++++++++++++
 spec/controllers/settings/exports_controller_spec.rb    | 14 ++++++++++++++
 spec/models/export_spec.rb                              | 17 +++++++++++++++++
 4 files changed, 65 insertions(+)
 create mode 100644 spec/controllers/settings/exports/blocked_accounts_controller_spec.rb
 create mode 100644 spec/controllers/settings/exports/following_accounts_controller_spec.rb
 create mode 100644 spec/controllers/settings/exports_controller_spec.rb
 create mode 100644 spec/models/export_spec.rb

(limited to 'spec')

diff --git a/spec/controllers/settings/exports/blocked_accounts_controller_spec.rb b/spec/controllers/settings/exports/blocked_accounts_controller_spec.rb
new file mode 100644
index 000000000..574d4d875
--- /dev/null
+++ b/spec/controllers/settings/exports/blocked_accounts_controller_spec.rb
@@ -0,0 +1,17 @@
+require 'rails_helper'
+
+describe Settings::Exports::BlockedAccountsController do
+  before do
+    sign_in Fabricate(:user), scope: :user
+  end
+
+  describe 'GET #index' do
+    it 'returns a csv of the blocking accounts' do
+      get :index, format: :csv
+
+      expect(response).to have_http_status(:success)
+      expect(response.content_type).to eq 'text/csv'
+      expect(response.headers['Content-Disposition']).to eq 'attachment; filename="blocking.csv"'
+    end
+  end
+end
diff --git a/spec/controllers/settings/exports/following_accounts_controller_spec.rb b/spec/controllers/settings/exports/following_accounts_controller_spec.rb
new file mode 100644
index 000000000..bf7680523
--- /dev/null
+++ b/spec/controllers/settings/exports/following_accounts_controller_spec.rb
@@ -0,0 +1,17 @@
+require 'rails_helper'
+
+describe Settings::Exports::FollowingAccountsController do
+  before do
+    sign_in Fabricate(:user), scope: :user
+  end
+
+  describe 'GET #index' do
+    it 'returns a csv of the following accounts' do
+      get :index, format: :csv
+
+      expect(response).to have_http_status(:success)
+      expect(response.content_type).to eq 'text/csv'
+      expect(response.headers['Content-Disposition']).to eq 'attachment; filename="following.csv"'
+    end
+  end
+end
diff --git a/spec/controllers/settings/exports_controller_spec.rb b/spec/controllers/settings/exports_controller_spec.rb
new file mode 100644
index 000000000..ff98f3ad9
--- /dev/null
+++ b/spec/controllers/settings/exports_controller_spec.rb
@@ -0,0 +1,14 @@
+require 'rails_helper'
+
+describe Settings::ExportsController do
+  before do
+    sign_in Fabricate(:user), scope: :user
+  end
+
+  describe 'GET #show' do
+    it 'returns http success' do
+      get :show
+      expect(response).to have_http_status(:success)
+    end
+  end
+end
diff --git a/spec/models/export_spec.rb b/spec/models/export_spec.rb
new file mode 100644
index 000000000..5cc62c266
--- /dev/null
+++ b/spec/models/export_spec.rb
@@ -0,0 +1,17 @@
+require 'rails_helper'
+
+describe Export do
+  describe 'to_csv' do
+    it 'returns a csv of the accounts' do
+      one = Account.new(username: 'one', domain: 'local.host')
+      two = Account.new(username: 'two', domain: 'local.host')
+      accounts = [one, two]
+
+      export = Export.new(accounts).to_csv
+      results = export.strip.split
+
+      expect(results.size).to eq 2
+      expect(results.first).to eq 'one@local.host'
+    end
+  end
+end
-- 
cgit