qa_server 7.5.0 → 7.8.0

data/lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/oclcfast_direct_validation.yml

@@ -26,12 +26,13 @@ search:
     replacements:
       maxRecords: '4'
 #  -
-#    query: War
+#    # This is an event at OCLCFAST web-search, but a meeting through OCLCFAST API.
+#    query: Bronze Night
 #    subauth: event_name
 #    replacements:
 #      maxRecords: '4'
   -
-    query: Festival
+    query: convention
     subauth: meeting
     replacements:
       maxRecords: '4'
@@ -71,7 +72,6 @@ search:
     replacements:
       maxRecords: '5'
   -
-    pending: true
     query: Sleep-overs
     subauth: concept
     subject_uri: "http://id.worldcat.org/fast/1120873"
@@ -79,6 +79,7 @@ search:
     replacements:
       maxRecords: '10'
   -
+    # This is a meeting in OCLCFAST_DIRECT, but event in OCLCFAST_LD4L_CACHE.
     pending: true
     query: People's International Tribunal on the Rights of Indigenous Hawaiians
     subauth: event
@@ -86,6 +87,13 @@ search:
     position: 3
     replacements:
       maxRecords: '8'
+  -
+    query: Science Fiction
+    subauth: meeting
+    subject_uri: "http://id.worldcat.org/fast/1406163"
+    position: 5
+    replacements:
+      maxRecords: '10'
   -
     pending: true
     query: University of Chicago Library
@@ -102,7 +110,6 @@ search:
     replacements:
       maxRecords: '20'
   -
-    pending: true
     query: Sjælland
     subauth: place
     result_size: 190
@@ -111,13 +118,12 @@ search:
     replacements:
       maxRecords: '10'
   -
-    pending: true
     query: Scream
     subauth: work
     subject_uri: "http://id.worldcat.org/fast/1358031"
-    position: 5
+    position: 10
     replacements:
-      maxRecords: '10'
+      maxRecords: '20'
 term:
   -
     identifier: '1914919'

data/lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/oclcfast_ld4l_cache_validation.yml

@@ -13,6 +13,9 @@ search:
   -
     query: tribunal
     subauth: event
+  -
+    query: convention
+    subauth: meeting
   -
     query: 'mark twain'
     subauth: person
@@ -32,7 +35,6 @@ search:
   #  Accuracy tests
   #------------------
   -
-    pending: true
     query: Tralee & Dingle Railway
     subject_uri: "http://id.worldcat.org/fast/1733583"
     position: 3
@@ -53,6 +55,14 @@ search:
     replacements:
       maxRecords: '8'
   -
+    query: Science Fiction
+    subauth: meeting
+    subject_uri: "http://id.worldcat.org/fast/1406163"
+    position: 5
+    replacements:
+      maxRecords: '10'
+  -
+    pending: true
     query: University of Chicago Library
     subauth: organization
     subject_uri: "http://id.worldcat.org/fast/539173"
@@ -78,10 +88,16 @@ search:
     query: Scream
     subauth: work
     subject_uri: "http://id.worldcat.org/fast/1358031"
-    position: 5
+    position: 10
+    replacements:
+      maxRecords: '20'
+  -
+    query: 'mark twain'
+    subauth: intangible
+    subject_uri: "http://id.worldcat.org/fast/1200561"
+    position: 3
     replacements:
       maxRecords: '10'
-
 term:
   -
     identifier: 'http://id.worldcat.org/fast/1914919'

data/lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/rda_registry_ld4l_cache_validation.yml

@@ -316,7 +316,6 @@ search:
     replacements:
       maxRecords: '20'
   -
-    pending: true
     query: polychrome
     subauth: colour_content
     position: 3
@@ -457,7 +456,6 @@ search:
     replacements:
       maxRecords: '20'
   -
-    pending: true
     query: unknown
     subauth: gender
     position: 3
@@ -556,7 +554,6 @@ search:
     replacements:
       maxRecords: '10'
   -
-    pending: true
     query: Hi 8
     subauth: video_format
     position: 3

