scide 0.0.6 → 0.0.7

data/Gemfile

@@ -12,6 +12,10 @@ group :development do
   gem 'shoulda'
   gem 'bundler', '~> 1.0.0'
   gem 'jeweler', '~> 1.6.4'
-  gem 'rcov', '>= 0'
+  gem 'simplecov'
   gem 'rdoc'
 end
+
+group :test do
+  gem 'active_support', '>= 2'
+end

data/Gemfile.lock

@@ -10,9 +10,9 @@ GEM
       bundler (~> 1.0)
       git (>= 1.2.5)
       rake
+    multi_json (1.0.3)
     paint (0.8.3)
     rake (0.9.2)
-    rcov (0.9.10)
     rdoc (3.9.4)
     rspec (2.6.0)
       rspec-core (~> 2.6.0)
@@ -23,6 +23,10 @@ GEM
       diff-lcs (~> 1.1.2)
     rspec-mocks (2.6.0)
     shoulda (2.11.3)
+    simplecov (0.5.3)
+      multi_json (~> 1.0.3)
+      simplecov-html (~> 0.5.3)
+    simplecov-html (0.5.3)
     upoj-rb (0.0.5)
       active_support (>= 2)
       paint
@@ -31,10 +35,11 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
+  active_support (>= 2)
   bundler (~> 1.0.0)
   jeweler (~> 1.6.4)
-  rcov
   rdoc
   rspec
   shoulda
+  simplecov
   upoj-rb (~> 0.0.4)

data/Rakefile

@@ -32,14 +32,6 @@ RSpec::Core::RakeTask.new do |t|
   # Put spec opts in a file named .rspec in root
 end
 
-#require 'rcov/rcovtask'
-#Rcov::RcovTask.new do |test|
-#  test.libs << 'test'
-#  test.pattern = 'test/**/test_*.rb'
-#  test.verbose = true
-#  test.rcov_opts << '--exclude "gems/*"'
-#end
-
 task :default => :test
 
 require 'rdoc/task'

data/TODO.md

@@ -0,0 +1,21 @@
+# TODO
+
+## Improvements/Maintenance
+
+* check format of keys, names, etc.
+* ensure that all data from configuration is duplicated
+* replace default hashes by HashWithIndifferentAccess
+* make all options/contents parameters optional
+* extract tempfile management into separate runner class
+* find a way to check whether a path is absolute (cross-platform)
+
+## New Features
+
+* allow to load configuration from hash
+* interactive configuration generation
+* save generated configurations
+
+## Specifications
+
+* find a way to stub puts (opts version & banner)
+* use rspec 2 shared examples

data/VERSION

@@ -1 +1 @@
-0.0.6
+0.0.7

data/lib/scide/commands/show.rb

@@ -17,12 +17,8 @@ module Scide
     #   ssh 127.0.0.1
     class Show < Scide::Command
 
-      def initialize contents, options = {}
-        super contents, options
-      end
-
-      # Returns a configuration fragment that will put
-      # this command GNU \Screen window without running it.
+      # Returns a configuration fragment that will show
+      # this command in a GNU \Screen window without running it.
       # This will use screen's <tt>stuff</tt> command to
       # put the text in the window.
       def to_screen

data/lib/scide/config.rb

@@ -47,14 +47,14 @@ module Scide
       Scide.fail :config_not_readable, "ERROR: configuration #{@file} is not readable" unless File.readable? @file
 
       begin
-        raw_config = File.open(@file, 'r').read
+        raw_config = load_config
       rescue StandardError => err
         Scide.fail :unexpected, "ERROR: could not read configuration #{@file}"
       end
 
       begin
-        @config = YAML::load raw_config
-      rescue StandardError => err
+        @config = parse_config raw_config
+      rescue SyntaxError => err
         Scide.fail :malformed_config, "ERROR: could not parse configuration #{@file}\n   #{err}"
       end
 
@@ -67,9 +67,9 @@ module Scide
       invalid_config 'projects configuration must be a hash' unless @config[:projects].nil? or @config[:projects].kind_of?(Hash)
 
       begin
-        @screen = @config[:screen]
+        @screen = @config[:screen] || HashWithIndifferentAccess.new
         @global = Scide::Global.new @config[:global]
-        @projects = @config[:projects].inject(HashWithIndifferentAccess.new) do |memo,obj|
+        @projects = (@config[:projects] || {}).inject(HashWithIndifferentAccess.new) do |memo,obj|
           memo[obj[0]] = Scide::Project.new @global, obj[0], obj[1]; memo
         end
       rescue ArgumentError => err
@@ -79,6 +79,16 @@ module Scide
 
     private
 
+    # Returns the contents of #file.
+    def load_config
+      File.open(@file, 'r').read
+    end
+
+    # Returns the parsed configuration.
+    def parse_config raw
+      YAML::load raw
+    end
+
     # Causes scide to fail with an <tt>invalid_config</tt> error (see Scide#fail).
     # Builds a complete error message containing the full path to the
     # configuration file and the given message.

data/lib/scide/global.rb

@@ -17,8 +17,9 @@ module Scide
     # * <tt>contents</tt> - The global options hash.
     def initialize contents
       raise ArgumentError, 'global configuration must be a hash' unless contents.kind_of? Hash
