smalruby-editor 0.0.5 → 0.0.6

data/spec/acceptance/text_editor/check.feature

@@ -0,0 +1,20 @@
+# encoding: utf-8
+# language: ja
+@javascript
+機能: Check - プログラムのエラーチェック
+  シナリオ: チェックボタンを押してプログラムの正しいかチェックできる
+    前提 "エディタ" 画面を表示する
+    かつ テキストエディタに "puts 'Hello, World!'" を入力済みである
+
+    もし "チェックボタン" をクリックする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+
+    ならば "メッセージ" に "エラー" を含まないこと
+
+    もし テキストエディタに "puts Hello, World!'" を入力済みである
+    かつ "チェックボタン" をクリックする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+
+    ならば "メッセージ" に "エラー" を含むこと
+    かつ "メッセージ" に "1行、19文字: syntax error, unexpected tSTRING_BEG, expecting keyword_do or '{' or '('" を含むこと
+    かつ "メッセージ" に "1行: unterminated string meets end of file" を含むこと

data/spec/acceptance/text_editor/load.feature

@@ -0,0 +1,74 @@
+# encoding: utf-8
+# language: ja
+@javascript
+機能: Load - プログラムのロード
+  シナリオ: プログラムを読み込む
+    前提 "エディタ" 画面を表示する
+
+    もし "files/01.rb" をロードする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+
+    ならば テキストエディタに "files/01.rb" を読み込むこと
+    かつ "プログラム名の入力欄" は "01.rb" であること
+
+  シナリオ: プログラムを入力済み状態でプログラムを読み込む
+    前提 "エディタ" 画面を表示する
+    かつ テキストエディタに "puts 'Hello, World!'" を入力済みである
+
+    もし "files/01.rb" をロードする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+
+    ならば テキストエディタに "files/01.rb" を読み込むこと
+    かつ "プログラム名の入力欄" は "01.rb" であること
+
+  シナリオ: 間違って画像を読み込もうとする
+    前提 "エディタ" 画面を表示する
+
+    もし "files/favicon.ico" をロードする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+
+    ならば テキストエディタのプログラムは "" であること
+    かつ "プログラム名の入力欄" は "" であること
+    かつ "メッセージ" に "favicon.icoはRubyのプログラムではありません" を含むこと
+
+  シナリオ: プログラムを修正後にロードボタンを押す
+    前提 "エディタ" 画面を表示する
+    かつ テキストエディタに "puts 'Hello, World!'" を入力済みである
+
+    もし "ロードボタン" をクリックする
+
+    ならば 確認メッセージ "まだセーブしていないのでロードするとプログラムが消えてしまうよ！それでもロードしますか？" を表示すること
+
+  シナリオ: プログラムを修正後にセーブしてからロードボタンを押す
+    前提 "エディタ" 画面を表示する
+    かつ テキストエディタに "puts 'Hello, World!'" を入力済みである
+    かつ プログラムの名前に "01.rb" を指定する
+    かつ "セーブボタン" をクリックする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+    かつ ダウンロードが完了するまで待つ
+    かつ 実際にはファイルをロードしないようにしておく
+
+    もし "ロードボタン" をクリックする
+
+    ならば 確認メッセージを表示しないこと
+
+  シナリオ: プログラムを修正後にプログラムをロードしてからロードボタンを押す
+    前提 "エディタ" 画面を表示する
+    かつ テキストエディタに "puts 'Hello, World!'" を入力済みである
+    かつ "files/01.rb" をロードする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+    かつ 実際にはファイルをロードしないようにしておく
+
+    もし "ロードボタン" をクリックする
+
+    ならば 確認メッセージを表示しないこと
+
+  シナリオ: プログラムを修正後に間違って画像をロードしてからロードボタンを押す
+    前提 "エディタ" 画面を表示する
+    かつ テキストエディタに "puts 'Hello, World!'" を入力済みである
+    かつ "files/favicon.ico" をロードする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+
+    もし "ロードボタン" をクリックする
+
+    ならば 確認メッセージ "まだセーブしていないのでロードするとプログラムが消えてしまうよ！それでもロードしますか？" を表示すること

