schubert-minglr 1.2.0 → 1.3.0

data/test/resources/attachment_test.rb

@@ -5,7 +5,7 @@ module Resources
   class AttachmentTest < Test::Unit::TestCase
   
     context Attachment do
-    
+
       context "configure" do
         
         should "append to the site url" do
@@ -21,14 +21,64 @@ module Resources
       
       context "fetch" do
         
-        should_eventually "download all found attachments for a given card number" do
+        should "download all found attachments for a given card number" do
+          card = Card.new
+          card.stubs(:number).returns(1)
+          
+          attachment1 = Attachment.new
+          attachment1.stubs(:url).returns("http://attachment1.url")
+          attachment1.stubs(:file_name).returns("attachment1.txt")
+          attachment2 = Attachment.new
+          attachment2.stubs(:url).returns("http://attachment2.url")
+          attachment2.stubs(:file_name).returns("attachment2.txt")
+
+          Attachment.stubs(:puts)
+          Card.expects(:find).with(1).returns(card)
+          Attachment.expects(:find).with(:all, :params => { :card_number => 1 }).returns([attachment1, attachment2])
+          Attachment.expects(:curl).with("curl --insecure --progress-bar --output attachment1.txt --user username:password http://attachment1.url")
+          Attachment.expects(:curl).with("curl --insecure --progress-bar --output attachment2.txt --user username:password http://attachment2.url")
+          
+          Attachment.fetch(1, "username", "password")
         end
       
       end
       
       context "attach" do
         
-        should_eventually "upload file for a given card number" do
+        setup do
+          options = { :username => "user",
+                      :password => "pass",
+                      :url => "proto://somehost.com:1234/projects/my_project" }
+          Base.configure(options)
+        end
+        
+        should "upload file for a given card number" do
+          file_name = File.join(File.dirname(__FILE__), "..", "..", "Rakefile")
+          
+          card = Card.new
+          card.stubs(:number).returns(1)
+          
+          fake_response = stub("Response", :status_code => 201)
+          fake_client = mock("HTTPClient")
+          fake_client.expects(:post).returns(fake_response)
+          fake_client.expects(:set_auth).with(nil, "username", "password")
+          
+          Card.expects(:find).with(1).returns(card)
+          HTTPClient.expects(:new).returns(fake_client)
+          Attachment.expects(:puts).with("File '#{file_name}' attached to card 1")
+
+          Attachment.attach(1, file_name, "username", "password")
+        end
+        
+        should "warn for invalid file" do          
+          card = Card.new
+          card.stubs(:number).returns(1)
+          
+          Card.expects(:find).with(1).returns(card)
+          File.expects(:exist?).with("myfile.txt").returns(false)
+          Attachment.expects(:warn).with("Unable to open file 'myfile.txt'")
+          
+          Attachment.attach(1, "myfile.txt", "username", "password")
         end
       
       end

data/test/resources/base_test.rb

@@ -30,10 +30,23 @@ module Resources
       
       context "print_collection" do
         
-        should_eventually "print values of attributes of objects separated by dashes and aligning columns of values" do
+        setup do
+          @object1 = stub("Object", :name => "Foo", :description => "Bar")
+          @object2 = stub("Object", :name => "Bar", :description => "Frobble")
+          @collection = [@object1, @object2]
+          @attributes = [:name, :description]
         end
         
-        should_eventually "allow for right adjusted values" do
+        should "print values of attributes of objects separated by dashes and aligning columns of values" do
+          Base.expects(:puts).with("Foo - Bar    ")
+          Base.expects(:puts).with("Bar - Frobble")
+          Base.print_collection(@collection, @attributes)
+        end
+        
+        should "allow for right adjusted values" do
+          Base.expects(:puts).with("Foo -     Bar")
+          Base.expects(:puts).with("Bar - Frobble")
+          Base.print_collection(@collection, @attributes, :right)
         end
         
       end

data/test/resources/card_test.rb

