Fix Ruby warnings in specs

- Wrap splat arguments in parentheses (ambiguous `*`)
- Wrap regexp in parentheses for raise_error (ambiguous `/`)
- Use .all.each instead of .all with block (block was ignored)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: yahonda

spec/plsql/procedure_spec.rb

@@ -1966,7 +1966,7 @@ def new_candidate(status)
 
       @fields = [:col1, :col2 ]
       @rows = (1..3).map { |i| ["row #{i}", i] }
-      plsql.typed_ref_cursor_table.insert_values *@rows
+      plsql.typed_ref_cursor_table.insert_values(*@rows)
       plsql.commit
 
     end

spec/plsql/schema_spec.rb

@@ -304,7 +304,7 @@ class TestModel < TestBaseModel
     end
 
     it "should log output to specified stream in case of exception" do
-      expect { plsql.test_dbms_output("test_dbms_output", true) }.to raise_error /Test Error/
+      expect { plsql.test_dbms_output("test_dbms_output", true) }.to raise_error(/Test Error/)
       expect(@buffer.string).to eq("DBMS_OUTPUT: test_dbms_output\n")
     end
 

spec/plsql/table_spec.rb

@@ -239,7 +239,7 @@
     end
 
     it "should insert many records with array of values" do
-      plsql.test_employees.insert_values *@employees_all_values
+      plsql.test_employees.insert_values(*@employees_all_values)
       expect(plsql.test_employees.all).to eq(@employees)
     end
 
@@ -332,15 +332,15 @@
       plsql.test_employees.update first_name: "Test", where: "employee_id = #{employee_id}"
       expect(plsql.test_employees.first(employee_id: employee_id)[:first_name]).to eq("Test")
       # all other records should not be changed
-      plsql.test_employees.all("WHERE employee_id > :1", employee_id) do |employee|
+      plsql.test_employees.all("WHERE employee_id > :1", employee_id).each do |employee|
         expect(employee[:first_name]).not_to eq("Test")
       end
     end
 
     it "should update all records in table" do
       plsql.test_employees.insert @employees
       plsql.test_employees.update first_name: "Test"
-      plsql.test_employees.all do |employee|
+      plsql.test_employees.all.each do |employee|
         expect(employee[:first_name]).to eq("Test")
       end
     end

spec/plsql/view_spec.rb

@@ -157,7 +157,7 @@
     end
 
     it "should insert many records with array of values" do
-      plsql.test_employees_v.insert_values *@employees_all_values
+      plsql.test_employees_v.insert_values(*@employees_all_values)
       expect(plsql.test_employees_v.all).to eq(@employees)
     end
 
@@ -230,15 +230,15 @@
       plsql.test_employees_v.update first_name: "Test", where: "employee_id = #{employee_id}"
       expect(plsql.test_employees_v.first(employee_id: employee_id)[:first_name]).to eq("Test")
       # all other records should not be changed
-      plsql.test_employees_v.all("WHERE employee_id > :1", employee_id) do |employee|
+      plsql.test_employees_v.all("WHERE employee_id > :1", employee_id).each do |employee|
         expect(employee[:first_name]).not_to eq("Test")
       end
     end
 
     it "should update all records in view" do
       plsql.test_employees_v.insert @employees
       plsql.test_employees_v.update first_name: "Test"
-      plsql.test_employees_v.all do |employee|
+      plsql.test_employees_v.all.each do |employee|
         expect(employee[:first_name]).to eq("Test")
       end
     end