Build: #1199 was successful Scheduled with changes by 4 people

Code commits

openmrs-core master

  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> e09fe8da30de6f004cb18d72d9284276c360c0bd

    Merge pull request #2151 from birgitta410/TRUNK-5069-4
    Starting tests for PatientServiceImpl.processDeath

  • Birgitta B <bboeckel@thoughtworks.com>

    Birgitta B <bboeckel@thoughtworks.com> 15b0f7ae56e8f5856f745c1c2780b9cff64ec0ee

    Starting tests for PatientServiceImpl.processDeath
    It's just too much to test this without the ability to mock Context - so using PowerMock runner for this one, although that will prevent jacoco from realising that the test coverage increased...

    • api/src/test/java/org/openmrs/api/PatientServiceImplUnitTest.java (version 15b0f7ae56e8f5856f745c1c2780b9cff64ec0ee)
    • api/src/test/java/org/openmrs/api/PatientServiceTest.java (version 15b0f7ae56e8f5856f745c1c2780b9cff64ec0ee)
    • api/src/test/java/org/openmrs/api/impl/PatientServiceImplTest.java (version 15b0f7ae56e8f5856f745c1c2780b9cff64ec0ee)
  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> c0514cca01d29f8f92cfaee238847a709a0fd6f3

    Merge pull request #2150 from teleivo/TRUNK-5064
    TRUNK-5064 Add test to PersonServiceImpl

  • teleivo <mrteleivo@gmail.com>

    teleivo <mrteleivo@gmail.com> 643fe740eb23294e256693e78e8511ee74d4df0f

    TRUNK-5064 Add test to PersonServiceImpl
    add test for unretireRelationshipType

    • api/src/test/java/org/openmrs/api/PersonServiceTest.java (version 643fe740eb23294e256693e78e8511ee74d4df0f)
  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> 16286e67f7183ca67df9c5ea1bd874d4a8796d01

    Merge pull request #2148 from teleivo/TRUNK-5117
    TRUNK-5117 Refactor PersonServiceImpl.getPersonAttributeTypes

  • teleivo <mrteleivo@gmail.com>

    teleivo <mrteleivo@gmail.com> f2e8d0e13458c4a037805fc3ea9a68dc6076a498

    TRUNK-5117 Refactor PersonServiceImpl.getPersonAttributeTypes
    * remove unnecessary check for empty attrNames since thats handled by
    the for loop
    * use getPersonAttributeType without getting it from the Context as
    also done for getPersonAttributeTypeByName

    • api/src/main/java/org/openmrs/api/impl/PersonServiceImpl.java (version f2e8d0e13458c4a037805fc3ea9a68dc6076a498)
  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> 9b85c7a32eddf9a761a87883823c91ae358d3dd0

    Merge pull request #2147 from teleivo/TRUNK-5117
    TRUNK-5117 Refactor PersonServiceImpl.getPersonAttributeTypes

  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> a08217036501a06e4ee5f1deecf32fbe0462889a

    Merge pull request #2146 from birgitta410/TRUNK-5069-3
    Refactor checkPatientIdentifiers: 10 years later, Java8 streams \o/

  • teleivo <mrteleivo@gmail.com>

    teleivo <mrteleivo@gmail.com> e8745aa482e83749526d640162d2172355439ee0

    TRUNK-5117 Refactor PersonServiceImpl.getPersonAttributeTypes
    * make helper method private, was not intentional
    * rename getGlobalProperty to plural since thats what it does
    * rename local variable to result to emphasize it gets returned
    * replace local var type Vector with ArrayList

    • api/src/main/java/org/openmrs/api/impl/PersonServiceImpl.java (version e8745aa482e83749526d640162d2172355439ee0)
  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> 003ef8cf142679fb584a7d04faed5dfc29a2c958

    Merge pull request #2145 from fruether/TRUNK-5064
    TRUNK-5117 Refactor PersonServiceImpl.getPersonAttributeTypes

  • freddy <fruether>

    freddy <fruether> fcdef64b636c1473159d8dab7c2149dcef090535

    TRUNK-5117 Refactor PersonServiceImpl.getPersonAttributeType

    • api/src/main/java/org/openmrs/api/impl/PersonServiceImpl.java (version fcdef64b636c1473159d8dab7c2149dcef090535)
  • Birgitta B <bboeckel@thoughtworks.com>

    Birgitta B <bboeckel@thoughtworks.com> 5964330d4326da889448883d55413285c0fc267d

    Refactor search for missing identifiers: 10 years later, Java8 streams \o/
    No more lists that get filled in a for loop

    • api/src/main/java/org/openmrs/api/impl/PatientServiceImpl.java (version 5964330d4326da889448883d55413285c0fc267d)
    • api/src/test/java/org/openmrs/api/PatientServiceImplUnitTest.java (version 5964330d4326da889448883d55413285c0fc267d)
    • api/src/test/java/org/openmrs/api/PatientServiceTest.java (version 5964330d4326da889448883d55413285c0fc267d)
  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> 5f7588610e3abb8aee9b5ca0aec4be40ce96d7b6

    Merge pull request #2142 from birgitta410/TRUNK-5069-2
    Write tests for PatientServiceImpl.checkPatientIdentifiers

  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> 40276e1f83aa02270dbb654b8ba78951d1a24689

    Merge pull request #2144 from fruether/test
    TRUNK-5064 Add tests to PersonServiceImpl

  • freddy <fruether>

    freddy <fruether> a11702bc405f0c6b821428e1db362808f0221593

    TRUNK-5064 Add tests to PersonServiceImpl
    Added tests to show the bugy TRUNK-5149 and TUNK-5148

    • api/src/test/java/org/openmrs/api/PersonServiceTest.java (version a11702bc405f0c6b821428e1db362808f0221593)
    • api/src/test/resources/org/openmrs/api/include/PersonServiceTest-PersonAttributeType.xml (version a11702bc405f0c6b821428e1db362808f0221593)
  • Birgitta B <bboeckel@thoughtworks.com>

    Birgitta B <bboeckel@thoughtworks.com> bd32c99aa62fdfc315c2d0d9c6822a53c9f27ea8

    Write tests for PatientServiceImpl.checkPatientIdentifiers
    Improves coverage slightly

    • api/src/main/java/org/openmrs/api/impl/PatientServiceImpl.java (version bd32c99aa62fdfc315c2d0d9c6822a53c9f27ea8)
    • api/src/test/java/org/openmrs/api/PatientServiceImplUnitTest.java (version bd32c99aa62fdfc315c2d0d9c6822a53c9f27ea8)
    • api/src/test/java/org/openmrs/api/PatientServiceTest.java (version bd32c99aa62fdfc315c2d0d9c6822a53c9f27ea8)
  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> c2f7c0d594e513b546f39f1e7e6bcfa82b6a3c6d

    Merge pull request #2141 from teleivo/TRUNK-5064
    TRUNK-5064 Add test to PersonServiceImpl

  • teleivo <mrteleivo@gmail.com>

    teleivo <mrteleivo@gmail.com> a91ee405f14b03614ec89f7123c873f3b469a3a2

    TRUNK-5064 Add test to PersonServiceImpl
    added tests for retireRelationshipType

    • api/src/test/java/org/openmrs/api/PersonServiceTest.java (version a91ee405f14b03614ec89f7123c873f3b469a3a2)
  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> ecfe01efeef9fe79fefb64cc4d254c4f25010b2b

    Merge pull request #2140 from fruether/TRUNK-5064
    TRUNK-5064 Add tests to PersonServiceImpl

  • freddy <fruether>

    freddy <fruether> 0ec45d50ae5e550874cee70f14cf416a1760a19a

    TRUNK-5064 Add tests to PersonServiceImpl

    • api/src/test/java/org/openmrs/api/PersonServiceTest.java (version 0ec45d50ae5e550874cee70f14cf416a1760a19a)
  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> 7ce201ac7adc5a5e84ad335e3efca8f480fdbe17

    Merge pull request #2139 from fruether/TRUNK-5064
    TRUNK-5064 Add tests to PersonServiceImpl

  • freddy <fruether>

    freddy <fruether> 1434a0409ec7e736405f7ac8128497b72c12a62f

    TRUNK-5064 Add tests to PersonServiceImpl

    • api/src/test/java/org/openmrs/api/PersonServiceTest.java (version 1434a0409ec7e736405f7ac8128497b72c12a62f)
  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> 7fef9da77042736b2b4dcce7ae340f0861638837

    Merge pull request #2138 from fruether/TRUNK-5064
    TRUNK-5064 Add tests to PersonServiceImpl

  • freddy <fruether>

    freddy <fruether> 3c5ab54ad6ea3624f1afb8373b968a2a632be679

    TRUNK-5064 Add tests to PersonServiceImpl
    TRUNK-5064 Add tests to PersonServiceImpl

    • api/src/test/java/org/openmrs/api/PersonServiceTest.java (version 3c5ab54ad6ea3624f1afb8373b968a2a632be679)
  • Ivo Ulrich <teleivo@users.noreply.github.com>

    Ivo Ulrich <teleivo@users.noreply.github.com> 3efee206ceae5d0dd41d9f3f8138f8895754e91c

    Merge pull request #2137 from birgitta410/TRUNK-5069
    Add unit tests for getDuplicatePatientsByAttributes calls without att…

  • Birgitta B <bboeckel@thoughtworks.com>

    Birgitta B <bboeckel@thoughtworks.com> b8d1ea53779c668db91596be7619ea43ad6904e7

    Add unit tests for getDuplicatePatientsByAttributes calls without attributes

    • api/src/test/java/org/openmrs/api/PatientServiceTest.java (version b8d1ea53779c668db91596be7619ea43ad6904e7)