homedir 3.0.alpha1 → 3.0.alpha2

data/.gitignore

@@ -11,3 +11,4 @@
 /pkg/
 /spec/reports
 /tmp/
+/.rbx/

data/.travis.yml

@@ -2,6 +2,7 @@ rvm:
   - jruby
   - 1.8.7
   - 1.9.2
+  - 1.9.3
 script: "bundle exec rake travis"
 notifications:
   irc: "irc.freenode.org#homedir"

data/.yardopts

@@ -0,0 +1,6 @@
+--protected
+--no-private
+--markup markdown
+-
+README.mkd
+

data/Guardfile

@@ -7,7 +7,7 @@ guard 'bundler' do
   # watch(/^.+\.gemspec/)
 end
 
-guard 'rspec', :version => 2, :cli => "--color --format doc" do
+guard 'rspec', :version => 2, :cli => "--color --format doc --fail-fast" do
   watch(%r{^spec/.+_spec\.rb$})
   watch(%r{^lib/(.+)\.rb$})     { |m| "spec/lib/#{m[1]}_spec.rb" }
   watch('spec/spec_helper.rb')  { "spec" }

data/LICENSE

@@ -0,0 +1,19 @@
+Copyright (c) 2012 Christian Höltje
+
+Permission is hereby granted, free of charge, to any person obtaining a
+copy of this software and associated documentation files (the "Software"),
+to deal in the Software without restriction, including without limitation
+the rights to use, copy, modify, merge, publish, distribute, sublicense,
+and/or sell copies of the Software, and to permit persons to whom the
+Software is furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included
+in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
+OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+DEALINGS IN THE SOFTWARE.

data/bin/homedir

@@ -0,0 +1,5 @@
+#!/usr/bin/env ruby
+require 'rubygems' if Gem.nil?
+require 'homedir/cli'
+
+Homedir::CLI.start

data/homedir.gemspec

@@ -2,24 +2,29 @@
 require File.expand_path('../lib/homedir/version', __FILE__)
 
 Gem::Specification.new do |gem|
-  gem.authors       = ["Christian Holtje"]
-  gem.email         = ["docwhat@gerf.org"]
-  gem.description   = %q{Your home directory package management system}
-  gem.summary       = %q{Manage your home manager}
-  gem.homepage      = "http://home-dir.org/"
+  gem.authors               = ["Christian Holtje"]
+  gem.email                 = ["docwhat@gerf.org"]
+  gem.description           = %q{Your home directory package management system}
+  gem.summary               = %q{Manage your home manager}
+  gem.homepage              = "http://home-dir.org/"
+  gem.license               = "MIT"
+  gem.required_ruby_version = ">= 1.8.7"
 
-  gem.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  gem.files         = `git ls-files`.split("\n")
-  gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  gem.name          = "homedir"
-  gem.require_paths = ["lib"]
-  gem.version       = Homedir::VERSION
+  #gem.post_install_message  = "Type 'homedir' to get started!"
+
+  gem.executables           = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+  gem.files                 = `git ls-files`.split("\n")
+  gem.test_files            = `git ls-files -- {test,spec,features}/*`.split("\n")
+  gem.name                  = "homedir"
+  gem.require_paths         = ["lib"]
+  gem.version               = Homedir::VERSION
 
   gem.add_development_dependency('rake')
   gem.add_development_dependency('rspec', ["~> 2.8"])
-  gem.add_development_dependency('fakefs', ["~> 0.4.0"])
   gem.add_development_dependency('factory_girl', ["~> 2.5.2"])
 
+  gem.add_development_dependency('yard')
+
   gem.add_development_dependency('guard')
   gem.add_development_dependency('guard-rspec')
   gem.add_development_dependency('guard-bundler')

data/lib/homedir/catalog.rb

