daigaku 0.3.0 → 0.4.0

data/lib/daigaku/terminal/solutions.rb

@@ -1,46 +1,41 @@
+require 'os'
+require_relative 'output'
+
 module Daigaku
   module Terminal
-
-    require 'os'
-    require_relative 'output'
-
     class Solutions < Thor
       include Terminal::Output
 
       desc 'solutions open [COURSE NAME]', 'Open the solutions folder of a course in a GUI window'
       def open(course_name = '')
-        begin
-          path = File.join(Daigaku.config.solutions_path, course_name)
-
-          unless Dir.exist?(path)
-            text = [
-              "The course directory \"#{File.basename(path)}\" is not available in",
-              "\"#{File.dirname(path)}\".\n",
-              'Hint:',
-              'Run "daigaku scaffold" to create empty solution files for all courses.'
-            ]
-            say_warning text.join("\n")
-
-            unless Loading::Courses.load(Daigaku.config.courses_path).empty?
-              Terminal::Courses.new.list
-            end
-
-            return
+        path = File.join(Daigaku.config.solutions_path, course_name)
+
+        unless Dir.exist?(path)
+          text = [
+            "The course directory \"#{File.basename(path)}\" is not available in",
+            "\"#{File.dirname(path)}\".\n",
+            'Hint:',
+            'Run "daigaku scaffold" to create empty solution files for all courses.'
+          ]
+          say_warning text.join("\n")
+
+          unless Loading::Courses.load(Daigaku.config.courses_path).empty?
+            Terminal::Courses.new.list
           end
 
-          if OS.windows?
-            system "explorer '#{path}'"
-          elsif OS.mac?
-            system "open '#{path}'"
-          elsif OS.linux?
-            system "xdg-open '#{path}'"
-          end
-        rescue ConfigurationError => e
-          say_warning e.message
+          return
         end
-      end
 
+        if OS.windows?
+          system "explorer '#{path}'"
+        elsif OS.mac?
+          system "open '#{path}'"
+        elsif OS.linux?
+          system "xdg-open '#{path}'"
+        end
+      rescue ConfigurationError => e
+        say_warning e.message
+      end
     end
-
   end
 end

data/lib/daigaku/terminal/welcome.rb

@@ -1,22 +1,21 @@
 module Daigaku
   module Terminal
-
     class Welcome
       include Terminal::Output
 
       def self.run
-        self.new.run
+        new.run
       end
 
       def self.about
-        self.new.about
+        new.about
       end
 
       def run
         empty_line
         say Terminal.text :welcome
         empty_line
-        say "For now, let's setup the daigaku paths."
+        say 'For now, let’s setup the daigaku paths.'
         Daigaku::Terminal::Setup.new.init
 
         show_setup_list_announcement
@@ -37,7 +36,7 @@ module Daigaku
         empty_line
         say Terminal.text :about
         empty_line
-        say %x{daigaku help}
+        say `daigaku help`
       end
 
       private
@@ -45,7 +44,7 @@ module Daigaku
       def show_setup_list_announcement
         command = 'daigaku setup list'
         text = [
-          "The courses path and solutions path have been added to your settings.",
+          'The courses path and solutions path have been added to your settings.',
           "Just list your current settings with the \"#{command}\" command:"
         ].join("\n")
 
@@ -56,7 +55,7 @@ module Daigaku
         command = 'daigaku courses list'
         text = [
           "Well done. Now, type \"#{command}\" to see what courses are",
-          "available in your daigaku folder:"
+          'available in your daigaku folder:'
         ].join("\n")
 
         get_command(command, text)
@@ -65,7 +64,7 @@ module Daigaku
       def show_courses_download_announcement
         command = 'daigaku courses download'
         text = [
-          "Oh! You don't have any courses, yet?",
+          'Oh! You don’t have any courses, yet?',
           "Just enter \"#{command}\" to download the basic Daigaku course:"
         ].join("\n")
 
@@ -75,7 +74,7 @@ module Daigaku
       def show_solutions_open_announcement
         command = 'daigaku solutions open'
         text = [
-          "When downloading a course, Daigaku scaffolds empty solution files",
+          'When downloading a course, Daigaku scaffolds empty solution files',
           "for your code on the fly.\n",
           "Type \"#{command}\" to open your solutions folder:"
         ].join("\n")
@@ -86,13 +85,12 @@ module Daigaku
       def show_learn_announcement
         command = 'daigaku learn'
         text = [
-          "Congratulations! You learned the first steps of using daigaku.",
+          'Congratulations! You learned the first steps of using daigaku.',
           "To continue and start learning Ruby type \"#{command}\":"
         ].join("\n")
 
         get_command(command, text)
       end
     end
-
   end
 end

data/lib/daigaku/test.rb

@@ -1,25 +1,24 @@
-module Daigaku
-  require 'fileutils'
+require 'fileutils'
 
+module Daigaku
   class Test
+    CODE_REGEX = /\[\['solution::code'\]\]/
 
     attr_reader :path
 
-    CODE_REGEX = /\[\['solution::code'\]\]/
-
     def initialize(path)
       @unit_path = path
-      @path = Dir[File.join(path, '*spec.rb')].first
+      @path      = Dir[File.join(path, '*spec.rb')].first
     end
 
     def run(solution_code)
-      spec_code = File.read(@path)
+      spec_code         = File.read(@path)
       patched_spec_code = insert_code(spec_code, solution_code.to_s)
 
       temp_spec = File.join(File.dirname(@path), "temp_#{File.basename(@path)}")
       create_temp_spec(temp_spec, patched_spec_code)
 
-      result = %x{ rspec --color --format j #{temp_spec} }
+      result = `rspec --color --format j #{temp_spec}`
       remove_file(temp_spec)
 
       TestResult.new(result)
@@ -33,14 +32,12 @@ module Daigaku
 
     def create_temp_spec(path, content)
       base_path = File.dirname(path)
-      FileUtils.mkdir_p(base_path) unless Dir.exist?(base_path)
+      FileUtils.makedirs(base_path) unless Dir.exist?(base_path)
       File.open(path, 'w') { |f| f.puts content }
     end
 
     def remove_file(path)
       FileUtils.rm(path) if File.exist?(path)
     end
-
   end
-
 end

data/lib/daigaku/test_result.rb

@@ -1,8 +1,7 @@
-module Daigaku
+require 'json'
 
+module Daigaku
   class TestResult
-    require 'json'
-
     attr_reader :examples, :example_count, :failure_count
 
     def initialize(result_json)
@@ -17,9 +16,9 @@ module Daigaku
 
       @examples = @result[:examples].map do |example|
         description = example[:full_description]
-        status = example[:status]
-        exception = example[:exception]
-        message = exception ? exception[:message] : nil
+        status      = example[:status]
+        exception   = example[:exception]
+        message     = exception ? exception[:message] : nil
 
         TestExample.new(description: description, status: status, message: message)
       end
@@ -47,10 +46,10 @@ module Daigaku
 
     def build_failed_summary
       message = examples.map do |example|
-        "#{example.description}\n#{example.status}: #{example.message}"
+        "#{example.description}\n#{example.status}: #{example.message}".strip
       end
 
-      summary = message.map(&:strip).join("\n" * 3)
+      message.join("\n" * 3)
     end
 
     def syntax_error_json
@@ -59,7 +58,7 @@ module Daigaku
         examples: [
           {
             status: 'failed',
-            exception: { message: ":( You got an error in your code!" }
+            exception: { message: ':( You got an error in your code!' }
           }
         ]
       }
@@ -67,20 +66,18 @@ module Daigaku
   end
 
   class TestExample
-
     attr_reader :description, :status, :message
 
-    EXAMPLE_PASSED_MESSAGE = "Your code passed this requirement."
+    EXAMPLE_PASSED_MESSAGE = 'Your code passed this requirement.'.freeze
 
-    def initialize(args = {})
-      @description = args[:description]
-      @status = args[:status]
-      @message = args[:message] || EXAMPLE_PASSED_MESSAGE
+    def initialize(description:, status:, message: nil)
+      @description = description
+      @status      = status
+      @message     = message || EXAMPLE_PASSED_MESSAGE
     end
 
     def passed?
       @status == 'passed'
     end
   end
-
 end

data/lib/daigaku/unit.rb

@@ -1,10 +1,9 @@
 module Daigaku
   class Unit
-
     attr_reader :title, :task
 
     def initialize(path)
-      @path = path
+      @path  = path
       @title = File.basename(path).gsub(/\_+/, ' ')
     end
 
@@ -23,6 +22,5 @@ module Daigaku
     def mastered?
       solution.verified?
     end
-
   end
 end

data/lib/daigaku/version.rb

@@ -1,3 +1,3 @@
 module Daigaku
-  VERSION = '0.3.0'
+  VERSION = '0.4.0'.freeze
 end

data/lib/daigaku/views.rb

@@ -23,7 +23,7 @@ module Daigaku
         curs_set(0) # invisible cursor
 
         height ||= lines
-        width ||= cols + 1
+        width  ||= cols + 1
 
         window = Daigaku::Window.new(height, width, top, left)
 
@@ -40,12 +40,13 @@ module Daigaku
 
       def sub_window_below_top_bar(window, top_bar)
         top_bar.show
-        top = top_bar.height
+
+        top        = top_bar.height
         sub_window = window.subwin(window.maxy - top, window.maxx, top, 0)
+
         sub_window.keypad(true)
         sub_window
       end
     end
-
   end
 end

data/lib/daigaku/views/chapters_menu.rb

@@ -2,9 +2,7 @@ require 'daigaku/views/menu'
 
 module Daigaku
   module Views
-
     class ChaptersMenu < Menu