@@ -5,16 +5,33 @@ module Resources
   class CardTest < Test::Unit::TestCase
   
     context Card do
-      
+
       context "create" do
         
-        should_eventually "create a card with options passed in" do
+        should "create a card with options passed in" do
+          card = Card.new
+          card.stubs(:number).returns(1)
+          Card.expects(:new).with(:name => "Name").returns(card)
+          Card.any_instance.expects(:save).returns(true)
+          Card.any_instance.expects(:reload)
+          Card.expects(:puts).with("Card #1 created")
+          Card.create({:name => "Name"})
         end
         
-        should_eventually "create a card with the status of new if status property is available" do
+        should "create a card with the status of new if status property is available" do
+          card = Card.new
+          card.stubs(:number).returns(1)
+          Card.expects(:new).with(:some_option => "value", :cp_status => "New").returns(card)
+          Card.any_instance.expects(:save).returns(true)
+          Card.any_instance.expects(:reload)
+          Card.expects(:puts).with("Card #1 created")
+          Card.create({:some_option => "value"}, "cp_status")
         end
         
-        should_eventually "warn if it is unable to create the card" do
+        should "warn if it is unable to create the card" do
+          Card.any_instance.expects(:save).returns(false)
+          Card.expects(:warn).with("Unable to create card")
+          Card.create
         end
         
       end
@@ -22,9 +39,44 @@ module Resources
       context "move" do
         
         should "move card from one state to the next as defined" do
+          card = Card.new
+          card.stubs(:card_type_name).returns("story")
+          card.stubs(:status).returns("New")
+          config = { :story_state_1 => "New > Dev", :status_property => "status"}
+          response = stub("Response", :attributes => { "status" => "completed" })
+          
+          Card.expects(:find).with(1).returns(card)
+          TransitionExecution.expects(:create).with(:transition => 'Dev', :card => 1).returns(response)
+          Card.expects(:puts).with("Moved card from New to Dev")
+          Card.move(1, {}, config)
+        end
+        
+        should "warn if no matching card type is found" do
+          card = Card.new
+          card.stubs(:card_type_name).returns("someothertype")
+          card.stubs(:status).returns("New")
+          config = { :story_state_1 => "New > Dev", :status_property => "status"}
+          
+          Card.expects(:find).with(1).returns(card)
+          Card.expects(:warn).with("No transitions defined for card of type someothertype")
+          Card.move(1, {}, config)
+        end
+        
+        should "warn if card has no status property" do
+          card = Card.new
+          card.stubs(:card_type_name).returns("story")
+          card.stubs(:current_status).returns("New")
+          config = { :story_state_1 => "New > Dev"}
+          
+          Card.expects(:find).with(1).returns(card)
+          Card.expects(:warn).with("No known status of card #1, cannot move!")
+          Card.move(1, {}, config)
         end
         
         should "warn if card cannot be found" do
+          Card.expects(:find).with(0).returns(nil)
+          Card.expects(:warn).with("No card #0 found to move")
+          Card.move(0)
         end
         
       end
@@ -57,27 +109,85 @@ module Resources
       
       context "print_card" do
         
-        should_eventually "print the details for a given card" do
+        should "print the details for a given card" do
+          status_property = "cp_status"
+          card = Card.new
+          card.expects(:to_s).with(status_property)
+          Card.expects(:find).with(1).returns(card)
+          Card.stubs(:puts)
+          Card.print_card(1, status_property)
+        end
+        
+        should "warn if it is not able to find the card" do
+          Card.expects(:find).with(0).returns(nil)
+          Card.expects(:warn).with("No card #0 found")
+          Card.print_card(0)
         end
         
       end
       
       context "update" do
         
-        should_eventually "update a card with the options passed in" do
+        should "update a card with the options passed in" do
+          card = mock("Card")
+          card.stubs(:number).returns(1)
+          card.expects(:foo=).with("bar")
+          card.expects(:baz=).with("frobble")
+          card.expects(:save)
+          Card.stubs(:puts)
+          Card.expects(:find).with(1).returns(card)
+          
+          Card.update(1, {:foo => "bar", :baz => "frobble"})
         end
         