+      raise ArgumentError, 'global options must be a hash' unless contents[:options].nil? or contents[:options].kind_of?(Hash)
 
-      @options = contents[:options] || {}
+      @options = contents[:options].try(:dup) || {}
 
       # default to home directory
       @path = contents[:path].try(:to_s) || File.expand_path('~')

data/lib/scide/overmind.rb

@@ -1,3 +1,5 @@
+require 'tempfile'
+
 module Scide
 
   # Utility class to run scide in a script.
@@ -56,7 +58,12 @@ module Scide
         puts screen.to_s.gsub(/^/, '   ')
         puts
       else
-        screen.run
+        file = Tempfile.new 'scide'
+        file.write screen.to_s
+        file.rewind
+        file.close
+        system screen.to_command(file.path)
+        file.unlink
       end
     end
   end

data/lib/scide/project.rb

@@ -12,6 +12,9 @@ module Scide
     # Project-specific options. Can be used by commands. See #initialize.
     attr_reader :options
 
+    # The GNU Screen windows of this project.
+    attr_reader :windows
+
     # Returns a project configuration.
     #
     # If not given in the project hash, #path is built by joining
@@ -33,19 +36,28 @@ module Scide
     # * <tt>path</tt> - The path where the project is located.
     def initialize global, key, contents
       raise ArgumentError, "project '#{key}' must be a hash" unless contents.kind_of? Hash
-      raise ArgumentError, "windows of project '#{key}' must be an array" unless contents[:windows].nil? or contents[:windows].kind_of?(Array)
+      raise ArgumentError, "windows of project '#{key}' must be an array" unless contents[:windows].kind_of?(Array)
       raise ArgumentError, "options of project '#{key}' must be a hash" unless contents[:options].nil? or contents[:options].kind_of?(Hash)
 
-      @global, @key = global, key
+      @key = key
 
       # path defaults to project key
       @path = contents[:path].try(:to_s) || key.to_s
-      # expand from home directory if not absolute
-      @path = File.join global.path, @path unless @path.match /^\//
+      unless path.match /^\//
+        # if not absolute
+        if global.path
+          # expand from global directory
+          @path = File.join global.path, @path
+        else
+          # or from home directory
+          @path = File.join File.expand_path('~'), @path
+        end
+      end
 
-      @options = global.options.dup.merge(contents[:options] || {})
-      @options[:name] ||= key
-      @options[:path] ||= @path
+      @options = global.options.dup
+      @options[:name] = key
+      @options[:path] = @path
+      @options.merge!(contents[:options] || {})
 
       @windows = contents[:windows].collect{ |w| Scide::Window.new self, w }
     end
@@ -58,7 +70,7 @@ module Scide
       String.new.tap do |s|
         @windows.each_with_index do |w,i|
           s << w.to_screen(i)
-          s << "\n" if i != @windows.length - 1
+          s << "\n" if i != (@windows.length - 1)
         end
       end
     end

data/lib/scide/screen.rb

@@ -1,5 +1,3 @@
-require 'tempfile'
-
 module Scide
 
   # Configuration of a GNU Screen session (windows for a specific project).
@@ -11,6 +9,9 @@ module Scide
     # The default screen hardstatus line.
     DEFAULT_HARDSTATUS = '%{= kG}[ %{G}%H %{g}][%= %{=kw}%?%-Lw%?%{r}(%{W}%n*%f%t%?(%u)%?%{r})%{w}%?%+Lw%?%?%= %{g}][%{B}%Y-%m-%d %{W}%c %{g}]'
 
+    # Options for this screen.
+    attr_accessor :options
+
     # Returns a screen configuration for the given project.
     #
     # ==== Arguments
@@ -25,17 +26,7 @@ module Scide
       raise ArgumentError, 'screen configuration must be a hash' unless options.nil? or options.kind_of?(Hash)
 
       @project = project
-      @options = options || {}
-    end
-
-    # Runs screen with this configuration.
-    #
-    # The configuration is saved to a temporary file, then removed.
-    def run
-      file = Tempfile.new 'scide'
-      save file.path
-      system to_command(file.path)
-      file.unlink
+      @options = options.try(:dup) || {}
     end
 
     # Returns the command that will be used to run screen with this configuration.
@@ -44,7 +35,7 @@ module Scide
     # * <tt>tmp_file</tt> - The temporary file in which the configuration will be stored.
     #   (Optional for dry-run.)
     def to_command tmp_file = 'TEMPORARY_FILE'
-      "cd #{@project.path} && #{binary} #{args} -c #{tmp_file}"
+      [ "cd #{@project.path} &&", binary, args, "-c #{tmp_file}" ].select(&:present?).join(' ')
     end
 
     # Verifies that the screen binary is there. If not, causes scide
@@ -64,28 +55,21 @@ module Scide
       end
     end
 
-    private
-
     # Returns the screen hardstatus line given as option, or
     # the default #DEFAULT_HARDSTATUS.
     def hardstatus
-      @options[:hardstatus] || DEFAULT_HARDSTATUS
+      @options[:hardstatus].try(:to_s) || DEFAULT_HARDSTATUS
     end
 
     # Returns the screen binary given as option, or the
     # default (<tt>screen</tt>).
     def binary
