t 3.1.0 → 4.1.0

data/lib/t/printable.rb

@@ -1,8 +1,8 @@
 module T
-  module Printable # rubocop:disable ModuleLength
-    LIST_HEADINGS = ['ID', 'Created at', 'Screen name', 'Slug', 'Members', 'Subscribers', 'Mode', 'Description'].freeze
-    TWEET_HEADINGS = ['ID', 'Posted at', 'Screen name', 'Text'].freeze
-    USER_HEADINGS = ['ID', 'Since', 'Last tweeted at', 'Tweets', 'Favorites', 'Listed', 'Following', 'Followers', 'Screen name', 'Name', 'Verified', 'Protected', 'Bio', 'Status', 'Location', 'URL'].freeze
+  module Printable # rubocop:disable Metrics/ModuleLength
+    LIST_HEADINGS = ["ID", "Created at", "Screen name", "Slug", "Members", "Subscribers", "Mode", "Description"].freeze
+    TWEET_HEADINGS = ["ID", "Posted at", "Screen name", "Text"].freeze
+    USER_HEADINGS = ["ID", "Since", "Last tweeted at", "Tweets", "Favorites", "Listed", "Following", "Followers", "Screen name", "Name", "Verified", "Protected", "Bio", "Status", "Location", "URL"].freeze
     MONTH_IN_SECONDS = 2_592_000
 
   private
@@ -12,73 +12,75 @@ module T
     end
 
     def build_long_tweet(tweet)
-      [tweet.id, ls_formatted_time(tweet), "@#{tweet.user.screen_name}", decode_full_text(tweet, options['decode_uris']).gsub(/\n+/, ' ')]
+      [tweet.id, ls_formatted_time(tweet), "@#{tweet.user.screen_name}", decode_full_text(tweet, options["decode_uris"]).gsub(/\n+/, " ")]
     end
 
     def build_long_user(user)
-      [user.id, ls_formatted_time(user), ls_formatted_time(user.status), user.statuses_count, user.favorites_count, user.listed_count, user.friends_count, user.followers_count, "@#{user.screen_name}", user.name, user.verified? ? 'Yes' : 'No', user.protected? ? 'Yes' : 'No', user.description.gsub(/\n+/, ' '), user.status? ? decode_full_text(user.status, options['decode_uris']).gsub(/\n+/, ' ') : nil, user.location, user.website.to_s]
+      [user.id, ls_formatted_time(user), ls_formatted_time(user.status), user.statuses_count, user.favorites_count, user.listed_count, user.friends_count, user.followers_count, "@#{user.screen_name}", user.name, user.verified? ? "Yes" : "No", user.protected? ? "Yes" : "No", user.description.gsub(/\n+/, " "), user.status? ? decode_full_text(user.status, options["decode_uris"]).gsub(/\n+/, " ") : nil, user.location, user.website.to_s]
     end
 
     def csv_formatted_time(object, key = :created_at)
       return nil if object.nil?
+
       time = object.send(key.to_sym).dup
-      time.utc.strftime('%Y-%m-%d %H:%M:%S %z')
+      time.utc.strftime("%Y-%m-%d %H:%M:%S %z")
     end
 
     def ls_formatted_time(object, key = :created_at, allow_relative = true)
-      return '' if object.nil?
+      return "" if object.nil?
+
       time = T.local_time(object.send(key.to_sym))
-      if allow_relative && options['relative_dates']
-        distance_of_time_in_words(time) + ' ago'
-      elsif time > Time.now - MONTH_IN_SECONDS * 6
-        time.strftime('%b %e %H:%M')
+      if allow_relative && options["relative_dates"]
+        "#{distance_of_time_in_words(time)} ago"
+      elsif time > Time.now - (MONTH_IN_SECONDS * 6)
+        time.strftime("%b %e %H:%M")
       else
-        time.strftime('%b %e  %Y')
+        time.strftime("%b %e  %Y")
       end
     end
 
     def print_csv_list(list)
-      require 'csv'
+      require "csv"
       say [list.id, csv_formatted_time(list), list.user.screen_name, list.slug, list.member_count, list.subscriber_count, list.mode, list.description].to_csv
     end
 
     def print_csv_tweet(tweet)
-      require 'csv'
-      say [tweet.id, csv_formatted_time(tweet), tweet.user.screen_name, decode_full_text(tweet, options['decode_uris'])].to_csv
+      require "csv"
+      say [tweet.id, csv_formatted_time(tweet), tweet.user.screen_name, decode_full_text(tweet, options["decode_uris"])].to_csv
     end
 
     def print_csv_user(user)
-      require 'csv'
+      require "csv"
       say [user.id, csv_formatted_time(user), csv_formatted_time(user.status), user.statuses_count, user.favorites_count, user.listed_count, user.friends_count, user.followers_count, user.screen_name, user.name, user.verified?, user.protected?, user.description, user.status? ? user.status.full_text : nil, user.location, user.website].to_csv
     end
 
     def print_lists(lists)
-      unless options['unsorted']
-        lists = case options['sort']
-        when 'members'
+      unless options["unsorted"]
+        lists = case options["sort"]
+        when "members"
           lists.sort_by(&:member_count)
-        when 'mode'
+        when "mode"
           lists.sort_by(&:mode)
-        when 'since'
+        when "since"
           lists.sort_by(&:created_at)
-        when 'subscribers'
+        when "subscribers"
           lists.sort_by(&:subscriber_count)
         else
           lists.sort_by { |list| list.slug.downcase }
         end
       end
-      lists.reverse! if options['reverse']
-      if options['csv']
-        require 'csv'
+      lists.reverse! if options["reverse"]
+      if options["csv"]
+        require "csv"
         say LIST_HEADINGS.to_csv unless lists.empty?
         lists.each do |list|
           print_csv_list(list)
         end
-      elsif options['long']
+      elsif options["long"]
         array = lists.collect do |list|
           build_long_list(list)
         end
-        format = options['format'] || Array.new(LIST_HEADINGS.size) { '%s' }
+        format = options["format"] || Array.new(LIST_HEADINGS.size) { "%s" }
         print_table_with_headings(array, LIST_HEADINGS, format)
       else
         print_attribute(lists, :full_name)
@@ -97,12 +99,14 @@ module T
 
     def print_table_with_headings(array, headings, format)
       return if array.flatten.empty?
+
       if STDOUT.tty?
         array.unshift(headings)
-        require 't/core_ext/kernel'
+        require "t/core_ext/kernel"
         array.collect! do |row|
           row.each_with_index.collect do |element, index|
             next if element.nil?
+
             Kernel.send(element.class.name.to_sym, format[index] % element)
           end
         end
@@ -114,14 +118,14 @@ module T
     end
 
     def print_message(from_user, message)
-      require 'htmlentities'
+      require "htmlentities"
 
-      case options['color']
-      when 'icon'
+      case options["color"]
+      when "icon"
         print_identicon(from_user, message)
         say
-      when 'auto'
-        say("   @#{from_user}", [:bold, :yellow])
+      when "auto"
+        say("   @#{from_user}", %i[bold yellow])
         print_wrapped(HTMLEntities.new.decode(message), indent: 3)
       else
         say("   @#{from_user}")
@@ -131,30 +135,30 @@ module T
     end
 
     def print_identicon(from_user, message)
-      require 'htmlentities'
-      require 't/identicon'
+      require "htmlentities"
+      require "t/identicon"
       icon = Identicon.for_user_name(from_user)
 
       # Save 6 chars for icon, ensure at least 3 lines long
-      lines = wrapped(HTMLEntities.new.decode(message), indent: 2, width: terminal_width - (6 + 5))
+      lines = wrapped(HTMLEntities.new.decode(message), indent: 2, width: Thor::Shell::Terminal.terminal_width - (6 + 5))
       lines.unshift(set_color("  @#{from_user}", :bold, :yellow))
-      lines.concat(Array.new([3 - lines.length, 0].max) { '' })
+      lines.concat(Array.new([3 - lines.length, 0].max) { "" })
 
-      $stdout.puts lines.zip(icon.lines).map { |x, i| "  #{i || '      '}#{x}" }
+      $stdout.puts(lines.zip(icon.lines).map { |x, i| "  #{i || '      '}#{x}" })
     end
 
     def wrapped(message, options = {})
       indent = options[:indent] || 0
-      width = options[:width] || terminal_width - indent
+      width = options[:width] || (Thor::Shell::Terminal.terminal_width - indent)
       paras = message.split("\n\n")
 
       paras.map! do |unwrapped|
-        unwrapped.strip.squeeze(' ').gsub(/.{1,#{width}}(?:\s|\Z)/) { ($& + 5.chr).gsub(/\n\005/, "\n").gsub(/\005/, "\n") }
+        unwrapped.strip.squeeze(" ").gsub(/.{1,#{width}}(?:\s|\Z)/) { (::Regexp.last_match(0) + 5.chr).gsub(/\n\005/, "\n").gsub(/\005/, "\n") }
       end
 
       lines = paras.inject([]) do |memo, para|
-        memo.concat(para.split("\n").map { |line| line.insert(0, ' ' * indent) })
-        memo.push ''
+        memo.concat(para.split("\n").map { |line| line.insert(0, " " * indent) })
+        memo.push ""
       end
 
       lines.pop
@@ -162,59 +166,59 @@ module T
     end
 
     def print_tweets(tweets)
-      tweets.reverse! if options['reverse']
-      if options['csv']
-        require 'csv'
+      tweets.reverse! if options["reverse"]
+      if options["csv"]
+        require "csv"
         say TWEET_HEADINGS.to_csv unless tweets.empty?
         tweets.each do |tweet|
           print_csv_tweet(tweet)
         end
-      elsif options['long']
+      elsif options["long"]
         array = tweets.collect do |tweet|
           build_long_tweet(tweet)
         end
-        format = options['format'] || Array.new(TWEET_HEADINGS.size) { '%s' }
+        format = options["format"] || Array.new(TWEET_HEADINGS.size) { "%s" }
         print_table_with_headings(array, TWEET_HEADINGS, format)
       else
         tweets.each do |tweet|
-          print_message(tweet.user.screen_name, decode_uris(tweet.full_text, options['decode_uris'] ? tweet.uris : nil))
+          print_message(tweet.user.screen_name, decode_uris(tweet.full_text, options["decode_uris"] ? tweet.uris : nil))
         end
       end
     end
 
-    def print_users(users) # rubocop:disable CyclomaticComplexity
-      unless options['unsorted']
-        users = case options['sort']
-        when 'favorites'
+    def print_users(users) # rubocop:disable Metrics/CyclomaticComplexity
+      unless options["unsorted"]
+        users = case options["sort"]
+        when "favorites"
           users.sort_by { |user| user.favorites_count.to_i }
-        when 'followers'
+        when "followers"
           users.sort_by { |user| user.followers_count.to_i }
-        when 'friends'
+        when "friends"
           users.sort_by { |user| user.friends_count.to_i }
-        when 'listed'
+        when "listed"
           users.sort_by { |user| user.listed_count.to_i }
-        when 'since'
+        when "since"
           users.sort_by(&:created_at)
-        when 'tweets'
+        when "tweets"
           users.sort_by { |user| user.statuses_count.to_i }
-        when 'tweeted'
-          users.sort_by { |user| user.status? ? user.status.created_at : Time.at(0) } # rubocop:disable BlockNesting
+        when "tweeted"
+          users.sort_by { |user| user.status? ? user.status.created_at : Time.at(0) } # rubocop:disable Metrics/BlockNesting
         else
           users.sort_by { |user| user.screen_name.downcase }
         end
       end
-      users.reverse! if options['reverse']
-      if options['csv']
-        require 'csv'
+      users.reverse! if options["reverse"]
+      if options["csv"]
+        require "csv"
         say USER_HEADINGS.to_csv unless users.empty?
         users.each do |user|
           print_csv_user(user)
         end
-      elsif options['long']
+      elsif options["long"]
         array = users.collect do |user|
           build_long_user(user)
         end
-        format = options['format'] || Array.new(USER_HEADINGS.size) { '%s' }
+        format = options["format"] || Array.new(USER_HEADINGS.size) { "%s" }
         print_table_with_headings(array, USER_HEADINGS, format)
       else
         print_attribute(users, :screen_name)

data/lib/t/rcfile.rb

@@ -1,13 +1,14 @@
-require 'singleton'
+require "singleton"
 
 module T
   class RCFile
     include Singleton
     attr_reader :path
-    FILE_NAME = '.trc'.freeze
+
+    FILE_NAME = ".trc".freeze
 
     def initialize
-      @path = File.join(File.expand_path('~'), FILE_NAME)
+      @path = File.join(File.expand_path("~"), FILE_NAME)
       @data = load_file
     end
 
@@ -17,7 +18,8 @@ module T
 
     def find(username)
       possibilities = Array(find_case_insensitive_match(username) || find_case_insensitive_possibilities(username))
-      raise(ArgumentError.new("Username #{username} is #{possibilities.empty? ? 'not found.' : 'ambiguous, matching ' + possibilities.join(', ')}")) unless possibilities.size == 1
+      raise(ArgumentError.new("Username #{username} is #{possibilities.empty? ? 'not found.' : "ambiguous, matching #{possibilities.join(', ')}"}")) unless possibilities.size == 1
+
       possibilities.first
     end
 
@@ -36,36 +38,36 @@ module T
     end
 
     def configuration
-      @data['configuration']
+      @data["configuration"]
     end
 
     def active_consumer_key
-      profiles[active_profile[0]][active_profile[1]]['consumer_key'] if active_profile?
+      profiles[active_profile[0]][active_profile[1]]["consumer_key"] if active_profile?
     end
 
     def active_consumer_secret
-      profiles[active_profile[0]][active_profile[1]]['consumer_secret'] if active_profile?
+      profiles[active_profile[0]][active_profile[1]]["consumer_secret"] if active_profile?
     end
 
     def active_profile
-      configuration['default_profile']
+      configuration["default_profile"]
     end
 
     def active_profile=(profile)
-      configuration['default_profile'] = [profile['username'], profile['consumer_key']]
+      configuration["default_profile"] = [profile["username"], profile["consumer_key"]]
       write
     end
 
     def active_secret
-      profiles[active_profile[0]][active_profile[1]]['secret'] if active_profile?
+      profiles[active_profile[0]][active_profile[1]]["secret"] if active_profile?
     end
 
     def active_token
-      profiles[active_profile[0]][active_profile[1]]['token'] if active_profile?
+      profiles[active_profile[0]][active_profile[1]]["token"] if active_profile?
     end
 
     def delete
-      File.delete(@path) if File.exist?(@path)
+      FileUtils.rm_f(@path)
     end
 
     def empty?
@@ -73,7 +75,7 @@ module T
     end
 
     def load_file
-      require 'yaml'
+      require "yaml"
       YAML.load_file(@path)
     rescue Errno::ENOENT
       default_structure
@@ -82,11 +84,10 @@ module T
     def path=(path)
       @path = path
       @data = load_file
-      @path
     end
 
     def profiles
-      @data['profiles']
+      @data["profiles"]
     end
 
     def reset
@@ -110,11 +111,11 @@ module T
     end
 
     def default_structure
-      {'configuration' => {}, 'profiles' => {}}
+      {"configuration" => {}, "profiles" => {}}
     end
 
     def write
-      require 'yaml'
+      require "yaml"
       File.open(@path, File::RDWR | File::TRUNC | File::CREAT, 0o0600) do |rcfile|
         rcfile.write @data.to_yaml
       end

data/lib/t/requestable.rb

@@ -1,4 +1,4 @@
-require 'twitter'
+require "twitter"
 
 module T
   module Requestable
@@ -6,7 +6,8 @@ module T
 
     def client
       return @client if @client
-      @rcfile.path = options['profile'] if options['profile']
+
+      @rcfile.path = options["profile"] if options["profile"]
       @client = Twitter::REST::Client.new do |config|
         config.consumer_key        = @rcfile.active_consumer_key
         config.consumer_secret     = @rcfile.active_consumer_secret