data/lib/qa_server/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module QaServer
-  VERSION = '7.5.0'
+  VERSION = '7.8.0'
 end

data/qa_server.gemspec

@@ -22,11 +22,11 @@ Gem::Specification.new do |spec|
   # Note: rails does not follow sem-ver conventions, it's
   # minor version releases can include breaking changes; see
   # http://guides.rubyonrails.org/maintenance_policy.html
-  spec.add_dependency 'rails', '~> 5.0' # Keep in sync with version supported by Hyrax
+  spec.add_dependency 'rails', '~> 5.2', '>= 5.2.5' # v5.2.5 required to address mimemagic gem yank
   spec.add_dependency 'useragent'
 
   # Required gems for QA and linked data access
-  spec.add_development_dependency 'qa', '~> 5.3'
+  spec.add_development_dependency 'qa', '~> 5.5', '>= 5.5.1' # questioning_authority
   spec.add_development_dependency 'linkeddata'
 
   # Produces dashboard charts on monitor status page
@@ -34,13 +34,14 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency 'better_errors' # provide debugging command line in
   spec.add_development_dependency 'binding_of_caller' # provides deep stack info used by better_errors
-  spec.add_development_dependency 'bixby', '~> 3.0.0' # rubocop styleguide
+  spec.add_development_dependency 'bixby', '~> 3.0', ">= 3.0.2" # rubocop styleguide
   # spec.add_development_dependency "capybara", '~> 3.29'
   # spec.add_development_dependency 'capybara-maleficent', '~> 0.3.0'
   # spec.add_development_dependency 'chromedriver-helper', '~> 2.1'
   spec.add_development_dependency 'deprecation'
-  spec.add_development_dependency 'engine_cart', '~> 2.0'
+  spec.add_development_dependency 'engine_cart', '~> 2.2'
   spec.add_development_dependency "factory_bot", '~> 4.4'
+  spec.add_development_dependency 'i18n-tasks'
   spec.add_development_dependency 'simplecov'
   spec.add_development_dependency 'sqlite3'
   spec.add_development_dependency 'rails-controller-testing', '~> 1'
@@ -48,7 +49,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rspec-its', '~> 1.1'
   spec.add_development_dependency 'rspec-rails', '~> 3.1'
   spec.add_development_dependency 'selenium-webdriver'
-  spec.add_development_dependency 'webdrivers', '~> 3.0'
+  spec.add_development_dependency 'webdrivers', '~> 4.4'
   spec.add_development_dependency 'webmock'
 
   ########################################################

data/spec/feature/accuracy_spec.rb

@@ -21,7 +21,7 @@ RSpec.describe 'Accuracy test' do # rubocop:disable RSpec/DescribeClass
         it "finds actual <= expected" do
           pending 'test is known to fail' if test_result[:pending]
           expect(test_result[:err_message]).to be_empty
-          expect(test_result[:request_data]).not_to be_empty
+          expect(test_result[:request_data]).to be_present
           expect(test_result[:actual]).not_to be_nil
           expect(test_result[:actual]).to be <= test_result[:expected]
         end

data/spec/i18n_spec.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require 'i18n/tasks'
+
+RSpec.describe I18n do
+  let(:i18n) { I18n::Tasks::BaseTask.new }
+  let(:missing_keys) { i18n.missing_keys }
+  let(:unused_keys) { i18n.unused_keys }
+  let(:inconsistent_interpolations) { i18n.inconsistent_interpolations }
+
+  it 'does not have missing keys' do
+    expect(missing_keys).to be_empty,
+                            "Missing #{missing_keys.leaves.count} i18n keys, run `i18n-tasks missing' to show them"
+  end
+
+  it 'does not have unused keys' do
+    expect(unused_keys).to be_empty,
+                           "#{unused_keys.leaves.count} unused i18n keys, run `i18n-tasks unused' to show them"
+  end
+
+  it 'files are normalized' do # rubocop:disable RSpec/ExampleLength
+    pending 'need to explore further normalization process'
+    non_normalized = i18n.non_normalized_paths
+    error_message = "The following files need to be normalized:\n" \
+                    "#{non_normalized.map { |path| "  #{path}" }.join("\n")}\n" \
+                    "Please run `i18n-tasks normalize' to fix"
+    expect(non_normalized).to be_empty, error_message
+  end
+
+  it 'does not have inconsistent interpolations' do
+    error_message = "#{inconsistent_interpolations.leaves.count} i18n keys have inconsistent interpolations.\n" \
+                    "Run `i18n-tasks check-consistent-interpolations' to show them"
+    expect(inconsistent_interpolations).to be_empty, error_message
+  end
+end

