Skip to content

Cover insert_fixture for CPK + CLOB#2699

Closed
yahonda wants to merge 1 commit into
rsim:masterfrom
yahonda:add-cpk-lob-insert-fixture-spec
Closed

Cover insert_fixture for CPK + CLOB#2699
yahonda wants to merge 1 commit into
rsim:masterfrom
yahonda:add-cpk-lob-insert-fixture-spec

Conversation

@yahonda

@yahonda yahonda commented May 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds the spec case originally listed as test #4 under the umbrella
issue #2694:

insert_fixture against a CPK + LOB table (covers the insert_fixture
call site).

The case was deferred when the umbrella PR landed because the
insert_fixture LOB write was independently broken with
ORA-01002: fetch out of sequence (#2696). That bug was fixed in
#2698 with a transaction wrap. Both fixes are now on master, so the
CPK + CLOB insert_fixture path is exercisable end-to-end.

Spec

A single it added to the existing CLOB / BLOB columns describe
block in composite_primary_key_spec.rb, reusing the cpk_docs /
CpkDoc scaffolding that #2697 already set up. No production code
changes.

Test plan

  • bundle exec rubocop
  • bundle exec rspec spec/active_record/connection_adapters/oracle_enhanced/composite_primary_key_spec.rb (43/43)
  • ORACLE_ENHANCED_PREPARED_STATEMENTS_FALSE=1 bundle exec rspec spec/active_record/connection_adapters/oracle_enhanced/composite_primary_key_spec.rb (43/43)

PR rsim#2697 added the cpk_docs setup and CPK + LOB create! coverage but
deferred insert_fixture, which was failing for an unrelated reason
later fixed by rsim#2698 (transaction wrap). Both fixes are now on master,
so add the missing case the umbrella issue listed.
@yahonda yahonda closed this May 5, 2026
@yahonda yahonda deleted the add-cpk-lob-insert-fixture-spec branch May 5, 2026 22:15
@yahonda yahonda mentioned this pull request May 5, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant