Skip to content

Commit 5415081

Browse files
yahondaclaude
andcommitted
Fix CI failure in TIMESTAMP WITH LOCAL TIME ZONE test
Remove unnecessary function execution that triggered ensure_tmp_tables_created, which is not needed to verify the type remapping in metadata. Also make setup idempotent. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent dcca897 commit 5415081

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

spec/plsql/procedure_spec.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2389,6 +2389,8 @@ def new_candidate(status)
23892389
describe "Function with table type containing TIMESTAMP WITH LOCAL TIME ZONE" do
23902390
before(:all) do
23912391
plsql.connect! CONNECTION_PARAMS
2392+
plsql.execute "DROP PACKAGE test_timestamp_ltz_pkg" rescue nil
2393+
plsql.execute "DROP TABLE test_timestamp_ltz" rescue nil
23922394
plsql.execute "CREATE TABLE test_timestamp_ltz (id NUMBER, ts TIMESTAMP WITH LOCAL TIME ZONE)"
23932395
plsql.execute <<-SQL
23942396
CREATE OR REPLACE PACKAGE test_timestamp_ltz_pkg IS
@@ -2413,7 +2415,6 @@ def new_candidate(status)
24132415
end
24142416

24152417
it "should remap TIMESTAMP(n) WITH LOCAL TIME ZONE to TIMESTAMP WITH LOCAL TIME ZONE" do
2416-
metadata = plsql.test_timestamp_ltz_pkg.test_fn.arguments
24172418
return_metadata = PLSQL::Procedure.find(plsql, :test_fn, "TEST_TIMESTAMP_LTZ_PKG")
24182419
fields = return_metadata.return[0][:element][:fields]
24192420
expect(fields[:ts][:data_type]).to eq("TIMESTAMP WITH LOCAL TIME ZONE")

0 commit comments

Comments
 (0)