dri 0.1.1 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e352230c90233014a517cb67addf17a051a2b2eb05c93f0e88bcaab45c40e723
-  data.tar.gz: 51bbd9e7f07e20668fa8b6161e41b3995ae47b8d1866be6ad81fd07fc0c37bf1
+  metadata.gz: 3ae68ce5f20bab397121604708b813e8f488316559f82392a389f1750e57c89c
+  data.tar.gz: 30b16b0291beb7e18cc74acb9c54100c869544bba883c79729f577c8bda5dd7f
 SHA512:
-  metadata.gz: 00b8bcfe9d1d884523f2f5d3963cfa8ab3b7f33f04f819fd17b253838bf899d67394de91524d3dfa40ff692b48c5768f039dc8b6ed44abbcd5faa14b1606eb60
-  data.tar.gz: 58a9c606cd89a5282f25b9fe89fcf16bcf9c70d53df868876884d10149e34786de5c3341d216dd227bbb3d44e16649e26032117c1f12e33e61de52168fd54ebe
+  metadata.gz: ad35cf45172de050ecd86c376527abc98ba535f9ad97f54f84f7be3875882c46ae897039d26a4c9f4db3c7df598608c81e8613518b7c7eeb44b7903610899406
+  data.tar.gz: a790667c5528fc3ad94f610b2add55b36a4fb5efd9bad3e8e10a78d44e58a8140721d4717a75def60ca829715389bebf001704a38a1a8127e1e6872228dd0fe6

data/.gitignore

@@ -6,9 +6,11 @@
 /pkg/
 /spec/reports/
 /tmp/
+/vendor/bundle
 
 # rspec failure tracking
 .rspec_status
 
 .dri_profile.yml
-handover_reports/*
+handover_reports/*
+.idea/

data/.gitlab-ci.yml

@@ -1,28 +1,47 @@
-image: ruby:3.0.2
+.job_base:
+  image: ruby:2.7
+  variables:
+    BUNDLE_PATH: vendor/bundle
+    BUNDLE_SUPPRESS_INSTALL_USING_MESSAGES: "true"
+  before_script:
+    - gem install bundler -v 2.3.9 --no-document
+    - bundle install
+  cache:
+    key:
+      files:
+        - dri.gemspec
+        - Gemfile.lock
+    paths:
+      - vendor/bundle
+  rules:
+    - if: '$CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH'
+    - if: '$CI_PIPELINE_SOURCE == "merge_request_event"'
+
+include:
+  - project: gitlab-org/quality/pipeline-common
+    ref: 0.3.4
+    file:
+      - /ci/gem-release.yml
 
 stages:
   - build
   - test
   - deploy
 
-before_script:
-  - gem install bundler:2.2.22
-  - bundle install
-
 build_gem:
   stage: build
+  extends: .job_base
   script:
     - gem build
 
-rspec:
+rubocop:
   stage: test
+  extends: .job_base
   script:
-    - bundle exec rspec
+    - bundle exec rubocop --color
 
-deploy:
-  stage: deploy
+rspec:
+  stage: test
+  extends: .job_base
   script:
-    - gem push dri*.gem
-  rules:
-    - if: '$CI_COMMIT_TAG'
-      when: always
+    - bundle exec rspec --color

data/.rubocop.yml

@@ -0,0 +1,53 @@
+inherit_gem:
+  gitlab-styles:
+    - rubocop-all.yml
+    - rubocop-gemspec.yml
+    - rubocop-lint.yml
+    - rubocop-naming.yml
+    - rubocop-performance.yml
+    - rubocop-rspec.yml
+    - rubocop-security.yml
+    - rubocop-style.yml
+
+AllCops:
+  TargetRubyVersion: 2.7
+
+RSpec/MultipleMemoizedHelpers:
+  Max: 25
+  AllowSubject: true
+
+CodeReuse/ActiveRecord:
+  Exclude:
+    - spec/**/*.rb
+
+Metrics/AbcSize:
+  Enabled: true
+  Max: 30
+
+Metrics/BlockLength:
+  Enabled: true
+  Exclude:
+    - spec/**/*.rb
+    - dri.gemspec
+
+Metrics/MethodLength:
+  Enabled: true
+  Max: 30
+
+Layout/BeginEndAlignment:
+  Enabled: true
+
+Layout/EmptyLinesAroundAttributeAccessor:
+  Enabled: true
+
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+
+Lint/HashCompareByIdentity:
+  Enabled: true
+
+Lint/RedundantSafeNavigation:
+  Enabled: true
+
+Style/ClassEqualityComparison:
+  Enabled: true

data/.ruby-version

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

data/.tool-versions

@@ -0,0 +1 @@
+ruby 3.0.2

data/Gemfile

@@ -1,7 +1,6 @@
+# frozen_string_literal: true
+
 source "https://rubygems.org"
 
 # Specify your gem's dependencies in dri.gemspec
 gemspec
-
-gem "rake", "~> 12.0"
-gem "rspec", "~> 3.0"

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    dri (0.1.0)
+    dri (0.1.3)
       httparty (~> 0.20.0)
       json (~> 2.6.1)
       markdown-tables (~> 1.1.1)
@@ -19,39 +19,88 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
+    activesupport (7.0.2.3)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
     addressable (2.8.0)
       public_suffix (>= 2.0.2, < 5.0)
+    ast (2.4.2)
+    concurrent-ruby (1.1.10)
     crack (0.4.5)
       rexml
     diff-lcs (1.5.0)
+    gitlab-styles (7.0.0)
+      rubocop (~> 0.91, >= 0.91.1)
+      rubocop-gitlab-security (~> 0.1.1)
+      rubocop-graphql (~> 0.10)
+      rubocop-performance (~> 1.9.2)
+      rubocop-rails (~> 2.9)
+      rubocop-rspec (~> 1.44)
     hashdiff (1.0.1)
     httparty (0.20.0)
       mime-types (~> 3.0)
       multi_xml (>= 0.5.2)
+    i18n (1.10.0)
+      concurrent-ruby (~> 1.0)
     json (2.6.1)
     markdown-tables (1.1.1)
     mime-types (3.4.1)
       mime-types-data (~> 3.2015)
     mime-types-data (3.2022.0105)
+    minitest (5.15.0)
     multi_xml (0.6.0)
+    parallel (1.22.1)
+    parser (3.1.1.0)
+      ast (~> 2.4.1)
     pastel (0.8.0)
       tty-color (~> 0.5)
     public_suffix (4.0.6)
-    rake (12.3.3)
+    rack (2.2.3)
+    rainbow (3.1.1)
+    rake (13.0.6)
+    regexp_parser (2.2.1)
     rexml (3.2.5)
-    rspec (3.11.0)
-      rspec-core (~> 3.11.0)
-      rspec-expectations (~> 3.11.0)
-      rspec-mocks (~> 3.11.0)
-    rspec-core (3.11.0)
-      rspec-support (~> 3.11.0)
-    rspec-expectations (3.11.0)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.2)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.2)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.11.0)
-    rspec-mocks (3.11.0)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.3)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.11.0)
-    rspec-support (3.11.0)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.3)
+    rubocop (0.93.1)
+      parallel (~> 1.10)
+      parser (>= 2.7.1.5)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8)
+      rexml
+      rubocop-ast (>= 0.6.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 2.0)
+    rubocop-ast (1.16.0)
+      parser (>= 3.1.1.0)
+    rubocop-gitlab-security (0.1.1)
+      rubocop (>= 0.51)
+    rubocop-graphql (0.14.2)
+      rubocop (>= 0.87, < 2)
+    rubocop-performance (1.9.2)
+      rubocop (>= 0.90.0, < 2.0)
+      rubocop-ast (>= 0.4.0)
+    rubocop-rails (2.9.1)
+      activesupport (>= 4.2.0)
+      rack (>= 1.1)
+      rubocop (>= 0.90.0, < 2.0)
+    rubocop-rspec (1.44.1)
+      rubocop (~> 0.87)
+      rubocop-ast (>= 0.7.1)
+    ruby-progressbar (1.11.0)
     strings (0.2.1)
       strings-ansi (~> 0.2)
       unicode-display_width (>= 1.5, < 3.0)
@@ -85,7 +134,9 @@ GEM
       pastel (~> 0.8)
       strings (~> 0.2.0)
       tty-screen (~> 0.8)
-    unicode-display_width (2.1.0)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (1.8.0)
     unicode_utils (1.4.0)
     webmock (3.14.0)
       addressable (>= 2.8.0)
@@ -98,10 +149,11 @@ PLATFORMS
 
 DEPENDENCIES
   dri!
-  rake (~> 12.0)
-  rspec (~> 3.0)
+  gitlab-styles (~> 7.0.0)
+  rake
+  rspec (~> 3.10.0)
   timecop (~> 0.9.1)
   webmock (~> 3.5)
 
 BUNDLED WITH
-   2.1.4
+   2.3.9

data/README.md

@@ -56,6 +56,9 @@ $ dri profile
   - failures
   - testcases
   - triaged
