quarantine 2.2.0 → 2.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c846ed3cf614eecefab5610bfb5d693444099fe90b6b429e4c697c16a014e21b
-  data.tar.gz: 5d17a24b4735f4c34af2d0902c0e3a7199750ce592694b5f9fe375928ada9d03
+  metadata.gz: 31ffed67bd1774918fd52c8bfe309398d43b8c218175f2a414725382df680ee7
+  data.tar.gz: aca3eab589422f555224ce2638e979aff7f6fc2f17d4b33ac91e2f0177d4fb9b
 SHA512:
-  metadata.gz: cb20ce19d6581b4836486176a080a6204a66f93d609e2b76a5d7bb91886b97e07466026fe45d0d7d4c16b8d2f072212a4ff982f42c40a55d68f42413276a92c4
-  data.tar.gz: bc093985a12b77f636d577b3fed3ab783f5fe206350a759a796097eee93976f42cb2fa199101a853f81d729aa109cce612f6deb4ba75748779281086e2591bca
+  metadata.gz: '0894a8c2561b741d3f9466d28e8424cde69ece0b7f228d700b485f748eb26531bd81ac549493a91641010ceb6b34eb1d7abbaf49475f4874d6f67db297d522ec'
+  data.tar.gz: 281ba613e8806b8d3f1e4318a65492435d971e0e89138cb87c44de2c7709a983672930e00cdfddb46bf245da4aa2e52804b2cf47d908fe9f0dd62eae3e7ac5fd

data/lib/quarantine/databases/google_sheets.rb

@@ -43,9 +43,8 @@ class Quarantine
         items.each do |item|
           cells = headers.map do |header|
             match = header.match(/^(extra_)?(.+)/)
-            extra, name = match[1..]
-            puts "header: #{header}, extra: #{extra}, name: #{name}"
-            value = extra ? item['extra_attributes'][name.to_sym] : item[name]
+            extra, name = match[1..2]
+            value = extra ? item['extra_attributes'][name] : item[name]
             value.to_s
           end
           row_idx = indexes[item['id']]
@@ -109,12 +108,12 @@ class Quarantine
         rows.map do |row|
           hash_row = Hash[headers.zip(row)]
           # TODO: use Google Sheets developer metadata to store type information
-          unless hash_row['id'].empty?
-            extra_values, base_values = hash_row.partition{|k, v| k.start_with?('extra_')}
-            base_hash = Hash[base_values]
-            base_hash['extra_attributes'] = Hash[extra_values]
-            base_hash
-          end
+          next nil if hash_row['id'].empty?
+
+          extra_values, base_values = hash_row.partition { |k, _v| k.start_with?('extra_') }
+          base_hash = Hash[base_values]
+          base_hash['extra_attributes'] = Hash[extra_values]
+          base_hash
         end.compact
       end
     end

data/lib/quarantine/rspec_adapter.rb

@@ -88,15 +88,15 @@ class Quarantine
           end
         end
         # The example failed, but it's not the final retry yet, so return nil.
-        return nil
+        return nil # rubocop:disable Style/RedundantReturn
       elsif metadata[:retry_attempts] > 0
         # The example passed this time, but failed one or more times before - the definition of a flaky test.
-        return [:quarantined, false]
+        return [:quarantined, false] # rubocop:disable Style/RedundantReturn
       elsif previously_quarantined
         # The example passed the first time, but it's already marked quarantined, so keep it that way.
-        return [:quarantined, true]
+        return [:quarantined, true] # rubocop:disable Style/RedundantReturn
       else
-        return [:passing, true]
+        return [:passing, true] # rubocop:disable Style/RedundantReturn
       end
     end
 

data/lib/quarantine/version.rb

@@ -1,3 +1,3 @@
 class Quarantine
-  VERSION = '2.2.0'.freeze
+  VERSION = '2.2.1'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: quarantine
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.2.1
 platform: ruby
 authors:
 - Flexport Engineering, Eric Zhu
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-04-17 00:00:00.000000000 Z
+date: 2021-04-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec