lita-team 1.0.0 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3063c4d97c93c2aec90fbca5c5cd0187be403e8b
-  data.tar.gz: c81a31b980fbac18ce25920033b69518657348d9
+  metadata.gz: 8ed605cdde13081275049d971e030093180a6289
+  data.tar.gz: f5f84bfbb4351c44541160802739958d607daf70
 SHA512:
-  metadata.gz: 3018309d640b32259fde30ccaf450515615fbd620398908b1ae0e5f3c887a4ad311f27f433f8e06b5e389efedcb0803f774cccf650eeaf9daeac0547118020c1
-  data.tar.gz: cde6b8fbcf5a6255981927ee0e50082398ada98d0ba33df1af6a58e331e40e8794160582445a549e0263c7071050f6a4a974bcaa22f9fb4e1f4f39576e77ce79
+  metadata.gz: d49dec67ce61f67e7e9dfd3cbeeda8126d17c16fb2b8c949b76062ca86b216f8a3740d02cc109214a6eefb5e9556ab68aade7bfd1a0c38a91629429707237d4d
+  data.tar.gz: e81b0780441ecb6c1cdb11da8fc5f03787badedc74b310d6f7eae6126c154fdfd4ddffc7de80eefbc410bf5406c1d64f2d54670aed1d4f4253cadc2e199e8954

data/.rubocop.yml

@@ -0,0 +1,243 @@
+AllCops:
+  Exclude:
+    - "vendor/**/*"
+    - "db/schema.rb"
+  UseCache: false
+Style/CollectionMethods:
+  Description: Preferred collection methods.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#map-find-select-reduce-size
+  Enabled: true
+  PreferredMethods:
+    collect: map
+    collect!: map!
+    find: detect
+    find_all: select
+    reduce: inject
+Style/DotPosition:
+  Description: Checks the position of the dot in multi-line method calls.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#consistent-multi-line-chains
+  Enabled: true
+  EnforcedStyle: trailing
+  SupportedStyles:
+  - leading
+  - trailing
+Style/FileName:
+  Description: Use snake_case for source file names.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#snake-case-files
+  Enabled: false
+  Exclude: []
+Style/GuardClause:
+  Description: Check for conditionals that can be replaced with guard clauses
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-nested-conditionals
+  Enabled: false
+  MinBodyLength: 1
+Style/IfUnlessModifier:
+  Description: Favor modifier if/unless usage when you have a single-line body.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#if-as-a-modifier
+  Enabled: false
+  MaxLineLength: 80
+Style/OptionHash:
+  Description: Don't use option hashes when you can use keyword arguments.
+  Enabled: false
+Style/PercentLiteralDelimiters:
+  Description: Use `%`-literal delimiters consistently
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#percent-literal-braces
+  Enabled: false
+  PreferredDelimiters:
+    "%": "()"
+    "%i": "()"
+    "%q": "()"
+    "%Q": "()"
+    "%r": "{}"
+    "%s": "()"
+    "%w": "()"
+    "%W": "()"
+    "%x": "()"
+Style/PredicateName:
+  Description: Check the names of predicate methods.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#bool-methods-qmark
+  Enabled: true
+  NamePrefix:
+  - is_
+  - has_
+  - have_
+  NamePrefixBlacklist:
+  - is_
+  Exclude:
+  - spec/**/*
+Style/RaiseArgs:
+  Description: Checks the arguments passed to raise/fail.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#exception-class-messages
+  Enabled: false
+  EnforcedStyle: exploded
+  SupportedStyles:
+  - compact
+  - exploded
+Style/SignalException:
+  Description: Checks for proper usage of fail and raise.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#fail-method
+  Enabled: false
+  EnforcedStyle: semantic
+  SupportedStyles:
+  - only_raise
+  - only_fail
+  - semantic
+Style/SingleLineBlockParams:
+  Description: Enforces the names of some block params.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#reduce-blocks
+  Enabled: false
+  Methods:
+  - reduce:
+    - a
+    - e
+  - inject:
+    - a
+    - e
+Style/SingleLineMethods:
+  Description: Avoid single-line methods.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-single-line-methods
+  Enabled: false
+  AllowIfMethodIsEmpty: true
+Style/StringLiterals:
+  Description: Checks if uses of quotes match the configured preference.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#consistent-string-literals
+  Enabled: true
+  EnforcedStyle: double_quotes
+  SupportedStyles:
+  - single_quotes
+  - double_quotes
+Style/StringLiteralsInInterpolation:
+  Description: Checks if uses of quotes inside expressions in interpolated strings
+    match the configured preference.
+  Enabled: true
+  EnforcedStyle: single_quotes
+  SupportedStyles:
+  - single_quotes
+  - double_quotes
+Style/TrailingCommaInArguments:
+  Description: 'Checks for trailing comma in argument lists.'
+  StyleGuide: 'https://github.com/bbatsov/ruby-style-guide#no-trailing-array-commas'
+  Enabled: false
+  EnforcedStyleForMultiline: no_comma
+  SupportedStyles:
+  - comma
+  - consistent_comma
+  - no_comma
+Style/TrailingCommaInLiteral:
+  Description: 'Checks for trailing comma in array and hash literals.'
+  StyleGuide: 'https://github.com/bbatsov/ruby-style-guide#no-trailing-array-commas'
+  Enabled: false
+  EnforcedStyleForMultiline: no_comma
+  SupportedStyles:
+  - comma
+  - consistent_comma
+  - no_comma
+Metrics/AbcSize:
+  Description: A calculated magnitude based on number of assignments, branches, and
+    conditions.
+  Enabled: false
+  Max: 15
+Metrics/ClassLength:
+  Description: Avoid classes longer than 100 lines of code.
+  Enabled: false
+  CountComments: false
+  Max: 100
+Metrics/ModuleLength:
+  CountComments: false
+  Max: 100
+  Description: Avoid modules longer than 100 lines of code.
+  Enabled: false
+Metrics/CyclomaticComplexity:
+  Description: A complexity metric that is strongly correlated to the number of test
+    cases needed to validate a method.
+  Enabled: false
+  Max: 6
+Metrics/MethodLength:
+  Description: Avoid methods longer than 10 lines of code.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#short-methods
+  Enabled: false
+  CountComments: false
+  Max: 10
+Metrics/ParameterLists:
+  Description: Avoid parameter lists longer than three or four parameters.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#too-many-params
+  Enabled: false
+  Max: 5
+  CountKeywordArgs: true
+Metrics/PerceivedComplexity:
+  Description: A complexity metric geared towards measuring complexity for a human
+    reader.
+  Enabled: false
+  Max: 7
+Lint/AssignmentInCondition:
+  Description: Don't use assignment in conditions.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#safe-assignment-in-condition
+  Enabled: false
+  AllowSafeAssignment: true
+Style/InlineComment:
+  Description: Avoid inline comments.
+  Enabled: false
+Style/AccessorMethodName:
+  Description: Check the naming of accessor methods for get_/set_.
+  Enabled: false
+Style/Alias:
+  Description: Use alias_method instead of alias.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#alias-method
+  Enabled: false
+Style/Documentation:
+  Description: Document classes and non-namespace modules.
+  Enabled: false
+Style/DoubleNegation:
+  Description: Checks for uses of double negation (!!).
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-bang-bang
+  Enabled: false
+Style/EachWithObject:
+  Description: Prefer `each_with_object` over `inject` or `reduce`.
+  Enabled: false
+Style/EmptyLiteral:
+  Description: Prefer literals to Array.new/Hash.new/String.new.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#literal-array-hash
+  Enabled: false
+Style/ModuleFunction:
+  Description: Checks for usage of `extend self` in modules.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#module-function
+  Enabled: false
+Style/OneLineConditional:
+  Description: Favor the ternary operator(?:) over if/then/else/end constructs.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#ternary-operator
+  Enabled: false
+Style/PerlBackrefs:
+  Description: Avoid Perl-style regex back references.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-perl-regexp-last-matchers
+  Enabled: false
+Style/Send:
+  Description: Prefer `Object#__send__` or `Object#public_send` to `send`, as `send`
+    may overlap with existing methods.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#prefer-public-send
+  Enabled: false
+Style/SpecialGlobalVars:
+  Description: Avoid Perl-style global variables.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-cryptic-perlisms
+  Enabled: false
+Style/VariableInterpolation:
+  Description: Don't interpolate global, instance and class variables directly in
+    strings.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#curlies-interpolate
+  Enabled: false
+Style/WhenThen:
+  Description: Use when x then ... for one-line cases.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#one-line-cases
+  Enabled: false
+Lint/EachWithObjectArgument:
+  Description: Check for immutable argument given to each_with_object.
+  Enabled: true
+Lint/HandleExceptions:
+  Description: Don't suppress exception.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#dont-hide-exceptions
+  Enabled: false
+Lint/LiteralInCondition:
+  Description: Checks of literals used in conditions.
+  Enabled: false
+Lint/LiteralInInterpolation:
+  Description: Checks for literals used in interpolation.
+  Enabled: false

