smalruby-editor 0.0.6 → 0.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d71a9e4754ddc023b99130576187b13e97cca3ce
-  data.tar.gz: 4ed1b62b217727d2a96bfa9c337049a8850e2684
+  metadata.gz: 27f4c696fc7ebd2507f3025d757ac3a956eb3e10
+  data.tar.gz: 70aec400f0cb75231ab0d19b5250eecab9e64ae5
 SHA512:
-  metadata.gz: bb60b2b55e43a4403b29441647b3eae78b6ae563be42d2b84c5c6f4971eccd10875fe131bac0013042d641a0aa5165b564472cf1cc3db5039fadb54ac688e9f5
-  data.tar.gz: 77272e287ef54a97a9c8fdf70e9da1c02a7a62eee9cf5e05df6413b0ae4285b6686616aa35020ebe64735e1ead930cb80e1d00840f193c4a24250bc49661f237
+  metadata.gz: 8d2a30f76ea5fbed2ffaf37b38c359641ac148d9a9d73e5bd3aff095eb50beff67a25204fad4d5e895604e5cc13345aadc85255fa7004384dbfc22d764380e93
+  data.tar.gz: 50ba103b5055f534b8b816ee4d2805f117ec88a3939265844fab51d2f3c15a5e5b1a73c48a039e5ce8731ff19130c9a94c6de7cb3cafa3e2de011efa7686d8d3

data/.rubocop.yml

@@ -1,6 +1,7 @@
 AllCops:
   Excludes:
     - db/seeds.rb
