checkoff 0.2.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 117a1e63ffe4109064607e746bc4c48b42d923d6
-  data.tar.gz: 4d428edd63053d7b021ad32d8c479b7f5fb54931
+SHA256:
+  metadata.gz: a75e010929b7dcb752bf9cffd0f424039cb4c54da94e44af30a8b8627f13eefb
+  data.tar.gz: f2410a5287cf99a90f23ed7052a18f09553a652b87b575a2b665e7717c0193fa
 SHA512:
-  metadata.gz: 96086eddeb94cfa9f01c37d9df640a71698f591ef1281f9a7ba1fcc40ab5e6a31f06bf40f7c987b741b64dbffa67f6dc63c3962f4434c3eea40e679fab9755a5
-  data.tar.gz: 320d92a0cc339558fabaf5372476764f721c73cbf5ace1a3366c3fb2c09ec52e38e2a7aef7c6d3e31957bdf21a594006cd579bbf6843764226b54dcf0dbd49a8
+  metadata.gz: b7f458362faa147dc0c6d6097b4c3e912ce73c9a5de729fdfedd61bb691b7f0825b35dbce99ad5e06e7b82a1945a576fde09455fb94655a4748995d251da62e2
+  data.tar.gz: 3e47840ed4da82d9859876773436ad678f44b02e9b8aa1de34ed7f2de6b0a1ea4cb4b25738c627bc2d7bca1de27ced995e9b57e6f6af3e52badf499bdf5f4cd5

data/.circleci/config.yml

@@ -0,0 +1,17 @@
+version: 2
+jobs:
+  build:
+    working_directory: ~/checkoff
+    docker:
+      - image: apiology/circleci-ruby:latest
+    steps:
+      - checkout
+      - type: cache-restore
+        key: checkoff-{{ checksum "checkoff.gemspec" }}-{{ checksum "Gemfile" }}
+      - run: gem install --no-document bundler
+      - run: bundle install --path vendor/bundle --jobs=4
+      - type: cache-save
+        key: checkoff-{{ checksum "checkoff.gemspec" }}-{{ checksum "Gemfile" }}
+        paths:
+          - "vendor/bundle"
+      - run: bundle exec rake --jobs=4 ci

data/.rubocop.yml

@@ -1,27 +1,27 @@
-# I like trailing commas in maps.  They let me insert new elements and
+# I like trailing commas in hashes.  They let me insert new elements and
 # see them as one line in a diff, not two.
-Style/TrailingCommaInLiteral:
+Style/TrailingCommaInHashLiteral:
+  EnforcedStyleForMultiline: comma
+
+Style/TrailingCommaInArrayLiteral:
   EnforcedStyleForMultiline: comma
 
 # I use keyword arguments for a poor man's dependency injection to cut
 # down on the magic in my tests.
-ParameterLists:
+Metrics/ParameterLists:
   CountKeywordArgs: false
 
-# If i'm using one function name and returning the contents of an
+# If I'm using one function name and returning the contents of an
 # attribute, that's OK.  The alternative would be this, which I find
 # confusing and often not really what I mean:
 #
 # attr_reader :something_else
 # alias_method :something, :something_else
-TrivialAccessors:
+Style/TrivialAccessors:
    ExactNameMatch: true
 
-Layout/MultilineMethodCallIndentation:
-  EnforcedStyle: indented
-
 #
-# Add 'XXX' to the standard list
+# Add 'XX X' to the standard list
 #
 Style/CommentAnnotation:
    Keywords:
@@ -37,3 +37,41 @@ Style/CommentAnnotation:
        W"
     - "XX\
        X"
+
+# https://stackoverflow.com/questions/40934345/rubocop-25-line-block-size-and-rspec-tests
+Metrics/BlockLength:
+  # Exclude DSLs
+  Exclude:
+    - 'Rakefile'
+    - '*.gemspec'
+    - '**/*.rake'
+    - 'spec/**/*.rb'
+    - 'feature/**/*.rb'
+
+# http://www.betterspecs.org/#single
+#
+# > in tests that are not isolated (e.g. ones that integrate with a
+# > DB, an external webservice, or end-to-end-tests), you take a
+# > massive performance hit to do the same setup over and over again,
+# > just to set a different expectation in each test. In these sorts
+# > of slower tests, I think it's fine to specify more than one
+# > isolated behavior.
+RSpec/MultipleExpectations:
+  Exclude:
+    - 'feature/**/*.rb'
+
+Style/StringLiterals:
+    EnforcedStyle: single_quotes
+    SupportedStyles:
+      - single_quotes
+      - double_quotes
+    ConsistentQuotesInMultiline: true
+
+AllCops:
+  TargetRubyVersion: 2.4
+
+
+require:
+  - rubocop-rspec
+  - rubocop-minitest
+  - rubocop-rake

data/.ruby-version

@@ -1 +1 @@
-ruby-2.3.6
+2.6.6

data/.travis.yml

@@ -1,11 +1,17 @@
+language: ruby
+cache: bundler
 before_install:
+  # https://bundler.io/blog/2019/05/14/
+  #   solutions-for-cant-find-gem-bundler-with-executable-bundle.html
+  - gem update --system
   - gem install bundler
-language: ruby
 rvm:
-  - ruby-2.5.0
-  - ruby-2.4.3
-  - ruby-2.3.6
-  - ruby-2.2.9
-#   - jruby-head # see https://github.com/apiology/quality/issues/59
-#   - jruby-9.0 # see https://github.com/apiology/quality/issues/59
-#   - ruby-head # Upstream bug - see https://travis-ci.org/apiology/checkoff/jobs/138626934
+  # JRuby error with native code build - possibly can be overcome?
+  # checking for main() in -lstdc++... RuntimeError: The compiler failed to generate
+  # https://travis-ci.com/apiology/pronto-punchlist/jobs/264907931
+  # - jruby-head
+  - ruby-head
+  - ruby-2.7
+  - ruby-2.6
+  - ruby-2.5
+  - ruby-2.4

data/Makefile

@@ -0,0 +1,27 @@
+.PHONY: spec feature
+
+all: localtest
+
+localtest:
+	@bundle exec rake localtest
+
+feature:
+	@bundle exec rake feature
+
+spec:
+	@bundle exec rake spec
+
+rubocop:
+	@bundle exec rake rubocop
+
+punchlist:
+	@bundle exec rake punchlist
+
+pronto:
+	@bundle exec rake pronto
+
+bigfiles:
+	@bundle exec rake bigfiles
+
+quality:
+	@bundle exec rake quality

data/README.md

@@ -75,7 +75,7 @@ And even gather counts for project metrics:
 ```bash
 $ checkoff view 'Personal Projects' 'Create demo' | jq 'to_entries | map(.value) | flatten | map(.name) | length'
 4
-$ 
+$
 ```
 
 ## Caching

data/checkoff.gemspec

@@ -1,7 +1,7 @@
-# frozen_string_literal: true
 # coding: ascii
+# frozen_string_literal: true
 
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'checkoff/version'
 
@@ -27,11 +27,11 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'cache_method'
   spec.add_runtime_dependency 'dalli'
 
-  spec.add_development_dependency 'bundler', '~> 1.12'
+  spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'minitest-profile'
   spec.add_development_dependency 'mocha'
-  spec.add_development_dependency 'quality'
-  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'quality', '~> 37'
+  spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rspec', '>=3.4'
   spec.add_development_dependency 'simplecov'
 end

data/coverage/.last_run.json

@@ -1,5 +1,5 @@
 {
   "result": {
-    "covered_percent": 98.65
+    "covered_percent": 85.99
   }
 }

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

@@ -63,7 +63,7 @@ module Checkoff
     def quickadd(workspace_name, task_name)
       workspace = @workspaces.workspace_by_name(workspace_name)
       @tasks.add_task(task_name,
-                      workspace_id: workspace.id)
+                      workspace_gid: workspace.gid)
     end
 
     def validate_args!(args)