-      @options[:binary] || 'screen'
+      @options[:binary].try(:to_s) || 'screen'
     end
 
     # Returns the screen command-line arguments given as options.
     def args
-      @options[:args]
-    end
-
-    # Saves this configuration to the file at the given path.
-    def save file
-      File.open(file, 'w'){ |f| f.write to_s }
+      @options[:args].try(:to_s)
     end
   end
 end

data/lib/scide/window.rb

@@ -3,6 +3,13 @@ module Scide
   # Configuration of a GNU Screen window (name, command).
   class Window
 
+    # The name of the window as it will shown in GNU Screen.
+    # See #initialize.
+    attr_reader :name
+
+    # The optional command that will be shown in this window.
+    attr_reader :command
+
     # Window-specific options. Can be used by commands. See #initialize.
     attr_reader :options
     

data/lib/scide.rb

@@ -54,6 +54,12 @@ module Scide
     @@exit_on_fail = exit_on_fail
   end
 
+  # Indicates whether scide is configured to exit on failure.
+  # See Scide.exit_on_fail=.
+  def self.exit_on_fail
+    @@exit_on_fail
+  end
+
   # Scide error. Can be raised if #exit_on_fail is set to false.
   class Error < StandardError
     attr_reader :condition

data/scide.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "scide"
-  s.version = "0.0.6"
+  s.version = "0.0.7"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["AlphaHydrae"]
@@ -26,6 +26,7 @@ Gem::Specification.new do |s|
     "LICENSE.txt",
     "README.md",
     "Rakefile",
+    "TODO.md",
     "VERSION",
     "bin/scide",
     "lib/scide.rb",
@@ -47,7 +48,19 @@ Gem::Specification.new do |s|
     "spec/commands/run_spec.rb",
     "spec/commands/show_spec.rb",
     "spec/commands/tail_spec.rb",
-    "spec/helper.rb"
+    "spec/config_spec.rb",
+    "spec/global_spec.rb",
+    "spec/helper.rb",
+    "spec/opts_spec.rb",
+    "spec/project_spec.rb",
+    "spec/results/config1.yml",
+    "spec/results/malformed_config.yml",
+    "spec/results/project1.screen",
+    "spec/results/screen1.screen",
+    "spec/scide_spec.rb",
+    "spec/screen_spec.rb",
+    "spec/version_spec.rb",
+    "spec/window_spec.rb"
   ]
   s.homepage = "http://github.com/AlphaHydrae/scide"
   s.licenses = ["MIT"]
@@ -64,7 +77,7 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<shoulda>, [">= 0"])
       s.add_development_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.6.4"])
-      s.add_development_dependency(%q<rcov>, [">= 0"])
+      s.add_development_dependency(%q<simplecov>, [">= 0"])
       s.add_development_dependency(%q<rdoc>, [">= 0"])
     else
       s.add_dependency(%q<upoj-rb>, ["~> 0.0.4"])
@@ -72,7 +85,7 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<shoulda>, [">= 0"])
       s.add_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_dependency(%q<jeweler>, ["~> 1.6.4"])
-      s.add_dependency(%q<rcov>, [">= 0"])
+      s.add_dependency(%q<simplecov>, [">= 0"])
       s.add_dependency(%q<rdoc>, [">= 0"])
     end
   else
@@ -81,7 +94,7 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<shoulda>, [">= 0"])
     s.add_dependency(%q<bundler>, ["~> 1.0.0"])
     s.add_dependency(%q<jeweler>, ["~> 1.6.4"])
-    s.add_dependency(%q<rcov>, [">= 0"])
+    s.add_dependency(%q<simplecov>, [">= 0"])
     s.add_dependency(%q<rdoc>, [">= 0"])
   end
 end

data/spec/command_spec.rb

@@ -33,7 +33,7 @@ describe Scide::Command do
     
     it "should resolve command with a string" do
       com = Scide::Command.resolve @window, 'SHOW fubar'
-      com.should be_a_kind_of(Scide::Commands::Show)
+      com.should be_a(Scide::Commands::Show)
     end
 
     it "should use second part of string as contents when resolving a command with a string" do
@@ -43,7 +43,7 @@ describe Scide::Command do
 
     it "should resolve command with a hash" do
       com = Scide::Command.resolve @window, :command => 'SHOW', :contents => 'fubar'
-      com.should be_a_kind_of(Scide::Commands::Show)
+      com.should be_a(Scide::Commands::Show)
     end
 
     it "should give duplicated window options to the resolved string command" do
@@ -61,14 +61,13 @@ describe Scide::Command do
     end
 
     it "should resolve camel-case command class names with a string" do
-      puts Scide::Commands::FuBar
       com = Scide::Command.resolve @window, 'FU_BAR'
-      com.should be_a_kind_of(Scide::Commands::FuBar)
+      com.should be_a(Scide::Commands::FuBar)
     end
 
     it "should resolve camel-case command class names with a hash" do
       com = Scide::Command.resolve @window, :command => 'FU_BAR'
-      com.should be_a_kind_of(Scide::Commands::FuBar)
+      com.should be_a(Scide::Commands::FuBar)
     end
 
     it "should raise an error when type of contents is unknown" do

data/spec/config_spec.rb

@@ -0,0 +1,109 @@
+require 'helper'
+require 'fileutils'
+
+describe Scide::Config do
+
+  it "should use the default config file if not given" do
+    conf = Scide::Config.new
+    conf.file.should == Scide::Config::DEFAULT_CONFIG_FILE
+  end
+
+  it "should use the given config file" do
+    conf = Scide::Config.new '/tmp/fubar'
+    conf.file.should == '/tmp/fubar'
+  end
+
+  it "should be empty when not loaded" do
+    conf = Scide::Config.new
+    conf.screen.should be_nil
+    conf.global.should be_nil
+    conf.projects.should be_nil
+  end
+
+  it "should be filled when loaded" do
+    file = File.join File.dirname(__FILE__), 'results', 'config1.yml'
+    conf = Scide::Config.new file
+    make_load conf
+    conf.screen.should be_a(HashWithIndifferentAccess)
+    conf.global.should be_a(Scide::Global)
+    conf.projects.should be_a(HashWithIndifferentAccess)
+    conf.projects[:project1].should be_a(Scide::Project)
+  end
+
+  [ true, false ].each do |exit_on_fail|
+    describe "Errors with exit on fail #{exit_on_fail}" do
+      before :each do
+        Scide.exit_on_fail = exit_on_fail
+      end
+
+      after :each do
+        Scide.exit_on_fail = true
+      end
+      
+      it SpecHelper.should_fail(exit_on_fail, :config_not_found, "when the config file does not exist") do
+        bad_file = "/tmp/IHateYouIfYouAddedThisFileJustToMakeMyTestFail"
+        conf = Scide::Config.new bad_file
+        load_should_fail(conf, :config_not_found)
+      end
+
+      it SpecHelper.should_fail(exit_on_fail, :config_not_readable, "if the config file is not readable") do
+        unreadable_file = Tempfile.new 'scide-unreadable'
+        FileUtils.chmod 0200, unreadable_file.path
+        conf = Scide::Config.new unreadable_file.path
+        load_should_fail(conf, :config_not_readable)
+        unreadable_file.unlink
+      end
+
+      it SpecHelper.should_fail(exit_on_fail, :unexpected, "if the config file cannot be read") do
+        file = File.join File.dirname(__FILE__), 'results', 'config1.yml'
+        conf = Scide::Config.new file
+        conf.stub!(:load_config){ raise 'bug' }
+        load_should_fail(conf, :unexpected)
+      end
+
+      it SpecHelper.should_fail(exit_on_fail, :malformed_config, "if the config file is not valid YAML") do
+        file = File.join File.dirname(__FILE__), 'results', 'malformed_config.yml'
+        conf = Scide::Config.new file
+        load_should_fail(conf, :malformed_config)
+      end
+
+      it SpecHelper.should_fail(exit_on_fail, :invalid_config, "if the configuration is not a hash") do
+        conf = Scide::Config.new
+        conf.stub!(:load_config){ "- 'a'\n- 'b'" }
+        load_should_fail(conf, :invalid_config)
+      end
+
+      it SpecHelper.should_fail(exit_on_fail, :invalid_config, "if the screen configuration is not a hash") do
+        conf = Scide::Config.new
+        conf.stub!(:load_config){ "screen:\n- 'a'\n- 'b'" }
+        load_should_fail(conf, :invalid_config)
+      end
+
+      it SpecHelper.should_fail(exit_on_fail, :invalid_config, "if the projects configuration is not a hash") do
+        conf = Scide::Config.new
+        conf.stub!(:load_config){ "projects:\n- 'a'\n- 'b'" }
+        load_should_fail(conf, :invalid_config)
+      end
+
+      it SpecHelper.should_fail(exit_on_fail, :invalid_config, "if the configuration is otherwise invalid") do
+        conf = Scide::Config.new
+        conf.stub!(:load_config){ "global:\n  a: true\nprojects:\n  a:\n    - 'a'\n    - 'b'" }
+        load_should_fail(conf, :invalid_config)
+      end
+    end
+  end
+
+  private
+
+  def make_load conf
+    SpecHelper.silence{ conf.load! }
+  end
+
+  def load_should_fail conf, condition
+    if Scide.exit_on_fail
+      lambda{ make_load conf }.should raise_error(SystemExit){ |err| err.status.should == Scide::EXIT[condition] }
+    else
+      lambda{ make_load conf }.should raise_error(Scide::Error){ |err| err.condition.should == condition }
+    end
+  end
+end

data/spec/global_spec.rb

@@ -0,0 +1,38 @@
+require 'helper'
+
+describe Scide::Global do
+
+  it "should take the given path if absolute" do
+    global = Scide::Global.new :path => '/tmp'
+    global.path.should == '/tmp'
+  end
+
+  it "should expand the given path from the home directory if relative" do
+    global = Scide::Global.new :path => 'fubar'
+    global.path.should == File.join(File.expand_path('~'), 'fubar')
+  end
+
+  it "should use the home directory if no path is given" do
+    global = Scide::Global.new({})
+    global.path.should == File.expand_path('~')
+  end
+
+  it "should duplicate the given options" do
+    options = { :a => 1, :b => true }
+    global = Scide::Global.new :options => options
+    global.options.should == options
+    global.options.should_not equal(options)
+  end
+
+  it "should raise an error if the contents are not a hash" do
+    [ nil, '   ', [] ].each do |not_hash|
+      lambda{ Scide::Global.new not_hash }.should raise_error(ArgumentError)
+    end
+  end
+
+  it "should raise an error if its options are not nil and not a hash" do
+    [ '   ', [] ].each do |not_hash|
+      lambda{ Scide::Global.new :options => not_hash }.should raise_error(ArgumentError)
+    end
+  end
+end