data/spec/presenters/qa_server/monitor_status/history_presenter_spec.rb

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+require 'spec_helper'
+
+RSpec.describe QaServer::MonitorStatus::HistoryPresenter do
+  let(:presenter) { described_class.new(parent: nil, historical_summary_data: historical_summary_data) }
+  # rubocop:disable Layout/ExtraSpacing
+  let(:historical_summary_data) do
+    # { 'auth_name' => { good: count_of_passing_tests, bad: count_of_failing_tests } }
+    {
+      'GOOD_AUTH'   => { good:    0, bad: 1000 },
+      'BARELY_GOOD' => { good:   49, bad:  951 },
+      'OK_AUTH'     => { good:   50, bad:  950 },
+      'STILL_OK'    => { good:   51, bad:  949 },
+      'BARELY_OK'   => { good:   99, bad:  901 },
+      'BAD_AUTH'    => { good:  100, bad:  900 },
+      'STILL_BAD'   => { good:  101, bad:  899 },
+      'REALLY_BAD'  => { good:  500, bad:  500 },
+      'HORRIBLE'    => { good: 1000, bad:    0 }
+    }
+    # rubocop:enable Layout/ExtraSpacing
+  end
+
+  describe '.failure_style_class' do
+    context 'returns NEUTRAL style' do
+      let(:expected_css_style) { "status-neutral" }
+      let(:zero_failure_entry) { ['GOOD_AUTH',   { good: 1000, bad:  0 }] } # rubocop:disable Layout/ExtraSpacing
+      let(:just_below_caution) { ['BARELY_GOOD', { good:  951, bad: 49 }] } # rubocop:disable Layout/ExtraSpacing
+
+      it 'when no failures' do
+        expect(presenter.failure_style_class(zero_failure_entry)).to eq expected_css_style
+      end
+
+      it 'when percent of failures is just below the CAUTION_THRESHOLD' do
+        expect(presenter.failure_style_class(just_below_caution)).to eq expected_css_style
+      end
+    end
+
+    context 'returns CAUTION style' do
+      let(:expected_css_style) { "status-unknown" }
+      let(:equal_caution)      { ['OK_AUTH',   { good: 950, bad: 50 }] }
+      let(:just_above_caution) { ['STILL_OK',  { good: 949, bad: 51 }] }
+      let(:just_below_warning) { ['BARELY_OK', { good: 901, bad: 99 }] }
+
+      it 'when percent of failures is equal to CAUTION_THRESHOLD' do
+        expect(presenter.failure_style_class(equal_caution)).to eq expected_css_style
+      end
+
+      it 'when percent of failures is just above CAUTION_THRESHOLD' do
+        expect(presenter.failure_style_class(just_above_caution)).to eq expected_css_style
+      end
+
+      it 'when percent of failures is just below WARNING_THRESHOLD' do
+        expect(presenter.failure_style_class(just_below_warning)).to eq expected_css_style
+      end
+    end
+
+    context 'returns WARNING style' do
+      let(:expected_css_style) { "status-bad" }
+      let(:equal_warning)      { ['BAD_AUTH',   { good: 900, bad:  100 }] }
+      let(:just_above_warning) { ['STILL_BAD',  { good: 899, bad:  101 }] }
+      let(:well_above_warning) { ['REALLY_BAD', { good: 500, bad:  500 }] }
+      let(:all_failures)       { ['HORRIBLE',   { good:   0, bad: 1000 }] }
+
+      it 'when percent of failures is equal to WARNING_THRESHOLD' do
+        expect(presenter.failure_style_class(equal_warning)).to eq expected_css_style
+      end
+
+      it 'when percent of failures is just above WARNING_THRESHOLD' do
+        expect(presenter.failure_style_class(just_above_warning)).to eq expected_css_style
+      end
+
+      it 'when percent of failures is well above WARNING_THRESHOLD' do
+        expect(presenter.failure_style_class(well_above_warning)).to eq expected_css_style
+      end
+
+      it 'when percent of failures is 100%' do
+        expect(presenter.failure_style_class(all_failures)).to eq expected_css_style
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: qa_server
 version: !ruby/object:Gem::Version
-  version: 7.5.0
+  version: 7.8.0
 platform: ruby
 authors:
 - E. Lynette Rayle
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-12-08 00:00:00.000000000 Z
+date: 2021-04-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -16,14 +16,20 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '5.2'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.2.5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '5.2'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.2.5
 - !ruby/object:Gem::Dependency
   name: useragent
   requirement: !ruby/object:Gem::Requirement
@@ -44,14 +50,20 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.3'
+        version: '5.5'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.5.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.3'
+        version: '5.5'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.5.1
 - !ruby/object:Gem::Dependency
   name: linkeddata
   requirement: !ruby/object:Gem::Requirement
@@ -114,14 +126,20 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '3.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.0.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '3.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.0.2
 - !ruby/object:Gem::Dependency
   name: deprecation
   requirement: !ruby/object:Gem::Requirement
@@ -142,14 +160,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '2.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '2.2'
 - !ruby/object:Gem::Dependency
   name: factory_bot
   requirement: !ruby/object:Gem::Requirement
@@ -164,6 +182,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '4.4'
+- !ruby/object:Gem::Dependency
+  name: i18n-tasks
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -268,14 +300,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '4.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '4.4'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -418,6 +450,7 @@ files:
 - app/views/qa_server/monitor_status/index.html.erb
 - app/views/qa_server/usage/index.html.erb
 - app/views/shared/_footer.html.erb
+- config/i18n-tasks.yml
 - config/locales/qa_server.en.yml
 - config/routes.rb
 - lib/generators/qa_server/assets_generator.rb
@@ -472,6 +505,8 @@ files:
 - lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/locdemographics_ld4l_cache_validation.yml
 - lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/locgenres_ld4l_cache_validation.yml
 - lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/locnames_ld4l_cache_validation.yml
+- lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/locnames_rwo2_ld4l_cache_validation.yml
+- lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/locnames_rwo3_ld4l_cache_validation.yml
 - lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/locnames_rwo_ld4l_cache_validation.yml
 - lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/locperformance_ld4l_cache_validation.yml
 - lib/generators/qa_server/templates/config/authorities/linked_data/scenarios/locsubjects_ld4l_cache_validation.yml
@@ -508,8 +543,10 @@ files:
 - spec/.gitignore
 - spec/cache_processors/qa_server/cache_expiry_service_spec.rb
 - spec/feature/accuracy_spec.rb
+- spec/i18n_spec.rb
 - spec/lib/configuration_spec.rb
 - spec/lib/qa_server_spec.rb
+- spec/presenters/qa_server/monitor_status/history_presenter_spec.rb
 - spec/rails_helper.rb
 - spec/services/qa_server/time_period_service_spec.rb
 - spec/services/qa_server/time_service_spec.rb
@@ -545,8 +582,10 @@ test_files:
 - spec/.gitignore
 - spec/cache_processors/qa_server/cache_expiry_service_spec.rb
 - spec/feature/accuracy_spec.rb
+- spec/i18n_spec.rb
 - spec/lib/configuration_spec.rb
 - spec/lib/qa_server_spec.rb
+- spec/presenters/qa_server/monitor_status/history_presenter_spec.rb
 - spec/rails_helper.rb
 - spec/services/qa_server/time_period_service_spec.rb
 - spec/services/qa_server/time_service_spec.rb