curationexperts / laevigata

94%

main: 97%

LAST BUILD ON BRANCH fix_redis_current

branch: fix_redis_current

CHANGE BRANCH

x

Reset

• fix_redis_current
• cache_junit_reporter
• dependabot/npm_and_yarn/axios-0.30.0
• dependabot/npm_and_yarn/babel/helpers-7.27.0
• dependabot/npm_and_yarn/babel/runtime-7.27.0
• dependabot/npm_and_yarn/brace-expansion-1.1.12
• dependabot/npm_and_yarn/cipher-base-1.0.6
• dependabot/npm_and_yarn/elliptic-6.6.1
• dependabot/npm_and_yarn/nanoid-3.3.8
• dependabot/npm_and_yarn/pbkdf2-3.1.3
• dependabot/npm_and_yarn/sha.js-2.4.12
• embargo_notifications
• fix_flaky_test
• fix_grad_service
• fix_redis_namespace_deprecations
• grad_dates
• homepage-nursing
• main
• speedier_tests
• spelling-misteak
• test_ci

Build # 2f4a3481-fc3c-44b3-b076-85704ed113e0

Build Type

Pull #2467

circleci

Committed by mark-dce

Commit Message

Remove deprecated calls to `Redis.current`

**ISSUE**
Our logs are being swamped with deprecation messages similar to:
```
`Redis.current` is deprecated and will be removed in 5.0.
  (called from: /Users/mark/.rbenv/versions/2.7.4/lib/ruby/gems/2.7.0/gems
   /hyrax-2.9.6/lib/hyrax/redis_event_store.rb:26:in `instance')
```

**RESOLUTION**
Refactor the redis initilizer and override Hyrax class methods that
call `Redis.current` in favor of a memoized call to `Redis.new`

**REFERENCES**
This commit was heavily influenced by
https://github.com/samvera/hyrax/commit/50cc8f0f

Pull Request Pull Request #2467: Remove deprecated calls to `Redis.current`

Run Details

17 of 17 new or added lines in 4 files covered. (100.0%)

193 existing lines in 28 files now uncovered.

2704 of 2866 relevant lines covered (94.35%)

52.91 hits per line