data/spec/helper.rb

@@ -1,5 +1,12 @@
 require 'rubygems'
 require 'bundler'
+require 'simplecov'
+
+# to silence streams
+require 'active_support/core_ext/kernel/reporting'
+
+# test coverage
+SimpleCov.start
 
 begin
   Bundler.setup(:default, :development)
@@ -12,6 +19,28 @@ end
 require 'rspec'
 require 'shoulda'
 
+class SpecHelper
+  def self.result name
+    File.open(File.join(File.dirname(__FILE__), 'results', "#{name}.screen"), 'r').read
+  end
+
+  def self.silence
+    silence_stream(STDOUT) do
+      silence_stream(STDERR) do
+        yield
+      end
+    end
+  end
+
+  def self.should_fail exit_on_fail, condition, msg
+    if exit_on_fail
+      "should exit with status #{Scide::EXIT[condition]} #{msg}"
+    else
+      "should raise an error with condition #{condition} #{msg}"
+    end
+  end
+end
+
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 require 'scide'

data/spec/opts_spec.rb

@@ -0,0 +1,42 @@
+require 'helper'
+
+describe Scide::Opts do
+
+  before :each do
+    @opts = Scide::Opts.new
+  end
+  
+  it "should add the help flag" do
+    [ '-h', '--help' ].each do |flag|
+      lambda{ parse!([flag]) }.should raise_error(SystemExit){ |err| err.status.should == 0 }
+    end
+  end
+
+  it "should add the usage flag" do
+    [ '-u', '--usage' ].each do |flag|
+      lambda{ parse!([flag]) }.should raise_error(SystemExit){ |err| err.status.should == 0 }
+    end
+  end
+
+  it "should add the version flag" do
+    expected_version = File.open(File.join(File.dirname(__FILE__), '..', 'VERSION'), 'r').read
+    lambda{ parse!([ '--version' ]) }.should raise_error(SystemExit){ |err| err.status.should == 0 }
+  end
+
+  it "should add the dry run flag" do
+    parse!([ '--dry-run' ])
+    @opts.funnel[:'dry-run'].should == true
+  end
+
+  it "should exit with status 2 for unknown arguments" do
+    [ '-x', '--fubar', '-4' ].each do |unknown|
+      lambda{ parse!([unknown]) }.should raise_error(SystemExit){ |err| err.status.should == 2 }
+    end
+  end
+
+  private
+
+  def parse! args
+    silence_stream(STDOUT){ silence_stream(STDERR){ @opts.parse! args } }
+  end
+end

data/spec/project_spec.rb

@@ -0,0 +1,122 @@
+require 'helper'
+
+describe Scide::Project do
+  
+  before :each do
+    @path = '/tmp'
+    @options = { :a => 1, :b => true }
+    @global = double('global')
+    @global.stub(:path){ @path }
+    @global.stub(:options){ @options }
+  end
+
+  it "should add the project key to the global path by default" do
+    pro = Scide::Project.new @global, 'fubar', :windows => []
+    pro.path.should == File.join(@path, pro.key)
+  end
+
+  it "should add its path to the global path if relative" do
+    pro = Scide::Project.new @global, 'fubar', :windows => [], :path => 'foo'
+    pro.path.should == File.join(@path, 'foo')
+  end
+
+  it "should use its path if absolute" do
+    pro = Scide::Project.new @global, 'fubar', :windows => [], :path => '/foo'
+    pro.path.should == '/foo'
+  end
+
+  it "should use its path if there is no global path" do
+    global_no_path = double('global')
+    global_no_path.stub(:path){ nil }
+    global_no_path.stub(:options){ { :a => 1, :b => true } }
+    pro = Scide::Project.new global_no_path, 'fubar', :windows => [], :path => '/foo'
+    pro.path.should == '/foo'
+  end
+
+  it "should expand its path if relative" do
+    global_relative = double('global')
+    global_relative.stub(:path){ nil }
+    global_relative.stub(:options){ { :a => 1, :b => true } }
+    pro = Scide::Project.new global_relative, 'fubar', :windows => []
+    pro.path.should == File.join(File.expand_path('~'), 'fubar')
+  end
+
+  it "should duplicate global options" do
+    @global.should_receive :path
+    @global.should_receive :options
+    pro = Scide::Project.new @global, 'fubar', :windows => []
+    pro.options.should_not equal(@options)
+  end
+
+  it "should add name and path to options" do
+    additional_options = { :name => 'fubar', :path => File.join(@path, 'fubar') }
+    pro = Scide::Project.new @global, 'fubar', :windows => []
+    pro.options.should == @options.merge(additional_options)
+  end
+
+  it "should override global name and path" do
+    global_with_name_and_path = double('global')
+    global_with_name_and_path.stub(:path){ @path }
+    global_with_name_and_path.stub(:options){ { :name => 'global', :path => '/global' } }
+    pro = Scide::Project.new global_with_name_and_path, 'fubar', :windows => []
+    pro.options[:name].should_not == 'global'
+    pro.options[:path].should_not == '/global'
+  end
+
+  it "should merge given options" do
+    additional_options = { :name => 'fubar', :path => File.join(@path, 'fubar') }
+    new_options = { :name => 'fubar2', :path => '/tmp2', :b => false, :c => 'foo' }
+    pro = Scide::Project.new @global, 'fubar', :windows => [], :options => new_options
+    pro.options.should == @options.merge(additional_options).merge(new_options)
+  end
+
+  describe 'Sample Configuration' do
+    before :each do
+      contents = {
+        :windows => [
+          'window1 EDIT',
+          { :name => 'window2', :command => 'SHOW', :contents => 'ssh example.com' },
+          'window3 TAIL file.txt',
+          { :string => 'window4 RUN ls -la' },
+          'window5'
+        ]
+      }
+      @project = Scide::Project.new @global, 'fubar', contents
+    end
+
+    it "should create five windows" do
+      @project.windows.length.should == 5
+      @project.windows.each{ |w| w.should be_a(Scide::Window) }
+    end
+
+    it "should create the correct commands" do
+      @project.windows[0].command.should be_a(Scide::Commands::Edit)
+      @project.windows[1].command.should be_a(Scide::Commands::Show)
+      @project.windows[2].command.should be_a(Scide::Commands::Tail)
+      @project.windows[3].command.should be_a(Scide::Commands::Run)
+      @project.windows[4].command.should be_nil
+    end
+
+    it "should create a GNU Screen configuration" do
+      @project.to_screen.should == SpecHelper.result(:project1)
+    end
+  end
+
+  it "should raise an error if the contents are not a hash" do
+    [ nil, '   ', [] ].each do |not_hash|
+      lambda{ Scide::Project.new @global, 'fubar', not_hash }.should raise_error(ArgumentError)
+    end
+  end
+
+  it "should raise an error if its windows are nil or not an array" do
+    [ nil, '   ', {} ].each do |not_array|
+      lambda{ Scide::Project.new @global, 'fubar', :windows => not_array }.should raise_error(ArgumentError)
+    end
+  end
+
+  it "should raise an error if its options are not nil and not a hash" do
+    [ '   ', [] ].each do |not_hash|
+      lambda{ Scide::Project.new @global, 'fubar', :windows => [], :options => not_hash }.should raise_error(ArgumentError)
+    end
+  end
+end