-
       private
 
       def before_enter(*args)
@@ -12,8 +10,8 @@ module Daigaku
       end
 
       def before_reenter(*args)
-        @course = args[0]
-        @chapter = args[1]
+        @course   = args[0]
+        @chapter  = args[1]
         @position = @course.chapters.find_index(@chapter)
       end
 
@@ -24,20 +22,20 @@ module Daigaku
       def interact_with(window)
         while char = window.getch
           case char
-            when KEY_UP
-              @position -= 1
-              broadcast(:reset_menu_position)
-            when KEY_DOWN
-              @position += 1
-              broadcast(:reset_menu_position)
-            when 10 # Enter
-              broadcast(:enter, @course, models[@position])
-              return
-            when 263 # Backspace
-              broadcast(:reenter, @course)
-              return
-            when 27 # ESC
-              exit
+          when KEY_UP
+            @position -= 1
+            broadcast(:reset_menu_position)
+          when KEY_DOWN
+            @position += 1
+            broadcast(:reset_menu_position)
+          when 10 # Enter
+            broadcast(:enter, @course, models[@position])
+            return
+          when 263 # Backspace
+            broadcast(:reenter, @course)
+            return
+          when 27 # ESC
+            exit
           end
 
           @position = items.length - 1 if @position < 0
@@ -53,8 +51,6 @@ module Daigaku
       def items
         models.map(&:title)
       end
-
     end
-
   end
 end

data/lib/daigaku/views/courses_menu.rb

@@ -2,9 +2,7 @@ require 'daigaku/views/menu'
 
 module Daigaku
   module Views
-
     class CoursesMenu < Menu
-
       private
 
       def header_text
@@ -14,17 +12,17 @@ module Daigaku
       def interact_with(window)
         while char = window.getch
           case char
-            when KEY_UP
-              @position -= 1
-              broadcast(:reset_menu_position)
-            when KEY_DOWN
-              @position += 1
-              broadcast(:reset_menu_position)
-            when 10 # Enter
-              broadcast(:enter, models[@position])
-              return
-            when 27 # ESC
-              exit
+          when KEY_UP
+            @position -= 1
+            broadcast(:reset_menu_position)
+          when KEY_DOWN
+            @position += 1
+            broadcast(:reset_menu_position)
+          when 10 # Enter
+            broadcast(:enter, models[@position])
+            return
+          when 27 # ESC
+            exit
           end
 
           @position = items.length - 1 if @position < 0
@@ -41,12 +39,11 @@ module Daigaku
         non_empty_courses = models.select { |course| !course.chapters.empty? }
 
         non_empty_courses.map do |course|
-          line = "#{course.title}"
+          line = course.title
           self.items_info <<= [(course.author ? "(by #{course.author})" : '')]
           line
         end
       end
     end
-
   end
 end

data/lib/daigaku/views/main_menu.rb

@@ -1,37 +1,37 @@
+require_relative 'subscriber'
+
 module Daigaku
   module Views
-
+    # Subscription: `first.subscribe(second)` means
+    # first subscribes second on the first's broadcast.
+    # second has to have method that is broadcasted.
     class MainMenu
       include Views
 
-      def initialize
-        courses_menu = Views::CoursesMenu.new
-        chapters_menu = Views::ChaptersMenu.new
-        units_menu = Views::UnitsMenu.new
-        task_view = Views::TaskView.new
+      attr_reader :courses_menu, :chapters_menu, :units_menu, :task_view
 
-        # Subscription: `first.subscribe(second)` means
-        # first subscribes second on the first's broadcast.
-        # second has to have method that is broadcasted.
+      def initialize
+        @courses_menu  = Views::CoursesMenu.new
+        @chapters_menu = Views::ChaptersMenu.new
+        @units_menu    = Views::UnitsMenu.new
+        @task_view     = Views::TaskView.new
 
-        # top down navigation
-        courses_menu.subscribe(chapters_menu, on: :enter)
-        chapters_menu.subscribe(units_menu, on: :enter)
-        units_menu.subscribe(task_view, on: :enter)
+        subscribe_events
+        courses_menu.enter
+      end
 
-        # bottom up navigation
-        chapters_menu.subscribe(courses_menu, on: :reenter)
-        units_menu.subscribe(chapters_menu, on: :reenter)
-        task_view.subscribe(units_menu, on: :reenter)
+      private
 
-        # position reset
-        courses_menu.subscribe(chapters_menu, on: :reset_menu_position)
-        courses_menu.subscribe(units_menu, on: :reset_menu_position)
-        chapters_menu.subscribe(units_menu, on: :reset_menu_position)
+      def subscribe_events
+        subscriber = Subscriber.new(
+          courses_menu:  courses_menu,
+          chapters_menu: chapters_menu,
+          units_menu:    units_menu,
+          task_view:     task_view
+        )
 
-        courses_menu.enter
+        subscriber.subscribe_events!
       end
     end
-
   end
 end