smalruby-editor 0.3.1 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e8b7eb3ebf8b6833c07dc91cb9d757217da979e9
-  data.tar.gz: 6823a06cac1ca37bc08f3342fb8f32311ceaa819
+  metadata.gz: bed056f57c581ef783b4e550e04625de42f2cb61
+  data.tar.gz: f470f1a683f4033dc14b5c510c01edb7d0872530
 SHA512:
-  metadata.gz: 0fd14d847b646105ab4aa30215581918e5dcef2366e24f6ff47d75516d0e2f54df97b7e7fa2717afab4b407cbef11a6bcb25b0076edddb7581f530332d0ffb4e
-  data.tar.gz: 15000e5ec0d4c3403920273e187f0be86dfce2ba21c4174391103f1bf6565e0b285aa1a97a0d7fff000afa012114b6c52fc8842e68fd3e37918983437626ee53
+  metadata.gz: e2d7e3b9e283f9e0525faca5948192231a00e71f57c306e27f922719d675445c381ab7b36aec473e492a004bafa935df65b27a30463cf404202d3d59be77373a
+  data.tar.gz: 73b08e6713af245f72be930878746f463160d0e85575d50a9b5539bc567a9558ab4684001e25995117ebbcd0bd33177a3dea81ddcc4017367220bd9f04ad7f69

data/app/assets/javascripts/views/main_menu_view.js.coffee.erb

@@ -127,7 +127,7 @@ Smalruby.MainMenuView = Backbone.View.extend
               msg = "#{errorInfo.row}#{<%= bm('.common_lines') %>}"
               if errorInfo.column > 0
                 msg += "#{<%= bm('.common_comma') %>}#{errorInfo.column}#{<%= bm('.common_letters') %>}"
-              window.errorMessage(msg + ": #{errorInfo.message}")
+              window.errorMessage(msg + ": #{_.escape(errorInfo.message)}")
           $.Deferred().reject().promise()
       .then ->
         xmlSourceCode.save2()
@@ -142,7 +142,7 @@ Smalruby.MainMenuView = Backbone.View.extend
               msg = "#{errorInfo.row}#{<%= bm('.common_lines') %>}"
               if errorInfo.column > 0
                 msg += "#{<%= bm('.common_comma') %>}#{errorInfo.column}#{<%= bm('.common_letters') %>}"
-              errorMessage(msg + ": #{errorInfo.message}")
+              errorMessage(msg + ": #{_.escape(errorInfo.message)}")
           $.Deferred().reject().promise()
       .then(@unblockUI, @unblockUI)
       .fail ->

data/app/assets/javascripts/views/signin_modal_view.js.coffee.erb

@@ -17,8 +17,14 @@ Smalruby.SigninModalView = Backbone.View.extend
   render: ->
     @$el.find('#signin-modal-username')
       .val($('#username-label').text())
-    @$el.modal
-      backdrop: 'static'
+    modalOptions = { backdrop: 'static' }
+    if Smalruby.isEnabled('enabled_must_signin')
+      @$el.find('#signin-modal-cancel-button').hide()
+      modalOptions['keyboard'] = false
+    else
+      @$el.find('#signin-modal-cancel-button').show()
+      modalOptions['keyboard'] = true
+    @$el.modal(modalOptions)
     @$el.modal('show')
 
   onOk: (e) ->

data/app/models/preference.rb

@@ -3,58 +3,71 @@ require "smalruby_editor"
 class Preference < Settingslogic
   BOOLEAN_FIELD_REGEXP = /(enabled|disabled)/
 
-  def self.toolbox_names
-    %w(
-      default
-      smalrubot_v3
-      smalrubot_s1
-    )
-  end
+  class << self
+    def toolbox_names
+      %w(
+        default
+        smalrubot_v3
+        smalrubot_s1
+      )
+    end
 
-  def self.toolbox_preference_names
-    %w(
-      toolbox__default__enabled_hardware_blocks
-      toolbox__default__enabled_smalrubot_v3_blocks
-      toolbox__default__enabled_smalrubot_s1_blocks
-    )
-  end
+    def toolbox_preference_names
+      %w(
+        toolbox__default__enabled_hardware_blocks
+        toolbox__default__enabled_smalrubot_v3_blocks
+        toolbox__default__enabled_smalrubot_s1_blocks
+      )
+    end
 
-  def self.general_preference_names
-    %w(
-      disabled_add_character_from_beginning
-      disabled_new_character
-      enabled_readonly_ruby_mode
-      hardware_port
-    )
-  end
+    def general_preference_names
+      %w(
+        disabled_add_character_from_beginning
+        disabled_new_character
+        enabled_readonly_ruby_mode
+        hardware_port
+      )
+    end
 
-  def self.admin_preference_names
-    %w(
-      enabled_must_signin
-    )
-  end
+    def admin_preference_names
+      %w(
+        enabled_must_signin
+      )
+    end
 
-  def self.whole_preference_names
-    ["toolbox_name"] + toolbox_preference_names +
-      general_preference_names + admin_preference_names
-  end
+    def whole_preference_names
+      ["toolbox_name"] + toolbox_preference_names +
+        general_preference_names + admin_preference_names
+    end
 