data/spec/results/config1.yml

@@ -0,0 +1,14 @@
+screen:
+  hardstatus: 'dummy'
+global:
+  path: '/tmp'
+projects:
+  project1:
+    windows:
+      - 'window1 EDIT'
+      - name: 'window2'
+        command: 'SHOW'
+        contents: 'ssh example.com'
+      - 'window3 TAIL file.txt'
+      - string: 'window4 RUN ls -la'
+      - 'window5'

data/spec/results/malformed_config.yml

@@ -0,0 +1,2 @@
+bad:
+	- yaml

data/spec/results/project1.screen

@@ -0,0 +1,9 @@
+screen -t window1 0
+stuff "$EDITOR\012"
+screen -t window2 1
+stuff "ssh example.com"
+screen -t window3 2
+stuff "tail -f file.txt\012"
+screen -t window4 3
+stuff "ls -la\012"
+screen -t window5 4

data/spec/results/screen1.screen

@@ -0,0 +1,14 @@
+startup_message off
+hardstatus on
+hardstatus alwayslastline
+hardstatus string 'dummy'
+
+screen -t window1 0
+stuff "$EDITOR\012"
+screen -t window2 1
+stuff "ssh example.com"
+screen -t window3 2
+stuff "tail -f file.txt\012"
+screen -t window4 3
+stuff "ls -la\012"
+screen -t window5 4

data/spec/scide_spec.rb

@@ -0,0 +1,38 @@
+require 'helper'
+
+describe Scide do
+
+  before :each do
+    Scide.exit_on_fail = true
+  end
+
+  after :each do
+    Scide.exit_on_fail = true
+  end
+  
+  it "should have the correct version" do
+    Scide::VERSION.should == File.open(File.join(File.dirname(__FILE__), '..', 'VERSION'), 'r').read
+  end
+
+  it "should exit on fail by default" do
+    lambda{ SpecHelper.silence{ Scide.fail nil, 'fubar' } }.should raise_error(SystemExit)
+  end
+
+  it "should exit with the correct status code" do
+    Scide::EXIT.each_pair do |condition,code|
+      lambda{ SpecHelper.silence{ Scide.fail condition, 'fubar' } }.should raise_error(SystemExit){ |err| err.status.should == code }
+    end
+  end
+
+  it "should raise an error on fail if configured to do so" do
+    Scide.exit_on_fail = false
+    lambda{ Scide.fail nil, 'fubar' }.should raise_error(Scide::Error)
+  end
+
+  it "should raise an error with the correct condition" do
+    Scide.exit_on_fail = false
+    Scide::EXIT.each_pair do |condition|
+      lambda{ Scide.fail condition, 'fubar' }.should raise_error(Scide::Error){ |err| err.condition.should == condition }
+    end
+  end
+end

data/spec/screen_spec.rb

@@ -0,0 +1,83 @@
+require 'helper'
+
+describe Scide::Screen do
+
+  before :each do
+    @project = double('project')
+    @project.stub(:path){ '/tmp' }
+    @project.stub(:to_screen){ 'fubar' }
+  end
+
+  it "should generate a command to run screen at the given project's path" do
+    screen = Scide::Screen.new(@project, {})
+    @project.should_receive :path
+    screen.to_command('fubar').should == 'cd /tmp && screen -c fubar'
+  end
+
+  it "should generate a command to run screen with the given options" do
+    screen = Scide::Screen.new @project, :binary => '/usr/bin/screen', :args => '-U'
+    @project.should_receive :path
+    screen.to_command('fubar').should == 'cd /tmp && /usr/bin/screen -U -c fubar'
+  end
+
+  it "should generate a screen configuration file" do
+    global = double('global')
+    global.stub(:path){ nil }
+    global.stub(:options){ {} }
+    contents = {
+      :windows => [
+        'window1 EDIT',
+        { :name => 'window2', :command => 'SHOW', :contents => 'ssh example.com' },
+        'window3 TAIL file.txt',
+        { :string => 'window4 RUN ls -la' },
+        'window5'
+      ]
+    }
+    project = Scide::Project.new(global, 'fubar', contents)
+    screen = Scide::Screen.new project, :hardstatus => 'dummy'
+    screen.to_s.should == SpecHelper.result(:screen1)
+  end
+
+  it "should use the default hardstatus line if not given" do
+    screen = Scide::Screen.new(@project, {})
+    screen.hardstatus.should == Scide::Screen::DEFAULT_HARDSTATUS
+  end
+
+  it "should use the given hardstatus line" do
+    screen = Scide::Screen.new @project, :hardstatus => 'fubar'
+    screen.hardstatus.should == 'fubar'
+  end
+
+  it "should use the default binary if not given" do
+    screen = Scide::Screen.new(@project, {})
+    screen.binary.should == 'screen'
+  end
+
+  it "should use the given binary" do
+    screen = Scide::Screen.new @project, :binary => '/usr/bin/screen'
+    screen.binary.should == '/usr/bin/screen'
+  end
+
+  it "should use the given args" do
+    screen = Scide::Screen.new @project, :args => '-U'
+    screen.args.should == '-U'
+  end
+
+  it "should duplicate the given options" do
+    options = { :a => 1, :b => true }
+    screen = Scide::Screen.new @project, options
+    screen.options.should == options
+    screen.options.should_not equal(options)
+  end
+
+  it "should exit with status 4 if the screen binary is not found" do
+    bad_binary = "IHateYouIfYouAddedThisBinaryJustToMakeMyTestFail"
+    lambda{ SpecHelper.silence{ Scide::Screen.new(@project, :binary => bad_binary).check_binary } }.should raise_error(SystemExit){ |err| err.status.should == 4 }
+  end
+
+  it "should raise an error if the options are not nil and not a hash" do
+    [ '   ', [] ].each do |not_hash|
+      lambda{ Scide::Screen.new @project, not_hash }.should raise_error(ArgumentError)
+    end
+  end
+end

data/spec/version_spec.rb

@@ -0,0 +1,8 @@
+require 'helper'
+
+describe 'Scide::VERSION' do
+
+  it "should be the correct version" do
+    Scide::VERSION.should == File.open(File.join(File.dirname(__FILE__), '..', 'VERSION'), 'r').read
+  end
+end

data/spec/window_spec.rb

