checkoff 0.4.0 → 0.5.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: abb481b28efc4dc6131fdc028fe0bffb8c441dc92c95474e23455f7a1fed8ede
-  data.tar.gz: 98e270c7be0078e5efa19425affc4692c3343d093e5ab96d027eb5b643d8b141
+  metadata.gz: eedcba2aa1cb33274b57228b924f8871d65e6d169b90d85b8328b207245c5d80
+  data.tar.gz: 075bd1131ea2a329637d77e08ce41e5589889c2207751634c7fe83fe7f8935d8
 SHA512:
-  metadata.gz: 2c6ddd7d5092290e0fbc953274f98c2b0b68262aef2201a378fa71a947152ee3c0d331c730127e5ba05616e7e9afc3261324a5d0b6babbaf63c4bd96115fcfa2
-  data.tar.gz: 3ba297866e0c3970e12a8f31b796359f58b93d02a2d7aae5aef04dd071e68a5d4d41a4c5c4e8b3ec5f38123b0f7072f1dd2ee98217fae9bd3d1a6fb2aa843012
+  metadata.gz: a7aaf83addcfd2371145991bd2672e0cd2ec9307359bcdd8fe606ff4e60eff09271b5fb08e031b448c540f29a2c3b7dbbddf2127f5a75c52ff8b4beacc8b3382
+  data.tar.gz: 581ccb6c3e27e608abf704751f06dcfd79ceb96b37f78131e603d9659961bf4269717aa5d6fd53bbf09f8cc71116e8ca859c27d327dbb13c05583552f0353a42

data/.rubocop.yml

@@ -67,9 +67,134 @@ Style/StringLiterals:
       - double_quotes
     ConsistentQuotesInMultiline: true
 
-AllCops:
-  TargetRubyVersion: 2.4
+Layout/BeginEndAlignment: # (new in 0.91)
+  Enabled: true
+Layout/EmptyLinesAroundAttributeAccessor: # (new in 0.83)
+  Enabled: true
+Layout/SpaceAroundMethodCallOperator: # (new in 0.82)
+  Enabled: true
+Lint/BinaryOperatorWithIdenticalOperands: # (new in 0.89)
+  Enabled: true
+Lint/ConstantDefinitionInBlock: # (new in 0.91)
+  Enabled: true
+Lint/DeprecatedOpenSSLConstant: # (new in 0.84)
+  Enabled: true
+Lint/DuplicateElsifCondition: # (new in 0.88)
+  Enabled: true
+Lint/DuplicateRequire: # (new in 0.90)
+  Enabled: true
+Lint/DuplicateRescueException: # (new in 0.89)
+  Enabled: true
+Lint/EmptyConditionalBody: # (new in 0.89)
+  Enabled: true
+Lint/EmptyFile: # (new in 0.90)
+  Enabled: true
+Lint/FloatComparison: # (new in 0.89)
+  Enabled: true
+Lint/IdentityComparison: # (new in 0.91)
+  Enabled: true
+Lint/MissingSuper: # (new in 0.89)
+  Enabled: true
+Lint/MixedRegexpCaptureTypes: # (new in 0.85)
+  Enabled: true
+Lint/OutOfRangeRegexpRef: # (new in 0.89)
+  Enabled: true
+Lint/RaiseException: # (new in 0.81)
+  Enabled: true
+Lint/SelfAssignment: # (new in 0.89)
+  Enabled: true
+Lint/StructNewOverride: # (new in 0.81)
+  Enabled: true
+Lint/TopLevelReturnWithArgument: # (new in 0.89)
+  Enabled: true
+Lint/TrailingCommaInAttributeDeclaration: # (new in 0.90)
+  Enabled: true
+Lint/UnreachableLoop: # (new in 0.89)
+  Enabled: true
+Lint/UselessMethodDefinition: # (new in 0.90)
+  Enabled: true
+Lint/UselessTimes: # (new in 0.91)
+  Enabled: true
+Style/AccessorGrouping: # (new in 0.87)
+  Enabled: true
+Style/BisectedAttrAccessor: # (new in 0.87)
+  Enabled: true
+Style/CaseLikeIf: # (new in 0.88)
+  Enabled: true
+Style/CombinableLoops: # (new in 0.90)
+  Enabled: true
+Style/ExplicitBlockArgument: # (new in 0.89)
+  Enabled: true
+Style/ExponentialNotation: # (new in 0.82)
+  Enabled: true
+Style/GlobalStdStream: # (new in 0.89)
+  Enabled: true
+Style/HashAsLastArrayItem: # (new in 0.88)
+  Enabled: true
+Style/HashEachMethods: # (new in 0.80)
+  Enabled: true
+Style/HashLikeCase: # (new in 0.88)
+  Enabled: true
+Style/HashTransformKeys: # (new in 0.80)
+  Enabled: true
+Style/HashTransformValues: # (new in 0.80)
+  Enabled: true
+Style/KeywordParametersOrder: # (new in 0.90)
+  Enabled: true
+Style/OptionalBooleanParameter: # (new in 0.89)
+  Enabled: true
+Style/RedundantAssignment: # (new in 0.87)
+  Enabled: true
+Style/RedundantFetchBlock: # (new in 0.86)
+  Enabled: true
+Style/RedundantFileExtensionInRequire: # (new in 0.88)
+  Enabled: true
+Style/RedundantRegexpCharacterClass: # (new in 0.85)
+  Enabled: true
+Style/RedundantRegexpEscape: # (new in 0.85)
+  Enabled: true
+Style/RedundantSelfAssignment: # (new in 0.90)
+  Enabled: true
+Style/SingleArgumentDig: # (new in 0.89)
+  Enabled: true
+Style/SlicingWithRange: # (new in 0.83)
+  Enabled: true
+Style/SoleNestedConditional: # (new in 0.89)
+  Enabled: true
+Style/StringConcatenation: # (new in 0.89)
+  Enabled: true
+Minitest/AssertInDelta: # (new in 0.10)
+  Enabled: true
+Minitest/AssertionInLifecycleHook: # (new in 0.10)
+  Enabled: true
+Minitest/AssertKindOf: # (new in 0.10)
+  Enabled: true
+Minitest/AssertOutput: # (new in 0.10)
+  Enabled: true
+Minitest/AssertPathExists: # (new in 0.10)
+  Enabled: true
+Minitest/AssertSilent: # (new in 0.10)
+  Enabled: true
+Minitest/LiteralAsActualArgument: # (new in 0.10)
+  Enabled: true
+Minitest/MultipleAssertions: # (new in 0.10)
+  Enabled: true
+Minitest/RefuteInDelta: # (new in 0.10)
+  Enabled: true
+Minitest/RefuteKindOf: # (new in 0.10)
+  Enabled: true
+Minitest/RefutePathExists: # (new in 0.10)
+  Enabled: true
+Minitest/TestMethodName: # (new in 0.10)
+  # This gives false positives when non-test methods exist...
+  Enabled: false
+Minitest/UnspecifiedException: # (new in 0.10)
+  Enabled: true
+RSpec/StubbedMock: # (new in 1.44)
+  Enabled: true
 