+    - spec/fixtures/**
 
 AsciiComments:
   Enabled: false

data/.travis.yml

@@ -8,7 +8,6 @@ before_script:
   - mysql -e 'create database myapp_test;'
 
 script:
-  - export SECRET_KEY_BASE='xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx'
   - export COVERALLS='1'
   - RAILS_ENV=test bundle exec rake db:migrate --trace
   - bundle exec rake db:test:prepare

data/README.rdoc

@@ -17,7 +17,7 @@ The Smalruby(smɔ́ːrúːbi) Project will provide a Ruby learning environment f
 requirements:
 
 * Windows or UNIX like OS (Mac OS X, Linux, etc...)
-* Ruby 2.0.0-p247 or higher.
+* Ruby 2.0.0-p353 or higher.
 * MySQL
 * Git
 

data/app/assets/javascripts/editor.js.coffee.erb

@@ -0,0 +1,139 @@
+# Place all the behaviors and hooks related to the matching controller here.
+# All this logic will automatically be available in application.js.
+# You can use CoffeeScript in this file: http://coffeescript.org/
+
+changed = false
+
+successMessage = (title, msg = '', selector = '#messages') ->
+  html = $('<div class="alert alert-success" style="display: none">')
+    .append("<h4><i class=\"icon-star\"></i>#{title}</h4>")
+    .append(msg)
+  $(selector).append(html)
+  html.fadeIn('slow')
+    .delay(3000)
+    .fadeOut('slow')
+
+errorMessage = (msg, selector = '#messages') ->
+  html = $('<div class="alert alert-error" style="display: none">')
+    .append('<button type="button" class="close" data-dismiss="alert">×</button>')
+    .append('<h4><i class="icon-exclamation-sign"></i>エラー</h4>')
+    .append(msg)
+  $(selector).append(html)
+  html.fadeIn('slow')
+    .delay(10000)
+    .fadeOut('slow')
+
+$ ->
+  saving = false
+
+  textEditor = ace.edit('text-editor')
+  textEditor.setTheme('ace/theme/github')
+  textEditor.setShowInvisibles(true)
+
+  textEditor.focus()
+  textEditor.gotoLine(0, 0)
+
+  textEditor.on 'change', (e) ->
+    changed = true
+
+  session = textEditor.getSession()
+  session.setMode('ace/mode/ruby')
+  session.setTabSize(2)
+  session.setUseSoftTabs(true)
+
+  $('#check-button').click (e) ->
+    e.preventDefault()
+    $.ajax
+      url: '/source_codes/check'
+      type: 'POST'
+      data:
+        source_code:
+          data: session.getDocument().getValue()
+      dataType: 'json'
+      success: (data, textStatus, jqXHR) ->
+        if data.length == 0
+          successMessage('チェックしました', 'ただし、プログラムを動かすとエラーが見つかるかもしれません。')
+        else
+          for errorInfo in data
+            do (errorInfo) ->
+              msg = "#{errorInfo.row}行"
+              if errorInfo.column > 0
+                msg += "、#{errorInfo.column}文字"
+              errorMessage(msg + ": #{errorInfo.message}")
+
+  $('#save-button').click (e) ->
+    e.preventDefault()
+    filename = $.trim($('#filename').val())
+    if filename.length <= 0
+      errorMessage('セーブする前にプログラムに名前をつけてね！')
+      $('#filename').focus()
+    else
+      afterSave = ->
+        saving = true
+        changed = false
+        successMessage('セーブしました')
+      $.ajax
+        url: '/source_codes/'
+        type: 'POST'
+        data:
+          source_code:
+            filename: filename
+            data: session.getDocument().getValue()
+        dataType: 'json'
+        success: (data, textStatus, jqXHR) ->
+          <% if Rails.env == 'standalone' %>
+          $.ajax
+            url: '/source_codes/write'
+            type: 'DELETE'
+            dataType: 'json'
+            success: (data, textStatus, jqXHR) ->
+              if data.source_code.error && confirm("前に#{filename}という名前でセーブしているけど本当にセーブしますか？\nセーブすると前に作成したプログラムは消えてしまうよ！")
+                $.ajax
+                  url: '/source_codes/write'
+                  type: 'DELETE'
+                  data:
+                    force: true
+                  dataType: 'json'
+                  success: (data, textStatus, jqXHR) ->
+                    afterSave()
+              else
+                afterSave()
+          <% else %>
+          afterSave()
+          $('#download-link').click()
+          <% end %>
+
+  $('#filename').keypress (e) ->
+    e = window.event if !e
+    if e.keyCode == 13
+      $('#save-button').click()
+      false
+    else
+      true
+
+  $('#load-button').click (e) ->
+    e.preventDefault()
+    if changed
+      return unless confirm('まだセーブしていないのでロードするとプログラムが消えてしまうよ！\nそれでもロードしますか？')
+    $(@).parent().find('input[name="source_code[file]"]').click()
+
+  $('#file-form').fileupload
+    dataType: 'json'
+    done: (e, data) ->
+      info = data.result.source_code
+      if info.error
+        errorMessage(info.filename + 'は' + info.error)
+      else
+        $('#filename').val(info.filename)
+        session.getDocument().setValue(info.data)
+        textEditor.focus()
+        textEditor.moveCursorTo(0, 0)
+        changed = false
+        successMessage('ロードしました')
+
+  window.onbeforeunload = (event) ->
+    if !saving && session.getDocument().getValue().length > 0
+      return '作成中のプログラムが消えてしまうよ！'
+    else
+      saving = false
+      return

data/app/controllers/source_codes_controller.rb

@@ -2,33 +2,41 @@
 require 'nkf'
 
 class SourceCodesController < ApplicationController
+  before_filter :check_whether_standalone, only: :write
+
   def check
     render json: SourceCode.new(source_code_params).check_syntax
   end
 
   def create
-    source_code = SourceCode.create!(source_code_params)
+    sc = SourceCode.create!(source_code_params)
     session[:source_code] = {
-      id: source_code.id,
-      digest: source_code.digest,
+      id: sc.id,
+      digest: sc.digest,
     }
-    render json: { source_code: { id: source_code.id } }
+    render json: { source_code: { id: sc.id } }
   end
 
   def download
-    source_code = SourceCode.find(session[:source_code][:id])
-    unless source_code.digest == session[:source_code][:digest]
-      fail ActiveRecord::RecordNotFound
-    end
-
     send_data(source_code.data,
-              filename: encode_filename(source_code.filename),
+              filename: url_encode_filename(source_code.filename),
               disposition: 'attachment',
               type: 'text/plain; charset=utf-8')
 
-    source_code.destroy
+    destroy_source_code_and_delete_session(source_code)
+  end
 
-    session[:source_code] = nil
+  def write
+    res = { source_code: { filename: source_code.filename } }
+
+    write_source_code(source_code)
+
+    destroy_source_code_and_delete_session(source_code)
+
+    render json: res
+  rescue => e
+    res[:source_code][:error] = e.message
+    render json: res
   end
 
   def load
@@ -45,6 +53,12 @@ class SourceCodesController < ApplicationController
 
   private
 
+  def check_whether_standalone
+    unless Rails.env == 'standalone'
+      fail "#{self.class.name}##{action_name}はstandaloneモードのみの機能です"
+    end
+  end
+
   def source_code_params
     params.require(:source_code).permit(:data, :filename)
   end
@@ -57,11 +71,36 @@ class SourceCodesController < ApplicationController
     }
   end
 
-  def encode_filename(filename)
+  def url_encode_filename(filename)
     if request.env['HTTP_USER_AGENT'] =~ /MSIE|Trident/
       return ERB::Util.url_encode(filename)
     else
       filename
     end
   end
+
+  def source_code
+    return @source_code if @source_code
+    sc = SourceCode.find(session[:source_code][:id])
+    unless sc.digest == session[:source_code][:digest]
+      fail ActiveRecord::RecordNotFound
+    end
+    @source_code = sc
+  end
+
+  def write_source_code(source_code)
+    path = Pathname("~/#{source_code.filename}").expand_path.to_s
+
+    fail 'すでに同じ名前のプログラムがあります' if File.exist?(path) && params[:force].blank?
+
+    File.open(path, 'w') do |f|
+      f.write(source_code.data)
+    end
+  end
+
+  def destroy_source_code_and_delete_session(source_code)
+    source_code.destroy
+
+    session[:source_code] = nil
+  end
 end

data/app/models/source_code.rb

@@ -7,6 +7,7 @@ require 'digest/sha2'
 # ソースコードを表現するモデル
 class SourceCode < ActiveRecord::Base
   validates :filename, presence: true
+  validate :validate_filename
   validates :data, presence: true, allow_blank: true
 
   # シンタックスをチェックする
@@ -39,4 +40,10 @@ class SourceCode < ActiveRecord::Base
                          RbConfig::CONFIG['RUBY_INSTALL_NAME'])
     Open3.capture3("#{ruby_cmd} -c #{path}")
   end
+
+  def validate_filename
+    if File.basename(filename) != filename
+      errors.add(:filename, 'includes directory separator(s)')
+    end
+  end
 end

data/app/views/editor/index.html.erb

@@ -1,14 +1,16 @@
 <div class="navbar navbar-inverse">
   <div class="navbar-inner">
     <ul class="nav">
-      <li class="active"><a id="ruby-tab" href="#">Ruby</a></li>
+      <li class="active"><a id="ruby-tab" href="#" onclick="window.event.preventDefault(); return false;">Ruby</a></li>
     </ul>
 
     <%= form_tag(source_codes_load_path, id: "file-form", class: "navbar-form pull-right", method: "post", multipart: true) do %>
       <button id="check-button" type="button" class="btn btn-primary">チェック</button>
       <input id="filename" type="text" class="span4" placeholder="プログラムの名前を入れてね（例：01.rb）">
       <button id="save-button" type="button" class="btn btn-primary">セーブ</button>
+      <% unless Rails.env == 'standalone' %>
       <%= link_to '', source_codes_download_path, id: 'download-link', style: 'display: none', 'data-method' => 'delete' %>
+      <% end %>
       <button id="load-button" class="btn btn-primary">ロード</button>
       <input id="load-file" type="file" name="source_code[file]" style="display: none">
     <% end %>
@@ -19,4 +21,3 @@
 </div>
 
 <div id="text-editor"></div>
-

data/bin/smalruby-editor

@@ -4,7 +4,6 @@
 require 'pathname'
 
 ENV['RAILS_ENV'] = 'standalone'
-ENV['SECRET_KEY_BASE'] ||= 'xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx'
 ENV['SMALRUBY_EDITOR_HOME'] ||= Pathname('~/.smalruby-editor').expand_path.to_s
 
 APP_PATH = File.expand_path('../../config/application',  __FILE__)

data/config/database.yml.mysql2

@@ -37,3 +37,12 @@ production:
   username: root
   password:
   host: localhost
+
+standalone:
+  adapter: mysql2
+  encoding: utf8
+  database: smalruby-editor_standalone
+  pool: 5
+  username: root
+  password:
+  host: localhost

data/config/database.yml.sqlite3

@@ -23,3 +23,9 @@ production:
   database: db/production.sqlite3
   pool: 5
   timeout: 5000
+
+standalone:
+  adapter: sqlite3
+  database: db/standalone.sqlite3
+  pool: 5
+  timeout: 5000

data/config/environments/standalone.rb

@@ -1,17 +1,15 @@
 require_relative 'production'
 
 SmalrubyEditor::Application.configure do
-  unless ENV['SMALRUBY_EDITOR_HOME']
-    fail 'you must set SMALRUBY_EDITOR_HOME environment variable.'
-  end
-
-  home_dir = SmalrubyEditor.create_home_directory
+  if ENV['SMALRUBY_EDITOR_HOME'].present?
+    home_dir = SmalrubyEditor.create_home_directory
 
-  config.paths.add 'config/database',
-                   with: home_dir.join('config/database.yml')
-  config.paths.add 'log', with: home_dir.join("log/#{Rails.env}.log")
-  config.paths.add 'tmp', with: home_dir.join('tmp')
-  config.paths.add 'tmp/cache', with: home_dir.join('tmp/cache')
+    config.paths.add('config/database',
+                     with: home_dir.join('config/database.yml'))
+    config.paths.add('log', with: home_dir.join("log/#{Rails.env}.log"))
+    config.paths.add('tmp', with: home_dir.join('tmp'))
+    config.paths.add('tmp/cache', with: home_dir.join('tmp/cache'))
+  end
 
   config.serve_static_assets = true
   config.log_level = :warn

data/config/initializers/secret_token.rb

@@ -9,4 +9,9 @@
 
 # Make sure your secret_key_base is kept private
 # if you're sharing your code publicly.
-SmalrubyEditor::Application.config.secret_key_base = ENV['SECRET_KEY_BASE']
+if Rails.env == 'production'
+  secret_key_base = ENV['SECRET_KEY_BASE']
+else
+  secret_key_base = '123456789012345678901234567890'
+end
+SmalrubyEditor::Application.config.secret_key_base = secret_key_base

data/config/routes.rb

@@ -6,6 +6,7 @@ SmalrubyEditor::Application.routes.draw do
   post 'source_codes/check'
   delete 'source_codes/download'
   post 'source_codes/load'
+  delete 'source_codes/write'
 
   # The priority is based upon order of creation: first created ->
   # highest priority.

data/lib/smalruby_editor/version.rb

@@ -1,3 +1,3 @@
 module SmalrubyEditor
-  VERSION = '0.0.6'
+  VERSION = '0.0.7'
 end

data/lib/tasks/gem.rake

@@ -1,5 +1,10 @@
 require 'bundler/gem_tasks'
 
-task 'build' => ['assets:clobber', 'assets:precompile']
+task 'assets:precompile:standalone' do
+  Rails.env = ENV['RAILS_ENV'] = 'standalone'
+  Rake::Task['assets:precompile'].invoke
+end
+
+task 'build' => ['assets:clobber', 'assets:precompile:standalone']
 
 

data/lib/tasks/rspec.rake

@@ -1,16 +1,16 @@
-begin
-  if defined? RSpec # otherwise fails on non-live environments
-    task(:spec).clear
-    desc "Run all specs/features in spec directory"
-    RSpec::Core::RakeTask.new(:spec => 'db:test:prepare') do |t|
-      t.pattern = './spec/{,**/}*{_spec.rb,.feature}'
-    end
-
-    namespace :spec do
-      desc "Run the code examples in spec/acceptance"
-      RSpec::Core::RakeTask.new(:acceptance => 'db:test:prepare') do |t|
-        t.pattern = './spec/acceptance/{,**/}*.feature'
-      end
-    end
-  end
-end
+begin
+  if defined? RSpec # otherwise fails on non-live environments
+    task(:spec).clear
+    desc "Run all specs/features in spec directory"
+    RSpec::Core::RakeTask.new(:spec => 'db:test:prepare') do |t|
+      t.pattern = './spec/{,**/}*{_spec.rb,.feature}'
+    end
+
+    namespace :spec do
+      desc "Run the code examples in spec/acceptance"
+      RSpec::Core::RakeTask.new(:acceptance => 'db:test:prepare') do |t|
+        t.pattern = './spec/acceptance/{,**/}*.feature'
+      end
+    end
+  end
+end

data/lib/tasks/rubocop.rake

@@ -1,3 +1,3 @@
 task :rubocop do
-  sh 'bundle exec rubocop --rails app lib bin config'
+  sh 'bundle exec rubocop --rails app lib bin config spec'
 end

data/spec/acceptance/standalone/save.feature

@@ -0,0 +1,32 @@
+# encoding: utf-8
+# language: ja
+@javascript
+@standalone
+機能: Save - プログラムのセーブ(standaloneモード)
+  シナリオ: セーブボタンを押してローカルマシンにプログラムを保存できる
+    前提 "エディタ" 画面を表示する
+    かつ テキストエディタに "puts 'Hello, World!'" を入力済みである
+    かつ プログラムの名前に "01.rb" を指定する
+
+    もし "セーブボタン" をクリックする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+
+    ならば ホームディレクトリに "01.rb" が存在すること
+    かつ ホームディレクトリの "01.rb" の内容が "puts 'Hello, World!'" であること
+    かつ "メッセージ" に "セーブしました" を含むこと
+
+  シナリオ: ローカルマシンに同じ名前のプログラムがある状態でセーブボタンを押す
+    前提 "エディタ" 画面を表示する
+    かつ ホームディレクトリに "puts 'Hello, World!'" という内容の "01.rb" が存在する
+    かつ テキストエディタに "n = 0" を入力済みである
+    かつ プログラムの名前に "01.rb" を指定する
+
+    もし "セーブボタン" をクリックする
+    かつ JavaScriptによるリクエストが終わるまで待つ
+
+    ならば 確認メッセージ "前に01.rbという名前でセーブしているけど本当にセーブしますか？\nセーブすると前に作成したプログラムは消えてしまうよ！" を表示すること
+    かつ ホームディレクトリの "01.rb" の内容が "puts 'Hello, World!'" であること
+
+    もし JavaScriptによるリクエストが終わるまで待つ
+
+    ならば "メッセージ" に "セーブしました" を含むこと

data/spec/acceptance/text_editor/check.feature

@@ -2,19 +2,24 @@
 # language: ja
 @javascript
 機能: Check - プログラムのエラーチェック
-  シナリオ: チェックボタンを押してプログラムの正しいかチェックできる
+  シナリオ: チェックボタンを押してプログラムにシンタックスエラーがないいことをチェックできる
     前提 "エディタ" 画面を表示する
     かつ テキストエディタに "puts 'Hello, World!'" を入力済みである
 
     もし "チェックボタン" をクリックする
     かつ JavaScriptによるリクエストが終わるまで待つ
 
-    ならば "メッセージ" に "エラー" を含まないこと
+    ならば "メッセージ" に "チェックしました" を含むこと
+    かつ "メッセージ" に "ただし、プログラムを動かすとエラーが見つかるかもしれません。" を含むこと
 
-    もし テキストエディタに "puts Hello, World!'" を入力済みである
-    かつ "チェックボタン" をクリックする
+  シナリオ: チェックボタンを押してプログラムにシンタックスエラーがあることをチェックできる
+    前提 "エディタ" 画面を表示する
+    かつ テキストエディタに "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

@@ -10,6 +10,9 @@
 
     ならば テキストエディタに "files/01.rb" を読み込むこと
     かつ "プログラム名の入力欄" は "01.rb" であること
+    かつ テキストエディタの 0 行目の 0 文字目にカーソルがあること
+    かつ テキストエディタにフォーカスがあること
+    かつ "メッセージ" に "ロードしました" を含むこと
 
   シナリオ: プログラムを入力済み状態でプログラムを読み込む
     前提 "エディタ" 画面を表示する
@@ -20,6 +23,9 @@
 
     ならば テキストエディタに "files/01.rb" を読み込むこと
     かつ "プログラム名の入力欄" は "01.rb" であること
+    かつ テキストエディタの 0 行目の 0 文字目にカーソルがあること
+    かつ テキストエディタにフォーカスがあること
+    かつ "メッセージ" に "ロードしました" を含むこと
 
   シナリオ: 間違って画像を読み込もうとする
     前提 "エディタ" 画面を表示する
