checkoff 0.5.0 → 0.5.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a75e010929b7dcb752bf9cffd0f424039cb4c54da94e44af30a8b8627f13eefb
-  data.tar.gz: f2410a5287cf99a90f23ed7052a18f09553a652b87b575a2b665e7717c0193fa
+  metadata.gz: 4ca9a96e298922b58c2d0912260c32e663591a1a8c9d331941d81387f66c5331
+  data.tar.gz: 96d002a7f29d566425c506ccbeddd002e85b7328377947a88942baa2c8a462e2
 SHA512:
-  metadata.gz: b7f458362faa147dc0c6d6097b4c3e912ce73c9a5de729fdfedd61bb691b7f0825b35dbce99ad5e06e7b82a1945a576fde09455fb94655a4748995d251da62e2
-  data.tar.gz: 3e47840ed4da82d9859876773436ad678f44b02e9b8aa1de34ed7f2de6b0a1ea4cb4b25738c627bc2d7bca1de27ced995e9b57e6f6af3e52badf499bdf5f4cd5
+  metadata.gz: 30328a895342c49a1e1e2ec59feb28e7a8ee1569f0210230cad181eaab9e8fee0c4a640fe4748447852d16dbd177cf77038a8cbc0f5b8f8be60ca3c497c5e71f
+  data.tar.gz: 8db7e5cd89c5ae2a9a826bbf142ef13b10fe181245964a049b395956c273537e296740935cdae8cfc0c324c5e0d056b2eb0f2576d13267b5ed36a013963d44d0

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": 85.99
+    "covered_percent": 83.74
   }
 }

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 }
-      current_section = membership.section.name
+      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)

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.5.0'
+  VERSION = '0.5.5'
 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 @@
-389
+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.5.0
+  version: 0.5.5
 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
@@ -235,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:
   - - ">="