+  - quarantines
+  - dequarantines
+  - featureflags
 - [4. publish](#4-publish)
   - report
 - [5. rm](#5-rm)
@@ -107,6 +110,25 @@ $ dri fetch triaged
 
 Fetches triaged failures which use the triage emoji specified in `dri profile`.
 
+```shell
+$ dri fetch quarantines
+```
+
+Fetches open quarantine Merge Requests to be reviewed
+
+```shell
+$ dri fetch dequarantines
+```
+
+Fetches open dequarantine Merge Requests to be reviewed
+
+```shell
+$ dri fetch featureflags
+```
+
+Fetches a list of today's feature flag changes, including the date and time in UTC of when the change occurred as well as a link to the corresponding issue from the feature-flag-log project.
+Results are organized by environment (production, staging, staging ref and preprod).
+
 #### 4. publish
 
 ```shell

data/Rakefile

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"
 
 RSpec::Core::RakeTask.new(:spec)
 
-task :default => :spec
+task default: :spec

data/bin/console

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require "bundler/setup"
 require "dri"

data/dri.gemspec

@@ -1,41 +1,44 @@
+# frozen_string_literal: true
+
 require_relative 'lib/dri/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "dri"
-  spec.license       = "MIT"
+  spec.name          = 'dri'
+  spec.license       = 'MIT'
   spec.version       = Dri::VERSION
-  spec.authors       = ["GitLab Quality"]
-  spec.email         = ["quality+dri@gitlab.com"]
+  spec.authors       = ['GitLab Quality']
+  spec.email         = ['quality+dri@gitlab.com']
 
-  spec.summary       = %q{CLI app to help triage GitLab QA pipelines}
+  spec.summary       = 'CLI app to help triage GitLab QA pipelines'
   spec.homepage      = 'https://gitlab.com/gitlab-org/quality/dri'
-  spec.required_ruby_version = Gem::Requirement.new(">= 2.5.0")
+  spec.required_ruby_version = Gem::Requirement.new('>= 2.7.0')
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
-  spec.files         = Dir.chdir(File.expand_path('..', __FILE__)) do
+  spec.files = Dir.chdir(File.expand_path(__dir__)) do
     `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   end
-  spec.bindir        = "exe"
+  spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
-  spec.add_dependency "tty-config", "~> 0.4.0"
-  spec.add_dependency "tty-font", "~> 0.5"
-  spec.add_dependency "tty-prompt", "~> 0.23.1"
-  spec.add_dependency "tty-spinner", "~> 0.9"
-  spec.add_dependency "tty-table", "~> 0.12.0"
+  spec.add_dependency 'httparty', '~> 0.20.0'
+  spec.add_dependency 'json', '~> 2.6.1'
+  spec.add_dependency 'markdown-tables', '~> 1.1.1'
+  spec.add_dependency 'pastel', '~> 0.8.0'
+  spec.add_dependency 'thor', '~> 1.0.1'
   spec.add_dependency 'tty-box', '~> 0.7.0'
+  spec.add_dependency 'tty-config', '~> 0.4.0'
+  spec.add_dependency 'tty-editor', '~> 0.6'
+  spec.add_dependency 'tty-font', '~> 0.5'
   spec.add_dependency 'tty-logger', '~> 0.6.0'
-  spec.add_dependency "tty-editor", "~> 0.6"
-  spec.add_dependency "pastel", "~> 0.8.0"
-  spec.add_dependency "thor", "~> 1.0.1"
-  spec.add_dependency "markdown-tables", "~> 1.1.1"  
-  spec.add_dependency 'json', '~> 2.6.1'
-  spec.add_dependency 'httparty', '~> 0.20.0'
+  spec.add_dependency 'tty-prompt', '~> 0.23.1'
+  spec.add_dependency 'tty-spinner', '~> 0.9'
+  spec.add_dependency 'tty-table', '~> 0.12.0'
 
-  spec.add_development_dependency "rake"
+  spec.add_development_dependency 'gitlab-styles', '~> 7.0.0'
+  spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec', '~> 3.10.0'
-  spec.add_development_dependency 'webmock', '~> 3.5'
   spec.add_development_dependency "timecop", "~> 0.9.1"
+  spec.add_development_dependency 'webmock', '~> 3.5'
 end

data/exe/dri

@@ -2,7 +2,7 @@
 # frozen_string_literal: true
 
 lib_path = File.expand_path('../lib', __dir__)
-$:.unshift(lib_path) if !$:.include?(lib_path)
+$LOAD_PATH.unshift(lib_path) unless $LOAD_PATH.include?(lib_path)
 require 'dri/cli'
 
 Signal.trap('INT') do
@@ -12,7 +12,7 @@ end
 
 begin
   Dri::CLI.start
-rescue Dri::CLI::Error => err
-  puts "ERROR: #{err.message}"
+rescue Dri::CLI::Error => e
+  puts "ERROR: #{e.message}"
   exit 1
 end

data/lib/dri/api_client.rb

@@ -3,14 +3,15 @@
 require "httparty"
 require "json"
 require "tty-config"
+require 'cgi'
 
 module Dri
-  class ApiClient
-    
-    API_URL              = 'https://gitlab.com/api/v4'
-    TESTCASES_PROJECT_ID = '11229385'
-    TRIAGE_PROJECT_ID    = '15291320'
-    GITLAB_PROJECT_ID    = '278964'
+  class ApiClient # rubocop:disable Metrics/ClassLength
+    API_URL                     = 'https://gitlab.com/api/v4'
+    TESTCASES_PROJECT_ID        = '11229385'
+    TRIAGE_PROJECT_ID           = '15291320'
+    GITLAB_PROJECT_ID           = '278964'
+    FEATURE_FLAG_LOG_PROJECT_ID = '15208716'
 
     def initialize(config)
       profile = config.read
@@ -50,6 +51,26 @@ module Dri
       fetch_json(url.join)
     end
 
+    # Fetch MRs
+    #
+    # @see https://docs.gitlab.com/ee/api/merge_requests.html for all passable options
+    #
+    # @param [Hash<String>] options
+    # @option options [String] state
+    # @option options [String] order_by
+    # @option options [String] sort
+    # @option options [String] milestone
+    # @option options [String] labels
+    def fetch_mrs(project_id:, **options)
+      uri = URI(API_URL)
+      uri.query = HTTParty::HashConversions.to_params(options)
+
+      # CGI.escape('gitlab-org/gitlab') => 'gitlab-org%2Fgitlab'
+      uri.path = File.join(uri.path, 'projects', CGI.escape(project_id), 'merge_requests')
+
+      fetch_json(uri.to_s)
+    end
+
     def fetch_current_triage_issue
       url = ["#{API_URL}/projects/"]
       url << "#{TRIAGE_PROJECT_ID}/issues?state=opened"
@@ -87,6 +108,16 @@ module Dri
       delete_json(url)
     end
 
+    def fetch_feature_flag_logs(date:, page:)
+      url = ["#{API_URL}/projects/"]
+      url << "#{FEATURE_FLAG_LOG_PROJECT_ID}/issues"
+      url << "?created_after=#{date}"
+      url << "&per_page=100"
+      url << "&page=#{page}"
+
+      fetch_json(url.join)
+    end
+
     private
 
     def post_json(url, body)
@@ -97,7 +128,7 @@ module Dri
         }
       }
 