@@ -30,6 +36,7 @@
     ならば テキストエディタのプログラムは "" であること
     かつ "プログラム名の入力欄" は "" であること
     かつ "メッセージ" に "favicon.icoはRubyのプログラムではありません" を含むこと
+    かつ "メッセージ" に "ロードしました" を含まないこと
 
   シナリオ: プログラムを修正後にロードボタンを押す
     前提 "エディタ" 画面を表示する
@@ -37,7 +44,7 @@
 
     もし "ロードボタン" をクリックする
 
-    ならば 確認メッセージ "まだセーブしていないのでロードするとプログラムが消えてしまうよ！それでもロードしますか？" を表示すること
+    ならば 確認メッセージ "まだセーブしていないのでロードするとプログラムが消えてしまうよ！\nそれでもロードしますか？" を表示すること
 
   シナリオ: プログラムを修正後にセーブしてからロードボタンを押す
     前提 "エディタ" 画面を表示する
@@ -71,4 +78,4 @@
 
     もし "ロードボタン" をクリックする
 
-    ならば 確認メッセージ "まだセーブしていないのでロードするとプログラムが消えてしまうよ！それでもロードしますか？" を表示すること
+    ならば 確認メッセージ "まだセーブしていないのでロードするとプログラムが消えてしまうよ！\nそれでもロードしますか？" を表示すること

data/spec/acceptance/text_editor/save.feature

@@ -11,6 +11,7 @@
     かつ JavaScriptによるリクエストが終わるまで待つ
 
     ならば "01.rb" をダウンロードすること
+    かつ "メッセージ" に "セーブしました" を含むこと
 
   シナリオ: 日本語のファイル名でプログラムを保存できる
     前提 "エディタ" 画面を表示する
@@ -21,6 +22,7 @@
     かつ JavaScriptによるリクエストが終わるまで待つ
 
     ならば "日本語.rb" をダウンロードすること
+    かつ "メッセージ" に "セーブしました" を含むこと
 
   シナリオ: ファイル名を入力せずにセーブボタンを押す
     前提 "エディタ" 画面を表示する
@@ -29,3 +31,4 @@
 
     ならば ダウンロードしないこと
     かつ "プログラム名の入力欄" にフォーカスがあること
+    かつ "メッセージ" に "セーブする前にプログラムに名前をつけてね！" を含むこと

data/spec/controllers/source_codes_controller_spec.rb

@@ -248,6 +248,179 @@ describe SourceCodesController do
     end
   end
 
+  describe 'プログラムをホームディレクトリに保存してサーバ上から削除する' \
+           ' (DELETE write)' do
+    let(:source_code) {
+      SourceCode.create!(filename: '01.rb', data: 'puts "Hello, World!"')
+    }
+    let(:params) { {} }
+    let(:_session) {
+      {
+        source_code: {
+          id: source_code.id,
+          digest: source_code.digest,
+        }
+      }
+    }
+
+    context 'RAILS_ENVがstandaloneの場合', set_standalone_rails_env: true do
+      let(:path) { Pathname("~/#{source_code.filename}").expand_path.to_s }
+
+      context 'セッションが正しい場合' do
+        shared_examples 'success writing' do
+          let(:target_file) {
+            f = double('File')
+            allow(f).to receive(:write)
+            f
+          }
+
+          before do
+            allow(File).to receive(:open).and_yield(target_file)
+            xhr :delete, :write, params, _session
+          end
+
+          specify { expect(response).to be_success }
+
+          specify 'ホームディレクトリにファイルを保存する' do
+            expect(File).to have_received(:open).with(path, 'w').once
+            expect(target_file)
+              .to have_received(:write).with(source_code.data).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
+          before do
+            allow(File).to receive(:exist?).with(path).and_return(false)
+          end
+
+          context 'ファイル名が半角英数字の場合' do
+            include_examples 'success writing'
+          end
+
+          context 'ファイル名が日本語の場合' do
+            let(:source_code) {
+              SourceCode.create!(filename: '日本語でも表現できる.rb',
+                                 data: 'puts "Hello, World!"')
+            }
+
+            include_examples 'success writing'
+          end
+        end
+
+        context '同じ名前のファイルが存在する場合' do
+          before do
+            allow(File).to receive(:exist?).with(path).and_return(true)
+          end
+
+          context '上書き保存モードの場合' do
+            let(:params) { { force: true } }
+
+            include_examples 'success writing'
+          end
+
+          context '上書き保存モードではない場合' do
+            before do
+              xhr :delete, :write, params, _session
+            end
+
+            specify 'プログラムをサーバ上から削除しない' do
+              expect(SourceCode).to have(1).records
+            end
+
+            specify 'セッションから[:source_code]を削除しない' do
+              expect(session[:source_code]).to eq(_session[:source_code])
+            end
+          end
+        end
+      end
+
+      context 'セッションが不正な場合' do
+        shared_examples 'raise exception' do
+          it {
+            expect {
+              xhr :delete, :write, params, _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
+
+    context 'RAILS_ENVがstandalone以外の場合' do
+      subject do
+        xhr :delete, :write, params, _session
+      end
+
+      it { expect { subject }.to raise_exception }
+
+      it 'プログラムをサーバ上から削除しない' do
+        begin
+          subject
+        rescue
+          expect(SourceCode).to have(1).records
+        end
+      end
+
+      it 'セッションから[:source_code]を削除しない' do
+        begin
+          subject
+        rescue
+          expect(session[:source_code]).to eq(_session[:source_code])
+        end
+      end
+    end
+  end
+
   describe 'プログラムを読み込む (POST load)' do
     before do
       post :load, source_code: { file: load_file }

data/spec/lib/smalruby_editor_spec.rb

@@ -18,7 +18,8 @@ describe SmalrubyEditor do
       end
 
       describe 'ホームディレクトリ' do
-        %w(config db log tmp tmp/cache tmp/pids tmp/sessions tmp/sockets).each do |dir|
+        %w(config db log tmp
+           tmp/cache tmp/pids tmp/sessions tmp/sockets).each do |dir|
           it "#{dir}ディレクトリを作成する" do
             expect(@home_dir.join(dir)).to be_directory
           end
@@ -54,7 +55,8 @@ describe SmalrubyEditor do
       end
     end
 
-    describe '引数を指定しない場合は環境変数SMALRUBY_EDITOR_HOMEに従う', create_home_directory: true do
+    describe '引数を指定しない場合は環境変数SMALRUBY_EDITOR_HOMEに従う',
+             create_home_directory: true do
       before(:all) do
         ENV['SMALRUBY_EDITOR_HOME'] = @home_dir.to_s
         SmalrubyEditor.create_home_directory

data/spec/models/source_code_spec.rb

@@ -2,6 +2,17 @@
 require 'spec_helper'
 
 describe SourceCode, 'Rubyのソースコードを表現するモデル' do
+  describe 'validation' do
+    describe 'filename' do
+      specify 'ディレクトリセパレータを含むことはできない' do
+        sc = SourceCode.new(filename: '/etc/passwd')
+        expect(sc).not_to be_valid
+        expect(sc.errors[:filename])
+          .to include('includes directory separator(s)')
+      end
+    end
+  end
+
   describe '#check_syntax', 'シンタックスをチェックする' do
     let(:source_code) {
       SourceCode.new(data: data)

data/spec/spec_helper.rb

@@ -24,9 +24,6 @@ Spork.prefork do
   # in spec/support/ and its subdirectories.
   Dir[Rails.root.join('spec/support/**/*.rb')].each { |f| require f }
 
-  load 'spec/steps/global_variable.rb', true
-  Dir.glob('spec/steps/**/*_steps.rb') { |f| load f, true }
-
   # Checks for pending migrations before tests are run.
   # If you are not using ActiveRecord, you can remove this line.
   ActiveRecord::Migration.check_pending! if defined?(ActiveRecord::Migration)
@@ -48,7 +45,7 @@ Spork.prefork do
     # If you're not using ActiveRecord, or you'd prefer not to run each of your
     # examples within a transaction, remove the following line or assign false
     # instead of true.
-    config.use_transactional_fixtures = true
+    config.use_transactional_fixtures = false
 
     # If true, the base class of anonymous controllers will be inferred
     # automatically. This will be the default behavior in future versions of
@@ -106,9 +103,40 @@ Spork.prefork do
 
     config.after(javascript: true) do
       page.execute_script('window.onbeforeunload = null')
-      if selenium?
-        FileUtils.rm_rf(downloads_dir)
-      end
+      FileUtils.rm_rf(downloads_dir) if selenium?
+    end
+
+    config.before(:all, javascript: true, standalone: true) do
+      expire_assets_cache
+    end
+
+    config.after(:all, javascript: true, standalone: true) do
+      expire_assets_cache
+    end
+
+    config.before(javascript: true, standalone: true) do
+      page.driver.restart
+
+      @_rails_env = Rails.env
+      Rails.env = ENV['RAILS_ENV'] = 'standalone'
+      @_home = ENV['HOME']
+      @_tmpdir = ENV['HOME'] = Dir.mktmpdir('smalruby-')
+    end
+
+    config.after(javascript: true, standalone: true) do
+      FileUtils.remove_entry_secure(@_tmpdir)
+      ENV['HOME'] = @_home
+      Rails.env = ENV['RAILS_ENV'] = @_rails_env
+
+      page.driver.restart
+    end
+
+    config.before :suite do
+      DatabaseRewinder.clean_all
+    end
+
+    config.after :each do
+      DatabaseRewinder.clean
     end
   end
 end

data/spec/steps/ace_steps.rb

@@ -4,8 +4,7 @@ text_editor = 'テキストエディタ'
 
 step 'テキストエディタにフォーカスがあること' do
   expect(page.evaluate_script(<<-JS)).to be_true
-    $('#{NAME_INFO[text_editor][:selector]} textarea.ace_text-input').get(0) ==
-      document.activeElement
+    ace.edit('#{NAME_INFO[text_editor][:id]}').isFocused()
   JS
 end
 

data/spec/steps/text_editor_steps.rb

@@ -37,12 +37,10 @@ step 'ダウンロードが完了するまで待つ' do
   if poltergeist?
     until page.response_headers['Content-Disposition'] ||
         (start_time + 5.seconds) < Time.now
-      sleep 1
+      sleep(1)
     end
   elsif selenium?
-    until downloads_dir.exist? || (start_time + 5.seconds) < Time.now
-      sleep 1
-    end
+    sleep(1) until downloads_dir.exist? || (start_time + 5.seconds) < Time.now
   end
 end
 
@@ -66,6 +64,12 @@ step 'ダウンロードしないこと' do
   end
 end
 
+step ':name にフォーカスを移す' do |name|
+  page.execute_script(<<-JS)
+    $('#{NAME_INFO[name][:selector]}').focus()
+  JS
+end
+
 step ':name にフォーカスがあること' do |name|
   # HACK: 現在のPhantomJSでは$(':focus')は動作しない
   # https://github.com/netzpirat/guard-jasmine/issues/48
@@ -75,6 +79,8 @@ step ':name にフォーカスがあること' do |name|
 end
 
 step ':filename をロードする' do |filename|
+  step '"ロードボタン" にフォーカスを移す'
+
   # HACK: input#load-file[type="file"]は非表示要素であるため、.show()し
   #   ないと見つけられずattach_fileが失敗する
   page.execute_script("$('#load-file').show()")
@@ -123,6 +129,7 @@ step '警告ダイアログの :name ボタンをクリックする' do |name|
 end
 
 step '確認メッセージ :message を表示すること' do |message|
+  message.gsub!('\n', "\n")
   if poltergeist?
     actual = page.evaluate_script('window.confirmMsg')
     page.execute_script('window.confirmMsg = null')
@@ -158,3 +165,19 @@ step '実際にはファイルをロードしないようにしておく' do
     JS
   end
 end
+
+step 'ホームディレクトリに :program という内容の :filename が存在する' do |program, filename|
+  File.open(Pathname("~/#{filename}").expand_path, 'w') do |f|
+    f.write(program)
+  end
+end
+
+step 'ホームディレクトリに :filename が存在すること' do |filename|
+  path = Pathname("~/#{filename}").expand_path
+  expect(path).to be_exist
+end
+
+step 'ホームディレクトリの :filename の内容が :program であること' do |filename, program|
+  path = Pathname("~/#{filename}").expand_path
+  expect(path.read).to eq(program)
+end

data/spec/support/assets.rb

@@ -0,0 +1,18 @@
+# -*- coding: utf-8 -*-
+
+# assetsのキャッシュを削除する
+#
+# ただし、このメソッドを呼び出した後で page.driver.restart を実行しなけ
+# ればブラウザ側のキャッシュが有効になったままとなる。
+def expire_assets_cache
+  env = Rails.application.assets
+  if Rails.application.config.cache_classes
+    env.instance_variable_set('@assets', {})
+    env.instance_variable_set('@digests', {})
+    env = env.instance_variable_get('@environment')
+  end
+  env.send(:expire_index!)
+  Dir.glob(Rails.root.join('tmp/cache/assets/test/sprockets/*')) { |f|
+    FileUtils.remove_entry_secure(f)
+  }
+end

data/spec/support/env.rb

@@ -0,0 +1,15 @@
+# -*- coding: utf-8 -*-
+
+require 'tmpdir'
+
+# Rails.envをstandaloneにする
+shared_context 'set standalone rails env', set_standalone_rails_env: true do
+  before do
+    @_rails_env = Rails.env
+    Rails.env = ENV['RAILS_ENV'] = 'standalone'
+  end
+
+  after do
+    Rails.env = ENV['RAILS_ENV'] = @_rails_env
+  end
+end

data/spec/turnip_helper.rb

@@ -0,0 +1,2 @@
+load 'spec/steps/global_variable.rb', true
+Dir.glob('spec/steps{,/**}/*_steps.rb') { |f| load f, true }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: smalruby-editor
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.7
 platform: ruby
 authors:
 - Kouji Takao
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-24 00:00:00.000000000 Z
+date: 2014-01-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -246,7 +246,7 @@ files:
 - app/assets/images/.keep
 - app/assets/images/favicon.ico
 - app/assets/javascripts/application.js
-- app/assets/javascripts/editor.js.coffee
+- app/assets/javascripts/editor.js.coffee.erb
 - app/assets/stylesheets/application.css
 - app/assets/stylesheets/editor.css.scss
 - app/assets/stylesheets/flatstrap-custom.css.scss
@@ -310,6 +310,7 @@ files:
 - public/robots.txt
 - smalruby-editor.gemspec
 - spec/acceptance/readme.md
+- spec/acceptance/standalone/save.feature
 - spec/acceptance/text_editor/base.feature
 - spec/acceptance/text_editor/check.feature
 - spec/acceptance/text_editor/load.feature
@@ -325,16 +326,19 @@ files:
 - spec/steps/ace_steps.rb
 - spec/steps/global_variable.rb
 - spec/steps/text_editor_steps.rb
+- spec/support/assets.rb
 - spec/support/capybara.rb
+- spec/support/env.rb
+- spec/turnip_helper.rb
 - vendor/assets/javascripts/.keep
 - vendor/assets/stylesheets/.keep
-- public/assets/application-abb819194306ad9fdda88e33fda0b06c.js
-- public/assets/application-abb819194306ad9fdda88e33fda0b06c.js.gz
+- public/assets/application-759ce4d1ac1e0e3991e6d350c8f98fc5.js
+- public/assets/application-759ce4d1ac1e0e3991e6d350c8f98fc5.js.gz
 - public/assets/application-f51ea0e777d97f12a8ce84308f31eb57.css
 - public/assets/application-f51ea0e777d97f12a8ce84308f31eb57.css.gz
 - public/assets/favicon-c7ae857bb9d06de8742ae2d337157e83.ico
 - public/assets/loading-e8e6dd7833131c92a6c3b9c8ccc6a6ac.gif
-- public/assets/manifest-0d0a1d57ad92e968912e72263d610a6e.json
+- public/assets/manifest-3e334cebc159781a2b475049ad366f6a.json
 - public/assets/progressbar-82023a146fba2a0f6d925629ed2b09c5.gif
 homepage: https://github.com/smalruby/smalruby-editor
 licenses:
@@ -395,6 +399,7 @@ specification_version: 4
 summary: A visual programming editor.
 test_files:
 - spec/acceptance/readme.md
+- spec/acceptance/standalone/save.feature
 - spec/acceptance/text_editor/base.feature
 - spec/acceptance/text_editor/check.feature
 - spec/acceptance/text_editor/load.feature
@@ -410,4 +415,7 @@ test_files:
 - spec/steps/ace_steps.rb
 - spec/steps/global_variable.rb
 - spec/steps/text_editor_steps.rb
+- spec/support/assets.rb
 - spec/support/capybara.rb
+- spec/support/env.rb
+- spec/turnip_helper.rb

data/app/assets/javascripts/editor.js.coffee

@@ -1,105 +0,0 @@
-# Place all the behaviors and hooks related to the matching controller here.
-# All this logic will automatically be available in application.js.
-# You can use CoffeeScript in this file: http://coffeescript.org/
-
-changed = false
-
-$ ->
-  saving = false
-
-  textEditor = ace.edit('text-editor')
-  textEditor.setTheme('ace/theme/github')
-  textEditor.setShowInvisibles(true)
-
-  textEditor.focus()
-  textEditor.gotoLine(0, 0)
-
-  textEditor.on('change', (e) ->
-    changed = true
-  )
-
-  session = textEditor.getSession()
-  session.setMode('ace/mode/ruby')
-  session.setTabSize(2)
-  session.setUseSoftTabs(true)
-
-  # FIXME: エディタの切り替え機能を実装するときに以下の処理を削除する。
-  #   なお、navbarの中のリンクの見た目をよくするためにa要素を使っているのだが、
-  #   デフォルトではRubyをクリックするとリンク先に遷移してしまう。そこでリンク
-  #   をクリックしてもリンク先に遷移しないようにしている。
-  $('ul.nav li a').click (e) ->
-    e.preventDefault()
-    false
-
-  $('#check-button').click (e) ->
-    e.preventDefault()
-    data =
-      source_code:
-        data: session.getDocument().getValue()
-    success = (data, textStatus, jqXHR) ->
-      for errorInfo in data
-        do (errorInfo) ->
-          msg = $('<div class="alert alert-error" style="display: none">')
-            .append('<button type="button" class="close" data-dismiss="alert">×</button>')
-            .append('<h4><i class="icon-exclamation-sign"></i>エラー</h4>')
-            .append("#{errorInfo.row}行")
-          if errorInfo.column > 0
-            msg.append("、#{errorInfo.column}文字")
-          msg.append(": #{errorInfo.message}")
-          $('#messages').append(msg)
-          msg.fadeIn('slow')
-    $.post('/source_codes/check', data, success, 'json')
-
-  $('#save-button').click (e) ->
-    e.preventDefault()
-    filename = $.trim($('#filename').val())
-    if filename.length <= 0
-      $('#filename').focus()
-    else
-      data =
-        source_code:
-          filename: filename
-          data: session.getDocument().getValue()
-      success = (data, textStatus, jqXHR) ->
-        saving = true
-        changed = false
-        $('#download-link').click()
-      $.post('/source_codes/', data, success, 'json')
-
-  $('#filename').keypress (e) ->
-    e = window.event if !e
-    if e.keyCode == 13
-      $('#save-button').click()
-      false
-    else
-      true
-
-  $('#load-button').click (e) ->
-    e.preventDefault()
-    if changed
-      return unless confirm('まだセーブしていないのでロードするとプログラムが消えてしまうよ！それでもロードしますか？')
-    $(@).parent().find('input[name="source_code[file]"]').click()
-
-  $('#file-form').fileupload(
-    dataType: 'json'
-    done: (e, data) ->
-      info = data.result.source_code
-      if info.error
-        msg = $('<div class="alert alert-error" style="display: none">')
-          .append('<button type="button" class="close" data-dismiss="alert">×</button>')
-          .append('<h4><i class="icon-exclamation-sign"></i>エラー</h4>')
-          .append(info.filename + 'は' + info.error)
-        $('#messages').append(msg)
-        msg.fadeIn('slow').delay(5000).fadeOut('slow')
-      else
-        $('#filename').val(info.filename)
-        session.getDocument().setValue(info.data)
-        changed = false
-  )
-
-  window.onbeforeunload = (event) ->
-    if !saving && session.getDocument().getValue().length > 0
-      return '作成中のプログラムが消えてしまいます。'
-    else
-      saving = false
-      return