data/spec/acceptance/text_editor/save.feature

@@ -0,0 +1,31 @@
+# encoding: utf-8
+# language: ja
+@javascript
+機能: Save - プログラムのセーブ
+  シナリオ: セーブボタンを押してプログラムを保存できる
+    前提 "エディタ" 画面を表示する
+    かつ テキストエディタに "puts 'Hello, World!'" を入力済みである
+    かつ プログラムの名前に "01.rb" を指定する
+
+    もし "セーブボタン" をクリックする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+
+    ならば "01.rb" をダウンロードすること
+
+  シナリオ: 日本語のファイル名でプログラムを保存できる
+    前提 "エディタ" 画面を表示する
+    かつ テキストエディタに "puts 'Hello, World!'" を入力済みである
+    かつ プログラムの名前に "日本語.rb" を指定する
+
+    もし "セーブボタン" をクリックする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+
+    ならば "日本語.rb" をダウンロードすること
+
+  シナリオ: ファイル名を入力せずにセーブボタンを押す
+    前提 "エディタ" 画面を表示する
+
+    もし "セーブボタン" をクリックする
+
+    ならば ダウンロードしないこと
+    かつ "プログラム名の入力欄" にフォーカスがあること

data/spec/controllers/editor_controller_spec.rb

@@ -3,71 +3,10 @@ require 'spec_helper'
 require 'nkf'
 
 describe EditorController do
-  describe "エディタ画面を表示する (GET 'index')" do
+  describe 'エディタ画面を表示する (GET index)' do
     it 'returns http success' do
-      get 'index'
+      get :index
       response.should be_success
     end
   end
-
-  describe "プログラムを保存する (GET 'save_file')" do
-    it 'アップロードされたプログラムをダウンロードできる' do
-      params = {
-        filename: '01.rb',
-        source: 'puts "Hello, World!"',
-      }.with_indifferent_access
-
-      expect(@controller).to receive(:send_data)
-        .with(params[:source], filename: params[:filename],
-              disposition: 'attachment',
-              type: 'text/plain; charset=utf-8')
-        .once.and_call_original
-
-      get 'save_file', params
-
-      expect(response).to be_success
-    end
-  end
-
-  describe "プログラムを読み込む (POST 'load_file')" do
-    before do
-      post 'load_file', load_file: load_file
-    end
-
-    describe '正常系' do
-      let(:load_file) { fixture_file_upload('files/01.rb', 'text/plain') }
-
-      it { expect(response).to be_success }
-
-      it 'アップロードされたプログラムの情報をJSON形式でダウンロードできる' do
-        load_file.rewind
-        file = {
-          name: load_file.original_filename,
-          type: load_file.content_type,
-          size: load_file.size,
-          source: load_file.read.force_encoding('utf-8'),
-        }
-        expect(response.body).to be_json_eql(file.to_json)
-      end
-    end
-
-    describe '異常系' do
-      context '画像をアップロードした場合' do
-        let(:load_file) { fixture_file_upload('files/favicon.ico',
-                                              'application/octet-stream') }
-
-        it { expect(response).to be_success }
-
-        it 'エラーを返す' do
-          file = {
-            name: load_file.original_filename,
-            error: 'Rubyのプログラムではありません',
-          }
-          file.each do |path, value|
-            expect(response.body).to include_json(value.to_json).at_path(path.to_s)
-          end
-        end
-      end
-    end
-  end
 end

data/spec/controllers/source_codes_controller_spec.rb

