qa_server 7.2.0 → 7.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 6126532673f0c180df4ffdad52f48e676a86f123
-  data.tar.gz: 41ffe57ca6dcae519aaba5860378ea018dd5f988
+SHA256:
+  metadata.gz: 8c0138684332f934add8000e85d76c86264162963e7df49fa170ee894783aa17
+  data.tar.gz: 2f9e19975f53fa4e0d34feb8e4423f203e0d46bafe6bc30af614459256a6b66f
 SHA512:
-  metadata.gz: 300bc7ce6d46fdd89c69a496753470c4de90687a272ad649a86559dd55f26af7a134e6cf9a8502eb4e210aa8268f6c27cc031d3d25a1e7227b1f559afbd4a8fe
-  data.tar.gz: a3110736087b71f9efa473ad65eb644b3e9cc6b4c305c091e74ed278447f9ab8aca12a7c0e7247b1f2213792fc922ddb9fceb2ad2d948fa86673525188ccc404
+  metadata.gz: bd9de3a942001f3e6eca2790441fa2ea2f8d386033c4af93cca230ac2ee3dbb357d44378535bdf521a89e196d29b6fd7308b1f1a6246ece9c1293c6c521deb7d
+  data.tar.gz: 933c7742a774144186e9a63f749a2b4d01c89c76b7849d8c18419e0ea95d0df28158ce5286ca694d516fa994f14796c8f668708e907ca2c037e880b101afc5e9

data/.rubocop_fixme.yml

@@ -12,3 +12,9 @@ Lint/UnusedMethodArgument:
 Style/SpecialGlobalVars:
   Exclude:
     - 'tasks/qa_server_dev.rake'
+
+Layout/AccessModifierIndentation:
+  EnforcedStyle: outdent
+
+#Layout/IndentationWidth:
+#  Enabled: false

data/.travis.yml

@@ -13,9 +13,9 @@ before_install:
   - google-chrome-stable --headless --disable-gpu --no-sandbox --remote-debugging-port=9222 http://localhost &
 
 rvm:
-  - 2.4.6
-  - 2.5.5
-  - 2.6.3
+  - 2.5.8
+  - 2.6.6
+  - 2.7.1
 
 env:
   global:
@@ -26,8 +26,7 @@ env:
   # It should be sufficient to test only the latest of the patch versions for a minor version, they
   # should be compatible across patch versions (only bug fixes are released in patch versions).
   matrix:
-    - "RAILS_VERSION=5.1.7"
-    - "RAILS_VERSION=5.2.3"
+    - "RAILS_VERSION=5.2.4"
 
 services:
   - redis-server

data/CHANGELOG.md

@@ -1,3 +1,41 @@
+### 7.5.1 (2020-12-08)
+
+* define missing i18n translations
+* add i18n-tasks gem for managing translations
+* add spec test to check for missing translations
+
+### 7.5.0 (2020-12-07)
+
+* UI improvements of check_status page
+  * pre-populate selection fields with most recently submitted authorities
+  * add ability to compare two authorities' accuracy tests in UI
+  * update check_status UI to require GO button click to start processing
+  * fix javascript to avoid referencing missing elements
+* accuracy test updates
+  * accuracy tests default to `context=false` since it is not required for testing
+  * allow accuracy tests through rspec to be marked pending (allows for better regression testing)
+  * add pending to failing validation tests such that accuracy_spec passes
+  * sync tests with production accuracy tests
+  * fix typo in `position` and a few syntax errors
+  * adjust result_size for validations where passing tests return less than 200 chars
+  * split locnames_rwo_ld4l_cache into 3 sets of tests of about 9 each to avoid time out
+  * add extended context for getty_aat_ld4l_cache and getty_tgn_ld4l_cache
+  * add authority locvocabs_ld4l_cache
+* add request_data to status_logger (e.g. query for search, identifier for term)
+* update to latest bixby v3.0
+
+### 7.4.0 (2020-04-13)
+
+* add size and complexity performance statistics (not auto-generated)
+
+### 7.3.0 (2020-04-08)
+
+* move generation of graphs to background jobs
+
+### 7.2.1 (2020-02-23)
+
+* fix graph fails generation when any label is empty string
+
 ### 7.2.0 (2020-02-22)
 
 * move graphs from assets to public directory