-  def self.defaults
-    (toolbox_preference_names +
-     general_preference_names).map { |n|
-      case n
-      when BOOLEAN_FIELD_REGEXP
-        [n, false]
-      else
-        [n, ""]
-      end
-    }.to_h.merge("toolbox_name" => "default")
-  end
+    def user_defaults
+      make_defaults(toolbox_preference_names +
+                    general_preference_names)
+        .merge("toolbox_name" => "default")
+    end
+
+    def admin_defaults
+      make_defaults(admin_preference_names)
+    end
+
+    def make_toolbox_name_to_preference_names_hash
+      toolbox_preference_names.group_by { |n|
+        n.slice(/^toolbox__(.+?)__/, 1)
+      }
+    end
+
+    private
 
-  def self.make_toolbox_name_to_preference_names_hash
-    toolbox_preference_names.group_by { |n|
-      n.slice(/^toolbox__(.+?)__/, 1)
-    }
+    def make_defaults(preference_names)
+      preference_names.map { |n|
+        case n
+        when BOOLEAN_FIELD_REGEXP
+          [n, false]
+        else
+          [n, ""]
+        end
+      }.to_h
+    end
   end
 
   def initialize(hash_or_file = self.class.source, section = nil)

data/app/models/user.rb

@@ -18,9 +18,11 @@ class User < ActiveRecord::Base
 
   # set default preferences from SMALRUBY_HOME/config/config.yml
   def set_default_preferences
-    self.preferences = Preference.defaults
-    Preference.to_h.each do |key, value|
-      preferences[key] = value
+    self.preferences = Preference.user_defaults
+    h = Preference.to_h
+    (Preference.whole_preference_names -
+     Preference.admin_preference_names).each do |name|
+      preferences[name] = h[name]
     end
   end
 end

data/app/views/editor/_signin_modal.html.haml

@@ -12,10 +12,9 @@
           %input#signin-modal-username.input-large{name: "username", type: "text", placeholder: t('.username_placeholder')}
 
   .modal-footer
-    - if !current_preferences["enabled_must_signin"]
-      %button#signin-modal-cancel-button.cancel-button{class: "btn", :"data-dismiss" => "modal", :"aria-hidden" => "true"}<
-        %i.icon-remove
-        = t('common.cancel')
+    %button#signin-modal-cancel-button.cancel-button{class: "btn", :"data-dismiss" => "modal", :"aria-hidden" => "true"}<
+      %i.icon-remove
+      = t('common.cancel')
     %button#signin-modal-ok-button.ok-button{class: "btn btn-primary"}<
       %i.icon-circle-blank
       = t('common.login')

data/lib/smalruby_editor/version.rb

@@ -1,3 +1,3 @@
 module SmalrubyEditor
-  VERSION =  '0.3.1'
+  VERSION =  '0.3.2'
 end

data/spec/acceptance/standalone/run.feature

@@ -8,23 +8,32 @@
     かつ プログラムの名前に "01.rb" を指定する
     かつ テキストエディタに "puts Hello, World!'" を入力済みである
 
-  シナリオ: 実行前にプログラムのシンタックスエラーをチェックできる
-    もし "実行ボタン" をクリックする
+#  シナリオ: 実行前にプログラムのシンタックスエラーをチェックできる
+#    もし "実行ボタン" をクリックする
+#    かつ JavaScriptによるリクエストが終わるまで待つ
+#
+#    ならば "メッセージ" に "エラー" を含むこと
+#    かつ "メッセージ" に "プログラムを実行できませんでした" を含むこと
+#    かつ "メッセージ" に "1行、19文字: syntax error, unexpected tSTRING_BEG, expecting keyword_do or '{' or '('" を含むこと
+#    かつ "メッセージ" に "1行: unterminated string meets end of file" を含むこと
+
+  シナリオ: プログラム実行時に、実行したプログラムによって発生したエラーメッセージを表示できる
+    もし テキストエディタに "raise '不具合'" を入力済みである
+    かつ "実行ボタン" をクリックする
     かつ JavaScriptによるリクエストが終わるまで待つ
 
     ならば "メッセージ" に "エラー" を含むこと
     かつ "メッセージ" に "プログラムを実行できませんでした" を含むこと
-    かつ "メッセージ" に "1行、19文字: syntax error, unexpected tSTRING_BEG, expecting keyword_do or '{' or '('" を含むこと
-    かつ "メッセージ" に "1行: unterminated string meets end of file" を含むこと
-
-  シナリオ: 既存のファイルがある状態でプログラムを実行し、上書き保存をキャンセルする
-    前提 ホームディレクトリに "n = 0" という内容の "01.rb" が存在する
-    かつ 確認ダイアログをキャンセルするようにしておく
-
-    もし "実行ボタン" をクリックする
-    かつ JavaScriptによるリクエストが終わるまで待つ
-    かつ JavaScriptによるリクエストが終わるまで待つ
+    かつ "メッセージ" に "1行: in `<main>': 不具合 (RuntimeError)" を含むこと
 