data/.ruby-version

@@ -1 +1 @@
-2.2.2
+2.3.0

data/.travis.yml

@@ -1,7 +1,7 @@
 language: ruby
 cache: bundler
 rvm:
-  - 2.2.2
+  - 2.3.0
 script: bundle exec rake
 before_install:
   - gem update --system

data/README.md

@@ -3,6 +3,7 @@
 [![Build Status](https://travis-ci.org/EdgarOrtegaRamirez/lita-team.svg?branch=master)](https://travis-ci.org/EdgarOrtegaRamirez/lita-team)
 [![Coverage Status](https://coveralls.io/repos/EdgarOrtegaRamirez/lita-team/badge.svg?branch=master)](https://coveralls.io/r/EdgarOrtegaRamirez/lita-team?branch=master)
 [![Code Climate](https://codeclimate.com/github/EdgarOrtegaRamirez/lita-team/badges/gpa.svg)](https://codeclimate.com/github/EdgarOrtegaRamirez/lita-team)
+[![Security](https://hakiri.io/github/EdgarOrtegaRamirez/lita-team/master.svg)](https://hakiri.io/github/EdgarOrtegaRamirez/lita-team/master)
 
 Create and manage the members of a team with Lita
 
@@ -11,7 +12,7 @@ Create and manage the members of a team with Lita
 Add lita-team to your Lita instance's Gemfile:
 
 ``` ruby
-gem "lita-team"
+gem 'lita-team'
 ```
 
 ## Usage

data/lib/lita/handlers/add_member_to_team.rb

@@ -0,0 +1,74 @@
+module Lita
+  module Handlers
+    class AddMemberToTeam < Handler
+      namespace :team
+      template_root File.expand_path("../../../../templates", __FILE__)
+
+      route(
+        /(\S*) team (\+1|add me|add (\S*))$/i,
+        :add,
+        command: true,
+        help: {
+          "<name> team +1" => "add me to team",
+          "<name> team add me" => "add me to team",
+          "<name> team add <member>" => "add member to team"
+        }
+      )
+
+      def add(response)
+        team_name = response.match_data[1]
+        if team = get_team(team_name)
+          if team[:limit] && team[:limit] <= team[:members].count
+            return response.reply(
+              render_template(:team_cannot_perform_operation,
+                              team_name: team_name)
+            )
+          end
+          member_name = (response.match_data[3] ||
+                         response.user.mention_name).delete("@")
+          if team[:members].key?(member_name.to_sym)
+            return response.reply(
+              t("member.already_in_team",
+                team_name: team_name,
+                member_name: member_name)
+            )
+          end
+          member = member_data(member_name)
+          count_was = team[:members].count
+          team[:members][member_name] = member
+          redis.set(team_name, MultiJson.dump(team))
+          response.reply(
+            render_template(:member_added_to_team,
+                            team_name: team_name,
+                            member_name: member[:name],
+                            count: count_was)
+          )
+        else
+          response.reply(render_template(:team_not_found, team_name: team_name))
+        end
+      end
+
+      private
+
+      def get_team(team_name)
+        data = redis.get(team_name)
+        MultiJson.load(data, symbolize_keys: true) if data
+      end
+
+      def member_data(member_name)
+        if lita_user = Lita::User.fuzzy_find(member_name)
+          {
+            id: lita_user.id,
+            name: lita_user.name,
+            mention_name: lita_user.mention_name,
+            confirmed: false
+          }
+        else
+          { name: member_name, confirmed: false }
+        end
+      end
+
+      Lita.register_handler(self)
+    end
+  end
+end

data/lib/lita/handlers/block_team.rb

@@ -0,0 +1,42 @@
+module Lita
+  module Handlers
+    class BlockTeam < Handler
+      namespace :team
+      template_root File.expand_path("../../../../templates", __FILE__)
+
+      route(
+        /(block|unblock) (\S*) team/i,
+        :toggle_block,
+        command: true,
+        help: {
+          "block <name> team" => "block team called <name>",
+          "unblock <name> team" => "unblock team called <name>"
+        }
+      )
+
+      def toggle_block(response)
+        team_name = response.match_data[2]
+        action = response.match_data[1]
+        if team = get_team(team_name)
+          team[:limit] = action == "block" ? team[:members].size : nil
+          redis.set(team[:name], MultiJson.dump(team))
+          response.reply(
+            render_template("team_#{action}ed".to_sym, team_name: team_name)
+          )
+        else
+          response.reply(
+            render_template(:team_not_found, team_name: team_name)
+          )
+        end
+      end
+
+      private
+
+      def get_team(team_name)
+        data = redis.get(team_name)
+        MultiJson.load(data, symbolize_keys: true) if data
+      end
+      Lita.register_handler(self)
+    end
+  end
+end

data/lib/lita/handlers/clear_team.rb

@@ -0,0 +1,38 @@
+module Lita
+  module Handlers
+    class ClearTeam < Handler
+      namespace :team
+      template_root File.expand_path("../../../../templates", __FILE__)
+
+      route(
+        /(\S*) team (clear|empty)/i,
+        :clear,
+        command: true,
+        help: {
+          "<name> team clear" => "clear team list",
+          "<name> team empty" => "clear team list"
+        }
+      )
+
+      def clear(response)
+        team_name = response.match_data[1]
+        if team = get_team(team_name)
+          team[:members] = {}
+          redis.set(team[:name], MultiJson.dump(team))
+          response.reply(t(:cleared, team_name: team_name))
+        else
+          response.reply(render_template(:team_not_found, team_name: team_name))
+        end
+      end
+
+      private
+
+      def get_team(team_name)
+        data = redis.get(team_name)
+        MultiJson.load(data, symbolize_keys: true) if data
+      end
+
+      Lita.register_handler(self)
+    end
+  end
+end

data/lib/lita/handlers/confirm_member.rb

@@ -0,0 +1,56 @@
+module Lita
+  module Handlers
+    class ConfirmMember < Handler
+      namespace :team
+      template_root File.expand_path("../../../../templates", __FILE__)
+
+      route(
+        /(\S*) team confirm (me|(\S*))$/i,
+        :confirm,
+        command: true,
+        help: {
+          "<name> team confirm me" => "confirm attendance",
+          "<name> team remove <member>" => "confirm member attendance"
+        }
+      )
+
+      def confirm(response)
+        team_name = response.match_data[1]
+        if team = get_team(team_name)
+          member_name = (response.match_data[3] ||
+                         response.user.mention_name).delete("@")
+          if member = team[:members][member_name.to_sym]
+            member[:confirmed] = true
+            unconfirmed = team[:members].count { |_k, v| !v[:confirmed] }
+            redis.set(team_name, MultiJson.dump(team))
+            response.reply(
+              render_template(:member_confirmed,
+                              team_name: team_name,
+                              member_name: member[:name],
+                              count: unconfirmed)
+            )
+          else
+            response.reply(
+              t("member.not_in_team",
+                team_name: team_name,
+                member_name: member_name)
+            )
+          end
+        else
+          response.reply(
+            render_template(:team_not_found, team_name: team_name)
+          )
+        end
+      end
+
+      private
+
+      def get_team(team_name)
+        data = redis.get(team_name)
+        MultiJson.load(data, symbolize_keys: true) if data
+      end
+
+      Lita.register_handler(self)
+    end
+  end
+end

data/lib/lita/handlers/create_team.rb

@@ -0,0 +1,38 @@
+module Lita
+  module Handlers
+    class CreateTeam < Handler
+      namespace :team
+      template_root File.expand_path("../../../../templates", __FILE__)
+
+      route(
+        /create (\S*) team/i,
+        :create,
+        command: true,
+        help: {
+          "create <name> team" => "create team called <name>"
+        }
+      )
+
+      def create(response)
+        team_name = response.match_data[1]
+        if redis.exists(team_name)
+          response.reply(
+            render_template(:team_already_exists, team_name: team_name)
+          )
+        else
+          data = {
+            name: team_name,
+            limit: nil,
+            location: nil,
+            icon: nil,
+            members: {}
+          }
+          redis.set(team_name, MultiJson.dump(data))
+          response.reply(render_template(:team_created, team_name: team_name))
+        end
+      end
+
+      Lita.register_handler(self)
+    end
+  end
+end

data/lib/lita/handlers/delete_team.rb

@@ -0,0 +1,26 @@
+module Lita
+  module Handlers
+    class DeleteTeam < Handler
+      namespace :team
+      template_root File.expand_path("../../../../templates", __FILE__)
+
+      route(/(delete|remove) (\S*) team/i, :delete, command: true, help:
+      {
+        "delete <name> team" => "delete team called <name>",
+        "remove <name> team" => "delete team called <name>"
+      })
+
+      def delete(response)
+        team_name = response.match_data[2]
+        if redis.exists(team_name)
+          redis.del(team_name)
+          response.reply(render_template(:team_deleted, team_name: team_name))
+        else
+          response.reply(render_template(:team_not_found, team_name: team_name))
+        end
+      end
+
+      Lita.register_handler(self)
+    end
+  end
+end

data/lib/lita/handlers/list_team.rb

@@ -0,0 +1,36 @@
+module Lita
+  module Handlers
+    class ListTeam < Handler
+      namespace :team
+      template_root File.expand_path("../../../../templates", __FILE__)
+
+      route(
+        /(\S*) team (list|show)/i,
+        :list,
+        command: true,
+        help: {
+          "<name> team list" => "list the people in the team",
+          "<name> team show" => "list the people in the team"
+        }
+      )
+
+      def list(response)
+        team_name = response.match_data[1]
+        if team = get_team(team_name)
+          response.reply(render_template(:list_team, team: team))
+        else
+          response.reply(render_template(:team_not_found, team_name: team_name))
+        end
+      end
+
+      private
+
+      def get_team(team_name)
+        data = redis.get(team_name)
+        MultiJson.load(data, symbolize_keys: true) if data
+      end
+
+      Lita.register_handler(self)
+    end
+  end
+end

data/lib/lita/handlers/list_teams.rb

@@ -0,0 +1,35 @@
+module Lita
+  module Handlers
+    class ListTeams < Handler
+      namespace :team
+      template_root File.expand_path("../../../../templates", __FILE__)
+
+      route(
+        /list teams/i,
+        :list,
+        command: true,
+        help: {
+          "list teams" => "list all teams"
+        }
+      )
+
+      def list(response)
+        keys = redis.keys
+        if keys.empty?
+          response.reply t(:no_teams_has_been_created)
+        else
+          teams = redis.mget(redis.keys("*")).map(&parse)
+          response.reply render_template(:list_teams, teams: teams)
+        end
+      end
+
+      private
+
+      def parse
+        -> (team) { MultiJson.load(team, symbolize_keys: true) }
+      end
+
+      Lita.register_handler(self)
+    end
+  end
+end