@@ -103,7 +103,7 @@ module Checkoff
       stderr.puts "  #{$PROGRAM_NAME} quickadd workspace task_name"
       stderr.puts
       stderr.puts "'project' can be set to a project name, or :my_tasks, " \
-                  ':my_tasks_upcoming, :my_tasks_new, or :my_tasks_today'
+                  ":my_tasks_upcoming, :my_tasks_new, or :my_tasks_today"
     end
 
     def view(workspace_name, project_name, section_name)

data/lib/checkoff/config_loader.rb

@@ -10,7 +10,7 @@ module Checkoff
     def self.load(sym)
       file = "#{sym}.yml"
       YAML.load_file(File.expand_path("~/.#{file}"))
-        .with_indifferent_access
+          .with_indifferent_access
     end
   end
 end

data/lib/checkoff/projects.rb

@@ -43,16 +43,18 @@ module Checkoff
     def projects_by_workspace_name(workspace_name)
       workspace = @workspaces.workspace_by_name(workspace_name)
       raise "Could not find workspace named #{workspace_name}" unless workspace
-      projects.find_by_workspace(workspace: workspace.id)
+
+      projects.find_by_workspace(workspace: workspace.gid)
     end
 
     def my_tasks(workspace_name)
       my_tasks = @config[:my_tasks]
-      id = @config[:my_tasks][workspace_name] unless my_tasks.nil?
-      if my_tasks.nil? || id.nil?
+      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
-      projects.find_by_id(id)
+
+      projects.find_by_id(gid)
     end
 
     def project(workspace_name, project_name)
@@ -75,7 +77,8 @@ module Checkoff
       {
         per_page: 100,
         options: {
-          fields: %w[name completed_at due_at due_on assignee_status tags],
+          fields: %w[name completed_at due_at due_on assignee_status tags
+                     memberships.project.gid memberships.section.name],
         },
       }
     end
@@ -83,10 +86,10 @@ module Checkoff
     def tasks_from_project(project, only_uncompleted: true, extra_fields: [])
       options = task_options
       options[:completed_since] = '9999-12-01' if only_uncompleted
-      options[:project] = project.id
+      options[:project] = project.gid
       options[:options][:fields] += extra_fields
-      client.tasks.find_all(options).to_a
+      client.tasks.find_all(**options).to_a
     end
-    cache_method :tasks_from_project, LONG_CACHE_TIME
+    cache_method :tasks_from_project, SHORT_CACHE_TIME
   end
 end

data/lib/checkoff/sections.rb

@@ -21,23 +21,17 @@ module Checkoff
 
     def_delegators :@projects, :client
 
-    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]
+    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
+      by_section[current_section] ||= []
+      by_section[current_section] << task
     end
 
-    def by_section(tasks)
-      current_section = nil
+    def by_section(tasks, project_gid)
       by_section = {}
       tasks.each do |task|
-        current_section, by_section = file_task_by_section(current_section,
-                                                           by_section, task)
+        file_task_by_section(by_section, task, project_gid)
       end
       by_section
     end
@@ -46,7 +40,7 @@ module Checkoff
     def tasks_by_section_for_project(project)
       raw_tasks = projects.tasks_from_project(project)
       active_tasks = projects.active_tasks(raw_tasks)
-      by_section(active_tasks)
+      by_section(active_tasks, project.gid)
     end
 
     def tasks_by_section_for_project_and_assignee_status(project,
@@ -54,9 +48,9 @@ module Checkoff
       raw_tasks = projects.tasks_from_project(project)
       by_assignee_status =
         projects.active_tasks(raw_tasks)
-          .group_by(&:assignee_status)
+                .group_by(&:assignee_status)
       active_tasks = by_assignee_status[assignee_status]
-      by_section(active_tasks)
+      by_section(active_tasks, project.gid)
     end
 
     def project_or_raise(workspace_name, project_name)
@@ -100,12 +94,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

@@ -42,15 +42,15 @@ module Checkoff
     end
 
     def add_task(name,
-                 workspace_id: default_workspace_id,
-                 assignee_id: default_assignee_id)
+                 workspace_gid: default_workspace_gid,
+                 assignee_gid: default_assignee_gid)
       @asana_task.create(client,
-                         assignee: assignee_id,
-                         workspace: workspace_id, name: name)
+                         assignee: assignee_gid,
+                         workspace: workspace_gid, name: name)
     end
 
