lifen_fhir 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4f268ed9c624f69baa94cf0c7b9ad04cf5778ffc
-  data.tar.gz: cfdeb52172b4385d3fce6a0a1f497931156669d2
+  metadata.gz: 883c936ea9075219b23c3cd2a999237d00096ab6
+  data.tar.gz: 62045d010b9ef604b067f17b319bcf57c712f176
 SHA512:
-  metadata.gz: '003841d9ab0e8df08ae247d1c5d7c9b05c42e095af4c6c298778e1de768527f0f697679f95249a516e39240093cfb79d0aa3e1e8711d0ea59c012bce2b6185d3'
-  data.tar.gz: f05094f969f7c04dd8b8b5ba7774c89c21c6e84870b44a291bcac42cf5a9e9c91c394389d733c662d5aa04306e381e29060fcbf0a5e663c610cd91f48299e007
+  metadata.gz: 6cc241e7aa995d4eeab7295ddf4ed87f305b91894e797be38f9e7eb1298b1571b33110e6b43d4ce9e988c0c918b3f3154510bce69ab18c6cc6fc401d2732cab0
+  data.tar.gz: b13b9722a50eab03eef4415dce10651f663b88cd9840dc677d4ee035d8129e912cb41fbb86dd67dde84c085e1907626739be02069cbac44751d87f15fb819c38

data/CHANGELOG.md

@@ -1,3 +1,13 @@
+0.6.0
+-----
+
+- Internal errors provides the client with a context that can be passed on issue tracker
+
+0.5.1
+-----
+
+- Fix missing fields issue in find_by_uuid for CommunicationRequest
+
 0.5.0
 -----
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    lifen_fhir (0.5.0)
+    lifen_fhir (0.6.0)
       faraday (>= 0.9)
       inflecto
       virtus (>= 1.0)

data/README.md

@@ -40,9 +40,21 @@ Optionnal configuration:
 - `proxy_url`: route all your requests via a proxy
 - `logger`: log all requests to a custom logger (on Rails, `Rails.logger` is selected automatically)
 
-### Internal
+### Managing errors
 
-#### Managing a Binary
+`LifenFhir::Error` provides a `context` that can be passed to your issue manager in order to have a better understanding of what is going wrong when an error happens. Here is an example with Sentry:
+
+```ruby
+begin
+  # some code using the gem
+rescue LifenFhir::Error => e
+  Raven.capture_exception(e)
+  Raven.extra_context(lifen: e.context)
+end
+```
+
+
+### Managing a Binary
 
 ```ruby
 binary = LifenFhir::Binary.new(uuid: "b7c7dae671b93e951ce6a4f530736276")
@@ -85,7 +97,7 @@ communication_request_request.send
 
 ```
 
-#### Custom Channels management
+### Custom Channels management
 
 ```ruby
 recipient = LifenFhir::Practitioner.new(uuid: "valid-user-uuid")
@@ -97,7 +109,7 @@ channel = recipient.create_address(type: "address", lines: ["39 rue Aboukir"], c
 channel = recipient.create_telecom(type: "telecom", system: "fax", value: "+33102030405")
 ```
 
-#### Managing Patients
+### Managing Patients
 
 ```ruby
 # To create a new patient
@@ -126,6 +138,10 @@ Once the new version is validated, the deployment follows those steps :
 6. publish the gem using `gem publish lifen_fhir-X.X.X.gem`
 7. run `bundle update lifen` in related projects :)
 
+## Contributing
+
+TBD
+
 ## License
 
 The MIT License (MIT)

data/lib/lifen_fhir/app_authenticated_client.rb

@@ -9,27 +9,21 @@ module LifenFhir
 
         case response.status
         when 400
-          raise Error, "Error 400, Unknown error, #{response_error(response, params)}"
+          raise_error("400 Error, Unknown error", response, params)
         when 401
-          raise InvalidSecretTokenError, "Error 401, Invalid app bearer, #{response_error(response, params)}"
-        when 403
-          raise UserAlreadyExistingError, "Error 403, User already existing, #{response_error(response, params)}"
+          raise_error("401 Error, Invalid app bearer", response, params)
         when 404
-          raise Error, "Error 404, Page not found, #{response_error(response, params)}"
+          raise_error("404 Error, Record not found", response, params)
         when 422
           json = JSON.parse response.body
 
           diagnostic = json["issue"][0]["diagnostics"]
 
-          raise Error, "Error 422, Unprocessable Entity, Diagnostic: '#{diagnostic}', #{response_error(response, params)}"
+          raise_error("422 Error, Unprocessable Entity, Diagnostic: '#{diagnostic}'", response, params)
         end
 
       end
 
-      def response_error(response, params)
-        "App Client, #{super(response, params)}"
-      end
-
       def bearer
         LifenFhir.configuration.application_access_token
       end

data/lib/lifen_fhir/client.rb

@@ -41,12 +41,7 @@ module LifenFhir
 
       def handle_errors(response, params)
         if response.status == 500
-
-          json = JSON.parse response.body
-
-          trace_id = json.fetch("X-B3-TraceId", "unknown")
-
-          raise Error, "Error 500, Internal server error (trace ID: #{trace_id}), #{response_error(response, params)}"
+          raise_error("500 Error, Internal server error", response, params)
         end
       end
 
@@ -88,6 +83,12 @@ module LifenFhir
         LifenFhir.configuration.proxy_url
       end
 
+      def raise_error(message, response, params)
+        message += " -- #{response.env.method.upcase} '#{response.env.url}'"
+
+        raise Error.new(message: message, response: response, params: params, client: "app", token: bearer)
+      end
+
       def logger
         LifenFhir.configuration.logger
       end
@@ -95,28 +96,18 @@ module LifenFhir
       def before_request
       end
 
-      def response_error(response, params)
-        params[:payload] = "filtered" if params.is_a?(Hash) and params.has_key? :payload
+      def use_and_remove_accept(params)
+        return params.delete :accept if params.is_a?(Hash) and params.has_key? :accept
 
-        "#{response.env.method.upcase} '#{response.env.url}' with params '#{params.inspect}' and bearer '#{trucanted_bearer}'"
+        "application/json"
       end
 
-      def trucanted_bearer
-        if m = /^(.{24})(.*)$/.match(bearer)
-          "#{m[1]}#{"*" * m[2].length}"
-        else
-          bearer
-        end
+      def before_request
       end
 
       def bearer
         raise "A bearer method must be defined"
       end
 
-      def use_and_remove_accept(params)
-        return params.delete :accept if params.is_a?(Hash) and params.has_key? :accept
-
-        "application/json"
-      end
   end
 end

data/lib/lifen_fhir/communication_request.rb

@@ -38,14 +38,22 @@ module LifenFhir
 
       self.status = json.fetch("status") { "unknown" }
 
-      self.binary = Binary.new.attributes_from_json(Array(json["payload"]).first)
+      if has_payload?(json)
+        self.binary = Binary.new.attributes_from_json(Array(json["payload"]).first)
+      end
 
-      self.category = Category.new.attributes_from_json(Array(json["category"]).first)
+      if has_category?(json)
+        self.category = Category.new.attributes_from_json(Array(json["category"]).first)
+      end
 
-      self.sender = Practitioner.new.attributes_from_json(json["sender"])
+      if has_sender?(json)
+        self.sender = Practitioner.new.attributes_from_json(json["sender"])
+      end
 
-      self.recipients = Array(json["recipient"]).map do |recipient_json|
-         Practitioner.new.attributes_from_json(recipient_json)
+      if has_recipient?(json)
+        self.recipients = Array(json["recipient"]).map do |recipient_json|
+           Practitioner.new.attributes_from_json(recipient_json)
+        end
       end
     end
 
@@ -85,5 +93,20 @@ module LifenFhir
         end
       end
 
+      def has_category?(json)
+        json.key?("category")
+      end
+
+      def has_sender?(json)
+        json.key?("sender")
+      end
+
+      def has_recipient?(json)
+        json.key?("recipient")
+      end
+
+      def has_payload?(json)
+        json.key?("payload")
+      end
   end
 end

data/lib/lifen_fhir/error.rb

@@ -1,27 +1,56 @@
 module LifenFhir
 
   class Error < StandardError
-  end
 
-  class UnauthorizedError < Error
-  end
+    attr_accessor :response, :params
 
-  class InvalidCredentialsError < Error
-  end
+    def initialize(message: , response: nil, params: {}, client: "app", token: nil)
+      self.response = response
+      self.params = params
 
-  class InvalidParamsError < Error
-  end
+      @client = client
+      @token = token
 
-  class InvalidSecretTokenError < Error
-  end
+      super(message)
+    end
 
-  class InvalidTokenError < Error
-  end
+    def context
+      return {} if response.nil?
 
-  class UserAlreadyExistingError < Error
-  end
+      {
+        url: response.env.url.to_s,
+        method: response.env.method.to_s,
+        token: trucanted_token,
+        payload_sent: filtered_params,
+        trace_id: extracted_trace_id,
+        payload_received: response.body,
+        client: @client
+      }
+
+    end
+
+    private
+
+      def extracted_trace_id
+        response.headers.fetch("X-B3-Traceid") { "unknown" }
+      end
+
+
+      def trucanted_token
+        if m = /^(.{24})(.*)$/.match(@token)
+          "#{m[1]}#{"*" * m[2].length}"
+        else
+          @token
+        end
+      end
+
+      def filtered_params
+        filtered_params = params
+
+        filtered_params[:payload] = "filtered" if filtered_params.is_a?(Hash) and filtered_params.has_key? :payload
 
-  class CantRefreshTokenError < Error
+        filtered_params
+      end
   end
 
-end
+end

data/lib/lifen_fhir/version.rb

@@ -1,3 +1,3 @@
 module LifenFhir
-  VERSION = "0.5.0"
+  VERSION = "0.6.0"
 end

data/lib/lifen_fhir.rb

@@ -22,7 +22,6 @@ module LifenFhir
   require 'lifen_fhir/logger'
   require 'lifen_fhir/error'
   require 'lifen_fhir/client'
-  require 'lifen_fhir/user_authenticated_client'
   require 'lifen_fhir/app_authenticated_client'
   require 'lifen_fhir/configuration'
   require 'lifen_fhir/base'

data/spec/cassettes/communication_request/find_by_uuid/valid_uuid_with_status_and_binary_only.yml

@@ -0,0 +1,70 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: https://develop.lifen.fr/fhir/CommunicationRequest/f1924980-6685-40eb-b07d-51f317365ae5
+    body:
+      encoding: UTF-8
+      string: "{}"
+    headers:
+      User-Agent:
+      - Faraday v0.12.1
+      Authorization:
+      - Bearer valid_application_access_token
+      Accept:
+      - application/json
+      Accept-Encoding:
+      - gzip;q=1.0,deflate;q=0.6,identity;q=0.3
+  response:
+    status:
+      code: 200
+      message: OK
+    headers:
+      Server:
+      - Apache-Coyote/1.1
+      X-B3-Traceid:
+      - 6d110814fbb41d26
+      X-B3-Spanid:
+      - 6d110814fbb41d26
+      X-Content-Type-Options:
+      - nosniff
+      X-Xss-Protection:
+      - 1; mode=block
+      Cache-Control:
+      - no-cache, no-store, max-age=0, must-revalidate
+      Pragma:
+      - no-cache
+      Expires:
+      - '0'
+      X-Powered-By:
+      - HAPI FHIR 2.4 REST Server (FHIR Server; FHIR 3.0.1/DSTU3)
+      Location:
+      - http://rc.lifen.fr/fhir/CommunicationRequest/f1924980-6685-40eb-b07d-51f317365ae5
+      Content-Type:
+      - application/json+fhir;charset=UTF-8
+      Transfer-Encoding:
+      - chunked
+      Date:
+      - Wed, 07 Jun 2017 08:34:57 GMT
+      Connection:
+      - close
+      Access-Control-Allow-Credentials:
+      - 'true'
+    body:
+      encoding: UTF-8
+      string: |-
+        {
+          "resourceType": "CommunicationRequest",
+          "id": "f1924980-6685-40eb-b07d-51f317365ae5",
+          "status": "cancelled",
+          "payload": [
+            {
+              "contentReference": {
+                "reference": "Binary/11e746b1-6cd7-1483-96ef-0242ac110004"
+              }
+            }
+          ]
+        }
+    http_version:
+  recorded_at: Wed, 07 Jun 2017 08:34:16 GMT
+recorded_with: VCR 3.0.3

data/spec/cassettes/communication_request/find_by_uuid/valid_uuid_with_status_only.yml

@@ -0,0 +1,63 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: https://develop.lifen.fr/fhir/CommunicationRequest/f1924980-6685-40eb-b07d-51f317365ae5
+    body:
+      encoding: UTF-8
+      string: "{}"
+    headers:
+      User-Agent:
+      - Faraday v0.12.1
+      Authorization:
+      - Bearer valid_application_access_token
+      Accept:
+      - application/json
+      Accept-Encoding:
+      - gzip;q=1.0,deflate;q=0.6,identity;q=0.3
+  response:
+    status:
+      code: 200
+      message: OK
+    headers:
+      Server:
+      - Apache-Coyote/1.1
+      X-B3-Traceid:
+      - 6d110814fbb41d26
+      X-B3-Spanid:
+      - 6d110814fbb41d26
+      X-Content-Type-Options:
+      - nosniff
+      X-Xss-Protection:
+      - 1; mode=block
+      Cache-Control:
+      - no-cache, no-store, max-age=0, must-revalidate
+      Pragma:
+      - no-cache
+      Expires:
+      - '0'
+      X-Powered-By:
+      - HAPI FHIR 2.4 REST Server (FHIR Server; FHIR 3.0.1/DSTU3)
+      Location:
+      - http://rc.lifen.fr/fhir/CommunicationRequest/f1924980-6685-40eb-b07d-51f317365ae5
+      Content-Type:
+      - application/json+fhir;charset=UTF-8
+      Transfer-Encoding:
+      - chunked
+      Date:
+      - Wed, 07 Jun 2017 08:34:57 GMT
+      Connection:
+      - close
+      Access-Control-Allow-Credentials:
+      - 'true'
+    body:
+      encoding: UTF-8
+      string: |-
+        {
+          "resourceType": "CommunicationRequest",
+          "id": "f1924980-6685-40eb-b07d-51f317365ae5",
+          "status": "cancelled"
+        }
+    http_version:
+  recorded_at: Wed, 07 Jun 2017 08:34:16 GMT
+recorded_with: VCR 3.0.3

data/spec/cassettes/errors/401.yml

@@ -0,0 +1,62 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: https://develop.lifen.fr/fhir/CommunicationRequest/valid_uuid
+    body:
+      encoding: UTF-8
+      string: "{}"
+    headers:
+      User-Agent:
+      - Faraday v0.12.1
+      Authorization:
+      - Bearer invalid_application_access_token
+      Accept:
+      - application/json
+      Accept-Encoding:
+      - gzip;q=1.0,deflate;q=0.6,identity;q=0.3
+  response:
+    status:
+      code: 401
+      message: Unauthorized
+    headers:
+      Server:
+      - Apache-Coyote/1.1
+      X-Content-Type-Options:
+      - nosniff
+      X-Xss-Protection:
+      - 1; mode=block
+      Cache-Control:
+      - no-cache, no-store, max-age=0, must-revalidate
+      Pragma:
+      - no-cache
+      Expires:
+      - '0'
+      X-Powered-By:
+      - HAPI FHIR 2.4 REST Server (FHIR Server; FHIR 3.0.1/DSTU3)
+      Content-Type:
+      - application/json+fhir;charset=UTF-8
+      Transfer-Encoding:
+      - chunked
+      Date:
+      - Mon, 05 Jun 2017 06:46:53 GMT
+      Connection:
+      - close
+      Access-Control-Allow-Credentials:
+      - 'true'
+    body:
+      encoding: UTF-8
+      string: |-
+        {
+          "resourceType": "OperationOutcome",
+          "issue": [
+            {
+              "severity": "error",
+              "code": "login",
+              "diagnostics": "Invalid access token"
+            }
+          ]
+        }
+    http_version: 
+  recorded_at: Mon, 05 Jun 2017 06:47:02 GMT
+recorded_with: VCR 3.0.3

data/spec/communication_request_spec.rb

@@ -151,6 +151,33 @@ describe LifenFhir::CommunicationRequest do
       expect(@communication_request.status).to eq "cancelled"
     end
 
+    context ':when only status, id and binary' do
+
+      it 'works' do
+        VCR.use_cassette "communication_request/find_by_uuid/valid_uuid_with_status_and_binary_only" do
+          @communication_request = LifenFhir::CommunicationRequest.find_by_uuid("f1924980-6685-40eb-b07d-51f317365ae5")
+        end
+
+        expect(@communication_request.status).to eq "cancelled"
+        expect(@communication_request.uuid).to eq "f1924980-6685-40eb-b07d-51f317365ae5"
+        expect(@communication_request.sender).to be_nil
+        expect(@communication_request.recipients.count).to eq(0)
+        expect(@communication_request.category.code).to eq("MEDICAL_REPORT")
+      end
+    end
+
+    context ':when only status and communication_request id' do
+
+      it 'works' do
+        VCR.use_cassette "communication_request/find_by_uuid/valid_uuid_with_status_only" do
+          @communication_request = LifenFhir::CommunicationRequest.find_by_uuid("f1924980-6685-40eb-b07d-51f317365ae5")
+        end
+
+        expect(@communication_request.status).to eq "cancelled"
+        expect(@communication_request.binary).to be_nil
+      end
+    end
+
     context 'with no defined status' do
       it 'works' do
         VCR.use_cassette "communication_request/find_by_uuid/valid_uuid_with_no_status" do
@@ -180,7 +207,7 @@ describe LifenFhir::CommunicationRequest do
           VCR.use_cassette "communication_request/find_by_uuid/invalid_uuid" do
             @communication_request = LifenFhir::CommunicationRequest.find_by_uuid("invalid_uuid")
           end
-        }.to raise_error LifenFhir::Error, "Error 404, Page not found, App Client, GET 'https://develop.lifen.fr/fhir/CommunicationRequest/invalid_uuid' with params '{}' and bearer 'valid_application_access******'"
+        }.to raise_error LifenFhir::Error
       end
 
     end

data/spec/error_spec.rb

@@ -0,0 +1,168 @@
+require 'spec_helper'
+
+describe LifenFhir::Error do
+
+  describe 'in real conditions' do
+
+    context '404' do
+
+      it 'extracts the correct context' do
+
+        begin
+          VCR.use_cassette "communication_request/find_by_uuid/invalid_uuid" do
+            communication_request = LifenFhir::CommunicationRequest.find_by_uuid("invalid_uuid")
+          end
+        rescue LifenFhir::Error => e
+
+          expect(e.to_s).to eq "404 Error, Record not found -- GET 'https://develop.lifen.fr/fhir/CommunicationRequest/invalid_uuid'"
+
+          current_context = e.context
+
+          expect(current_context[:client]).to eq "app"
+          expect(current_context[:url]).to eq "https://develop.lifen.fr/fhir/CommunicationRequest/invalid_uuid"
+          expect(current_context[:method]).to eq "get"
+          expect(current_context[:token]).to eq "valid_application_access******"
+          expect(current_context[:payload_sent]).to eq({})
+          expect(current_context[:trace_id]).to eq "unknown"
+          expect(current_context[:payload_received]).to include('"diagnostics": "Resource CommunicationRequest/invalid_uuid is not known"')
+        end
+
+      end
+
+    end
+
+    context '400' do
+
+      let(:address) {
+        LifenFhir::Address.new(
+          lines: ["M et Mme Potter", "39 rue d'Aboukir"],
+          city: "paris",
+          postal_code: "75002",
+          country: "France"
+        )
+      }
+
+      let(:patient) {
+        patient = LifenFhir::Patient.new(
+          first_name: ["Pierre", "Henri"],
+          last_name: "Potter",
+          birth_date: Date.new(1974, 12, 25),
+          address: address
+        )
+      }
+
+      it 'extracts the correct context' do
+        patient.last_name = nil
+
+        begin
+          VCR.use_cassette("patient/create/invalid_attributes") do
+            patient.save
+          end
+        rescue LifenFhir::Error => e
+
+          expect(e.to_s).to eq "400 Error, Unknown error -- POST 'https://develop.lifen.fr/fhir/Patient'"
+
+          current_context = e.context
+
+          expect(current_context[:client]).to eq "app"
+          expect(current_context[:url]).to eq "https://develop.lifen.fr/fhir/Patient"
+          expect(current_context[:method]).to eq "post"
+          expect(current_context[:token]).to eq "valid_application_access******"
+          expect(current_context[:payload_sent]["birthDate"]).to eq("1974-12-25")
+          expect(current_context[:trace_id]).to eq "unknown"
+          expect(current_context[:payload_received]).to include('"diagnostics": "Patient\'s family name is mandatory."')
+        end
+      end
+
+    end
+
+    context '401' do
+
+      before do
+        LifenFhir.configure do |config|
+          config.application_access_token = "invalid_application_access_token"
+        end
+      end
+
+      it 'extracts the correct context' do
+        begin
+          VCR.use_cassette "errors/401" do
+            communication_request = LifenFhir::CommunicationRequest.find_by_uuid("valid_uuid")
+          end
+        rescue LifenFhir::Error => e
+
+          expect(e.to_s).to eq "401 Error, Invalid app bearer -- GET 'https://develop.lifen.fr/fhir/CommunicationRequest/valid_uuid'"
+
+          current_context = e.context
+
+          expect(current_context[:client]).to eq "app"
+          expect(current_context[:url]).to eq "https://develop.lifen.fr/fhir/CommunicationRequest/valid_uuid"
+          expect(current_context[:method]).to eq "get"
+          expect(current_context[:token]).to eq "invalid_application_acce********"
+          expect(current_context[:payload_sent]).to eq({})
+          expect(current_context[:trace_id]).to eq "unknown"
+          expect(current_context[:payload_received]).to include('"diagnostics": "Invalid access token"')
+        end
+
+      end
+    end
+
+    context '422' do
+      let(:practitioner) { LifenFhir::Practitioner.new(uuid: "11e5c85e-9bc0-4c6e-9b29-deb9993a92c0") }
+
+      it 'extracts the correct context' do
+        begin
+          VCR.use_cassette "practitioner/create_channel/address/invalid_attributes" do
+            practitioner.create_address(type: "address", lines: [], city: "Paris", postal_code: "75002", country: "France")
+          end
+        rescue LifenFhir::Error => e
+
+          expect(e.to_s).to eq "422 Error, Unprocessable Entity, Diagnostic: ''line' is mandatory in address data.' -- POST 'https://develop.lifen.fr/fhir/Practitioner/11e5c85e-9bc0-4c6e-9b29-deb9993a92c0/$add-address'"
+
+          current_context = e.context
+
+          expect(current_context[:client]).to eq "app"
+          expect(current_context[:url]).to eq "https://develop.lifen.fr/fhir/Practitioner/11e5c85e-9bc0-4c6e-9b29-deb9993a92c0/$add-address"
+          expect(current_context[:method]).to eq "post"
+          expect(current_context[:token]).to eq "valid_application_access******"
+          expect(current_context[:payload_sent]["resourceType"]).to eq("Practitioner")
+          expect(current_context[:trace_id]).to eq "unknown"
+          expect(current_context[:payload_received]).to include('"diagnostics": "\'line\' is mandatory in address data."')
+        end
+      end
+    end
+
+    context '500' do
+      let(:medium)                    { LifenFhir::Medium.new(uuid: "valid-medium-uuid-1") }
+      let(:sender)                    { LifenFhir::Practitioner.new(uuid: "valid-sender-uuid") }
+      let(:recipient)                 { LifenFhir::Practitioner.new(uuid: "valid-recipient-uuid-1") }
+      let(:binary)                    { LifenFhir::Binary.new(uuid: "invalid-binary-uuid") }
+      let(:communication_request)     { LifenFhir::CommunicationRequest.new(sender: sender, recipients: [recipient], medium: [medium], binary: binary) }
+
+      it 'extracts the correct context' do
+        begin
+          VCR.use_cassette "communication_request/send/unknown_binary" do
+            communication_request.send
+          end
+        rescue LifenFhir::Error => e
+
+          expect(e.to_s).to eq "500 Error, Internal server error -- POST 'https://develop.lifen.fr/fhir/CommunicationRequest'"
+
+          current_context = e.context
+
+          expect(current_context[:client]).to eq "app"
+          expect(current_context[:url]).to eq "https://develop.lifen.fr/fhir/CommunicationRequest"
+          expect(current_context[:method]).to eq "post"
+          expect(current_context[:token]).to eq "valid_application_access******"
+          expect(current_context[:payload_sent][:resourceType]).to eq("CommunicationRequest")
+          expect(current_context[:trace_id]).to eq "unknown"
+          expect(current_context[:payload_received]).to include('"diagnostics": "Failed to call access method"')
+        end
+      end
+
+
+    end
+
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lifen_fhir
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Leonard Sellam
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-05 00:00:00.000000000 Z
+date: 2017-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -217,7 +217,6 @@ files:
 - lib/lifen_fhir/patient.rb
 - lib/lifen_fhir/practitioner.rb
 - lib/lifen_fhir/telecom.rb
-- lib/lifen_fhir/user_authenticated_client.rb
 - lib/lifen_fhir/version.rb
 - lifen_fhir.gemspec
 - spec/binary_spec.rb
@@ -226,6 +225,8 @@ files:
 - spec/cassettes/communication_request/find_by_uuid/invalid_uuid.yml
 - spec/cassettes/communication_request/find_by_uuid/valid_uuid_with_a_status.yml
 - spec/cassettes/communication_request/find_by_uuid/valid_uuid_with_no_status.yml
+- spec/cassettes/communication_request/find_by_uuid/valid_uuid_with_status_and_binary_only.yml
+- spec/cassettes/communication_request/find_by_uuid/valid_uuid_with_status_only.yml
 - spec/cassettes/communication_request/send/invalid_medium.yml
 - spec/cassettes/communication_request/send/patient/invalid_attributes_binary.yml
 - spec/cassettes/communication_request/send/patient/valid_attributes_binary.yml
@@ -234,6 +235,7 @@ files:
 - spec/cassettes/communication_request/send/unknown_recipient.yml
 - spec/cassettes/communication_request/send/valid_attributes.yml
 - spec/cassettes/communication_request/send/valid_attributes_binary.yml
+- spec/cassettes/errors/401.yml
 - spec/cassettes/logger/invalid.yml
 - spec/cassettes/logger/valid.yml
 - spec/cassettes/patient/create/invalid_attributes.yml
@@ -251,6 +253,7 @@ files:
 - spec/cassettes/practitioner/find_by_rpps/wrong_rpps.yml
 - spec/category_spec.rb
 - spec/communication_request_spec.rb
+- spec/error_spec.rb
 - spec/logger_spec.rb
 - spec/patient_spec.rb
 - spec/practitioner_spec.rb
@@ -288,6 +291,8 @@ test_files:
 - spec/cassettes/communication_request/find_by_uuid/invalid_uuid.yml
 - spec/cassettes/communication_request/find_by_uuid/valid_uuid_with_a_status.yml
 - spec/cassettes/communication_request/find_by_uuid/valid_uuid_with_no_status.yml
+- spec/cassettes/communication_request/find_by_uuid/valid_uuid_with_status_and_binary_only.yml
+- spec/cassettes/communication_request/find_by_uuid/valid_uuid_with_status_only.yml
 - spec/cassettes/communication_request/send/invalid_medium.yml
 - spec/cassettes/communication_request/send/patient/invalid_attributes_binary.yml
 - spec/cassettes/communication_request/send/patient/valid_attributes_binary.yml
@@ -296,6 +301,7 @@ test_files:
 - spec/cassettes/communication_request/send/unknown_recipient.yml
 - spec/cassettes/communication_request/send/valid_attributes.yml
 - spec/cassettes/communication_request/send/valid_attributes_binary.yml
+- spec/cassettes/errors/401.yml
 - spec/cassettes/logger/invalid.yml
 - spec/cassettes/logger/valid.yml
 - spec/cassettes/patient/create/invalid_attributes.yml
@@ -313,6 +319,7 @@ test_files:
 - spec/cassettes/practitioner/find_by_rpps/wrong_rpps.yml
 - spec/category_spec.rb
 - spec/communication_request_spec.rb
+- spec/error_spec.rb
 - spec/logger_spec.rb
 - spec/patient_spec.rb
 - spec/practitioner_spec.rb

data/lib/lifen_fhir/user_authenticated_client.rb

@@ -1,39 +0,0 @@
-module LifenFhir
-  class UserAuthenticatedClient < Client
-
-    def initialize(token)
-      @token = token
-    end
-
-    attr_reader :token
-
-    private
-
-      def handle_errors(response, params)
-        super(response, params)
-
-        case response.status
-        when 400
-          raise InvalidParamsError, "Error 400, Invalid params, #{response_error(response, params)}"
-        when 401
-          raise UnauthorizedError, "Error 401, Token is not valid, #{response_error(response, params)}"
-        when 403
-          raise Error, "Error 403, Action is forbidden, #{response_error(response, params)}"
-        end
-
-      end
-
-      def response_error(response, params)
-        "User Client, #{super(response, params)}"
-      end
-
-      def bearer
-        token.value
-      end
-
-      def before_request
-        token.refresh_once_if_needed
-      end
-
-  end
-end