trello_cli 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9f6f168bd026bf25fdcd47f9f589a5837e1339d8
-  data.tar.gz: dcff0a71503befe5a480088b17ea2835e9f9d548
+  metadata.gz: 67d3269ab8c44f68e1ecdadb52a096162df122d3
+  data.tar.gz: 0995679927099f04e253b604d145f8d414665c18
 SHA512:
-  metadata.gz: 081b31895dd730abd67700638f2155f0b50c6c120f59bac61d99f7fa62c67eb7e7e3a21d57e4dbb5b556dd87b4fd165ce4f3dfacc0896ee2495d30f79162f1df
-  data.tar.gz: 70fbeb8cbf808f989c9e30c56bfd5b6529772636cfec2a04636d213aa639d979780221890bfecf6c4b00c441493e580c9115529a88254961e1a3168a911aeec2
+  metadata.gz: a52aaeac10cf59ac1fdf54e614c83bcd3cbf0703ce545e3b699cdbc146ac74705847dd9b4701e1ab5d771c0789f31dac790d23349de5de56b6f8fe9c716dd21c
+  data.tar.gz: 01a9dcce612760fd26d7e3c3b83efff6cb683df5d9d394d73c81dd9430aab96a374d01303d3a7610064353b9bdbbdad13ffbd1e152e46a7adba33b1f60ae689b

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 0.3.0:
+
+* Add ability to assign member to card on creation
+* Updated to ruby-trello 1.1.1
+
 ## 0.2.1:
 
 * Bug fix in listing closed boards

data/lib/trello_cli.rb

@@ -1,4 +1,5 @@
 require "trello_cli/cli"
+require "trello_cli/errors"
 require "trello_cli/requests"
 require "trello_cli/version"
 

data/lib/trello_cli/cli/card/create.rb

@@ -4,7 +4,7 @@ module TrelloCli
       class Create
 
         def initialize
-          @options = {}
+          @options = { members: [] }
         end
 
         def run
@@ -42,6 +42,10 @@ module TrelloCli
               @options[:list_id] = l
             end
 
+            opts.on("-m", "--member [MEMBER_NAME]", "Member name to assign to card") do |m|
+              @options[:members] << m
+            end
+
             opts.on("-n", "--name [NAME]", "Name Of Card") do |n|
               @options[:name] = n
             end

data/lib/trello_cli/cli/commands/card.rb

@@ -8,6 +8,9 @@ module TrelloCli
         def create
           ensure_credential_envs_set
           TrelloCli::CLI::Card::Create.new.run
+        rescue TrelloCli::Errors => e
+          puts "error: #{e}"
+          exit 1
         end
 
         def list

data/lib/trello_cli/errors.rb

@@ -0,0 +1,4 @@
+module TrelloCli
+  class Errors < StandardError
+  end
+end

data/lib/trello_cli/requests.rb

@@ -1,6 +1,7 @@
 require 'trello'
 
 require 'trello_cli/requests/shared'
+require 'trello_cli/requests/members'
 
 require 'trello_cli/requests/create_card'
 require 'trello_cli/requests/move_card'

data/lib/trello_cli/requests/create_card.rb

@@ -9,13 +9,34 @@ module TrelloCli
       end
 
       def create(args)
-        card = Trello::Card.new 'name'    => args[:name],
-                                'desc'    => args[:description],
-                                'idBoard' => args[:board_id],
-                                'idList'  => args[:list_id]
+        @board_id = args[:board_id]
+
+        args[:members].each do |member_username|
+          unless members.member_exists_in_board? member_username
+            msg = "member '#{member_username}' does not exist in board '#{@board_id}'."
+            raise Errors.new msg
+          end
+        end
+
+        member_ids = args[:members].map do |member_username|
+          m = members.find_member_by_username_in_board member_username
+          m.attributes[:id]
+        end
+
+        card = Trello::Card.new 'name'      => args[:name],
+                                'desc'      => args[:description],
+                                'idBoard'   => @board_id,
+                                'idList'    => args[:list_id],
+                                'idMembers' => member_ids.join(",")
         card.save
       end
 
+      private
+
+      def members
+        @members ||= Members.new @board_id
+      end
+
     end
   end
 end

data/lib/trello_cli/requests/members.rb

@@ -0,0 +1,41 @@
+module TrelloCli
+  module Requests
+    class Members
+
+      include Shared
+
+      def initialize(board_id)
+        connect_to_trello
+
+        @board_id = board_id
+      end
+
+      def member_exists_in_board?(member_username)
+        !!find_member_by_username_in_board(member_username)
+      end
+
+      def find_member_by_username_in_board(member_username)
+        members.each do |m|
+          return m if m.attributes[:username] == member_username
+        end
+
+        false
+      end
+
+      private
+
+      def member_id?(name)
+        Trello::Member.find(name)
+      end
+
+      def members
+        @members ||= board.members
+      end
+
+      def board
+        @board ||= Trello::Board.find @board_id
+      end
+
+    end
+  end
+end

data/lib/trello_cli/version.rb

@@ -1,3 +1,3 @@
 module TrelloCli
-  VERSION = "0.2.1"
+  VERSION = "0.3.0"
 end

data/spec/requests/create_card_spec.rb

@@ -5,20 +5,39 @@ describe TrelloCli::Requests::CreateCard do
     Trello.should_receive(:configure)
   end
 
+  let(:members_mock) { double "members" }
+  let(:trello_card_mock) { double 'trello card' }
+  let(:member_struct) { Struct.new(:attributes) }
+
+  before do
+    @args = { :name        => 'name',
+              :description => 'desc',
+              :board_id    => '123',
+              :list_id     => '321',
+              :members     => ["user1", "user2"] }
+    @options = { 'name'      => @args[:name],
+                 'desc'      => @args[:description],
+                 'idBoard'   => @args[:board_id],
+                 'idList'    => @args[:list_id],
+                 'idMembers' => "id1,id2" }
+  end
+
   it "should create the requested card" do
-    args = { :name        => 'name',
-             :description => 'desc',
-             :board_id    => '123',
-             :list_id     => '321' }
-    options = { 'name'    => args[:name],
-                'desc'    => args[:description],
-                'idBoard' => args[:board_id],
-                'idList'  => args[:list_id] }
-    trello_card_mock = double 'trello card'
+    TrelloCli::Requests::Members.should_receive(:new).
+                                 with("123").
+                                 and_return members_mock
+    @args[:members].each do |m|
+      members_mock.should_receive(:member_exists_in_board?).with(m).and_return true
+    end
+    members_mock.should_receive(:find_member_by_username_in_board).
+                 with("user1").and_return member_struct.new(id: "id1")
+    members_mock.should_receive(:find_member_by_username_in_board).
+                 with("user2").and_return member_struct.new(id: "id2")
+
     Trello::Card.should_receive(:new).
-                 with(options).
+                 with(@options).
                  and_return trello_card_mock
     trello_card_mock.should_receive(:save)
-    subject.create args
+    subject.create @args
   end
 end

data/spec/requests/members_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe TrelloCli::Requests::Members do
+  before do
+    Trello.should_receive(:configure)
+  end
+
+  let(:subject) { TrelloCli::Requests::Members.new "123" }
+  let(:board_mock) { double "board" }
+  let(:member_object) { Struct.new(:attributes) }
+  let(:member1) { member_object.new(username: "member1") }
+  let(:member2) { member_object.new(username: "member2") }
+  let(:members) { [member1, member2] }
+ 
+  before do
+    Trello::Board.should_receive(:find).with("123").and_return board_mock
+    board_mock.should_receive(:members).and_return members
+  end
+
+  describe "#member_exists_in_board?" do
+    it "should return true if the member exists" do
+      subject.member_exists_in_board?("member1").should == true
+    end
+    it "should return false if the member does not exists" do
+      subject.member_exists_in_board?("member999").should == false
+    end
+  end
+
+  describe "#find_member_by_username_in_board" do
+    it "should return the member object for a given username" do
+      subject.find_member_by_username_in_board("member1").should == member1
+    end
+  end
+
+end

data/trello_cli.gemspec

@@ -18,8 +18,9 @@ Gem::Specification.new do |gem|
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
 
-  gem.add_development_dependency "rspec"
-  gem.add_development_dependency "rake"
+  gem.add_development_dependency "rspec", "~> 2.14"
+  gem.add_development_dependency "rake", "~> 10.1"
 
-  gem.add_runtime_dependency 'ruby-trello', '0.5.1'
+  gem.add_runtime_dependency 'json', '1.8.0' # Added to resolve version conflict
+  gem.add_runtime_dependency 'ruby-trello', '1.1.1'
 end

metadata

@@ -1,57 +1,71 @@
 --- !ruby/object:Gem::Specification
 name: trello_cli
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Brett Weaver
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-21 00:00:00.000000000 Z
+date: 2014-12-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.14'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.14'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '10.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '10.1'
+- !ruby/object:Gem::Dependency
+  name: json
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.8.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.8.0
 - !ruby/object:Gem::Dependency
   name: ruby-trello
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.5.1
+        version: 1.1.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.5.1
+        version: 1.1.1
 description: Simple Trello Command Line Interface
 email:
 - brett@weav.net
@@ -82,11 +96,13 @@ files:
 - lib/trello_cli/cli/list.rb
 - lib/trello_cli/cli/list/list.rb
 - lib/trello_cli/cli/run.rb
+- lib/trello_cli/errors.rb
 - lib/trello_cli/requests.rb
 - lib/trello_cli/requests/create_card.rb
 - lib/trello_cli/requests/list_boards.rb
 - lib/trello_cli/requests/list_cards.rb
 - lib/trello_cli/requests/list_lists.rb
+- lib/trello_cli/requests/members.rb
 - lib/trello_cli/requests/move_card.rb
 - lib/trello_cli/requests/shared.rb
 - lib/trello_cli/version.rb
@@ -97,6 +113,7 @@ files:
 - spec/requests/list_boards_spec.rb
 - spec/requests/list_card_spec.rb
 - spec/requests/list_lists_spec.rb
+- spec/requests/members_spec.rb
 - spec/requests/move_card_spec.rb
 - spec/requests/shared_spec.rb
 - spec/spec_helper.rb
@@ -132,6 +149,7 @@ test_files:
 - spec/requests/list_boards_spec.rb
 - spec/requests/list_card_spec.rb
 - spec/requests/list_lists_spec.rb
+- spec/requests/members_spec.rb
 - spec/requests/move_card_spec.rb
 - spec/requests/shared_spec.rb
 - spec/spec_helper.rb