-        should_eventually "print out the details of the card after updating" do
+        should "print out the details of the card after updating" do
+          card = mock("Card")
+          card.stubs(:number).returns(1)
+          card.stubs(:foo=)
+          card.stubs(:baz=)
+          card.stubs(:save)
+          Card.stubs(:find).with(1).returns(card)
+          
+          Card.expects(:puts).with("Card #1 updated\n\n")
+          Card.expects(:puts).with(card.to_s)          
+          Card.update(1, {:foo => "bar", :baz => "frobble"})
         end
         
-        should_eventually "warn if it is not able to find the card" do
+        should "warn if it is not able to find the card" do
+          Card.expects(:find).with(0).returns(nil)
+          Card.expects(:warn).with("Unable to update card #0")
+          Card.update(0)
         end
         
       end
       
       context "to_s" do
         
-        should_eventually "format the details of a given card including attachments" do
+        should "format the details of a given card including attachments" do
+          Resources::Base.site = "http://foo.bar"
+          
+          attachment = Attachment.new
+          attachment.stubs(:file_name).returns("My File Name")
+          attachment.stubs(:url).returns("http://some.url")
+          Attachment.expects(:find).with(:all, :params => { :card_number => 1 }).returns([attachment])
+          
+          card = Card.new
+          card.stubs(:number).returns(1)
+          card.stubs(:name).returns("My Name")
+          card.stubs(:card_type_name).returns("My Type")
+          card.stubs(:description).returns("My Description")
+          
+          expected_output = <<-EOS
+     Number: 1
+       Name: My Name
+       Type: My Type
+     Status: 
+Description: My Description
+
+Attachments:
+  * My File Name: http://some.url
+          EOS
+          assert_equal expected_output, card.to_s
         end
         
       end

data/test/resources/project_test.rb

@@ -18,6 +18,24 @@ module Resources
         end
       
       end
+      
+      context "print_all" do
+        
+        should "print list of all projects" do
+          project1 = Project.new
+          project2 = Project.new
+          Project.expects(:find).with(:all).returns([project1, project2])
+          Project.expects(:print_collection).with([project1, project2], [:name, :description])
+          Project.print_all          
+        end
+        
+        should "print warning if no projects are found" do
+          Project.expects(:find).with(:all).returns([])
+          Project.expects(:warn).with("No projects found")
+          Project.print_all
+        end
+      
+      end
     
     end
 

data/test/resources/user_test.rb

@@ -24,13 +24,6 @@ module Resources
         end
       
       end
-      
-      context "find_user_id_for_user" do
-        
-        should_eventually "return the owner_id of the user matching the username" do
-        end
-        
-      end
     
     end
 

data/test/test_helper.rb

@@ -1,3 +1,5 @@
+MINGLR_ENV = "test"
+
 require "rubygems"
 require "test/unit"
 begin

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: schubert-minglr
 version: !ruby/object:Gem::Version 
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors: 
 - Michael Schubert
@@ -11,7 +11,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-07-26 00:00:00 -07:00
+date: 2009-07-27 00:00:00 -07:00
 default_executable: 
 dependencies: []
 
@@ -35,12 +35,17 @@ files:
 - VERSION.yml
 - bin/minglr
 - bin/mtx
+- cucumber.yml
+- features/cards.feature
+- features/step_definitions/minglr_steps.rb
+- features/step_definitions/shared_steps.rb
+- features/users.feature
 - lib/minglr.rb
 - lib/minglr/action.rb
 - lib/minglr/config_parser.rb
 - lib/minglr/extensions/array.rb
-- lib/minglr/input_cache.rb
-- lib/minglr/mtx_options_parser.rb
+- lib/minglr/mtx/input_cache.rb
+- lib/minglr/mtx/options_parser.rb
 - lib/minglr/options_parser.rb
 - lib/minglr/resources/attachment.rb
 - lib/minglr/resources/base.rb
