polisher 0.3 → 0.4

data/spec/event_handlers_spec.rb

@@ -16,59 +16,107 @@ require 'fileutils'
 
 require File.dirname(__FILE__) + '/spec_helper'
 
-BUILD_VERSION='fc11'
+include EventHandlers
 
-describe "Polisher::EventHandlers" do
+describe "EventHandlers" do
+
+  before(:all) do
+    @gem = Project.create! :name => 'rubygem-polisher'
+    @gem.primary_source = Source.new :name => 'polisher', :source_type => 'gem', :uri => 'http://rubygems.org/gems/polisher-%{version}.gem'
+    @gem_event0 = Event.create! :project => @gem, :process => 'download_sources'
+    @gem_event1 = Event.create! :project => @gem, :process => 'create_rpm_package', :process_options => "mock=fedora-12-x86_64"
+
+    @gem_event2 = Event.create! :project => @gem, :process => 'create_rpm_package',
+                                :process_options => "spec=#{ARTIFACTS_DIR}/templates/polisher.spec.tpl;mock=fedora-12-x86_64"
+
+    @gem_event3 = Event.create! :project => @gem, :process => 'update_repo', :process_options => "#{ARTIFACTS_DIR}/repos/fedora-ruby"
+
+    @project = Project.create :name => 'ruby-activerecord'
+    @project.sources << Source.new(:name => 'activerecord', :source_type => 'file',
+          :uri     => 'http://rubyforge.org/frs/download.php/%{level}/activerecord-%{version}.tgz')
+
+    @project_event0 = Event.create! :project => @project, :process => 'download_sources', :process_options => "level=28874"
+    @project_event1 = Event.create! :project => @project, :process => 'create_rpm_package',
+                                    :process_options => "spec=#{ARTIFACTS_DIR}/templates/polisher-projects.spec.tpl;mock=fedora-12-x86_64"
+
+    @project_event2 = Event.create! :project => @project, :process => 'update_repo', :process_options => "#{ARTIFACTS_DIR}/repos/fedora-ruby"
+  end
  
   before(:each) do
-     FileUtils.rm_rf(ARTIFACTS_DIR)
-     FileUtils.mkdir_p(ARTIFACTS_DIR + '/gems')
-     FileUtils.mkdir_p(ARTIFACTS_DIR + '/repos')
-     FileUtils.mkdir_p(ARTIFACTS_DIR + '/SOURCES')
-     FileUtils.mkdir_p(ARTIFACTS_DIR + '/SPECS')
-     FileUtils.mkdir_p(ARTIFACTS_DIR + '/templates')
-
-     unless defined? @gem
-       @gem = ManagedGem.create :name => 'polisher', :source_id => 1
-     end
+    FileUtils.rm_rf(ARTIFACTS_DIR) if File.directory? ARTIFACTS_DIR
+    FileUtils.mkdir_p(ARTIFACTS_DIR + '/repos')
+    FileUtils.mkdir_p(ARTIFACTS_DIR + '/SOURCES')
+    FileUtils.mkdir_p(ARTIFACTS_DIR + '/SPECS')
+    FileUtils.mkdir_p(ARTIFACTS_DIR + '/templates')
+
+    File.write(ARTIFACTS_DIR + '/templates/polisher.spec.tpl', POLISHER_GEM2RPM_TEST_TEMPLATE)
+    File.write(ARTIFACTS_DIR + '/templates/polisher-projects.spec.tpl', POLISHER_ERB_TEST_TEMPLATE)
+  end
+
+  after(:each) do
+    FileUtils.rm_rf(ARTIFACTS_DIR) if File.directory? ARTIFACTS_DIR
   end
 
-  it "should correctly create package" do
-     create_package(@gem)
-     File.exists?(ARTIFACTS_DIR + '/gems/polisher-0.1.gem').should == true
-     File.exists?(ARTIFACTS_DIR + '/SOURCES/polisher-0.1.gem').should == true
+  it "should download sources" do
+    download_sources(@gem_event0, "0.3")
+    File.exists?(ARTIFACTS_DIR + '/SOURCES/polisher-0.3.gem').should == true
+
+    download_sources(@project_event0, "2.0.1")
+    File.exists?(ARTIFACTS_DIR + '/SOURCES/activerecord-2.0.1.tgz').should == true
+  end
+
+  it "should correctly create a gem based package" do
+     download_sources(@gem_event0, "0.3")
+     create_rpm_package(@gem_event1, "0.3")
      File.exists?(ARTIFACTS_DIR + '/SPECS/rubygem-polisher.spec').should == true
-     File.exists?(ARTIFACTS_DIR + "/SRPMS/rubygem-polisher-0.1-1.#{BUILD_VERSION}.src.rpm").should == true
-     File.exists?(ARTIFACTS_DIR + "/RPMS/noarch/rubygem-polisher-0.1-1.#{BUILD_VERSION}.noarch.rpm").should == true
+     File.exists?(ARTIFACTS_DIR + "/SRPMS/rubygem-polisher-0.3-1.fc11.src.rpm").should == true
+     File.exists?(ARTIFACTS_DIR + "/RPMS/noarch/rubygem-polisher-0.3-1.fc12.noarch.rpm").should == true
   end
 
-  it "should correctly create package using template" do
-    File.write(ARTIFACTS_DIR + '/templates/polisher.spec.tpl', POLISHER_TEST_TEMPLATE)
-    create_package(@gem, '/polisher.spec.tpl')
+  it "should correctly create a gem based package using template" do
+    download_sources(@gem_event0, "0.3")
+    create_rpm_package(@gem_event2, '0.3')
     File.exists?(ARTIFACTS_DIR + '/SPECS/rubygem-polisher.spec').should == true
     File.read_all(ARTIFACTS_DIR + '/SPECS/rubygem-polisher.spec').should =~ /.*by polisher.*/
   end
 
+  it "should correctly create an upstream based project package" do
+    download_sources(@project_event0, "2.0.1")
+    create_rpm_package(@project_event1, "2.0.1", :release => 3)
+
+    File.exists?(ARTIFACTS_DIR + '/SPECS/ruby-activerecord.spec').should == true
+    File.exists?(ARTIFACTS_DIR + "/SRPMS/ruby-activerecord-2.0.1-3.fc11.src.rpm").should == true
+    File.exists?(ARTIFACTS_DIR + "/RPMS/noarch/ruby-activerecord-2.0.1-3.fc12.noarch.rpm").should == true
+    File.exists?(ARTIFACTS_DIR + "/RPMS/noarch/ruby-activerecord-subpkg-2.0.1-3.fc12.noarch.rpm").should == true
+    File.exists?(ARTIFACTS_DIR + "/RPMS/noarch/another-subpkg-2.0.1-3.fc12.noarch.rpm").should == true
+  end
+
   it "should correctly update repository" do