+AllCops:
+  TargetRubyVersion: 2.5
 
 require:
   - rubocop-rspec

data/.travis.yml

@@ -10,8 +10,13 @@ rvm:
   # checking for main() in -lstdc++... RuntimeError: The compiler failed to generate
   # https://travis-ci.com/apiology/pronto-punchlist/jobs/264907931
   # - jruby-head
-  - ruby-head
+
+  # These are currently pending ruby-asana fixes:
+  #
+  # https://github.com/Asana/ruby-asana/pull/99
+  #
+  # - ruby-head
+  # - ruby-3.0
   - ruby-2.7
   - ruby-2.6
   - ruby-2.5
-  - ruby-2.4

data/checkoff.gemspec

@@ -10,12 +10,12 @@ Gem::Specification.new do |spec|
   spec.version       = Checkoff::VERSION
   spec.authors       = ['Vince Broz']
   spec.email         = ['vince@broz.cc']
-
   spec.summary       = 'Command-line and gem client for Asana (unofficial)'
   spec.homepage      = 'http://github.com/apiology/checkoff'
   spec.license       = 'MIT'
+  spec.required_ruby_version = '>= 2.5'
 
-  spec.files         = `git ls-files -z`.split("\x0").reject do |f|
+  spec.files = `git ls-files -z`.split("\x0").reject do |f|
     f.match(%r{^(test|spec|features)/})
   end
   spec.bindir        = 'exe'
@@ -23,14 +23,14 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_runtime_dependency 'activesupport'
-  spec.add_runtime_dependency 'asana'
+  spec.add_runtime_dependency 'asana', '>0.10.0'
   spec.add_runtime_dependency 'cache_method'
   spec.add_runtime_dependency 'dalli'
 
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'minitest-profile'
   spec.add_development_dependency 'mocha'
-  spec.add_development_dependency 'quality', '~> 37'
+  spec.add_development_dependency 'quality', '~> 38'
   spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rspec', '>=3.4'
   spec.add_development_dependency 'simplecov'

data/coverage/.last_run.json

@@ -1,5 +1,5 @@
 {
   "result": {
-    "covered_percent": 88.54
+    "covered_percent": 83.74
   }
 }

data/lib/checkoff.rb

@@ -4,4 +4,5 @@ require 'checkoff/version'
 require 'checkoff/workspaces'
 require 'checkoff/projects'
 require 'checkoff/sections'