-    def default_assignee_id
-      @config[:default_assignee_id]
+    def default_assignee_gid
+      @config[:default_assignee_gid]
     end
   end
 end

data/lib/checkoff/version.rb

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

data/lib/checkoff/workspaces.rb

@@ -12,11 +12,12 @@ module Checkoff
     def client
       @client ||= @asana_client.new do |c|
         c.authentication :access_token, @config[:personal_access_token]
+        c.default_headers 'asana-enable' => 'string_ids,new_sections'
       end
     end
 
-    def default_workspace_id
-      @config[:default_workspace_id]
+    def default_workspace_gid
+      @config[:default_workspace_gid]
     end
 
     def workspace_by_name(workspace_name)

data/lib/tasks/ci.rake

@@ -1,3 +1,4 @@
 # frozen_string_literal: true
 
+desc 'Run tasks to be done during a continuous integration (CI) build'
 task ci: :localtest

data/lib/tasks/clear_metrics.rake

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+desc 'Rebaseline quality thresholds to last commit'
 task :clear_metrics do |_t|
   ret =
     system('git checkout coverage/.last_run.json metrics/*_high_water_mark')

data/lib/tasks/localtest.rake

@@ -1,3 +1,4 @@
 # frozen_string_literal: true
 
+desc 'Standard build when running on a workstation'
 task localtest: %i[clear_metrics test quality]

data/metrics/bigfiles_high_water_mark

@@ -1 +1 @@
-459
+389

data/metrics/mdl_high_water_mark

@@ -1 +1 @@
-8
+7

data/metrics/punchlist_high_water_mark

@@ -1,2 +1 @@
-11
-
+3

data/metrics/pycodestyle_high_water_mark

@@ -0,0 +1 @@
+0

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: checkoff
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Vince Broz
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2018-04-26 00:00:00.000000000 Z
+date: 2020-10-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -70,16 +70,16 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.12'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.12'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: minitest-profile
   requirement: !ruby/object:Gem::Requirement
@@ -112,30 +112,30 @@ dependencies:
   name: quality
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '37'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '37'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -164,7 +164,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
+description:
 email:
 - vince@broz.cc
 executables:
@@ -172,6 +172,7 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".circleci/config.yml"
 - ".gitignore"
 - ".rubocop.yml"
 - ".ruby-version"
@@ -179,12 +180,12 @@ files:
 - CODE_OF_CONDUCT.md
 - Gemfile
 - LICENSE.txt
+- Makefile
 - README.md
 - Rakefile
 - bin/console
 - bin/setup
 - checkoff.gemspec
-- circle.yml
 - coverage/.last_run.json
 - docs/example_project.png
 - exe/checkoff
@@ -193,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
@@ -216,6 +218,7 @@ files:
 - metrics/mdl_high_water_mark
 - metrics/pep8_high_water_mark
 - metrics/punchlist_high_water_mark
+- metrics/pycodestyle_high_water_mark
 - metrics/rails_best_practices_high_water_mark
 - metrics/rubocop_high_water_mark
 - metrics/scalastyle_high_water_mark
@@ -224,7 +227,7 @@ homepage: http://github.com/apiology/checkoff
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -239,9 +242,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.14
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: Command-line and gem client for Asana (unofficial)
 test_files: []

data/circle.yml

@@ -1,6 +0,0 @@
-test:
-  override:
-    - bundle exec rake localtest
-general:
-  artifacts:
-    - "coverage"