@@ -0,0 +1,296 @@
+# -*- coding: utf-8 -*-
+require 'spec_helper'
+
+describe SourceCodesController do
+  describe 'プログラムが正しいかチェックする (XHR POST check)' do
+    before do
+      params = {
+        source_code: {
+          data: data,
+        }
+      }.with_indifferent_access
+
+      xhr :post, :check, params
+    end
+
+    context 'プログラムが正しい場合' do
+      let(:data) { 'puts "Hello, World!"' }
+
+      it { expect(response).to be_success }
+
+      it do
+        expected = parse_json(SourceCode.new(data: data).check_syntax.to_json)
+        expect(parse_json(response.body)).to eq(expected)
+      end
+    end
+
+    context 'プログラムが正しくない場合' do
+      let(:data) { 'puts Hello, World!"' }
+
+      it { expect(response).to be_success }
+
+      it do
+        expected = parse_json(SourceCode.new(data: data).check_syntax.to_json)
+        expect(parse_json(response.body)).to eq(expected)
+      end
+    end
+  end
+
+  describe 'プログラムを保存する (XHR POST create)' do
+    let(:params) {
+      {
+        source_code: {
+          filename: '01.rb',
+          data: 'puts "Hello, World!"',
+        }
+      }
+    }
+
+    describe 'レスポンス' do
+      subject {
+        xhr :post, :create, params
+        response
+      }
+
+      it { should be_success }
+      it {
+        ids = SourceCode.all.map(&:id)
+        expect(parse_json(subject.body))
+          .to eq('source_code' => {
+                   'id' => SourceCode.find(SourceCode.all.map(&:id) - ids)
+                     .first.id
+                 })
+      }
+    end
+
+    it 'アップロードされたプログラムを保存できる' do
+      expect {
+        xhr :post, :create, params
+      }.to change { SourceCode.count }.by(1)
+    end
+
+    describe 'アップロードされたプログラム' do
+      subject {
+        ids = SourceCode.all.map(&:id)
+        xhr :post, :create, params
+        SourceCode.find((SourceCode.all.map(&:id) - ids).first)
+      }
+
+      its(:data) { should eq(params[:source_code][:data]) }
+      its(:filename) { should eq(params[:source_code][:filename]) }
+    end
+
+    describe 'session' do
+      before {
+        ids = SourceCode.all.map(&:id)
+        xhr :post, :create, params
+        @created_source_code =
+          SourceCode.find((SourceCode.all.map(&:id) - ids).first)
+      }
+
+      subject { session }
+
+      it '[:source_code][:id]はアップロードしたプログラムのIDである' do
+        expect(subject[:source_code][:id]).to eq(@created_source_code.id)
+      end
+
+      it '[:source_code][:digest]はアップロードしたプログラムのSHA256のハッシュ値である' do
+        expect(subject[:source_code][:digest])
+          .to eq(@created_source_code.digest)
+      end
+    end
+  end
+
+  describe 'プログラムをダウンロードしてサーバ上から削除する (DELETE download)' do
+    let(:source_code) {
+      SourceCode.create!(filename: '01.rb', data: 'puts "Hello, World!"')
+    }
+
+    context 'セッションが正しい場合' do
+      let(:_session) {
+        {
+          source_code: {
+            id: source_code.id,
+            digest: source_code.digest,
+          }
+        }
+      }
+
+      context 'ファイル名が半角英数字の場合' do
+        before do
+          allow(@controller).to receive(:send_data).and_call_original
+          delete :download, {}, _session
+        end
+
+        specify 'プログラムをダウンロードする' do
+          expect(@controller).to have_received(:send_data)
+            .with(source_code.data,
+                  filename: source_code.filename,
+                  disposition: 'attachment',
+                  type: 'text/plain; charset=utf-8')
+            .once
+        end
+
+        specify 'プログラムをサーバ上から削除する' do
+          expect(SourceCode).to have(0).records
+        end
+
+        specify 'セッションから[:source_code]を削除する' do
+          expect(session[:source_code]).to be_nil
+        end
+      end
+
+      context 'ファイル名が日本語の場合' do
+        # rubocop:disable LineLength
+        USER_AGENT = {
+          IE11: 'Mozilla/5.0 (Windows NT 6.3; WOW64; Trident/7.0; Touch; rv:11.0) like Gecko',
+          IE10: 'Mozilla/5.0 (compatible; MSIE 10.0; Windows NT 6.2; Trident/6.0; Touch)',
+          Chrome20: 'Mozilla/5.0 (Windows NT 5.1) AppleWebKit/536.11 (KHTML, like Gecko) Chrome/20.0.1132.57 Safari/536.11',
+          Firefox14: 'Mozilla/5.0 (Windows NT 5.1; rv:14.0) Gecko/20100101 Firefox/14.0.1',
+        }
+        # rubocop:enable LineLength
+
+        let(:source_code) {
+          SourceCode.create!(filename: '日本語.rb', data: 'puts "Hello, World!"')
+        }
+
+        [:IE10, :IE11].each do |browser|
+          context "ブラウザが#{browser}の場合" do
+            before do
+              @request.env['HTTP_USER_AGENT'] = USER_AGENT[browser]
+              allow(@controller).to receive(:send_data).and_call_original
+              delete :download, {}, _session
+            end
+
+            specify 'プログラムをダウンロードする' do
+              expect(@controller).to have_received(:send_data)
+                .with(source_code.data,
+                      filename: ERB::Util.url_encode(source_code.filename),
+                      disposition: 'attachment',
+                      type: 'text/plain; charset=utf-8')
+                .once
+            end
+          end
+        end
+
+        [:Chrome20, :Firefox14].each do |browser|
+          context "ブラウザが#{browser}の場合" do
+            before do
+              @request.env['HTTP_USER_AGENT'] = USER_AGENT[browser]
+              allow(@controller).to receive(:send_data).and_call_original
+              delete :download, {}, _session
+            end
+
+            specify 'プログラムをダウンロードする' do
+              expect(@controller).to have_received(:send_data)
+                .with(source_code.data,
+                      filename: source_code.filename,
+                      disposition: 'attachment',
+                      type: 'text/plain; charset=utf-8')
+                .once
+            end
+          end
+        end
+      end
+    end
+
+    context 'セッションが不正な場合' do
+      shared_examples 'raise exception' do
+        it {
+          expect {
+            delete :download, {}, _session
+          }.to raise_exception
+        }
+      end
+
+      context 'セッションに[:source_code]がない場合' do
+        let(:_session) { {} }
+
+        include_examples 'raise exception'
+      end
+
+      context 'セッションに[:source_code][:id]がない場合' do
+        let(:_session) {
+          {
+            source_code: {
+              digest: source_code.digest,
+            }
+          }
+        }
+
+        include_examples 'raise exception'
+      end
+
+      context 'セッションに[:source_code][:digest]がない場合' do
+        let(:_session) {
+          {
+            source_code: {
+              id: source_code.id,
+            }
+          }
+        }
+
+        include_examples 'raise exception'
+      end
+
+      context 'セッションの[:source_code][:digest]が不正な場合' do
+        let(:_session) {
+          {
+            source_code: {
+              id: source_code.id,
+              digest: 'invalid_digest',
+            }
+          }
+        }
+
+        include_examples 'raise exception'
+      end
+    end
+  end
+
+  describe 'プログラムを読み込む (POST load)' do
+    before do
+      post :load, source_code: { file: load_file }
+    end
+
+    describe '正常系' do
+      let(:load_file) { fixture_file_upload('files/01.rb', 'text/plain') }
+
+      it { expect(response).to be_success }
+
+      it 'アップロードされたプログラムの情報をJSON形式でダウンロードできる' do
+        load_file.rewind
+        expected = {
+          source_code: {
+            filename: load_file.original_filename,
+            type: load_file.content_type,
+            size: load_file.size,
+            data: load_file.read.force_encoding('utf-8'),
+          }
+        }.to_json
+        expect(response.body).to be_json_eql(expected)
+      end
+    end
+
+    describe '異常系' do
+      context '画像をアップロードした場合' do
+        let(:load_file) {
+          fixture_file_upload('files/favicon.ico', 'application/octet-stream')
+        }
+
+        it { expect(response).to be_success }
+
+        it 'エラーを返す' do
+          info = {
+            filename: load_file.original_filename,
+            error: 'Rubyのプログラムではありません',
+          }
+          info.each do |path, value|
+            expect(response.body)
+              .to include_json(value.to_json).at_path("source_code/#{path}")
+          end
+        end
+      end
+    end
+  end
+end