pulibrary / allsearch_api

100%

main: 100%

LAST BUILD ON BRANCH rom-rb-2

branch: rom-rb-2

CHANGE BRANCH

x

Reset

• rom-rb-2
• 377-multiple-onliine-results
• add-rake-to-gemfile
• bundle-update-06162025
• catalog-rack-app
• coverage
• dependabot/bundler/activerecord-8.0.2.1
• dependabot/bundler/nokogiri-1.18.9
• dependabot/bundler/rack-3.1.14
• dependabot/bundler/rack-3.1.16
• dependabot/bundler/rack-3.1.17
• dependabot/bundler/rack-3.1.18
• dependabot/bundler/rack-session-2.1.1
• dependabot/bundler/rexml-3.4.2
• dependabot/bundler/thor-1.4.0
• dpul-rack
• engineering_search_spike
• fqdn_deploy_files
• i360_staff_name_link
• i371_engineering_librarian_bug
• main
• migrate-some-more-rails-controllers
• missing-gem
• pry-byebug
• remove-action-view
• replace-all-controllers
• rubocop-plugins
• sandbergja-patch-1
• self_hosted_runner_experiment
• semantic_logger

Build # 65c01928-9796-4303-af7f-1820b3d43cfe

Build Type

Pull #389

circleci

Committed by sandbergja

Commit Message

Replace uses of the Banner ActiveRecord model with Rom-RB

The ActiveRecord model is still used in tests and a database migration,
maybe those could be migrated to Rom-RB in a separate PR?

A Rom Container is created that uses the same database connections as
ActiveRecord.  This allows rom to use the same database transactions
as activerecord, allowing tests (which use db transactions to stay
isolated from one another) to still pass.

Pull Request Pull Request #389: Replace uses of the Banner ActiveRecord model with Rom-RB

Run Details

39 of 39 new or added lines in 4 files covered. (100.0%)

1060 of 1065 relevant lines covered (99.53%)

89.5 hits per line