@@ -0,0 +1,122 @@
+require 'helper'
+
+describe Scide::Window do
+
+  before :each do
+    @options = { :a => 1, :b => true }
+    @project = double('project')
+    @project.stub(:options){ @options }
+  end
+
+  describe "String Initialization" do
+    it "should take the given name" do
+      win = Scide::Window.new @project, 'fubar'
+      win.name.should == 'fubar'
+    end
+
+    it "should create the correct command" do
+      win = Scide::Window.new @project, 'fubar EDIT'
+      win.command.should be_a(Scide::Commands::Edit)
+    end
+
+    it "should be initializable with a name and a command with no contents" do
+      lambda{ Scide::Window.new @project, 'fubar EDIT' }.should_not raise_error(ArgumentError)
+    end
+
+    it "should be initializable with a name and a command with contents" do
+      lambda{ Scide::Window.new @project, 'fubar TAIL file.txt' }.should_not raise_error(ArgumentError)
+    end
+
+    it "should raise an error when initialized without a name" do
+      lambda{ Scide::Window.new @project, '   ' }.should raise_error(ArgumentError)
+    end
+
+    it "should raise an error for unknown commands" do
+      lambda{ Scide::Window.new @project, 'fubar EDT' }.should raise_error(ArgumentError)
+    end
+  end
+
+  describe "Hash Initialization" do
+    it "should take the given name" do
+      win = Scide::Window.new @project, :name => 'fubar'
+      win.name.should == 'fubar'
+    end
+
+    it "should create the correct command" do
+      win = Scide::Window.new @project, :name => 'fubar', :command => 'EDIT'
+      win.command.should be_a(Scide::Commands::Edit)
+    end
+
+    it "should be initializable with a name and a command with no contents" do
+      lambda{ Scide::Window.new @project, :name => 'fubar', :command => 'EDIT' }.should_not raise_error(ArgumentError)
+    end
+
+    it "should be initializable with a name and a command with contents" do
+      lambda{ Scide::Window.new @project, :name => 'fubar', :command => 'EDIT', :contents => 'file.txt' }.should_not raise_error(ArgumentError)
+    end
+
+    it "should raise an error when initialized without a name" do
+      lambda{ Scide::Window.new @project, :fubar => true }.should raise_error(ArgumentError)
+    end
+
+    it "should raise an error for unknown commands" do
+      lambda{ Scide::Window.new @project, :name => 'fubar', :command => 'EDT' }.should raise_error(ArgumentError)
+    end
+
+    describe "String Option" do
+      
+      it "should take the given name" do
+        win = Scide::Window.new @project, :string => 'fubar'
+        win.name.should == 'fubar'
+      end
+
+      it "should create the correct command" do
+        win = Scide::Window.new @project, :string => 'fubar EDIT'
+        win.command.should be_a(Scide::Commands::Edit)
+      end
+
+      it "should be initializable with a name and a command with no contents" do
+        lambda{ Scide::Window.new @project, :string => 'fubar EDIT' }.should_not raise_error(ArgumentError)
+      end
+
+      it "should be initializable with a name and a command with contents" do
+        lambda{ Scide::Window.new @project, :string => 'fubar EDIT file.txt' }.should_not raise_error(ArgumentError)
+      end
+
+      it "should raise an error when initialized without a name" do
+        lambda{ Scide::Window.new @project, :string => '   ' }.should raise_error(ArgumentError)
+      end
+
+      it "should raise an error for unknown commands" do
+        lambda{ Scide::Window.new @project, :string => 'fubar EDT' }.should raise_error(ArgumentError)
+      end
+    end
+  end
+
+  it "should raise an error when type of contents is unknown" do
+    lambda{ Scide::Window.new(@project, []) }.should raise_error(ArgumentError)
+  end
+
+  it "should duplicate project options" do
+    @project.should_receive :options
+    win = Scide::Window.new @project, 'fubar'
+    win.options.should == @options
+    win.options.should_not equal(@options)
+  end
+
+  it "should merge its options to project options" do
+    window_options = { :b => false, :c => 'foo' }
+    @project.should_receive :options
+    win = Scide::Window.new @project, :name => 'fubar', :options => window_options
+    win.options.should == @options.merge(window_options)
+  end
+
+  it "should generate a GNU Screen window" do
+    # without a command, hash initialization
+    win = Scide::Window.new @project, :name => 'fubar'
+    win.to_screen(0).should == 'screen -t fubar 0'
+    # with a command, string initialization
+    win = Scide::Window.new @project, 'fubar EDIT file.txt'
+    win.to_screen(0).should == %|screen -t fubar 0\n#{win.command.to_screen}|
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: scide
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.7
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-10-02 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: upoj-rb
-  requirement: &2156558600 !ruby/object:Gem::Requirement
+  requirement: &2157640360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 0.0.4
   type: :runtime
   prerelease: false
-  version_requirements: *2156558600
+  version_requirements: *2157640360
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2156558080 !ruby/object:Gem::Requirement
+  requirement: &2157639760 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2156558080
+  version_requirements: *2157639760
 - !ruby/object:Gem::Dependency
   name: shoulda
-  requirement: &2156557480 !ruby/object:Gem::Requirement
+  requirement: &2157639160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2156557480
+  version_requirements: *2157639160
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &2156556880 !ruby/object:Gem::Requirement
+  requirement: &2157638560 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,10 +54,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *2156556880
+  version_requirements: *2157638560
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &2156556280 !ruby/object:Gem::Requirement
+  requirement: &2157637960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -65,10 +65,10 @@ dependencies:
         version: 1.6.4
   type: :development
   prerelease: false
-  version_requirements: *2156556280
+  version_requirements: *2157637960
 - !ruby/object:Gem::Dependency
-  name: rcov
-  requirement: &2156555680 !ruby/object:Gem::Requirement
+  name: simplecov
+  requirement: &2157637360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2156555680
+  version_requirements: *2157637360
 - !ruby/object:Gem::Dependency
   name: rdoc
-  requirement: &2156555080 !ruby/object:Gem::Requirement
+  requirement: &2157636760 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,7 +87,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2156555080
+  version_requirements: *2157636760
 description: Utility to generate GNU screen configuration files.
 email: hydrae.alpha@gmail.com
 executables:
@@ -105,6 +105,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- TODO.md
 - VERSION
 - bin/scide
 - lib/scide.rb
@@ -126,7 +127,19 @@ files:
 - spec/commands/run_spec.rb
 - spec/commands/show_spec.rb
 - spec/commands/tail_spec.rb
+- spec/config_spec.rb
+- spec/global_spec.rb
 - spec/helper.rb
+- spec/opts_spec.rb
+- spec/project_spec.rb
+- spec/results/config1.yml
+- spec/results/malformed_config.yml
+- spec/results/project1.screen
+- spec/results/screen1.screen
+- spec/scide_spec.rb
+- spec/screen_spec.rb
+- spec/version_spec.rb
+- spec/window_spec.rb
 homepage: http://github.com/AlphaHydrae/scide
 licenses:
 - MIT
@@ -142,7 +155,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1145488263121204374
+      hash: 1323190818279805743
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: