From 6675535b5cb3a44e0301c2ef73b37e81f476def6 Mon Sep 17 00:00:00 2001 From: Alex Morgun <87077843+oleksii-morgun@users.noreply.github.com> Date: Tue, 26 Mar 2024 16:40:02 -0400 Subject: [PATCH 1/2] [#MHV-54372] remove 556 transition feature flags (#15990) Co-authored-by: Alex Morgun <87077843+oleksii-morgun-va@users.noreply.github.com> --- config/features.yml | 6 ------ 1 file changed, 6 deletions(-) diff --git a/config/features.yml b/config/features.yml index 068190f7557..d9e65f1cdb8 100644 --- a/config/features.yml +++ b/config/features.yml @@ -73,12 +73,6 @@ features: actor_type: user description: Enables access to the 10-10EZR application in prod for the purposes of conducting user reasearch enable_in_development: true - cerner_transition_556_t30: - actor_type: user - description: This will control the content that will be in effect 30 days prior to transition. - cerner_transition_556_t5: - actor_type: user - description: This will control the content that will be in effect 5 days prior to transition. cerner_override_653: actor_type: user description: This will show the Cerner facility 653 as `isCerner`. From c2d9c5c2157801f9a285276c2d6a172c42d11550 Mon Sep 17 00:00:00 2001 From: Tony Gravell <146013972+agravell047@users.noreply.github.com> Date: Tue, 26 Mar 2024 18:08:32 -0400 Subject: [PATCH 2/2] removed cache busting for mobile rx (#16099) * removed cache busting for mobile rx * removed failing test for mobile * fixed gemfile --- lib/rx/client.rb | 10 ++++---- .../request/prescriptions_request_spec.rb | 23 ++++++++++--------- 2 files changed, 16 insertions(+), 17 deletions(-) diff --git a/lib/rx/client.rb b/lib/rx/client.rb index c3a9fe4780f..4e62136e59a 100644 --- a/lib/rx/client.rb +++ b/lib/rx/client.rb @@ -18,6 +18,7 @@ class Client < Common::Client::Base client_session Rx::ClientSession CACHE_TTL = 3600 * 1 # 1 hour cache + CACHE_TTL_ZERO = 0 def request(method, path, params = {}, headers = {}, options = {}) super(method, path, params, headers, options) @@ -31,7 +32,7 @@ def request(method, path, params = {}, headers = {}, options = {}) # @return [Common::Collection[Prescription]] # def get_active_rxs - Common::Collection.fetch(::Prescription, cache_key: cache_key('getactiverx'), ttl: CACHE_TTL) do + Common::Collection.fetch(::Prescription, cache_key: cache_key('getactiverx'), ttl: CACHE_TTL_ZERO) do perform(:get, 'prescription/getactiverx', nil, token_headers).body end end @@ -53,7 +54,7 @@ def get_active_rxs_with_details # @return [Common::Collection[Prescription]] # def get_history_rxs - Common::Collection.fetch(::Prescription, cache_key: cache_key('gethistoryrx'), ttl: CACHE_TTL) do + Common::Collection.fetch(::Prescription, cache_key: cache_key('gethistoryrx'), ttl: CACHE_TTL_ZERO) do perform(:get, 'prescription/gethistoryrx', nil, token_headers).body end end @@ -123,10 +124,7 @@ def get_tracking_history_rx(id) # @return [Faraday::Env] # def post_refill_rxs(ids) - if (result = perform(:post, 'prescription/rxrefill', ids, token_headers)) - Common::Collection.bust([cache_key('getactiverx'), cache_key('gethistoryrx')]) - end - result + perform(:post, 'prescription/rxrefill', ids, token_headers) end ## diff --git a/modules/mobile/spec/request/prescriptions_request_spec.rb b/modules/mobile/spec/request/prescriptions_request_spec.rb index bc820f34fe1..3f6f269a6b7 100644 --- a/modules/mobile/spec/request/prescriptions_request_spec.rb +++ b/modules/mobile/spec/request/prescriptions_request_spec.rb @@ -125,20 +125,21 @@ end end - context 'prescription cache is present on refill' do - it 'flushes prescription cache on refill' do - set_cache + # Temporarily removing test until we can figure out how to handle cache + # context 'prescription cache is present on refill' do + # it 'flushes prescription cache on refill' do + # set_cache - VCR.use_cassette('mobile/rx_refill/prescriptions/refills_prescriptions') do - put '/mobile/v0/health/rx/prescriptions/refill', params: { ids: %w[21530889 21539942] }, - headers: sis_headers - end + # VCR.use_cassette('mobile/rx_refill/prescriptions/refills_prescriptions') do + # put '/mobile/v0/health/rx/prescriptions/refill', params: { ids: %w[21530889 21539942] }, + # headers: sis_headers + # end - get '/mobile/v0/health/rx/prescriptions', headers: sis_headers + # get '/mobile/v0/health/rx/prescriptions', headers: sis_headers - assert_requested :get, upstream_mhv_history_url, times: 1 - end - end + # assert_requested :get, upstream_mhv_history_url, times: 1 + # end + # end end describe 'GET /mobile/v0/health/rx/prescriptions', :aggregate_failures do