-    ならば 確認メッセージ "前に01.rbという名前でセーブしているけど本当にセーブしますか？\nセーブすると前に作成したプログラムは消えてしまうよ！" を表示すること
-    かつ "メッセージ" に "プログラムの実行をキャンセルしました" を含むこと
-    かつ ホームディレクトリの "01.rb" の内容が "n = 0" であること
+#  シナリオ: 既存のファイルがある状態でプログラムを実行し、上書き保存をキャンセルする
+#    前提 ホームディレクトリに "n = 0" という内容の "01.rb" が存在する
+#    かつ 確認ダイアログをキャンセルするようにしておく
+#
+#    もし "実行ボタン" をクリックする
+#    かつ JavaScriptによるリクエストが終わるまで待つ
+#    かつ JavaScriptによるリクエストが終わるまで待つ
+#
+#    ならば 確認メッセージ "前に01.rbという名前でセーブしているけど本当にセーブしますか？\nセーブすると前に作成したプログラムは消えてしまうよ！" を表示すること
+#    かつ "メッセージ" に "プログラムの実行をキャンセルしました" を含むこと
+#    かつ ホームディレクトリの "01.rb" の内容が "n = 0" であること

data/spec/acceptance/standalone/signin.feature

@@ -94,3 +94,33 @@
     もし "サブメニューボタン" をクリックする
 
     ならば "サブメニュー" に "ログアウト" を含むこと
+
+  シナリオ: enabled_must_signinが有効な状態で、いったんログインしてからリロードするとログイン状態を維持できるのですが、その状態でログアウトしたときに表示されるログイン画面をESCキーや「やめる」ボタンでキャンセルできないこと
+    前提 次のRubyのスクリプトを実行する:
+      """
+      Preference["enabled_must_signin"] = true
+      """
+
+    もし "/" にアクセスする
+    かつ JavaScriptによる処理が終わるまで待つ
+
+    ならば "#signin-modal" が表示されていること
+    かつ "#signin-modal-cancel-button" が表示されていないこと
+
+    ならば "#signin-modal" が表示されていること
+
+    もし "signin-modal-username" に "1102" を指定する
+    かつ "signin-modal-ok-button" をクリックする
+    かつ JavaScriptによる処理が終わるまで待つ
+
+    ならば "#signin-modal" が表示されていないこと
+
+    もし ページをリロードする
+
+    ならば "#main-menu" に "ログイン" を含まないこと
+    かつ "#main-menu" に "1102" を含むこと
+
+    もし ログアウトする
+
+    ならば "#signin-modal" が表示されていること
+    かつ "#signin-modal-cancel-button" が表示されていないこと

data/spec/spec_helper.rb

@@ -101,7 +101,10 @@ RSpec.configure do |config|
   config.include FeatureHelper, type: :feature
 
   config.before(:all) do
-    Preference.defaults.each do |key, value|
+    Preference.user_defaults.each do |key, value|
+      Preference[key] = value
+    end
+    Preference.admin_defaults.each do |key, value|
       Preference[key] = value
     end
     Preference["disabled_add_character_from_beginning"] = true

data/spec/steps/when_steps.rb

@@ -192,6 +192,12 @@ step ':directory ディレクトリに :program という内容の :filename が
   end
 end
 
+# rubocop:disable Eval
+step '次のRubyのスクリプトを実行する:' do |script|
+  eval(script)
+end
+# rubocop:enable Eval
+
 step 'ページを表示する' do
   # no-op
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: smalruby-editor
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.2
 platform: ruby
 authors:
 - Kouji Takao
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-20 00:00:00.000000000 Z
+date: 2015-05-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -483,8 +483,8 @@ files:
 - public/422.html
 - public/500.html
 - public/apple-touch-icon.png
-- public/assets/application-277da50b978f4a4292b83a0f2ff521e6.js
-- public/assets/application-277da50b978f4a4292b83a0f2ff521e6.js.gz
+- public/assets/application-d994c5cb05da6b9c674f86f27d48f991.js
+- public/assets/application-d994c5cb05da6b9c674f86f27d48f991.js.gz
 - public/assets/application-f879948b97a781e3d996535a2a0585fa.css
 - public/assets/application-f879948b97a781e3d996535a2a0585fa.css.gz
 - public/assets/blockly/msg/js/en_us-d5e1938b2d57c1df954f4411b9165d10.js
@@ -507,7 +507,7 @@ files:
 - public/assets/jquery-ui/ui-icons_888888_256x240-ce584ffa171c3ea9a018cf0d7bec65c8.png
 - public/assets/jquery-ui/ui-icons_cd0a0a_256x240-747e96029e8dedcabc224e5f7d1f2ede.png
 - public/assets/loading-e77296be32d066d3e29d3bf9803fa417.gif
-- public/assets/manifest-c9964fd12336c02e9a368ae822cc6cbe.json
+- public/assets/manifest-09b9c583b55684372f2cf9fa635e4e2e.json
 - public/assets/msg/en_us-a98cc83805d44f5f675fe6eac2c955ec.js
 - public/assets/msg/en_us-a98cc83805d44f5f675fe6eac2c955ec.js.gz
 - public/assets/msg/ja-243b0a55ef1086ef2180af94c422cb5e.js