-     create_package(@gem)
-     update_repo(@gem, 'fedora-ruby')
+     download_sources(@gem_event0, "0.3")
+     create_rpm_package(@gem_event1, '0.3')
+     update_yum_repo(@gem_event3, '0.3')
+
+     template = ARTIFACTS_DIR + '/templates/polisher-projects.spec.tpl'
+     File.write(template, POLISHER_ERB_TEST_TEMPLATE)
+     download_sources(@project_event0, "2.0.1")
+     create_rpm_package(@project_event1, "2.0.1", :release => 3)
+     update_yum_repo(@project_event2, '2.0')
+
      File.directory?(ARTIFACTS_DIR + '/repos/fedora-ruby/noarch').should == true
      File.directory?(ARTIFACTS_DIR + '/repos/fedora-ruby/repodata').should == true
-     File.exists?(ARTIFACTS_DIR + '/repos/fedora-ruby/noarch/rubygem-polisher.rpm').should == true
+     File.exists?(ARTIFACTS_DIR + "/repos/fedora-ruby/noarch/rubygem-polisher-0.3-1.fc12.noarch.rpm").should == true
+     File.exists?(ARTIFACTS_DIR + "/repos/fedora-ruby/noarch/ruby-activerecord-2.0.1-3.fc12.noarch.rpm").should == true
+     File.exists?(ARTIFACTS_DIR + "/repos/fedora-ruby/noarch/ruby-activerecord-subpkg-2.0.1-3.fc12.noarch.rpm").should == true
+     File.exists?(ARTIFACTS_DIR + "/repos/fedora-ruby/noarch/another-subpkg-2.0.1-3.fc12.noarch.rpm").should == true
      File.exists?(ARTIFACTS_DIR + '/repos/fedora-ruby/repodata/repomd.xml').should == true
      File.exists?(ARTIFACTS_DIR + '/repos/fedora-ruby/repodata/primary.xml.gz').should == true
      File.exists?(ARTIFACTS_DIR + '/repos/fedora-ruby/repodata/other.xml.gz').should == true
      File.exists?(ARTIFACTS_DIR + '/repos/fedora-ruby/repodata/filelists.xml.gz').should == true
   end
 
-  it "should correctly notify email recipients" do
-     # TODO test notify subscribers
-     #notify_subscribers(@gem, [...])
-  end
-
 end
 
-POLISHER_TEST_TEMPLATE =
+POLISHER_GEM2RPM_TEST_TEMPLATE =
 %q{# Generated from <%= File::basename(format.gem_path) %> by polisher -*- rpm-spec -*-
 %define ruby_sitelib %(ruby -rrbconfig -e "puts Config::CONFIG['sitelibdir']")
 %define gemdir %(ruby -rubygems -e 'puts Gem::dir' 2>/dev/null)
@@ -162,3 +210,91 @@ rm -rf %{buildroot}
 * <%= Time.now.strftime("%a %b %d %Y") %> <%= packager %> - <%= spec.version %>-1
 - Initial package
 }
+
+# copied w/ changes from http://cvs.fedoraproject.org/viewvc/rpms/ruby-activerecord/F-13/ruby-activerecord.spec?revision=1.6&view=co
+POLISHER_ERB_TEST_TEMPLATE =
+%q{
+%{!?ruby_sitelib: %define ruby_sitelib %(ruby -rrbconfig -e "puts Config::CONFIG['sitelibdir']")}
+%define rname activerecord
+# Only run the tests on distros that have sqlite3
+%define with_check 0
+
+Name:           ruby-%{rname}
+Version:        <%= version %>
+Release:        <%= release %>%{?dist}
+Summary:        Implements the ActiveRecord pattern for ORM
+
+Group:          Development/Languages
+
+License:        MIT
+URL:            http://rubyforge.org/projects/activerecord/
+Source0:        http://rubyforge.org/frs/download.php/28874/activerecord-%{version}.tgz
+BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
+
+BuildArch:      noarch
+BuildRequires:  ruby >= 1.8
+%if %with_check
+BuildRequires:  ruby(active_support) = 2.0.1
+BuildRequires:  ruby(sqlite3)
+%endif
+Requires:       ruby(abi) = 1.8
+Requires:       ruby(active_support) = 2.0.1
+Provides:       ruby(active_record) = %{version}
+
+%description
+Implements the ActiveRecord pattern (Fowler, PoEAA) for ORM. It ties
+database tables and classes together for business objects, like Customer or
+Subscription, that can find, save, and destroy themselves without resorting
+to manual SQL.
+
+%package subpkg
+Summary: Test sub package
+Group:   Development/Languages
+
+%package -n another-subpkg
+Summary: Another test sub package
+Group:   Development/Languages
+
+%description subpkg
+Test subpackage
+
+%description -n another-subpkg
+Another est subpackage
+
+%prep
+%setup -q -n %{rname}-%{version}
+chmod 0644 README
+
+%build
+
+%install
+rm -rf $RPM_BUILD_ROOT
+install -d $RPM_BUILD_ROOT/%{ruby_sitelib}/
+cp -pr lib/* $RPM_BUILD_ROOT/%{ruby_sitelib}/
+find $RPM_BUILD_ROOT/%{ruby_sitelib} -type f | xargs chmod a-x
+
+%check
+%if %with_check
+cd test
+ruby -I "connections/native_sqlite3" base_test.rb
+%endif
+
+%clean
+rm -rf $RPM_BUILD_ROOT
+
+%files
+%defattr(-,root,root,-)
+%{ruby_sitelib}/activerecord.rb
+%{ruby_sitelib}/active_record.rb
+%{ruby_sitelib}/active_record
+%doc README CHANGELOG examples/
+
+%files subpkg
+%defattr(-,root,root,-)
+%doc README
+
+%files -n another-subpkg
+%defattr(-,root,root,-)
+%doc CHANGELOG
+
+}

data/spec/gem_adapter_spec.rb

@@ -0,0 +1,89 @@
+# ruby gem polisher gem adapter spec
+#
+# Copyright (C) 2010 Red Hat, Inc.
+# Written by Mohammed Morsi <mmorsi@redhat.com>
+#
+# This program is free software, you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License
+# as published by the Free Software Foundation, either version 3
+# of the License, or (at your option) any later version.
+# 
+# You should have received a copy of the the GNU Affero
+# General Public License, along with Polisher. If not, see 
+# <http://www.gnu.org/licenses/>
+
+require 'uri'
+require 'json'
+require 'net/http'
+
+require File.dirname(__FILE__) + '/spec_helper'
+
+describe "Polisher::GemAdapter" do
+  it "should successfully subscribe/unsubscribe to updates" do
+     gem = Source.new :name => "polisher", :source_type => "gem", 
+                      :uri => "http://rubygems.org/downloads/polisher-0.3.gem"
+
+     Polisher::GemAdapter.subscribe(gem,  
+                                    "http://projects.morsi.org/polisher/demo/gems/released/1", 
+                                    POLISHER_CONFIG["gem_api_key"])
+     Polisher::GemAdapter.subscribed?(gem, POLISHER_CONFIG["gem_api_key"]).should == true
+     Polisher::GemAdapter.unsubscribe(gem,  
+                                    "http://projects.morsi.org/polisher/demo/gems/released/1", 
+                                    POLISHER_CONFIG["gem_api_key"])
+     Polisher::GemAdapter.subscribed?(gem, POLISHER_CONFIG["gem_api_key"]).should == false
+  end
+
+  it "should raise error if subscribe source, callback_url, or api_key is invalid" do
+     lambda { 
+       Polisher::GemAdapter.subscribe(nil, 
+                                      "http://projects.morsi.org/polisher/demo/gems/released/1", 
+                                      POLISHER_CONFIG['gem_api_key'])
+     }.should raise_error(ArgumentError)
+
+     gem = Source.new :name => "polisher", :source_type => "gem", 
+                      :uri => "http://rubygems.org/downloads/polisher-0.3.gem"
+     lambda { 
+       Polisher::GemAdapter.subscribe(gem, 42, POLISHER_CONFIG['gem_api_key']) 
+     }.should raise_error(ArgumentError)
+
+     lambda {
+       Polisher::GemAdapter.subscribe(gem, 
+                                      "http://projects.morsi.org/polisher/demo/gems/released/1", 
+                                      nil)
+     }.should raise_error(ArgumentError)
+  end
+
+  it "should raise error if subscription target is invalid" do
+    gem = Source.new :name => "polisher", :source_type => "gem", 
+                     :uri => "http://non.existant/downloads/polisher-0.3.gem"
+    lambda {
+      Polisher::GemAdapter.subscribe(gem,  
+                                     "http://projects.morsi.org/polisher/demo/gems/released/1", 
+                                     POLISHER_CONFIG["gem_api_key"])
+    }.should raise_error(RuntimeError)
+
+    gem = Source.new :name => "polisher", :source_type => "gem", 
+                     :uri => "http://morsi.org/downloads/polisher-0.3.gem"
+    lambda {
+      Polisher::GemAdapter.subscribe(gem,  
+                                     "http://projects.morsi.org/polisher/demo/gems/released/1", 
+                                     POLISHER_CONFIG["gem_api_key"])
+    }.should raise_error(RuntimeError)
+  end
+
+  it "should successfully get remote gem info" do
+     gem = Source.new :name => "polisher", :source_type => "gem", 
+                      :uri => "http://rubygems.org/downloads/polisher-0.3.gem"
+     info = Polisher::GemAdapter.get_info(gem)
+     info["name"].should == "polisher"
+  end
+
+  it "should raise error if get info target is invalid" do
+    gem = Source.new :name => "polisher", :source_type => "gem", 
+                     :uri => "http://invalid.uri/downloads/polisher-0.3.gem"
+     lambda {
+       Polisher::GemAdapter.get_info(gem)
+     }.should raise_error(RuntimeError)
+  end
+
+end

data/spec/models_spec.rb

@@ -7,177 +7,664 @@
 # it under the terms of the GNU Affero General Public License
 # as published by the Free Software Foundation, either version 3
 # of the License, or (at your option) any later version.
-# 
+#
 # You should have received a copy of the the GNU Affero
-# General Public License, along with Polisher. If not, see 
+# General Public License, along with Polisher. If not, see
 # <http://www.gnu.org/licenses/>
 
 require File.dirname(__FILE__) + '/spec_helper'
 
+describe "Polisher::Project" do
+  before(:each) do
+    FileUtils.rm_rf(ARTIFACTS_DIR) if File.directory? ARTIFACTS_DIR
+    FileUtils.mkdir_p(ARTIFACTS_DIR)
+  end
 
-describe "Polisher::Source" do
-   it "should not be valid if name or uri is missing" do
-      src = Source.new :name => 'foo', :uri => 'bar'
-      src.valid?.should be(true)
+  after(:each) do
+    FileUtils.rm_rf(ARTIFACTS_DIR) if File.directory? ARTIFACTS_DIR
+  end
 
-      src.name = nil
-      src.valid?.should be(false)
-      src.name = 'foo'
+  it "should not be valid if name is missing" do
+    project = Project.new :name => 'foo'
+    project.should be_valid
 
-      src.uri = nil
-      src.valid?.should be(false)
-   end
+    project.name = nil
+    project.should_not be_valid
+  end
 
-   it "should clean uri properly" do
-      src = Source.new :uri => 'http://example.com/'
-      src.clean_uri!
-      src.uri.should == 'http://example.com'
-   end
+  it "should not be valid if duplicate name exists" do
+    Project.create! :name => 'dup-project-name-test'
+    project = Project.new :name => 'dup-project-name-test'
+    project.should_not be_valid
+  end
+
+  it "should provide access to all dependencies and dependent projects" do
+    project1 = Project.create! :name => "project-dependency-tree1"
+    project2 = Project.create! :name => "project-dependency-tree2"
+    project3 = Project.create! :name => "project-dependency-tree3"
+    project4 = Project.create! :name => "project-dependency-tree4"
+    project5 = Project.create! :name => "project-dependency-tree5"
+
+    pd1 = ProjectDependency.create! :project => project2, :depends_on_project => project1
+    pd2 = ProjectDependency.create! :project => project3, :depends_on_project => project2
+    pd3 = ProjectDependency.create! :project => project4, :depends_on_project => project2
+    pd4 = ProjectDependency.create! :project => project4, :depends_on_project => project5
+
+    deps = project1.dependencies
+    deps.size.should == 0
+    depts = project1.dependents
+    depts.size.should == 1
+    depts[0].project.id.should == project2.id
+
+    deps = project2.dependencies
+    deps.size.should == 1
+    deps[0].depends_on_project.id.should == project1.id
+    depts = project2.dependents
+    depts.size.should == 2
+    depts[0].project.id.should == project3.id
+    depts[1].project.id.should == project4.id
+
+    deps = project3.dependencies
+    deps.size.should == 1
+    deps[0].depends_on_project.id.should == project2.id
+    depts = project3.dependents
+    depts.size.should == 0
+
+    deps = project4.dependencies
+    deps.size.should == 2
+    deps[0].depends_on_project.id.should == project2.id
+    deps[1].depends_on_project.id.should == project5.id
+    depts = project4.dependents
+    depts.size.should == 0
+
+    deps = project5.dependencies
+    deps.size.should == 0
+    depts = project5.dependents
+    depts.size.should == 1
+    depts[0].project.id.should == project4.id
+  end
+
+  it "should download all sources" do
+    project = Project.create! :name => 'project-dl-test100'
+
+    # see FIXME in Project::download_to
+    source1 = Source.create!  :name => 'jffi-spec', :uri => 'http://mo.morsi.org/files/jruby/jffi.spec', :source_type => 'spec'
+    source2 = Source.create!  :name => 'joni-spec', :uri => 'http://mo.morsi.org/files/jruby/joni.spec', :source_type => 'spec'
+
+    ps1 = ProjectSourceVersion.create! :project => project, :source => source1
+    ps2 = ProjectSourceVersion.create! :project => project, :source => source2, :project_version => "1.5"
+
+    project.download_to :dir => ARTIFACTS_DIR
+
+    File.size?(ARTIFACTS_DIR + '/joni.spec').should_not be_nil
+    File.size?(ARTIFACTS_DIR + '/jffi.spec').should_not be_nil
+  end
+
+  it "should download all sources for specified version" do
+    project = Project.new :name => 'project-dl-test100'
+    source1 = Source.new  :name => 'jffi-spec', :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/jffi.spec', :source_type => 'spec'
+    source2 = Source.new  :name => 'joni-spec', :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/joni.spec', :source_type => 'spec'
+
+    ps1 = ProjectSourceVersion.new :project => project, :source => source1, :project_version => "1.6", :source_uri_params => "cluster=jruby;filetype=spec"
+    ps2 = ProjectSourceVersion.new :project => project, :source => source2, :project_version => "1.5", :source_uri_params => "cluster=jruby;dir=files"
+    project.project_source_versions << ps1 << ps2
+
+    project.download_to :version => "1.5", :dir => ARTIFACTS_DIR
+
+    File.size?(ARTIFACTS_DIR + '/joni.spec').should_not be_nil
+    File.size?(ARTIFACTS_DIR + '/jffi.spec').should be_nil
+  end
+
+  it "should return all events for the specified version" do
+    project = Project.new :name => 'project-dl-test100'
+    event1  = Event.new :version_qualifier => "=", :version => "1.5"
+    event2  = Event.new :version_qualifier => ">", :version => "1.6"
+    project.events << event1 << event2
+
+    events = project.events_for_version("1.5")
+    events.size.should == 1
+    events.include?(event1).should be_true
+    events.include?(event2).should be_false
+  end
+
+  it "should return all project_source_versions for the specified version" do
+    project = Project.new :name => 'project-dl-test100'
+    source1 = Source.new  :name => 'jffi-spec', :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/jffi.spec', :source_type => 'spec'
+    source2 = Source.new  :name => 'joni-spec', :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/joni.spec', :source_type => 'spec'
+    source3 = Source.new  :name => 'jruby-spec', :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/jruby.spec', :source_type => 'spec'
+
+    ps1 = ProjectSourceVersion.new :project => project, :source => source1, :project_version => '1.6', :source_uri_params => "cluster=jruby;filetype=spec"
+    ps2 = ProjectSourceVersion.new :project => project, :source => source2, :project_version => "1.5", :source_uri_params => "cluster=jruby;dir=files"
+    ps3 = ProjectSourceVersion.new :project => project, :source => source3, :source_uri_params => "cluster=jruby;filetype=spec"
+    project.project_source_versions << ps1 << ps2 << ps3
+
+    project_sources = project.project_source_versions_for_version("1.5")
+    project_sources.size.should == 2
+    project_sources.include?(ps1).should be_false
+    project_sources.include?(ps2).should be_true
+    project_sources.include?(ps3).should be_true
+  end
+
+  it "should return all sources for the specified version with formatted uris" do
+    project = Project.new :name => 'project-dl-test100'
+    source1 = Source.new  :name => 'jffi-spec',  :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/jffi.spec',  :source_type => 'spec'
+    source2 = Source.new  :name => 'joni-spec',  :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/joni.spec',  :source_type => 'spec'
+    source3 = Source.new  :name => 'jruby-spec', :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/jruby.spec', :source_type => 'spec'
+
+    ps1 = ProjectSourceVersion.new :project => project, :source => source1, :project_version => "1.6", :source_uri_params => "cluster=jruby;filetype=spec"
+    ps2 = ProjectSourceVersion.new :project => project, :source => source2, :project_version => "1.5", :source_uri_params => "cluster=jruby;dir=files"
+    ps3 = ProjectSourceVersion.new :project => project, :source => source3, :source_uri_params => "cluster=jruby;dir=files"
+    project.project_source_versions << ps1 << ps2 << ps3
+
+    sources = project.sources_for_version("1.5")
+    sources.size.should == 2
+    sources.include?(source1).should be_false
+    sources.include?(source2).should be_true
+    sources.include?(source3).should be_true
+
+    sources[0].uri.should == "http://mo.morsi.org/files/jruby/joni.spec"
+    sources[1].uri.should == "http://mo.morsi.org/files/jruby/jruby.spec"
+  end
+
+  it "should provide access to primary source" do
+    project = Project.create! :name => 'primary-source-project-test'
+    source1 = Source.create!(:name => 'primary-source-test1', :source_type => 'file', :uri => 'http://foo1.foo')
+    source2 = Source.create!(:name => 'primary-source-test2', :source_type => 'file', :uri => 'http://foo2.foo')
+    project.sources << source1
+    project.primary_source= source2
+
+    primary_src = project.primary_source
+    primary_src.should_not be_nil
+    primary_src.name.should == source2.name
+  end
+
+  it "should provide access to all dependencies for the specified version" do
+    project1 = Project.create! :name => "project-dependency-tree10"
+    project2 = Project.create! :name => "project-dependency-tree20"
+    project3 = Project.create! :name => "project-dependency-tree30"
+    project4 = Project.create! :name => "project-dependency-tree40"
+
+    pd1 = ProjectDependency.create! :project => project1, :depends_on_project => project2, :project_version => "1.0.0"
+    pd2 = ProjectDependency.create! :project => project1, :depends_on_project => project3, :project_version => "2.0.0"
+    pd2 = ProjectDependency.create! :project => project1, :depends_on_project => project4
+
+    deps = project1.dependencies_for_version("1.0.0")
+    deps.size.should == 2
+    deps[0].depends_on_project.id.should == project2.id
+    deps[1].depends_on_project.id.should == project4.id
+  end
+
+  it "should return all versions which the project is configured for" do
+    project = Project.new :name => 'project-dl-test100'
+    event1  = Event.new :version_qualifier => "=", :version => "1.5"
+    event2  = Event.new :version_qualifier => ">", :version => "1.6"
+    project.events << event1 << event2
+
+    source1 = Source.new  :name => 'jffi-spec',  :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/jffi.spec',  :source_type => 'spec'
+    source2 = Source.new  :name => 'joni-spec',  :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/joni.spec',  :source_type => 'spec'
+    source3 = Source.new  :name => 'jruby-spec', :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/jruby.spec', :source_type => 'spec'
+    ps1 = ProjectSourceVersion.new :project => project, :source => source1, :source_uri_params => "cluster=jruby;filetype=spec"
+    ps2 = ProjectSourceVersion.new :project => project, :source => source2, :project_version => "1.5", :source_uri_params => "cluster=jruby;dir=files"
+    ps3 = ProjectSourceVersion.new :project => project, :source => source3, :project_version => "1.7", :source_uri_params => "cluster=jruby;dir=files"
+    project.project_source_versions << ps1 << ps2 << ps3
+
+    projectd = Project.new :name => 'project-version-test99'
+    project.project_dependencies << ProjectDependency.new(:project => project, :depends_on_project => projectd, :project_version => "9.9")
+
+    versions = project.versions
+    versions.size.should == 4
+    versions.include?("1.5").should be_true
+    versions.include?("1.6").should be_true
+    versions.include?("1.7").should be_true
+    versions.include?("9.9").should be_true
+  end
 
+  it "should trigger dependencies and events upon release" do
+    project   = Project.create :name => 'project-release-test-199'
+
+    Event.create :project => project,
+                 :process => "project_released_test_handlerAAA",
+                 :version_qualifier => '=',
+                 :version => "5.6"
+
+    Event.create :project => project,
+                 :process => "project_released_test_handlerBBB",
+                 :version_qualifier => '>',
+                 :version => "7.9"
+
+    projectd1 = Project.create :name => 'project-release-test-299'
+    Event.create :project => projectd1,
+                 :process => "project_released_test_handlerDDD"
+    Event.create :project => projectd1,
+                 :process => "project_released_test_handlerEEE",
+                 :version_qualifier => "=",
+                 :version => "1.9"
+
+    projectd2 = Project.create :name => 'project-release-test-399'
+    Event.create :project => projectd2,
+                 :process => "project_released_test_handlerFFF",
+                 :version_qualifier => "=",
+                 :version => "1.9"
+    Event.create :project => projectd2,
+                 :process => "project_released_test_handlerGGG",
+                 :version_qualifier => "=",
+                 :version => "6.5"
+
+    projectd3 = Project.create :name => 'project-release-test-499'
+    Event.create :project => projectd3,
+                 :process => "project_released_test_handlerCCC"
+
+    pd1 = ProjectDependency.new :project => project, :depends_on_project => projectd1, :depends_on_project_params => "jim=bo;john=mark"
+    pd2 = ProjectDependency.new :project => project, :depends_on_project => projectd2, :project_version => "5.6", :depends_on_project_version => "6.5"
+    pd3 = ProjectDependency.new :project => project, :depends_on_project => projectd3, :project_version => "7.9"
+    project.project_dependencies << pd1 << pd2 << pd3
+
+    project.released_version("5.6", :joe => 'bob', :sam => 'sue')
+
+    $project_released_test_handler_flags.include?('AAA').should be_true
+    $project_released_test_handler_flags.include?('BBB').should be_false
+    $project_released_test_handler_flags.include?('CCC').should be_false
+    $project_released_test_handler_flags.include?('DDD').should be_true
+    $project_released_test_handler_flags.include?('EEE').should be_true
+    $project_released_test_handler_flags.include?('FFF').should be_false
+    $project_released_test_handler_flags.include?('GGG').should be_true
+    $project_released_test_handler_flags.include?('joe=bob').should be_true
+    $project_released_test_handler_flags.include?('sam=sue').should be_true
+    $project_released_test_handler_flags.include?('jim=bo').should be_true
+    $project_released_test_handler_flags.include?('john=mark').should be_true
+  end
 end
 
+describe "Polisher::Source" do
+  before(:each) do
+    FileUtils.rm_rf(ARTIFACTS_DIR) if File.directory? ARTIFACTS_DIR
+    FileUtils.mkdir_p(ARTIFACTS_DIR)
+  end
+
+  after(:each) do
+    FileUtils.rm_rf(ARTIFACTS_DIR) if File.directory? ARTIFACTS_DIR
+  end
 
-describe "Polisher::ManagedGem" do
-  it "should not be valid if name or source is missing" do
-      gem = ManagedGem.new :name => 'foo', :source_id => 1
-      gem.valid?.should be(true)
+  it "should not be valid if name, source_type, or uri is missing or invalid" do
+      source = Source.new :uri => 'uri', :name => "foo", :source_type => "gem"
+      source.should be_valid
 
-      gem.name = nil
-      gem.valid?.should be(false)
-      gem.name = 'foo'
+      source.uri = nil
+      source.should_not be_valid
+      source.uri = 'uri'
 
-      gem.source_id = nil
-      gem.valid?.should be(false)
+      source.source_type = nil
+      source.should_not be_valid
+      source.source_type = 'bar'
+      source.should_not be_valid
+      source.source_type = 'file'
+
+      source.name = nil
+      source.should_not be_valid
   end
 
-  it "should not be valid if name is not unique within source scope" do
-     gem1 = ManagedGem.create :name => 'foo', :source_id => 1
-     gem2 = ManagedGem.new :name => 'foo', :source_id => 1
-     gem2.valid?.should be(false)
+  it "should not be valid if name or uri is not unique in scope" do
+      source1 = Source.create! :uri => 'uri', :name => "foo", :source_type => "gem"
+      source2 = Source.new :uri => 'uri', :name => "bar", :source_type => "archive"
+      source2.should_not be_valid
+
+      source3 = Source.new :uri => 'zaz', :name => "foo", :source_type => "archive"
+      source3.should_not be_valid
   end
 
-  it "should generate correct source uri from gem uri" do
-     ManagedGem.uri_to_source_uri('http://rubygems.org/downloads/polisher-0.1.gem').
-         should == 'http://rubygems.org'
+  it "should return filename generated from uri" do
+     source = Source.new(
+        :uri => 'http://mo.morsi.org/files/jruby/joni-123.spec?addition=foo&params=bar',
+        :name => "joni-spec", :source_type => "spec")
+     source.filename.should == "joni-123.spec"
   end
 
-  it "should successfully subscribe/unsubscribe to updates" do
-     gem = ManagedGem.new :name => "polisher", :source_id => 1
-     gem.subscribe :callback_url => "http://projects.morsi.org/polisher/demo/gems/updated/1"
-     gem.subscribed?.should == true
-     gem.unsubscribe :callback_url => "http://projects.morsi.org/polisher/demo/gems/updated/1"
-     gem.subscribed?.should == false
+  it "should return all project_source_versions for the specified version" do
+    project1 = Project.new :name => 'project-src-p100'
+    project2 = Project.new :name => 'project-src-p101'
+    project3 = Project.new :name => 'project-src-p102'
+    source = Source.new  :name => 'jffi-spec', :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/jffi.spec', :source_type => 'spec'
+
+    ps1 = ProjectSourceVersion.new :project => project1, :source => source, :source_uri_params => "cluster=jruby;filetype=spec"
+    ps2 = ProjectSourceVersion.new :project => project2, :source => source, :project_version => "1.5", :source_uri_params => "cluster=jruby;dir=files"
+    ps3 = ProjectSourceVersion.new :project => project3, :source => source, :project_version => "1.6", :source_uri_params => "cluster=jruby;dir=files"
+    source.project_source_versions << ps1 << ps2
+
+    project_sources = source.project_source_versions_for_version("1.5")
+    project_sources.size.should == 2
+    project_sources.include?(ps1).should be_true
+    project_sources.include?(ps2).should be_true
+    project_sources.include?(ps3).should be_false
   end
 
-  it "should successfully get remote gem info" do
-     gem = ManagedGem.new :name => "polisher", :source_id => 1
-     info = gem.get_info
-     info["name"].should == "polisher"
+  it "should return all projects for the specified version" do
+    project1 = Project.new :name => 'project-src-p100'
+    project2 = Project.new :name => 'project-src-p101'
+    project3 = Project.new :name => 'project-src-p102'
+    source = Source.new  :name => 'jffi-spec', :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/jffi.spec', :source_type => 'spec'
+
+    ps1 = ProjectSourceVersion.new :project => project1, :source => source, :source_uri_params => "cluster=jruby;filetype=spec"
+    ps2 = ProjectSourceVersion.new :project => project2, :source => source, :source_version => "1.5", :source_uri_params => "cluster=jruby;dir=files"
+    ps3 = ProjectSourceVersion.new :project => project3, :source => source, :source_version => "1.6", :source_uri_params => "cluster=jruby;dir=files"
+    source.project_source_versions << ps1 << ps2 << ps3
+
+    projects = source.project_source_versions_for_version("1.5")
+    projects.size.should == 2
+    projects.collect { |p| p.source_version }.include?(nil).should be_true
+    projects.collect { |p| p.source_version }.include?("1.5").should be_true
+    projects.collect { |p| p.source_version }.include?("1.6").should be_false
+
+    projects.collect { |p| p.project.name }.include?("project-src-p100").should be_true
+    projects.collect { |p| p.project.name }.include?("project-src-p101").should be_true
+    projects.collect { |p| p.project.name }.include?("project-src-p102").should be_false
   end
 
-  it "should successfully download gem" do
-     FileUtils.rm_rf(ARTIFACTS_DIR) if File.directory? ARTIFACTS_DIR
-     FileUtils.mkdir_p(ARTIFACTS_DIR)
+  it "should return all versions which the source is configured for" do
+    project1 = Project.new :name => 'project-src-p100'
+    project2 = Project.new :name => 'project-src-p101'
+    project3 = Project.new :name => 'project-src-p102'
+    source = Source.new  :name => 'jffi-spec', :uri => 'http://mo.morsi.org/%{dir}/%{cluster}/jffi.spec', :source_type => 'spec'
+
+    ps1 = ProjectSourceVersion.new :project => project1, :source => source, :source_uri_params => "cluster=jruby;filetype=spec"
+    ps2 = ProjectSourceVersion.new :project => project2, :source => source, :source_version => "1.5", :source_uri_params => "cluster=jruby;dir=files"
+    ps3 = ProjectSourceVersion.new :project => project3, :source => source, :source_version => "1.6", :source_uri_params => "cluster=jruby;dir=files"
+    source.project_source_versions << ps1 << ps2 << ps3
+
+    versions = source.versions
+    versions.size.should == 2
+    versions.include?("1.5").should be_true
+    versions.include?("1.6").should be_true
+  end
 
-     gem = ManagedGem.new :name => "polisher", :source_id => 1
-     path = gem.download_to(:dir => ARTIFACTS_DIR, :version => 0.1)
-     File.size?(ARTIFACTS_DIR + '/polisher-0.1.gem').should_not be_nil
-     FileUtils.rm(ARTIFACTS_DIR + '/polisher-0.1.gem')
-     path.should == ARTIFACTS_DIR + '/polisher-0.1.gem'
+  it "should format the uri w/ the specified variables" do
+    source = Source.new :uri => "http://%{var1}.%{var2}/%{another}?%{yet_another}=%{even_more}&%{last}"
+    source.format_uri! :var1 => 'val1', :var2 => 'val2',
+                       :another => 'foo', :yet_another => 'bar',
+                       :even_more => 'something', :last => '123'
 
+    source.uri.should == "http://val1.val2/foo?bar=something&123"
+  end
 
-     gem.download_to(:path => ARTIFACTS_DIR + '/my.gem', :version => 0.1)
-     File.size?(ARTIFACTS_DIR + '/my.gem').should_not be_nil
+  it "should be downloadable" do
+     source = Source.new(
+        :uri => 'http://mo.morsi.org/files/jruby/joni.spec',
+        :name => "joni-spec", :source_type => "spec")
+     path = source.download_to(:dir => ARTIFACTS_DIR)
+     File.size?(ARTIFACTS_DIR + '/joni.spec').should_not be_nil
+     FileUtils.rm(ARTIFACTS_DIR + '/joni.spec')
+     path.should == ARTIFACTS_DIR + '/joni.spec'
+
+     source.download_to(:path => ARTIFACTS_DIR + '/joni.spec')
+     File.size?(ARTIFACTS_DIR + '/joni.spec').should_not be_nil
+  end
 
-     gem.download_to(:dir => ARTIFACTS_DIR)
-     File.size?(ARTIFACTS_DIR + '/polisher-0.1.gem').should_not be_nil
+  it "should permit a parameterized download" do
+     source = Source.new(
+        :uri => 'http://mo.morsi.org/files/%{group}/%{name}.spec',
+        :name => "joni-spec", :source_type => 'spec')
+     path = source.download_to(:dir => ARTIFACTS_DIR, :group => "jruby", :name => "joni")
+     File.size?(ARTIFACTS_DIR + '/joni.spec').should_not be_nil
+     FileUtils.rm(ARTIFACTS_DIR + '/joni.spec')
+     path.should == ARTIFACTS_DIR + '/joni.spec'
+
+     source.download_to(:path => ARTIFACTS_DIR + '/joni.spec')
+     File.size?(ARTIFACTS_DIR + '/joni.spec').should_not be_nil
   end
 
+  it "should raise an exception if download source uri or destination path is invalid" do
+     source = Source.new(
+        :uri => 'http://invalid.uri',
+        :name => 'invalid-source1', :source_type => 'file')
+     lambda {
+       path = source.download_to(:dir => ARTIFACTS_DIR)
+     }.should raise_error(RuntimeError)
+
+     source = Source.new(
+        :uri => 'http://mo.morsi.org/files/jruby/joni.spec',
+        :name => 'invalid-source2', :source_type => 'spec')
+     lambda {
+       path = source.download_to(:dir => '/')
+     }.should raise_error(RuntimeError)
+
+     lambda {
+       path = source.download_to(:dir => '/nonexistantfoobar')
+     }.should raise_error(RuntimeError)
+  end
 end
 
-describe "Polisher::Event" do
+describe "Polisher::ProjectSourceVersion" do
+
+   it "should default primary_source to false" do
+    project = Project.create! :name => 'project-source-valid-testproj0'
+    source  = Source.create!  :name => 'project-source-valid-testsource0',
+                              :uri  => 'http://presvts234', :source_type => 'file'
+    ps = ProjectSourceVersion.create! :project => project, :source => source
+    ps.primary_source.should be_false
+   end
+
+   it "should default versions to nil" do
+    project = Project.create! :name => 'project-source-valid-testproj10'
+    source  = Source.create!  :name => 'project-source-valid-testsource10',
+                              :uri  => 'http://presvts098', :source_type => 'file'
+    ps = ProjectSourceVersion.create! :project => project, :source => source, :project_version => "", :source_version => ""
+    ps.project_version.should be_nil
+    ps.source_version.should be_nil
+   end
+
+   it "should not be valid if a project id/version is associated w/ a source multiple times" do
+    project = Project.create! :name => 'project-source-valid-testproj1'
+    source  = Source.create!  :name => 'project-source-valid-testsource1',
+                              :uri  => 'http://presvts581', :source_type => 'file'
+
+    ps1 = ProjectSourceVersion.create! :project => project, :source => source, :project_version => '1.6'
+    ps2 = ProjectSourceVersion.new :project => project, :source => source, :project_version => '1.6'
+
+    ps2.valid?.should be_false
+   end
+
+   it "should not be valid if a project id/version is associated w/ multiple primary sources" do
+    project = Project.create! :name => 'project-source-valid-testproj2'
+    source1 = Source.create!  :name => 'project-source-valid-testsource100',
+                              :uri  => 'http://presvts690', :source_type => 'file'
+    source2 = Source.create!  :name => 'project-source-valid-testsource200',
+                              :uri  => 'http://presvts456', :source_type => 'file'
+
+    ps1 = ProjectSourceVersion.create! :project => project, :source => source1, :project_version => '1.6',
+                                 :primary_source => true
+
+    ps2 = ProjectSourceVersion.new :project => project, :source => source2, :project_version => '1.6',
+                                 :primary_source => true
 
-   it "should not be valid if managed_gem or process is missing" do
-      gem = ManagedGem.create :name => 'valid-event-test-gem1', :source_id => 1
+    ps2.valid?.should be_false
+   end
+
+end
+
+describe "Polisher::ProjectDependency" do
+
+  it "should default versions to nil" do
+    project1 = Project.create! :name => 'project-dep-valid-testproj10'
+    project2 = Project.create! :name => 'project-dep-valid-testproj20'
+
+    pd = ProjectDependency.create! :project => project1, :depends_on_project => project2
+    pd.project_version.should be_nil
+    pd.depends_on_project_version.should be_nil
+  end
+
+  it "should not be valid if missing project or depends_on_project" do
+    project1 = Project.create! :name => 'project-dep-valid-testproj30'
+    project2 = Project.create! :name => 'project-dep-valid-testproj40'
+
+    pd = ProjectDependency.new
+    pd.should_not be_valid
+
+     
+    pd = ProjectDependency.new :project => project1
+    pd.should_not be_valid
+
+    pd = ProjectDependency.new :depends_on_project => project2
+    pd.should_not be_valid
+
+    pd = ProjectDependency.new :project => project1, :depends_on_project => project2
+    pd.should be_valid
+  end
+
+  it "should not be valid if project/depends_on_project/project_version/depends_on_project_version are not unique" do
+    project1 = Project.create! :name => 'project-dep-valid-testproj50'
+    project2 = Project.create! :name => 'project-dep-valid-testproj60'
+
+    pd = ProjectDependency.create! :project => project1, :depends_on_project => project2, :project_version => "1.0.0", :depends_on_project_version => "2.0.1"
 
-      event = Event.new :managed_gem_id => gem.id, :process => 'create_repo'
-      event.valid?.should be(true)
+    pd2 = ProjectDependency.new :project => project1, :depends_on_project => project2, :project_version => "1.0.0", :depends_on_project_version => "2.0.1"
+    pd2.should_not be_valid
 
-      event.managed_gem = nil
-      event.valid?.should be(false)
-      event.managed_gem = gem
+    pd2.depends_on_project_version = "2.0.2"
+    pd2.should be_valid
+  end
+
+end
+
+describe "Polisher::Event" do
+
+   it "should return list of supported event handlers" do
+     processes = Event::processes
+     processes.size.should == 3
+     processes.include?("download_sources").should be_true
+     processes.include?("create_rpm_package").should be_true
+     processes.include?("update_yum_repo").should be_true
+   end
+
+   it "should not be valid if process is missing" do
+      project = Project.create! :name => 'valid-event-test-project0'
+      event = Event.new :project_id => project.id, :process => 'create_repo'
+      event.should be_valid
 
       event.process = nil
-      event.valid?.should be(false)
+      event.should_not be_valid
+   end
+
+   it "should not be valid if project is missing" do
+      project = Project.create! :name => 'valid-event-test-project1'
+      event = Event.new :process => 'create_repo'
+      event.should_not be_valid
+      event.project = project
+      event.should be_valid
    end
 
    it "should not be valid with invalid version qualifier" do
-      gem = ManagedGem.create :name => 'valid-event-test-gem2', :source_id => 1
+      project = Project.create! :name => 'valid-event-test-project2'
 
-      event = Event.new :managed_gem_id => gem.id, :process => 'create_repo', :version_qualifier => ">", :gem_version => 5
-      event.valid?.should be(true)
+      event = Event.new :project_id => project.id, :process => 'create_repo', :version_qualifier => ">", :version => 5
+      event.should be_valid
 
       event.version_qualifier = '=='
-      event.valid?.should be(false)
+      event.should_not be_valid
    end
 
    it "should not be valid if version/qualifier are not both present or nil" do
-      gem = ManagedGem.create :name => 'valid-event-test-gem3', :source_id => 1
- 
-      event = Event.new :managed_gem_id => gem.id, :process => 'create_repo', :version_qualifier => ">"
-      event.valid?.should be(false)
- 
-      event = Event.new :managed_gem_id => gem.id, :process => 'create_repo', :gem_version => 5
-      event.valid?.should be(false)
-
-      event = Event.new :managed_gem_id => gem.id, :process => 'create_repo', :gem_version => 5, :version_qualifier => '<='
-      event.valid?.should be(true)
+      project = Project.create! :name => 'valid-event-test-project3'
+
+      event = Event.new :project_id => project.id, :process => 'create_repo', :version_qualifier => ">"
+      event.should_not be_valid
+
+      event = Event.new :project_id => project.id, :process => 'create_repo', :version => 5
+      event.should_not be_valid
+
+      event = Event.new :project_id => project.id, :process => 'create_repo', :version => 5, :version_qualifier => '<='
+      event.should be_valid
    end
 
    it "should correctly resolve version qualifiers" do
-      event = Event.new :version_qualifier => nil
+      event = Event.new :version_qualifier => nil, :version => "5"
       event.applies_to_version?('1.1').should be(true)
+      event.applies_to_version?('1.5.3').should be(true)
 
-      event = Event.new :version_qualifier => "=", :gem_version => "5.3"
+      event = Event.new :version_qualifier => "=", :version => "5.3"
       event.applies_to_version?('1.2').should be(false)
       event.applies_to_version?('5.3').should be(true)
+      event.applies_to_version?('5.3.0').should be(false)
+      event.applies_to_version?('5.3.1').should be(false)
       event.applies_to_version?('7.9').should be(false)
 
-      event = Event.new :version_qualifier => ">", :gem_version => "1.9"
-      event.applies_to_version?('1.8').should be(false)
-      event.applies_to_version?('1.9').should be(false)
+      event = Event.new :version_qualifier => ">", :version => "1.9.2"
+      event.applies_to_version?('1.8.1').should be(false)
+      event.applies_to_version?('1.9.1').should be(false)
+      event.applies_to_version?('1.9.3').should be(true)
       event.applies_to_version?('2.0').should be(true)
 
-      event = Event.new :version_qualifier => "<", :gem_version => "0.6"
+      event = Event.new :version_qualifier => "<", :version => "0.6"
       event.applies_to_version?('0.5').should be(true)
       event.applies_to_version?('0.6').should be(false)
       event.applies_to_version?('0.7').should be(false)
 
-      event = Event.new :version_qualifier => ">=", :gem_version => "1.9"
+      event = Event.new :version_qualifier => ">=", :version => "1.9"
       event.applies_to_version?('1.8').should be(false)
       event.applies_to_version?('1.9').should be(true)
       event.applies_to_version?('2.0').should be(true)
 
-      event = Event.new :version_qualifier => "<=", :gem_version => "0.6"
-      event.applies_to_version?('0.5').should be(true)
-      event.applies_to_version?('0.6').should be(true)
-      event.applies_to_version?('0.7').should be(false)
+      event = Event.new :version_qualifier => "<=", :version => "0.6.4"
+      event.applies_to_version?('0.5.2').should be(true)
+      event.applies_to_version?('0.6.1').should be(true)
+      event.applies_to_version?('0.6.6').should be(false)
+      event.applies_to_version?('0.7.4').should be(false)
+   end
+
+   it "should raise error if trying to compare invalid versions" do
+      event = Event.new
+
+      # nil event version and version_qualifier is permitted
+      #lambda {
+      #  event.applies_to_version?('0.5.2')
+      #}.should raise_error(ArgumentError)
+
+      event.version = '0.7'
+      lambda {
+        event.applies_to_version?(111)
+      }.should raise_error(ArgumentError)
    end
 
    it "should successfully run event process" do
-      gem = ManagedGem.new :name => "foobar"
-      event = Event.new :managed_gem => gem, :process => "test_event_run_method", :process_options => "a;b;c"
-      event.run
-
-      $test_event_run_hash[:gem].should_not be_nil
-      $test_event_run_hash[:first].should_not be_nil
-      $test_event_run_hash[:second].should_not be_nil
-      $test_event_run_hash[:third].should_not be_nil
-
-      $test_event_run_hash[:gem].name.should == "foobar"
-      $test_event_run_hash[:first].should == "a"
-      $test_event_run_hash[:second].should == "b"
-      $test_event_run_hash[:third].should == "c"
+      project = Project.new :name => "foobar"
+      event = Event.new :project => project, :process => "test_event_run_method"
+      event.run(:version => "5.0", :key1 => "val1", :some => "thing", :answer => 42)
+
+      $test_event_run_hash[:event].should_not be_nil
+      $test_event_run_hash[:project].should_not be_nil
+      $test_event_run_hash[:version].should_not be_nil
+      $test_event_run_hash[:key1].should_not be_nil
+      $test_event_run_hash[:some].should_not be_nil
+      $test_event_run_hash[:answer].should_not be_nil
+
+      $test_event_run_hash[:event].process == "test_event_run_method"
+      $test_event_run_hash[:project].name.should == "foobar"
+      $test_event_run_hash[:version].should == "5.0"
+      $test_event_run_hash[:key1].should == "val1"
+      $test_event_run_hash[:some].should == "thing"
+      $test_event_run_hash[:answer].should == 42
+   end
+
+   it "should raise an exception if running event w/out specifying version" do
+      project = Project.new :name => "foobar"
+      event = Event.new :project => project, :process => "test_event_run_method"
+      lambda {
+        event.run
+      }.should raise_error(ArgumentError)
+   end
+
+   it "should raise an exception if running event process that doesn't correspond to a method" do
+      project = Project.new :name => "foobar"
+      event = Event.new :project => project, :process => "non_existant_method"
+      lambda {
+        event.run :version => "5"
+      }.should raise_error(ArgumentError)
+   end
+
+   it "should raise an exception if event process being run does" do
+      project = Project.new :name => "foobar"
+      event = Event.new :project => project, :process => "error_generating_method"
+      lambda {
+        event.run :version => "5"
+      }.should raise_error(RuntimeError)
    end
 end
 
@@ -185,9 +672,50 @@ end
 $test_event_run_hash = {}
 
 # helper method, invoked in Event::run spec
-def test_event_run_method(gem, first, second, third)
-  $test_event_run_hash[:gem] = gem
-  $test_event_run_hash[:first]  = first
-  $test_event_run_hash[:second] = second
-  $test_event_run_hash[:third]  = third
+def test_event_run_method(event, version, args = {})
+  $test_event_run_hash[:event]   = event
+  $test_event_run_hash[:project] = event.project
+  $test_event_run_hash[:version] = version
+  $test_event_run_hash[:key1]   = args[:key1]
+  $test_event_run_hash[:some]   = args[:some]
+  $test_event_run_hash[:answer] = args[:answer]
+end
+
+def error_generating_method(event, version, args = {})
+  raise RuntimeError
+end
+
+$project_released_test_handler_flags = []
+
+def project_released_test_handlerAAA(event, version, args = {})
+  $project_released_test_handler_flags << "AAA"
+end
+
+def project_released_test_handlerBBB(event, version, args = {})
+  $project_released_test_handler_flags << "BBB"
+end
+
+def project_released_test_handlerCCC(event, version, args = {})
+  $project_released_test_handler_flags << "CCC"
+end
+
+def project_released_test_handlerDDD(event, version, args = {})
+  $project_released_test_handler_flags << "DDD"
+
+  args.each { |k,v|
+    $project_released_test_handler_flags << "#{k}=#{v}"
+  }
 end
+
+def project_released_test_handlerEEE(event, version, args = {})
+  $project_released_test_handler_flags << "EEE"
+end
+
+def project_released_test_handlerFFF(event, version, args = {})
+  $project_released_test_handler_flags << "FFF"
+end
+
+def project_released_test_handlerGGG(event, version, args = {})
+  $project_released_test_handler_flags << "GGG"
+end
+