-      response =  HTTParty.post(url, options)      
+      response = HTTParty.post(url, options)
       handle_response(response)
     end
 
@@ -107,12 +138,12 @@ module Dri
     end
 
     def handle_response(response)
-      #puts response.to_json
+      # puts response.to_json
       response
     end
 
     def fetch_json(url)
-      response =  HTTParty.get(url, headers: header)
+      response = HTTParty.get(url, headers: header)
 
       if response.code != 200
         puts "Response error code \"#{response.code}\" - Unable to sync with GitLab"
@@ -122,4 +153,4 @@ module Dri
       handle_response(JSON.parse(response.body))
     end
   end
-end
+end

data/lib/dri/cli.rb

@@ -25,14 +25,14 @@ module Dri
     end
 
     class_option :no_color, type: :boolean, default: false,
-                              desc: "Disable colorization in output"
+                            desc: "Disable colorization in output"
 
     desc 'version', 'dri version'
     def version
       require_relative 'version'
       puts "v#{Dri::VERSION}"
     end
-    map %w(--version -v) => :version
+    map %w[--version -v] => :version
 
     require_relative 'commands/rm'
     register Dri::Commands::Rm, 'rm', 'rm [SUBCOMMAND]', 'Remove triage-related items'

data/lib/dri/command.rb

@@ -1,7 +1,10 @@
 # frozen_string_literal: true
+
 require_relative 'command'
 require_relative 'api_client'
 
+require 'dri/refinements/truncate'
+
 require "tty-config"
 require "pastel"
 require 'forwardable'
@@ -11,7 +14,6 @@ module Dri
     extend Forwardable
 
     def_delegators :command, :run
-    attr_reader :config, :emoji, :token, :username, :timezone, :profile
 
     def pastel(**options)
       Pastel.new(**options)
@@ -54,10 +56,10 @@ module Dri
     end
 
     def verify_config_exists
-      if !config.exist?
-        logger.error "Oops, could not find a configuration. Try using #{add_color('dri init', :yellow)} first."
-        exit 1
-      end
+      return if config.exist?
+
+      logger.error "Oops, could not find a configuration. Try using #{add_color('dri init', :yellow)} first."
+      exit 1
     end
 
     def add_color(str, *color)