@@ -0,0 +1,47 @@
+require 'homedir/errors'
+
+module Homedir
+  class Catalog < Set
+
+    # @param {String} name The name of the {Homedir::Package package} to find.
+    # @return {Homedir::Package} The package that matches the `name`
+    def find_by_name name
+      found = select { |pkg| pkg.name == name }
+      raise "Invalid package object(s): #{found.inspect}" unless found.length <= 1
+      found.first
+    end
+
+    # Returns all dependencies for `package` including `package` itself.
+    #
+    # @raise [Homedir::MissingDependendency] if a dependency cannot be found.
+    # @param {Homedir::Package} package The package to get dependencies for.
+    # @return {Set} The {Homedir::Package Package} objects that `package` depends on.
+    def find_all_dependencies package
+      found = Set.new([package])
+      find_all_dependencies_not_found found, package
+      return found
+    end
+
+    private
+
+    # This is the method that does the work for {#find_all_dependencies}.
+    #
+    # @param {Set} found The set of all found packages.
+    # @param {Homedir::Package} package The package to get dependencies for. It is assumed it is already in `found`
+    def find_all_dependencies_not_found found, package
+      package.dependencies.each do |dependency_name|
+        dependency = find_by_name dependency_name
+
+        raise MissingDependencyError.new(
+          "The package '#{package}' requires '#{dependency_name}' which cannot be found."
+        ) if dependency.nil?
+
+        if not found.include? dependency
+          found << dependency
+          find_all_dependencies_not_found(found, dependency)
+        end
+      end
+      return nil
+    end
+  end
+end

data/lib/homedir/cli.rb

@@ -0,0 +1,55 @@
+require 'thor'
+
+module Homedir
+  # The command line interface for {Homedir}.
+  #
+  # See the source or the command line tool `bin/homedir` for more
+  # information
+  class CLI < Thor
+
+    desc "list", "List all known packages."
+    method_option(
+      :remote,
+      :type => :boolean,
+      :default => false,
+      :aliases => '-r',
+      :description => "Queries the remote server")
+    def list
+      # FIXME: list needs to do something
+      puts "Not implemented yet"
+    end
+
+    desc "info PACKAGE", "Describe a specific PACKAGE in detail."
+    def info(package_name)
+      # FIXME: info needs to do something
+      puts "Not implemented yet #{package_name.inspect}"
+    end
+
+    desc "enable PACKAGE...", "Enable PACKAGE, making it available in your home directory."
+    def enable(*packages)
+      # FIXME: info needs to do something
+      puts "Not implemented yet #{packages.inspect}"
+    end
+
+    desc "disable PACKAGE...", "Disable PACKAGE, removing it from your home directory."
+    def disable(*packages)
+      # FIXME: info needs to do something
+      puts "Not implemented yet #{packages.inspect}"
+    end
+
+    desc "create PACKAGE...", "Create PACKAGE, prompting you for information."
+    method_option(
+      :directory,
+      :type => :string,
+      :default => "#{ENV['HOME']}/.homedir/package",
+      :aliases => "-d",
+      :description => "Where to save the DIRECTORY."
+    )
+    def create(package_name)
+      # FIXME: info needs to do something
+      puts "Not implemented yet #{package_name.inspect}"
+    end
+
+  end
+end
+

data/lib/homedir/errors.rb

@@ -0,0 +1,17 @@
+
+module Homedir
+  # When possible, all exceptions raised by Homedir classes
+  # will be subclassed from {Homedir::Error}.
+  class Error < StandardError
+  end
+
+  # If a dependency is missing from a {Homedir::Package package} (it isn't
+  # available in a {Homedir::Catalog Catalog}) then this exception is raised.
+  class MissingDependencyError < Error
+  end
+
+  # The name assigned to a {Homedir::Package package} is not legal. Check the
+  # message for the list of acceptable characters in a name.
+  class InvalidNameError < Error
+  end
+end

data/lib/homedir/package.rb

@@ -1,28 +1,78 @@
-require 'homedir'
+require 'homedir/errors'
 require 'pathname'
 require 'fileutils'
 require 'yaml'
 
+# A package contains all the meta information about the package:
+#
+# * name, description, etc.
+# * pre and post install instructions.
+# * location of the package directory.
+#
 module Homedir
   class Package