data/Rakefile

@@ -9,7 +9,7 @@ end
 Bundler::GemHelper.install_tasks
 
 # Set up the test application prior to running jasmine tasks.
-task :setup_test_server do
+task setup_test_server: :environment do
   require 'engine_cart'
   EngineCart.load_application!
 end

data/app/assets/stylesheets/qa_server/_check-status.scss

@@ -25,6 +25,23 @@ div#example-url-warning {
   color: #4d0000;
 }
 
+div#authority-selector {
+  margin-top: 20px;
+  margin-bottom: 0px;
+  margin-left: 20px;
+  margin-right: 20px;
+  width: 40%;
+}
+
+div#comparison-authority-selector {
+  display: none;
+  margin-top: 20px;
+  margin-bottom: 0px;
+  margin-left: 20px;
+  margin-right: 20px;
+  width: 40%;
+}
+
 div.status-section {
   margin-top: 50px;
 }
@@ -43,6 +60,20 @@ table, th, td {
   padding: 5px;
 }
 
+th.comparison-header {
+  background-color: #2e2c2c;
+}
+
+th.comparison-before {
+  text-align: center;
+  color: #f4f360;
+}
+
+th.comparison-after {
+  text-align: center;
+  color: #7df34c;
+}
+
 th {
   background-color: #a1b1c4;
   color: black;
@@ -52,6 +83,11 @@ td.position {
   text-align: center;
 }
 
+th.bold-left-border,
+td.bold-left-border {
+  border-left: 2px solid black;
+}
+
 .status-good {
   text-align: center;
   background-color: #ccffcc;

data/app/cache_processors/concerns/qa_server/cache_keys.rb

@@ -2,15 +2,10 @@
 # This module sets up the keys used to identify data in Rails.cache
 module QaServer
   module CacheKeys
-    SCENARIO_RUN_TEST_DATA_CACHE_KEY = "QaServer--CacheKeys--scenario_run_test_data"
     SCENARIO_RUN_SUMMARY_DATA_CACHE_KEY = "QaServer--CacheKeys--scenario_run_summary_data"
     SCENARIO_RUN_FAILURE_DATA_CACHE_KEY = "QaServer--CacheKeys--scenario_run_failure_data"
     SCENARIO_RUN_HISTORY_DATA_CACHE_KEY = "QaServer--CacheKeys--scenario_run_history_data"
-    SCENARIO_RUN_HISTORY_GRAPH_CACHE_KEY = "QaServer--CacheKeys--scenario_run_history_graph"
 
     PERFORMANCE_DATATABLE_DATA_CACHE_KEY = "QaServer--Cache--performance_datatable_data"
-    PERFORMANCE_GRAPH_HOURLY_DATA_CACHE_KEY = "QaServer--CacheKeys--performance_graph_hourly_data"
-    PERFORMANCE_GRAPH_DAILY_DATA_CACHE_KEY = "QaServer--CacheKeys--performance_graph_daily_data"
-    PERFORMANCE_GRAPH_MONTHLY_DATA_CACHE_KEY = "QaServer--CacheKeys--performance_graph_monthly_data"
   end
 end

data/app/cache_processors/qa_server/cache_expiry_service.rb

@@ -8,6 +8,11 @@ module QaServer
         cache_expires_at - QaServer::TimeService.current_time
       end
 
+      def end_of_hour_expiry
+        ct = QaServer::TimeService.current_time
+        ct.end_of_hour - ct
+      end
+
       # @param key [String] cache key
       # @param force [Boolean] if true, forces cache to regenerate by returning true; otherwise, uses cache expiry to determine whether cache has expired
       # @return [Boolean] true if cache has expired or is being forced to expire
@@ -19,15 +24,15 @@ module QaServer
         force
       end
 
-      private
+    private
 
-        # @return [ActiveSupport::TimeWithZone] DateTime at which cache should expire
-        def cache_expires_at
-          offset = QaServer.config.hour_offset_to_expire_cache
-          offset_time = QaServer::TimeService.current_time
-          offset_time = offset_time.tomorrow unless (offset_time + 5.minutes).hour < offset
-          offset_time.beginning_of_day + offset.hours - 5.minutes
-        end
+      # @return [ActiveSupport::TimeWithZone] DateTime at which cache should expire
+      def cache_expires_at
+        offset = QaServer.config.hour_offset_to_expire_cache
+        offset_time = QaServer::TimeService.current_time
+        offset_time = offset_time.tomorrow unless (offset_time + 5.minutes).hour < offset
+        offset_time.beginning_of_day + offset.hours - 5.minutes
+      end
     end
   end
 end

data/app/cache_processors/qa_server/job_id_cache.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+# Provide service methods for getting a list of all authorities and scenarios for an authority.
+module QaServer
+  class JobIdCache
+    class << self
+      # Is the passed in job_id the active one for the job_key?
+      # @param job_key [String] key unique to the job being run (e.g. "QaServer::Jobs::MonitorTestsJob")
+      # @param job_id [String] UUID for job running the tests
+      # @param expires_in [ActiveSupport::Duration]  This should be at least as long as the expected job run time to avoid multiple instances of the job running at the same time.
+      # @note When job completes, call reset_job_id to invalidate the cache
+      def active_job_id?(job_key:, job_id:, expires_in: 30.minutes)
+        cached_job_id = Rails.cache.fetch(cache_key(job_key), expires_in: expires_in, race_condition_ttl: 5.minutes) { job_id }
+        cached_job_id == job_id
+      end
+
+      # Delete cache for job id for the job represented by job_key.  Call this when the job completes.
+      # @param job_key [String] key unique to the job being run (e.g. "QaServer::Jobs::MonitorTestsJob")
+      def reset_job_id(job_key:)
+        Rails.cache.delete(cache_key(job_key))
+      end
+
+    private
+
+      def cache_key(job_key)
+        "#{job_key}-job_id"
+      end
+    end
+  end
+end

data/app/cache_processors/qa_server/performance_cache.rb

@@ -49,45 +49,45 @@ module QaServer
       cache_to_write = nil # free cache for garbage collection
     end
 
-    private
+  private
 
-      def swap_cache_hash
-        cache_to_write = @cache
-        @cache = {} # reset main cache so new items after write begins are cached in the main cache
-        QaServer.config.performance_cache_logger.debug("#{self.class}##{__method__} - cache memory BEFORE write: #{ObjectSpace.memsize_of(cache_to_write)}")
-        cache_to_write
-      end
+    def swap_cache_hash
+      cache_to_write = @cache
+      @cache = {} # reset main cache so new items after write begins are cached in the main cache
+      QaServer.config.performance_cache_logger.debug("#{self.class}##{__method__} - cache memory BEFORE write: #{ObjectSpace.memsize_of(cache_to_write)}")
+      cache_to_write
+    end
 
-      def log(id:)
-        return if QaServer.config.suppress_logging_performance_datails?
-        Rails.logger.debug("*** performance data for id: #{id} ***")
-        Rails.logger.debug(@cache[id].to_yaml)
-      end
+    def log(id:)
+      return if QaServer.config.suppress_logging_performance_datails?
+      Rails.logger.debug("*** performance data for id: #{id} ***")
+      Rails.logger.debug(@cache[id].to_yaml)
+    end
 
-      def incomplete?(entry)
-        required_keys.each { |k| return true unless entry.key? k }
-        false
-      end
+    def incomplete?(entry)
+      required_keys.each { |k| return true unless entry.key? k }
+      false
+    end
 
-      def required_keys
-        [:dt_stamp,
-         :authority,
-         :action,
-         :action_time_ms,
-         :size_bytes,
-         :retrieve_time_ms,
-         :graph_load_time_ms,
-         :normalization_time_ms]
-      end
+    def required_keys
+      [:dt_stamp,
+       :authority,
+       :action,
+       :action_time_ms,
+       :size_bytes,
+       :retrieve_time_ms,
+       :graph_load_time_ms,
+       :normalization_time_ms]
+    end
 
-      def log_write_all(prefix, size_before, cache_size)
-        if size_before.positive?
-          QaServer.config.performance_cache_logger.debug("#{prefix} 0 of #{size_before} performance data records were saved") if size_before == cache_size
-          QaServer.config.performance_cache_logger.debug("#{prefix} #{size_before - cache_size} of #{size_before} performance data records were saved") if size_before > cache_size
-        else
-          QaServer.config.performance_cache_logger.debug("#{prefix} 0 of 0 performance data records were saved")
-        end
-        QaServer.config.performance_cache_logger.debug("#{prefix} - cache memory AFTER write: #{ObjectSpace.memsize_of @cache}")
+    def log_write_all(prefix, size_before, cache_size)
+      if size_before.positive?
+        QaServer.config.performance_cache_logger.debug("#{prefix} 0 of #{size_before} performance data records were saved") if size_before == cache_size
+        QaServer.config.performance_cache_logger.debug("#{prefix} #{size_before - cache_size} of #{size_before} performance data records were saved") if size_before > cache_size
+      else
+        QaServer.config.performance_cache_logger.debug("#{prefix} 0 of 0 performance data records were saved")
       end
+      QaServer.config.performance_cache_logger.debug("#{prefix} - cache memory AFTER write: #{ObjectSpace.memsize_of @cache}")
+    end
   end
 end

data/app/cache_processors/qa_server/performance_day_graph_cache.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+# Cache the datetime_stamp of the last time the performance day graph was generated.  Calls job to generate the graph if expired.
+module QaServer
+  class PerformanceDayGraphCache
+    class << self
+      # Generates graphs for the past 24 hours for :search, :fetch, and :all actions for each authority.
+      # @param force [Boolean] if true, run the tests even if the cache hasn't expired; otherwise, use cache if not expired
+      def generate_graphs(force: false)
+        Rails.cache.fetch(cache_key, expires_in: next_expiry, race_condition_ttl: 30.seconds, force: force) do
+          QaServer.config.monitor_logger.debug("(QaServer::PerformanceDayGraphCache) - KICKING OFF PERFORMANCE DAY GRAPH GENERATION (force: #{force})")
+          QaServer::PerformanceDayGraphJob.perform_later
+          "Graphs generation initiated at #{QaServer::TimeService.current_time}"
+        end
+      end
+
+    private
+
+      def cache_key
+        "QaServer::PerformanceDayGraphCache.generate_graphs--latest_generation_initiated"
+      end
+
+      def next_expiry
+        QaServer::CacheExpiryService.end_of_hour_expiry
+      end
+    end
+  end
+end

data/app/cache_processors/qa_server/performance_month_graph_cache.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+# Cache the datetime_stamp of the last time the performance month graph was generated.  Calls job to generate the graph if expired.
+module QaServer
+  class PerformanceMonthGraphCache
+    class << self
+      # Generates graphs for the past 30 days for :search, :fetch, and :all actions for each authority.
+      # @param force [Boolean] if true, run the tests even if the cache hasn't expired; otherwise, use cache if not expired
+      def generate_graphs(force: false)
+        Rails.cache.fetch(cache_key, expires_in: next_expiry, race_condition_ttl: 30.seconds, force: force) do
+          QaServer.config.monitor_logger.debug("(QaServer::PerformanceMonthGraphCache) - KICKING OFF PERFORMANCE MONTH GRAPH GENERATION (force: #{force})")
+          QaServer::PerformanceMonthGraphJob.perform_later
+          "Graphs generation initiated at #{QaServer::TimeService.current_time}"
+        end
+      end
+
+    private
+
+      def cache_key
+        "QaServer::PerformanceMonthGraphCache.generate_graphs--latest_generation_initiated"
+      end
+
+      def next_expiry
+        QaServer::CacheExpiryService.cache_expiry
+      end
+    end
+  end
+end

data/app/cache_processors/qa_server/performance_year_graph_cache.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+# Cache the datetime_stamp of the last time the performance year graph was generated.  Calls job to generate the graph if expired.
+module QaServer
+  class PerformanceYearGraphCache
+    class << self
+      # Generates graphs for the 12 months for :search, :fetch, and :all actions for each authority.
+      # @param force [Boolean] if true, run the tests even if the cache hasn't expired; otherwise, use cache if not expired
+      def generate_graphs(force: false)
+        Rails.cache.fetch(cache_key, expires_in: next_expiry, race_condition_ttl: 30.seconds, force: force) do
+          QaServer.config.monitor_logger.debug("(QaServer::PerformanceYearGraphCache) - KICKING OFF PERFORMANCE YEAR GRAPH GENERATION (force: #{force})")
+          QaServer::PerformanceYearGraphJob.perform_later
+          "Graphs generation initiated at #{QaServer::TimeService.current_time}"
+        end
+      end
+
+    private
+
+      def cache_key
+        "QaServer::PerformanceYearGraphCache.generate_graphs--latest_generation_initiated"
+      end
+
+      def next_expiry
+        QaServer::CacheExpiryService.cache_expiry
+      end
+    end
+  end
+end

data/app/cache_processors/qa_server/scenario_history_cache.rb

@@ -21,15 +21,15 @@ module QaServer
         end
       end
 
-      private
+    private
 
-        def cache_key_for_historical_data
-          SCENARIO_RUN_HISTORY_DATA_CACHE_KEY
-        end
+      def cache_key_for_historical_data
+        SCENARIO_RUN_HISTORY_DATA_CACHE_KEY
+      end
 
-        def next_expiry
-          QaServer::CacheExpiryService.cache_expiry
-        end
+      def next_expiry
+        QaServer::CacheExpiryService.cache_expiry
+      end
     end
   end
 end

data/app/cache_processors/qa_server/scenario_history_graph_cache.rb

@@ -2,31 +2,26 @@
 # Generate graphs for the past 30 days using cached data.  Graphs are generated only if the cache has expired.
 module QaServer
   class ScenarioHistoryGraphCache
-    class_attribute :graphing_service
-    self.graphing_service = QaServer::HistoryGraphingService
-
     class << self
-      include QaServer::CacheKeys
-
-      HISTORICAL_GRAPH_FILENAME = 'historical_side_stacked_bar.png'
-
       # Generates graphs for the past 30 days for :search, :fetch, and :all actions for each authority.
       # @param force [Boolean] if true, run the tests even if the cache hasn't expired; otherwise, use cache if not expired
       def generate_graph(data:, force: false)
-        return unless QaServer::CacheExpiryService.cache_expired?(key: cache_key_for_force, force: force, next_expiry: next_expiry)
-        QaServer.config.monitor_logger.debug("(QaServer::ScenarioHistoryGraphCache) - GENERATING historical summary graph (force: #{force})")
-        graphing_service.generate_graph(data)
+        Rails.cache.fetch(cache_key, expires_in: next_expiry, race_condition_ttl: 30.seconds, force: force) do
+          QaServer.config.monitor_logger.debug("(QaServer::ScenarioHistoryGraphCache) - KICKING OFF HISTORY GRAPH GENERATION (force: #{force})")
+          QaServer::HistoryGraphJob.perform_later(data: data)
+          "Graph generation initiated at #{QaServer::TimeService.current_time}"
+        end
       end
 
-      private
+    private
 
-        def cache_key_for_force
-          "#{SCENARIO_RUN_HISTORY_GRAPH_CACHE_KEY}--force"
-        end
+      def cache_key
+        "QaServer::ScenarioHistoryGraphCache.generate_graph--latest_generation_initiated"
+      end
 
-        def next_expiry
-          QaServer::CacheExpiryService.cache_expiry
-        end
+      def next_expiry
+        QaServer::CacheExpiryService.cache_expiry
+      end
     end
   end
 end