@@ -53,7 +58,11 @@ files:
 - minglrconfig.sample
 - tasks/commit.sample.rake
 - tasks/svn.sample.rake
+- test/action_test.rb
+- test/commands_test.rb
+- test/config_parser_test.rb
 - test/extensions/array_test.rb
+- test/options_parser_test.rb
 - test/resources/attachment_test.rb
 - test/resources/base_test.rb
 - test/resources/card_test.rb
@@ -61,8 +70,9 @@ files:
 - test/resources/property_definition_test.rb
 - test/resources/user_test.rb
 - test/test_helper.rb
-has_rdoc: true
+has_rdoc: false
 homepage: http://github.com/schubert/minglr
+licenses: 
 post_install_message: PostInstall.txt
 rdoc_options: 
 - --charset=UTF-8
@@ -83,12 +93,16 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.2.0
+rubygems_version: 1.3.5
 signing_key: 
-specification_version: 2
+specification_version: 3
 summary: command line user tool for Mingle (http://mingle.thoughtworks.com/mingle-agile-project-management)
 test_files: 
+- test/action_test.rb
+- test/commands_test.rb
+- test/config_parser_test.rb
 - test/extensions/array_test.rb
+- test/options_parser_test.rb
 - test/resources/attachment_test.rb
 - test/resources/base_test.rb
 - test/resources/card_test.rb

data/lib/minglr/input_cache.rb

@@ -1,22 +0,0 @@
-class InputCache
-  class << self
-    def put(key, content)
-      File.open(file_pathname(key), File::CREAT | File::WRONLY | File::TRUNC) { |file| file.write content }
-    end
-    
-    def get(key)
-      if content = File.read(file_pathname(key))
-        return nil if content.blank?
-        content
-      end
-    rescue
-      nil
-    end
-    
-    protected
-    
-    def file_pathname(key)
-      File.join(Dir::tmpdir, "#{key.to_s.gsub(/[^\w]/, '')}.entry")
-    end
-  end
-end

data/lib/minglr/mtx_options_parser.rb

@@ -1,53 +0,0 @@
-class MtxOptionsParser
-  def self.parse(args, *required_by_command)
-    uri_options = {}
-    command_options = {}
-
-    parser = OptionParser.new do |opts|
-      opts.banner = "Usage: mtx [options]"
-      opts.on("--transition TRANSITION", "Transition name.") do |transition|
-        command_options[:transition] = transition
-      end
-
-      opts.on("--card CARD", "Card number.") do |card|
-        command_options[:card] = card
-      end
-
-      opts.on("--properties ARGS", Array, "User-entered properties and values for the transition in array format. Must be an even number of comma-delimited values, like \"A,B,'C with spaces','D with spaces'\".") do |args|
-        command_options[:properties] = args.in_groups_of(2).map { |key, value| {'name' => key, 'value' => value} }
-      end
-      
-      opts.on("--comment COMMENT", "Transition comment. This may be required depending on your transition settings.") do |comment|
-        command_options[:comment] = comment
-      end
-
-      opts.on("--username USERNAME", "Mingle username.") do |username|
-        uri_options[:username] = username
-      end
-
-      opts.on("--password PASSWORD", "Mingle password.") do |password|
-        uri_options[:password] = password
-      end
-
-      opts.on("--host_port HOST_PORT", "Host and port.") do |host_and_port|
-        uri_options[:host_and_port] = host_and_port
-      end
-
-      opts.on("--project PROJECT", "Project name.") do |project|
-        uri_options[:project] = project
-      end
-    end
-    
-    parser.parse! args
-
-    ([:project, :host_and_port] | required_by_command).each do |arg|
-      unless command_options[arg] || uri_options[arg]
-        # TODO: let commands handle their own errors
-        $stderr.puts "Missing command-line argument --#{arg.to_s}, use --help for command-line options."
-        exit 1
-      end
-    end
-
-    [uri_options, command_options]
-  end
-end