-    REQUIRED_OPTIONS = [
+    # A list of required values. If a value in this
+    # list isn't set, then the object isn't {#valid? valid}.
+    # @see #valid?
+    REQUIRED_VALUES = [
       :name,
       :description,
       :directory,
     ].freeze
-    DEFAULT_OPTIONS = {
-      :name         => nil,
-      :description  => nil,
-      :dependencies => [],
-      :post_install => nil,
-      :pre_install  => nil,
-      :directory    => nil,
+    DEFAULT_VALUES = {
+      :name           => nil,
+      :description    => nil,
+      :dependencies   => [],
+      :post_install   => nil,
+      :pre_uninstall  => nil,
+      :directory      => nil,
     }.freeze
-    attr_accessor *DEFAULT_OPTIONS.keys
 
+    # The name of the package.
+    #
+    # * It must match the regex: `[a-zA-Z0-9_-]`.
+    # * It must match the directory it'll be written-to/read-from
+    #
+    # @return {String} The name of the package
+    attr_reader :name
 
+    # A description of the package for humans.
+    #
+    # @return {String} The description of the package
+    attr_accessor :description
+
+    # A list of package names this package depends on.
+    #
+    # When assigning a list, they can either be {Homedir::Package Package} objects or strings.
+    # @return [Enumerable] A list of package {#name names}.
+    attr_reader :dependencies
+
+    # A script that should run after install.
+    #
+    # @return {String}
+    attr_accessor :post_install
+
+    # A script to run before install.
+    attr_accessor :pre_uninstall
+
+    # The directory where the package is stored.
+    #
+    # * The directory's basename must match {#name}.
+    #
+    # @return {Pathname}
+    attr_reader :directory
+
+    # Create a new {Homedir::Package Package} instance.
+    #
+    # @param {Hash} options
+    # @option options [String] :name The name of the package.
+    # @option options [String] :description The description of the package.
+    # @option options [Enumerable] :dependencies A list of {Homedir::Package packages} or {String strings}.
+    # @option options [Pathname] :directory The directory the packages is located at.
+    # @option options [String] :post_install Commands to run after installing the package.
+    # @option options [String] :pre_uninstall Commands to run before uninstalling the package.
     def initialize(options = {})
-      options = DEFAULT_OPTIONS.merge(options)
+      options = DEFAULT_VALUES.merge(options)
 
       # Set the options
       options.each do |k,v|
@@ -30,19 +80,47 @@ module Homedir
       end
     end
 
+    # The hashes are based on the name
+    # @return {Fixnum} The hash
+    def hash
+      @name.hash
+    end
+
+    # Returns true if the packages have the same name.
+    #
+    # @param {Homedir::Package} other The package to check for equality to.
+    # @return {Boolean} `true` if the `name` matches.
+    def eql?(other)
+      if other.respond_to? :name
+        other.name == @name
+      else
+        false
+      end
+    end
+
+    # {include:#dependencies}
     def dependencies= value
       @dependencies = value.map { |p| p.to_s }
     end
 
+    # {include:#directory}
     def directory= value
       @directory = value.nil? ? nil : Pathname.new(value)
     end
 
+    # {include:#name}
+    # @param {String} value The new name for the package.
+    # @raise [Homedir::InvalidNameError] If the name doesn't match the regexp `/^[0-9a-zA-Z_-]+$/`
     def name= value
-      raise "Invalid name: #{value.inspect}" unless value.nil? || value =~ /^[0-9a-zA-Z_-]+$/
+      raise InvalidNameError.new(
+        "The name #{value.inspect} must only contain numbers, letters, `_` and `-`"
+      ) unless value.nil? || value =~ /^[0-9a-zA-Z_-]+$/
       @name = value
     end
 
+    # Saves the package info to {#directory}`/.homedir/` if it {#valid? valid}.
+    #
+    # @see #valid?
     def save!
       raise "The package is not valid: #{self}" unless valid?
 
@@ -54,12 +132,30 @@ module Homedir
       end
     end
 
+    # Verifies the package is valid to save.
+    #
+    # @return {Boolean} `true` if the package is valid.
     def valid?
-      REQUIRED_OPTIONS.reduce(true) do |is_valid, opt|
-        is_valid && !instance_variable_get("@#{opt}").nil?
+      errors = []
+      REQUIRED_VALUES.each do |value|
+        errors << "#{value} must not be empty." if instance_variable_get("@#{value}").nil?
       end
+      # FIXME: This should check that .directory.basename == .name
+      @errors = errors
+      @errors.length == 0
+    end
+
+    # A list of errors
+    #
+    # This list is only filled after {#.valid?} is run.
+    # @return [Array] An array of strings describing the problems.
+    def errors
+      @errors || []
     end
 
+    # Returns the {#name} attribute
+    #
+    # @return {String} The {#name} of the package.
     def to_s
       @name
     end

data/lib/homedir/version.rb

@@ -1,4 +1,4 @@
 
 module Homedir
-  VERSION = '3.0.alpha1'
+  VERSION = '3.0.alpha2'
 end

data/lib/homedir.rb

@@ -1,3 +1,6 @@
+# AUTHOR::    Christian Höltje <docwhat@gerf.org>
+# COPYRIGHT:: Copyright (c) 2012 Christian Höltje
+# LICENSE::   The MIT License. See the file LICENSE distributed with this source.
 require 'homedir/package'
 
 module Homedir

data/spec/lib/homedir/catalog_spec.rb

@@ -0,0 +1,75 @@
+require 'spec_helper'
+require 'homedir/catalog'
+require 'homedir/errors'
+
+
+describe Homedir::Catalog do
+  def mkpkg(name, dependencies=[])
+    double(name).tap do |pkg|
+      pkg.stub(:name) { name }
+      pkg.stub(:dependencies) { dependencies }
+    end
+  end
+
+  describe ".add" do
+    it "adds the package to the catalog" do
+      pkg = mkpkg('somename')
+      subject.add pkg
+      subject.should include(pkg)
+    end
+  end
+
+  describe ".find_all_dependencies" do
+    let(:pkg) { mkpkg('pkg', ['pkg2']) }
+
+    context "in an acyclic graph" do
+      before(:each) do
+        subject.add mkpkg('pkg2', ['pkg3'])
+        subject.add mkpkg('pkg3')
+        subject.add mkpkg('lonelypkg')
+      end
+
+      it "should all dependencies" do
+        dependents = subject.find_all_dependencies(pkg)
+        dependents.size.should == 3
+      end
+
+      it "should be packages" do
+        dependents = subject.find_all_dependencies(pkg)
+        dependents.each do |p|
+          p.should respond_to(:name)
+        end
+      end
+    end
+
+    context "in a cyclic graph" do
+      before(:each) do
+        subject.add pkg
+        subject.add mkpkg('pkg2', [pkg.name])
+      end
+
+      it "should return only two" do
+        dependents = subject.find_all_dependencies(pkg)
+        dependents.size.should == 2
+      end
+    end
+
+    context "with a missing dependency" do
+      it "should raise an error" do
+        expect { subject.find_all_dependencies(pkg) }.to raise_error(Homedir::MissingDependencyError)
+      end
+    end
+  end
+
+  describe ".find_by_name" do
+    let(:pkg) { mkpkg('a-name') }
+    before(:each) do
+      subject.add(pkg)
+    end
+
+    it "return pkg" do
+      subject.find_by_name(pkg.name).should == pkg
+    end
+  end
+end
+

data/spec/lib/homedir/package_spec.rb

@@ -1,13 +1,10 @@
 require 'spec_helper'
 require 'homedir/package'
+require 'homedir/errors'
 require 'pathname'
 
 describe Homedir::Package do
 
-  context "loading a v3 package from disk" do
-    describe "#read_from_directory"
-  end
-
   describe "#new" do
     context "creating a package from scratch" do
       describe "dependencies" do
@@ -36,24 +33,52 @@ describe Homedir::Package do
     end
   end
 
-  describe "#from_directory" do
-    it "should read its configuration from a directory"
+  describe ".hash" do
+    it "should be the same for two identically named packages" do
+      pkg1 = build(:package, :name => 'samename')
+      pkg2 = build(:package, :name => pkg1.name)
+      pkg1.hash.should == pkg2.hash
+    end
+  end
+
+  describe ".eql?" do
+    it "should be true for identically named packages" do
+      pkg1 = build(:package, :name => 'samename')
+      pkg2 = build(:package, :name => pkg1.name)
+      pkg1.should eql(pkg2)
+    end
   end
 
   describe ".name" do
-    it "should not accept an invalid name" do
+    it "cannot have spaces" do
       pkg = build(:package)
-      expect { pkg.name = "a name" }.to raise_error
+      expect { pkg.name = "a name" }.to raise_error(Homedir::InvalidNameError)
     end
   end
 
   describe ".valid?" do
-    it "should require a name" do
-      build(:package, :name => nil).should_not be_valid
+    context "name" do
+      it "is required" do
+        build(:package, :name => nil).should_not be_valid
+      end
+
+      it "should be shown in .errors" do
+        pkg = build(:package, :name => nil)
+        pkg.valid?
+        pkg.errors.join.should include("name must not be empty")
+      end
     end
 
-    it "should require a description" do
-      build(:package, :description => nil).should_not be_valid
+    context "description" do
+      it "should require a description" do
+        build(:package, :description => nil).should_not be_valid
+      end
+
+      it "should be shown in .errors" do
+        pkg = build(:package, :description=> nil)
+        pkg.valid?
+        pkg.errors.join.should include("description must not be empty")
+      end
     end
   end
 

data/spec/spec_helper.rb

@@ -1,12 +1,19 @@
-require 'fakefs'
-require 'fakefs/spec_helpers'
+require 'tmpdir'
 require 'factory_girl'
 require 'factories'
 
+def within_a_tmpdir
+  Dir.mktmpdir('homedir-spec-') do |dir|
+    Dir.chdir(dir)
+    yield
+  end
+end
+
 RSpec.configure do |config|
   # Allows using build(), create(), etc. without the "FactoryGirl." part.
   config.include FactoryGirl::Syntax::Methods
 
-  # resets the fake filesystem after each example
-  config.include FakeFS::SpecHelpers
+  config.around(:each) do |example|
+    within_a_tmpdir(&example)
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: homedir
 version: !ruby/object:Gem::Version
-  version: 3.0.alpha1
+  version: 3.0.alpha2
   prerelease: 4
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-16 00:00:00.000000000Z
+date: 2012-02-18 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70278115028520 !ruby/object:Gem::Requirement
+  requirement: &70240106090360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70278115028520
+  version_requirements: *70240106090360
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70278115027880 !ruby/object:Gem::Requirement
+  requirement: &70240106089100 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,32 +32,32 @@ dependencies:
         version: '2.8'
   type: :development
   prerelease: false
-  version_requirements: *70278115027880
+  version_requirements: *70240106089100
 - !ruby/object:Gem::Dependency
-  name: fakefs
-  requirement: &70278115024500 !ruby/object:Gem::Requirement
+  name: factory_girl
+  requirement: &70240106087000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 2.5.2
   type: :development
   prerelease: false
-  version_requirements: *70278115024500
+  version_requirements: *70240106087000
 - !ruby/object:Gem::Dependency
-  name: factory_girl
-  requirement: &70278115023820 !ruby/object:Gem::Requirement
+  name: yard
+  requirement: &70240106085920 !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 2.5.2
+        version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70278115023820
+  version_requirements: *70240106085920
 - !ruby/object:Gem::Dependency
   name: guard
-  requirement: &70278115023440 !ruby/object:Gem::Requirement
+  requirement: &70240106084680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70278115023440
+  version_requirements: *70240106084680
 - !ruby/object:Gem::Dependency
   name: guard-rspec
-  requirement: &70278115022940 !ruby/object:Gem::Requirement
+  requirement: &70240106083380 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70278115022940
+  version_requirements: *70240106083380
 - !ruby/object:Gem::Dependency
   name: guard-bundler
-  requirement: &70278115022440 !ruby/object:Gem::Requirement
+  requirement: &70240106081940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70278115022440
+  version_requirements: *70240106081940
 - !ruby/object:Gem::Dependency
   name: rb-fsevent
-  requirement: &70278115022020 !ruby/object:Gem::Requirement
+  requirement: &70240106080280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,10 +98,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70278115022020
+  version_requirements: *70240106080280
 - !ruby/object:Gem::Dependency
   name: growl
-  requirement: &70278115021580 !ruby/object:Gem::Requirement
+  requirement: &70240106078640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -109,31 +109,38 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70278115021580
+  version_requirements: *70240106078640
 description: Your home directory package management system
 email:
 - docwhat@gerf.org
-executables: []
+executables:
+- homedir
 extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
 - .travis.yml
+- .yardopts
 - Gemfile
 - Guardfile
+- LICENSE
 - README.mkd
 - Rakefile
+- bin/homedir
 - homedir.gemspec
 - lib/homedir.rb
+- lib/homedir/catalog.rb
+- lib/homedir/cli.rb
+- lib/homedir/errors.rb
 - lib/homedir/package.rb
-- lib/homedir/repository.rb
 - lib/homedir/version.rb
 - spec/factories.rb
+- spec/lib/homedir/catalog_spec.rb
 - spec/lib/homedir/package_spec.rb
-- spec/lib/homedir/repository_spec.rb
 - spec/spec_helper.rb
 homepage: http://home-dir.org/
-licenses: []
+licenses:
+- MIT
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -143,10 +150,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
-      version: '0'
-      segments:
-      - 0
-      hash: -3141288371879137994
+      version: 1.8.7
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -161,6 +165,7 @@ specification_version: 3
 summary: Manage your home manager
 test_files:
 - spec/factories.rb
+- spec/lib/homedir/catalog_spec.rb
 - spec/lib/homedir/package_spec.rb
-- spec/lib/homedir/repository_spec.rb
 - spec/spec_helper.rb
+has_rdoc: 

data/lib/homedir/repository.rb

@@ -1,27 +0,0 @@
-require 'pathname'
-require 'homedir/package'
-
-class Homedir::Repository
-  attr_reader :directory, :package_loader, :packages, :name
-
-  def initialize(directory, options={})
-    options = {
-      :package_loader => Homedir::Package,
-    }.merge(options)
-    @directory = Pathname.new(directory)
-    @name = @directory.basename.to_s
-    @package_loader = options[:package_loader]
-  end
-
-  def scan
-    packages = []
-    directory.each_entry do |entry|
-      next if entry.to_s.start_with? '.'
-      path = directory + entry
-      packages << package_loader.load_directory(path)
-    end
-    @packages = packages.freeze
-    return @packages
-  end
-
-end

data/spec/lib/homedir/repository_spec.rb

@@ -1,62 +0,0 @@
-require 'spec_helper'
-require 'homedir/repository'
-require 'pathname'
-
-describe Homedir::Repository do
-  subject { Homedir::Repository.new directory, :package_loader => package_loader }
-  let(:package_loader) do
-    double().tap do |pl|
-      # :package is a placeholder for a Package instance
-      pl.stub(:load_directory) { :package }
-    end
-  end
-  let(:directory) { Pathname.new('/') } # an absolute directory
-  let(:package_dirs) { (1..10).map { |n| directory + "package-#{n}" } }
-
-  before(:each) do
-    # We need to make the directories exist so it can be scaned
-    package_dirs.each { |p| p.mkdir }
-  end
-
-  describe "#new" do
-    subject { Homedir::Repository }
-
-    it "should take a directory name" do
-      subject.new directory.to_s
-    end
-
-    it "should accept a pathname object" do
-      subject.new directory
-    end
-
-    it "should accept a Package class" do
-      subject.new directory, :package_loader => double()
-    end
-  end
-
-  describe ".scan" do
-    it "should load each package" do
-      package_dirs.each do |p|
-        package_loader.should_receive(:load_directory).with(p)
-      end
-      subject.scan.size.should == package_dirs.size
-    end
-  end
-
-  describe ".packages" do
-    it "should contain package objects" do
-      subject.scan
-      subject.packages.each do |p|
-        p.should == :package
-      end
-    end
-  end
-
-  describe ".name" do
-    it "should return the basename of the directory" do
-      repo = Homedir::Repository.new Pathname.new('/a/b/my-name')
-      repo.name.should == 'my-name'
-    end
-  end
-
-end