Api 42322 extract org web service bean from local bgs

[mchristiansonVA]

Summary

• This work is behind a feature toggle (flipper): NO
• Extracts OrgWebService into its own file.
• Updates User to use OrgWebService for calls to findPoaHistoryByPtcpntId
• Updates User and POA specs.

Related issue(s)

• API-42322

Testing done

• New code is covered by unit tests
• Tested local calls to POA v1 & v2
• Rspec

What areas of the site does it impact?

POA v1 & v2 calls to BGS findPoaHistoryByPtcpntId updated to use OrgWebService

Acceptance criteria

• I fixed|updated|added unit tests and integration tests for each feature (if applicable).
• No error nor warning in the console.
• Events are being sent to the appropriate logging solution
• Documentation has been updated (link to documentation)
• No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
• Feature/bug has a monitor built into Datadog (if applicable)
• If app impacted requires authentication, did you login to a local build and verify all authenticated routes work as expected
• I added a screenshot of the developed feature

[stiehlrod]

@mchristiansonVA Did you check the Veteran module for usage of the org web service? @rockwellwindsor-va pointed this out to me on another PR.

[FonzMP]

we're calling for the WSDL because of the lack of Service / Bean in the definitions file. Should be able to replace theOrgWebService with the below. Starts at :136

      ##
      # OrgWebServiceBean
      #
      module OrgWebServiceBean
        DEFINITION =
          Bean.new(
            path: 'OrgWebServiceBean',
            namespaces: Namespaces.new(
              target: 'http://org.services.vetsnet.vba.va.gov/',
              data: nil
            )
          )
      end

      module OrgWebService
        DEFINITION =
          Service.new(
            bean: OrgWebServiceBean::DEFINITION,
            path: 'OrgWebService'
          )
      end

[FonzMP]

can use bean_name here for the endpoint so it's changeable in one place.

[FonzMP]

can use bean_name here for the endpoint so it's changeable in one place.

Would be a nice fix but not critical to functionality.

[FonzMP]