removed cache busting for mobile rx (#16099)

* removed cache busting for mobile rx * removed failing test for mobile * fixed gemfile
department-of-veterans-affairs · Mar 26, 2024 · c2d9c5c · c2d9c5c
1 parent 6675535
commit c2d9c5c
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 17 deletions.
diff --git 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
@@ -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