+require 'checkoff/subtasks'
 require 'checkoff/tasks'

data/lib/checkoff/cli.rb

@@ -15,11 +15,13 @@ module Checkoff
   class CLI
     attr_reader :sections, :stderr
 
-    def initialize(workspaces: Checkoff::Workspaces.new,
-                   projects: Checkoff::Projects.new,
-                   sections: Checkoff::Sections.new(projects: projects),
-                   tasks: Checkoff::Tasks.new,
-                   stderr: STDERR,
+    def initialize(config: Checkoff::ConfigLoader.load(:asana),
+                   workspaces: Checkoff::Workspaces.new(config: config),
+                   projects: Checkoff::Projects.new(config: config),
+                   sections: Checkoff::Sections.new(config: config,
+                                                    projects: projects),
+                   tasks: Checkoff::Tasks.new(config: config),
+                   stderr: $stderr,
                    kernel: Kernel)
       @workspaces = workspaces
       @projects = projects
@@ -87,9 +89,10 @@ module Checkoff
     def parse_args(args)
       mode = args[0]
       subargs = OpenStruct.new
-      if mode == 'view'
+      case mode
+      when 'view'
         parse_view_args(subargs, args)
-      elsif mode == 'quickadd'
+      when 'quickadd'
         parse_quickadd_args(subargs, args)
       else
         raise
@@ -118,9 +121,10 @@ module Checkoff
     def run(args)
       validate_args!(args)
       command, subargs = parse_args(args)
-      if command == 'view'
+      case command
+      when 'view'
         view(subargs.workspace, subargs.project, subargs.section)
-      elsif command == 'quickadd'
+      when 'quickadd'
         quickadd(subargs.workspace, subargs.task_name)
       else
         raise

data/lib/checkoff/config_loader.rb

@@ -3,7 +3,6 @@
 require 'yaml'
 require 'active_support/core_ext/hash'
 
-# frozen_string_literal: true
 module Checkoff
   # Load configuration file
   class ConfigLoader

data/lib/checkoff/projects.rb

@@ -1,6 +1,7 @@
+# frozen_string_literal: true
+
 require_relative 'config_loader'
 
-# frozen_string_literal: true
 require 'cache_method'
 require 'asana'
 
@@ -25,7 +26,7 @@ module Checkoff
 
     def initialize(config: Checkoff::ConfigLoader.load(:asana),
                    asana_client: Asana::Client,
-                   workspaces: Checkoff::Workspaces.new)
+                   workspaces: Checkoff::Workspaces.new(config: config))
       @config = config
       @asana_client = asana_client
       @workspaces = workspaces
@@ -50,9 +51,7 @@ module Checkoff
     def my_tasks(workspace_name)
       my_tasks = @config[:my_tasks]
       gid = @config[:my_tasks][workspace_name] unless my_tasks.nil?
-      if my_tasks.nil? || gid.nil?
-        raise "Please define [:my_tasks][#{workspace_name}] in config file"
-      end
+      raise "Please define [:my_tasks][#{workspace_name}] in config file" if my_tasks.nil? || gid.nil?
 
       projects.find_by_id(gid)
     end

data/lib/checkoff/sections.rb

@@ -13,7 +13,8 @@ module Checkoff
 
     attr_reader :projects, :time
 
-    def initialize(projects: Checkoff::Projects.new,
+    def initialize(config: Checkoff::ConfigLoader.load(:asana),
+                   projects: Checkoff::Projects.new(config: config),
                    time: Time)
       @projects = projects
       @time = time
@@ -22,8 +23,11 @@ module Checkoff
     def_delegators :@projects, :client
 
     def file_task_by_section(by_section, task, project_gid)
-      membership = task.memberships.find { |m| m.project.gid == project_gid }
+      membership = task.memberships.find { |m| m['project']['gid'] == project_gid }
+      raise "Could not find task in project_gid #{project_gid}: #{task}" if membership.nil?
+
       current_section = membership.section.name
+      current_section = nil if current_section == '(no section)'
       by_section[current_section] ||= []
       by_section[current_section] << task
     end
@@ -43,6 +47,27 @@ module Checkoff
       by_section(active_tasks, project.gid)
     end
 
+    def legacy_file_task_by_section(current_section, by_section, task)
+      if task.name =~ /:$/
+        current_section = task.name
+        by_section[current_section] = []
+      else
+        by_section[current_section] ||= []
+        by_section[current_section] << task
+      end
+      [current_section, by_section]
+    end
+
+    def legacy_by_section(tasks)
+      current_section = nil
+      by_section = {}
+      tasks.each do |task|
+        current_section, by_section =
+          legacy_file_task_by_section(current_section, by_section, task)
+      end
+      by_section
+    end
+
     def tasks_by_section_for_project_and_assignee_status(project,
                                                          assignee_status)
       raw_tasks = projects.tasks_from_project(project)
@@ -50,7 +75,7 @@ module Checkoff
         projects.active_tasks(raw_tasks)
                 .group_by(&:assignee_status)
       active_tasks = by_assignee_status[assignee_status]
-      by_section(active_tasks, project.gid)
+      legacy_by_section(active_tasks)
     end
 
     def project_or_raise(workspace_name, project_name)
@@ -64,11 +89,12 @@ module Checkoff
 
     def tasks_by_section(workspace_name, project_name)
       project = project_or_raise(workspace_name, project_name)
-      if project_name == :my_tasks_new
+      case project_name
+      when :my_tasks_new
         tasks_by_section_for_project_and_assignee_status(project, 'inbox')
-      elsif project_name == :my_tasks_today
+      when :my_tasks_today
         tasks_by_section_for_project_and_assignee_status(project, 'today')
-      elsif project_name == :my_tasks_upcoming
+      when :my_tasks_upcoming
         tasks_by_section_for_project_and_assignee_status(project, 'upcoming')
       else
         tasks_by_section_for_project(project)
@@ -94,12 +120,6 @@ module Checkoff
     end
     cache_method :section_task_names, SHORT_CACHE_TIME
 
-    # Returns all subtasks, including section headers
-    def raw_subtasks(task)
-      task.subtasks(projects.task_options)
-    end
-    cache_method :raw_subtasks, LONG_CACHE_TIME
-
     def task_due?(task)
       if task.due_at
         Time.parse(task.due_at) <= time.now

data/lib/checkoff/subtasks.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+require 'forwardable'
+
+module Checkoff
+  # Query different subtasks of Asana tasks
+  class Subtasks
+    MINUTE = 60
+    LONG_CACHE_TIME = MINUTE * 15
+    SHORT_CACHE_TIME = MINUTE * 5
+
+    extend Forwardable
+
+    def file_task_by_section(current_section, by_section, task)
+      if task.name =~ /:$/
+        current_section = task.name
+        by_section[current_section] = []
+      else
+        by_section[current_section] ||= []
+        by_section[current_section] << task
+      end
+      [current_section, by_section]
+    end
+
+    def by_section(tasks)
+      current_section = nil
+      by_section = {}
+      tasks.each do |task|
+        current_section, by_section = file_task_by_section(current_section,
+                                                           by_section, task)
+      end
+      by_section
+    end
+
+    # Returns all subtasks, including section headers
+    def raw_subtasks(task)
+      task.subtasks(projects.task_options)
+    end
+    cache_method :raw_subtasks, LONG_CACHE_TIME
+  end
+end

data/lib/checkoff/tasks.rb

@@ -15,7 +15,7 @@ module Checkoff
     SHORT_CACHE_TIME = MINUTE * 5
 
     def initialize(config: Checkoff::ConfigLoader.load(:asana),
-                   sections: Checkoff::Sections.new,
+                   sections: Checkoff::Sections.new(config: config),
                    asana_task: Asana::Resources::Task)
       @config = config
       @sections = sections

data/lib/checkoff/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Checkoff
-  VERSION = '0.4.0'
+  VERSION = '0.5.4'
 end

data/lib/tasks/quality.rake

@@ -3,7 +3,7 @@
 require 'quality/rake/task'
 
 Quality::Rake::Task.new do |task|
-  task.skip_tools = ['reek']
+  task.skip_tools = %w[reek cane eslint jscs pycodestyle rails_best_practices flake8]
   task.output_dir = 'metrics'
-  task.exclude_files = ['docs/example_project.png']
+  task.exclude_files = ['docs/example_project.png', '.rubocop.yml']
 end

data/metrics/bigfiles_high_water_mark

@@ -1 +1 @@
-390
+414

data/metrics/rubocop_high_water_mark

@@ -1 +1 @@
-2
+4

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: checkoff
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.4
 platform: ruby
 authors:
 - Vince Broz
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-10-17 00:00:00.000000000 Z
+date: 2020-12-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -28,16 +28,16 @@ dependencies:
   name: asana
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ">"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.10.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ">"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.10.0
 - !ruby/object:Gem::Dependency
   name: cache_method
   requirement: !ruby/object:Gem::Requirement
@@ -114,14 +114,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '37'
+        version: '38'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '37'
+        version: '38'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -194,6 +194,7 @@ files:
 - lib/checkoff/config_loader.rb
 - lib/checkoff/projects.rb
 - lib/checkoff/sections.rb
+- lib/checkoff/subtasks.rb
 - lib/checkoff/tasks.rb
 - lib/checkoff/version.rb
 - lib/checkoff/workspaces.rb
@@ -234,7 +235,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="