* 4759 - WIP unit test for this issue
* WIP updating unit test for ResourceIdListStep
* WIP candidate fix for ResourceIdListStep batch size problem
* Enhance unit test with ArgumentCaptor.
* WIP Parameterized the ResourceIdListStepTest and added assertions
* Adding changelog for this issue
* Updating changelog message for 5055
* Code formatting and import cleanup
---------
Co-authored-by: Luke deGruchy <luke.degruchy@smilecdr.com>
* relegate surefire config to parent pom pluginManagement
* back out build fix to see if stack trace now shows up in logs
* back out the backed out fix.
---------
Co-authored-by: Ken Stevens <ken@smilecdr.com>
* extracted ResponseBundleBuilder and started writing unit tests. Currently at 71% coverage.
* asert bundle entries
* assert first entry id
* unknown size
* coverage up to 78%
* test filter nulls
* test null id check
* up to 91% test coverage
* 96% test coverage
* 98% test coverage. this is final state
* fix cyclic dep
* switch to record
* split up large method
* split up large method
* consolidate params
* consolidate params
* IntelliJ Warnings
* cleanup
* cleanup
* extract methods
* extract methods
* param names
* variable name
* formatting
* extract method
* cleanup
* javadoc
* upstream consolidation
* parameter names
* TODO
* TODO
* review feedback
---------
Co-authored-by: Ken Stevens <ken@smilecdr.com>
* Partial implementation after this has been paused: Move trimming of value and system from BaseSearchParamExtractor to ResourceIndexedSearchParamToken. Do not throw Exceptions in TokenPredicateBuilder. Ensure that the hash computing in the search param token takes the 200+ token value into account. Other search param tokens are not yet implemented.
* in ResourceIndexedSearchParamToken,changed hardcoded length to constant MAX_LENGTH;
in BaseSearchParamExtractor, added check in createTokenIndexIfNotBlank so that the return value is null if system or value is blank.
* added test for TokenPredicateBuilder to verify that long values exceeds the max length does not cause exception.
added test in FhirSystemDaoR4SearchTest to verify searchParamToken with longer system and value than max length can be created successfully. And hash value will be calculated before truncation
* fixed a logical error in previous commits
* changed existing tests to accommodate for the changes
* modified existing tests
* added changelog
* changed truncation method name and added comments
Co-authored-by: michaelabuckley <michaelabuckley@gmail.com>
* modified changelog
Co-authored-by: michaelabuckley <michaelabuckley@gmail.com>
* combined redundant tests
---------
Co-authored-by: tyner <tyner.guo@smilecdr.com>
Co-authored-by: TynerGjs <132295567+TynerGjs@users.noreply.github.com>
Co-authored-by: michaelabuckley <michaelabuckley@gmail.com>
* Reindex Batch Jobs stuck in QUEUED status after SearchParameter update - test added
* Reindex Batch Jobs stuck in QUEUED status after SearchParameter update - fix
* Reindex Batch Jobs stuck in QUEUED status after SearchParameter update - fix test
* Reindex Batch Jobs stuck in QUEUED status after SearchParameter update - send BatchJobWorkNotification after commit
* Reindex Batch Jobs stuck in QUEUED status after SearchParameter update - changelog fix
Co-authored-by: michaelabuckley <michaelabuckley@gmail.com>
---------
Co-authored-by: michaelabuckley <michaelabuckley@gmail.com>
* Allow PageMethodBinding to bind also for POST
* Add changelog
* Adjust test
---------
Co-authored-by: juan.marchionatto <juan.marchionatto@smilecdr.com>
* GenericClient POST paging support - implementation and test
* GenericClient POST paging support - fixes
* GenericClient POST paging support - fixes
* GenericClient POST paging support - failed tests fix
* GenericClient POST paging support - HAPI version bump
* Added unit test and bug fix to ensure no two partitions with the same ID can be created.
* Added changelogs, and changed the equals operator.
* Made required changes to ensure updating of partition is not affected
* Changed unit test and its location to ensure functionality.
* changing for loop to findById to avoid iteration over all partitions.
* Only allows findById when list is not empty, deals with that edge case.
* Replaced null with isPresent in if statement to fix issues regarding the creation of links on different partitions
* Removed the findAll command within the function. To avoid searching twice.
* Edited the changelogs to properly describe the issue that we have at hand
* Edited the changelogs to properly describe the issue that we have at hand pt 2, wording changes.
* log specific error message + test
* changelog
* remove draft tests
* remove another comment i left behind
* update issue number in changelog
* update error location
* update error location
* update test
* change to exception
* address review comments: add test with submit and also update change log
* format test nicer
---------
Co-authored-by: justindar <justin.dar@smilecdr.com>
* Implemented fixes, created test for JPA side
* Added test and getter for test for register listener
* refactored test names to make them more descriptive
* Modified tests
* resolved comments
* created changlog
* changed test to use MAX_MANAGED_PARAM_COUNT, and created the corresponding getter
* start building tests
* add test
* Basic test and fix bug
* remove tests, fix existing test
* fix up tests
* fix up tests
---------
Co-authored-by: Ken Stevens <ken@smilecdr.com>
* First commit: Patch bad implementation of RuleBulkExportImpl to fix this use case.
* Fix changelog. Fix comments on unit tests.
* Fix TODO with correct issue number.
* Default search mode to MATCH when creating SearchBundleEntryParts
* Add changelog
* Use enum name instead of explicit constant
---------
Co-authored-by: juan.marchionatto <juan.marchionatto@smilecdr.com>
* validate-code on remote-terminology service returns NullPointerException - tests
* validate-code on remote-terminology service returns NullPointerException - fixed
* validate-code on remote-terminology service returns NullPointerException - added changelog
* validate-code on remote-terminology service returns NullPointerException - tests readability improvement and fixes
* validate-code on remote-terminology service returns NullPointerException - fixes
* Ensure StepExecutor only logs a stack trace and error if the retries have been exhausted. Otherwise, just log a debug.
* Accept code reviewer suggestion to amend log message
Co-authored-by: michaelabuckley <michaelabuckley@gmail.com>
---------
Co-authored-by: michaelabuckley <michaelabuckley@gmail.com>
* added failing test
* added support for warning messages to batch2
* added change log
* fixed errors from merge, made warning work with new batch2
* fixed message parse, changed interface method to default, formatting changes
* added changelog
* code review changes
* code review change
* added new implementation for warning message processing, fixed migration
* changed column to lob, as text is not well supported
* code review changes
---------
Co-authored-by: Steven Li <steven@smilecdr.com>
* Unit tests for validation after install and uninstall IG
* Fix validation result cached after install and uninstall IG
* Fix validation result cached after install and uninstall IG - updated changelog
* Fix validation result cached after uninstalling IG - updated changelog
* Fix validation result cached after uninstalling IG - fixed unit tests
* Fix validation result cached after uninstalling IG - fixed changelog
* Fix validation result cached after uninstalling IG - test fixes
Tadgh
Brenin Rhodes
Steve Corbett
JasonRoberts-smile
James Agnew
Ken Stevens
Thomas Papke
Yusuf Daniju
Luke deGruchy
volodymyr-korzh
jmarchionatto
TipzCM
StevenXLi
LalithE
jdar8
TynerGjs
dotasek
michaelabuckley
dependabot[bot]