3llo 1.0.0.pre.rc.0 → 1.3.1.pre.rc.0

data/lib/3llo/api/checklist.rb

@@ -4,20 +4,14 @@ module Tr3llo
       extend self
 
       def create(card_id, data)
-        params = data.merge(key: api_key, token: api_token)
+        req_path = Utils.build_req_path("/cards/#{card_id}/checklists")
 
-        client.post("/cards/#{card_id}/checklists", params)
+        client.post(req_path, {}, data)
       end
 
       def get(checklist_id)
-        payload =
-          JSON.parse(
-            client.get(
-              "/checklists/#{checklist_id}",
-              key: api_key,
-              token: api_token
-            )
-          )
+        req_path = Utils.build_req_path("/checklists/#{checklist_id}")
+        payload = client.get(req_path, {})
 
         checklist_id, checklist_name = payload.fetch_values("id", "name")
         checklist_shortcut = Entities.make_shortcut(:checklist, checklist_id)
@@ -31,24 +25,20 @@ module Tr3llo
       end
 
       def update(checklist_id, data)
-        params = data.merge(key: api_key, token: api_token)
+        req_path = Utils.build_req_path("/checklists/#{checklist_id}")
 
-        client.put("/checklists/#{checklist_id}", params)
+        client.put(req_path, {}, data)
       end
 
       def delete(checklist_id)
-        client.delete("/checklists/#{checklist_id}", token: api_token, key: api_key)
+        req_path = Utils.build_req_path("/checklists/#{checklist_id}")
+
+        client.delete(req_path, {}, {})
       end
 
       def list_by_card_id(card_id)
-        payload =
-          JSON.parse(
-            client.get(
-              "/cards/#{card_id}/checklists",
-              key: api_key,
-              token: api_token
-            )
-          )
+        req_path = Utils.build_req_path("/cards/#{card_id}/checklists")
+        payload = client.get(req_path, {})
 
         payload.map do |checklist_payload|
           checklist_id, checklist_name = checklist_payload.fetch_values("id", "name")
@@ -56,8 +46,8 @@ module Tr3llo
 
           items =
             checklist_payload
-            .fetch("checkItems", [])
-            .map do |item_payload|
+              .fetch("checkItems", [])
+              .map do |item_payload|
               item_id, item_name, item_state = item_payload.fetch_values("id", "name", "state")
               item_shortcut = Entities.make_shortcut(:check_item, item_id)
 
@@ -74,14 +64,8 @@ module Tr3llo
       end
 
       def get_item(card_id, item_id)
-        item_payload =
-          JSON.parse(
-            client.get(
-              "/cards/#{card_id}/checkItem/#{item_id}",
-              key: api_key,
-              token: api_token
-            )
-          )
+        req_path = Utils.build_req_path("/cards/#{card_id}/checkItem/#{item_id}")
+        item_payload = client.get(req_path, {})
 
         item_id, item_name, item_state = item_payload.fetch_values("id", "name", "state")
         item_shortcut = Entities.make_shortcut(:check_item, item_id)
@@ -90,35 +74,29 @@ module Tr3llo
       end
 
       def create_item(checklist_id, name)
-        client.post(
-          "/checklists/#{checklist_id}/checkItems",
-          key: api_key,
-          token: api_token,
+        req_path = Utils.build_req_path("/checklists/#{checklist_id}/checkItems")
+        payload = {
           name: name,
           pos: "bottom",
-          state: false
-        )
+          state: "false"
+        }
+
+        client.post(req_path, {}, payload)
       end
 
       def update_item(card_id, check_item_id, data)
-        params = data.merge(key: api_key, token: api_token)
+        req_path = Utils.build_req_path("/cards/#{card_id}/checkItem/#{check_item_id}")
 
-        client.put("/cards/#{card_id}/checkItem/#{check_item_id}", params)
+        client.put(req_path, {}, data)
       end
 
       def delete_item(card_id, item_id)
-        client.delete("/cards/#{card_id}/checkItem/#{item_id}", token: api_token, key: api_key)
-      end
-
-      private
+        req_path = Utils.build_req_path("/cards/#{card_id}/checkItem/#{item_id}")
 
-      def api_key
-        Application.fetch_configuration!().api_key
+        client.delete(req_path, {}, {})
       end
 
-      def api_token
-        Application.fetch_configuration!().api_token
-      end
+      private
 
       def client
         Application.fetch_client!()

data/lib/3llo/api/label.rb

@@ -0,0 +1,64 @@
+module Tr3llo
+  module API
+    module Label
+      extend self
+
+      def find_all_by_board(board_id)
+        req_path =
+          Utils.build_req_path(
+            "/boards/#{board_id}/labels"
+          )
+
+        client
+          .get(req_path, {})
+          .reject { |label| label["name"].empty? }
+          .map do |label_payload|
+            make_struct(label_payload)
+          end
+      end
+
+      def find(label_id)
+        req_path = Utils.build_req_path("/labels/#{label_id}")
+        label_payload = client.get(req_path, {})
+
+        make_struct(label_payload)
+      end
+
+      def create(name:, color:, board_id:)
+        req_path = Utils.build_req_path("/labels")
+        payload = {
+          "name" => name,
+          "color" => color,
+          "idBoard" => board_id
+        }
+
+        client.post(req_path, {}, payload)
+      end
+
+      def update(label_id, data)
+        req_path = Utils.build_req_path("/labels/#{label_id}")
+
+        client.put(req_path, {}, data)
+      end
+
+      def delete(label_id)
+        req_path = Utils.build_req_path("/labels/#{label_id}")
+
+        client.delete(req_path, {}, {})
+      end
+
+      private
+
+      def make_struct(payload)
+        id, name, color = payload.fetch_values("id", "name", "color")
+        shortcut = Entities.make_shortcut(:label, id)
+
+        Entities::Label.new(id: id, shortcut: shortcut, name: name, color: color)
+      end
+
+      def client
+        Application.fetch_client!()
+      end
+    end
+  end
+end

data/lib/3llo/api/list.rb

@@ -4,26 +4,33 @@ module Tr3llo
       extend self
 
       def find_all_by_board(board_id)
-        JSON.parse(
-          client.get(
+        req_path =
+          Utils.build_req_path(
             "/boards/#{board_id}/lists",
-            list: true,
-            key: api_key,
-            token: api_token
+            {"list" => "true"}
           )
-        ).map do |list_payload|
-          make_struct(list_payload)
-        end
+
+        client
+          .get(req_path, {})
+          .map do |list_payload|
+            make_struct(list_payload)
+          end
       end
 
       def archive_cards(list_id)
-        JSON.parse(
-          client.post(
-            "/lists/#{list_id}/archiveAllCards",
-            key: api_key,
-            token: api_token
-          )
-        )
+        req_path = Utils.build_req_path("/lists/#{list_id}/archiveAllCards")
+
+        client.post(req_path, {}, {})
+      end
+
+      def create(name, board_id)
+        req_path = Utils.build_req_path("/lists")
+        payload = {
+          "name" => name,
+          "idBoard" => board_id
+        }
+
+        client.post(req_path, {}, payload)
       end
 
       private
@@ -38,14 +45,6 @@ module Tr3llo
       def client
         Application.fetch_client!()
       end
-
-      def api_key
-        Application.fetch_configuration!().api_key
-      end
-
-      def api_token
-        Application.fetch_configuration!().api_token
-      end
     end
   end
 end

data/lib/3llo/api/token.rb

@@ -4,10 +4,10 @@ module Tr3llo
       extend self
 
       def verify(key, token)
-        client.get("/tokens/#{token}", key: key)
+        client.get("/tokens/#{token}?key=#{key}", {})
 
         true
-      rescue HTTP::Client::RequestError
+      rescue RemoteServer::RequestError
         false
       end
 

data/lib/3llo/api/user.rb

@@ -4,31 +4,19 @@ module Tr3llo
       extend self
 
       def find(user_id)
-        url = "/members/#{user_id}"
+        client = Application.fetch_client!()
+        req_path = Utils.build_req_path("/members/#{user_id}")
 
-        make_struct(
-          JSON.parse(
-            client.get(
-              url,
-              key: key,
-              token: token
-            )
-          )
-        )
+        make_struct(client.get(req_path, {}))
       end
 
       def find_all_by_board(board_id)
-        url = "/board/#{board_id}/members"
+        client = Application.fetch_client!()
+        req_path = Utils.build_req_path("/board/#{board_id}/members")
 
-        JSON.parse(
-          client.get(
-            url,
-            key: key,
-            token: token
-          )
-        ).map do |user_payload|
-          make_struct(user_payload)
-        end
+        client
+          .get(req_path, {})
+          .map { |user_payload| make_struct(user_payload) }
       end
 
       private
@@ -39,18 +27,6 @@ module Tr3llo
 
         Entities::User.new(id, shortcut, username)
       end
-
-      def key
-        Application.fetch_configuration!().api_key
-      end
-
-      def token
-        Application.fetch_configuration!().api_token
-      end
-
-      def client
-        Application.fetch_client!()
-      end
     end
   end
 end

data/lib/3llo/api.rb

@@ -1,6 +1,7 @@
-require '3llo/api/board'
-require '3llo/api/card'
-require '3llo/api/list'
-require '3llo/api/user'
-require '3llo/api/checklist'
-require '3llo/api/token'
+require "3llo/api/board"
+require "3llo/api/card"
+require "3llo/api/list"
+require "3llo/api/label"
+require "3llo/api/user"
+require "3llo/api/checklist"
+require "3llo/api/token"

data/lib/3llo/application.rb

@@ -2,10 +2,10 @@ module Tr3llo
   module Application
     extend self
 
-    DEFAULT_CONFIG_FILE_PATH = "~/.3llo.config.json"
+    DEFAULT_CONFIG_FILE_PATH = "~/.3llo.config.json".freeze
 
     def start(args)
-      $container = Container.new()
+      $application = Container.new()
 
       interface = register_interface!()
       option_parser = build_option_parser()
@@ -13,12 +13,12 @@ module Tr3llo
 
       options = parse_cli_args!(option_parser, args)
 
-      if options.key?(:help)
+      if options.has_key?(:help)
         execute_help!(option_parser, interface)
         exit
       end
 
-      if options.key?(:configure)
+      if options.has_key?(:configure)
         execute_configure!(interface)
         exit
       end
@@ -28,7 +28,7 @@ module Tr3llo
 
       print_help!(interface)
 
-      load_configuration!(config_file, interface)
+      load_configuration!(config_file)
       register_registry!()
 
       load_user!(interface)
@@ -64,7 +64,7 @@ module Tr3llo
       parser.parse!(args, into: options)
 
       options
-    rescue OptionParser::InvalidArgument, OptionParser::InvalidOption => exception
+    rescue OptionParser::InvalidArgument, OptionParser::InvalidOption
       {}
     end
 
@@ -85,19 +85,21 @@ module Tr3llo
     end
 
     def register_api_client!()
-      $container.register(:api_client, Tr3llo::HTTP::Client)
+      remote_server = Tr3llo::RemoteServer.new("https://api.trello.com/1")
+
+      $application.register(:api_client, remote_server)
     end
 
     def register_interface!()
       prompt = TTY::Prompt.new()
-      $container.register(:interface, Tr3llo::Interface.new(prompt, $stdout))
+      $application.register(:interface, Tr3llo::Interface.new(prompt, $stdout))
     end
 
-    def load_configuration!(config_file, interface)
+    def load_configuration!(config_file)
       config_path = File.expand_path(config_file)
 
       config =
-        if File.exists?(config_path)
+        if File.exist?(config_path)
           JSON.load(File.read(config_path))
         else
           {}
@@ -110,17 +112,25 @@ module Tr3llo
 
       configuration.finalize!()
 
-      $container.register(:configuration, configuration)
+      $application.register(:configuration, configuration)
     rescue KeyError => exception
       command_string = "3llo --configure"
-      abort(Utils.paint("#{exception.key.inspect} has not been configured. Please run #{command_string.inspect} to set up configuration.", "red"))
+
+      abort(
+        Utils.paint(
+          "#{exception.key.inspect} has not been configured. " \
+            "Please run #{command_string.inspect} to set up configuration.",
+          "red"
+        )
+      )
     end
 
     def get_config_entry(config, key, env_key)
       config.fetch(key) do
         if ENV.has_key?(env_key)
           Utils.deprecate!(
-            "Setting #{env_key.inspect} as an environment variable is deprecated. It will be removed in the future versions of 3llo. Please use config file instead."
+            "Setting #{env_key.inspect} as an environment variable is deprecated. " \
+              "It will be removed in the future versions of 3llo. Please use config file instead."
           )
 
           ENV.fetch(env_key)
@@ -131,37 +141,37 @@ module Tr3llo
     end
 
     def register_registry!()
-      $container.register(:registry, Tr3llo::Registry.new)
+      $application.register(:registry, Tr3llo::Registry.new)
     end
 
     def register_board!(board)
-      $container.register(:board, board)
+      $application.register(:board, board)
     end
 
     def fetch_board!()
-      $container.resolve(:board)
+      $application.resolve(:board)
     rescue ::Container::KeyNotFoundError
       raise BoardNotSelectedError
     end
 
     def fetch_user!()
-      $container.resolve(:user)
+      $application.resolve(:user)
     end
 
     def fetch_configuration!()
-      $container.resolve(:configuration)
+      $application.resolve(:configuration)
     end
 
     def fetch_interface!()
-      $container.resolve(:interface)
+      $application.resolve(:interface)
     end
 
     def fetch_client!()
-      $container.resolve(:api_client)
+      $application.resolve(:api_client)
     end
 
     def fetch_registry!()
-      $container.resolve(:registry)
+      $application.resolve(:registry)
     end
 
     def load_user!(interface)
@@ -172,7 +182,7 @@ module Tr3llo
         interface.puts("You're logged in as #{decorated_username}")
       end
 
-      $container.register(:user, user)
+      $application.register(:user, user)
     end
   end
 end

data/lib/3llo/command/board/add.rb

@@ -0,0 +1,29 @@
+module Tr3llo
+  module Command
+    module Board
+      module Add
+        extend self
+
+        def execute()
+          interface = Application.fetch_interface!()
+
+          interface.print_frame do
+            name = interface.input.ask("Name:", required: true)
+            desc = interface.input.ask("Description:")
+
+            default_lists =
+              interface.input.yes?("With default set of lists to the board (To Do, Doing, Done)?") do |question|
+                question.default false
+                question.positive "Y"
+                question.negative "N"
+              end
+
+            API::Board.create(name: name, desc: desc, default_lists: default_lists)
+
+            interface.puts("Board has been created.")
+          end
+        end
+      end
+    end
+  end
+end

data/lib/3llo/command/board.rb

@@ -1,6 +1,4 @@
-require '3llo/command/board/list'
-require '3llo/command/board/select'
-require '3llo/command/board/invalid'
+Tr3llo::Utils.require_directory(File.dirname(__FILE__) + "/board/*.rb")
 
 module Tr3llo
   module Command
@@ -9,21 +7,21 @@ module Tr3llo
 
       def execute(subcommand, args)
         case subcommand
-        when 'list'
+        when "list"
           user = Application.fetch_user!()
 
           Command::Board::List.execute(user[:id])
-        when 'select'
-          board_key, _ = args
+        when "select"
+          board_key, = args
           Utils.assert_string!(board_key, "board key is missing")
 
           Command::Board::Select.execute(board_key)
+        when "add"
+          Command::Board::Add.execute()
         else
           handle_invalid_subcommand(subcommand, args)
         end
-      rescue InvalidCommandError => exception
-        Command::Board::Invalid.execute(exception.message)
-      rescue InvalidArgumentError => exception
+      rescue InvalidCommandError, InvalidArgumentError => exception
         Command::Board::Invalid.execute(exception.message)
       end
 

data/lib/3llo/command/card/add.rb

@@ -14,17 +14,21 @@ module Tr3llo
           interface = Application.fetch_interface!()
 
           interface.print_frame do
-            list_id = interface.input.select(
-              "Choose the list this card should belong to:",
-              list_options
-            )
+            if list_options.any?
+              list_id = interface.input.select(
+                "Choose the list this card should belong to:",
+                list_options
+              )
 
-            name = interface.input.ask("Name:", required: true)
-            description = interface.input.ask("Description:")
+              name = interface.input.ask("Name:", required: true)
+              description = interface.input.multiline("Description:").join("")
 
-            API::Card.create(name, description, list_id)
+              API::Card.create(name, description, list_id)
 
-            interface.puts("Card has been created.")
+              interface.puts("Card has been created.")
+            else
+              interface.puts("There is no list on board.")
+            end
           end
         end
       end

data/lib/3llo/command/card/add_label.rb

@@ -0,0 +1,56 @@
+module Tr3llo
+  module Command
+    module Card
+      module AddLabel
+        extend self
+
+        def execute(key, board_id)
+          card_id = Entities.parse_id(:card, key)
+          assert_card_id!(card_id, key)
+
+          card = API::Card.find(card_id)
+
+          interface = Application.fetch_interface!()
+
+          interface.print_frame do
+            label_ids = select_labels(interface, card, board_id)
+
+            add_labels(card_id, label_ids)
+            interface.puts("Chosen labels have been added to the card.")
+          end
+        end
+
+        private
+
+        def assert_card_id!(card_id, key)
+          raise InvalidArgumentError.new("#{key.inspect} is not a valid card key") unless card_id
+        end
+
+        def add_labels(card_id, label_ids)
+          API::Card.add_labels(card_id, label_ids)
+        end
+
+        def select_labels(interface, card, board_id)
+          label_options =
+            API::Label.find_all_by_board(board_id)
+              .map { |label| [label.name, label.id] }
+              .to_h()
+
+          preselected_label_ids =
+            card.labels.flat_map do |label|
+              index = label_options.find_index { |_label_name, label_id| label_id == label.id }
+
+              index ? [index + 1] : []
+            end
+
+          interface.input.multi_select(
+            "Choose the labels to add to this card:",
+            label_options,
+            default: preselected_label_ids,
+            enum: "."
+          )
+        end
+      end
+    end
+  end
+end

data/lib/3llo/command/card/assign.rb

@@ -33,14 +33,14 @@ module Tr3llo
         def select_user(interface, card, board_id)
           user_options =
             API::User.find_all_by_board(board_id)
-            .map { |user| [user.username, user.id] }
-            .to_h()
+              .map { |user| [user.username, user.id] }
+              .to_h()
 
           member_ids =
             card.members.flat_map do |member|
               index = user_options.find_index { |_username, user_id| user_id == member.id }
 
-              if index then [index + 1] else [] end
+              index ? [index + 1] : []
             end
 
           interface.input.multi_select(