From 8372ea824f501fdbbcc4d04ad151e2831d447540 Mon Sep 17 00:00:00 2001 From: Mahima Singh <105724608+smahima27@users.noreply.github.com> Date: Thu, 4 Dec 2025 16:19:34 +0530 Subject: [PATCH] Fixed spec tests --- spec/unit/pool_manager_spec.rb | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/spec/unit/pool_manager_spec.rb b/spec/unit/pool_manager_spec.rb index c7b44c0..abe5555 100644 --- a/spec/unit/pool_manager_spec.rb +++ b/spec/unit/pool_manager_spec.rb @@ -426,39 +426,39 @@ EOT end end - describe '#is_permanent_error?' do + describe '#permanent_error?' do before do expect(subject).not_to be_nil end it 'identifies template not found errors as permanent' do - expect(subject.is_permanent_error?('template not found', 'RuntimeError')).to be(true) + expect(subject.permanent_error?('template not found', 'RuntimeError')).to be(true) end it 'identifies invalid path errors as permanent' do - expect(subject.is_permanent_error?('invalid path specified', 'ArgumentError')).to be(true) + expect(subject.permanent_error?('invalid path specified', 'ArgumentError')).to be(true) end it 'identifies permission denied errors as permanent' do - expect(subject.is_permanent_error?('permission denied', 'SecurityError')).to be(true) + expect(subject.permanent_error?('permission denied', 'SecurityError')).to be(true) end it 'identifies ArgumentError class as permanent' do - expect(subject.is_permanent_error?('some argument error', 'ArgumentError')).to be(true) + expect(subject.permanent_error?('some argument error', 'ArgumentError')).to be(true) end it 'identifies network errors as transient' do - expect(subject.is_permanent_error?('connection timeout', 'Timeout::Error')).to be(false) + expect(subject.permanent_error?('connection timeout', 'Timeout::Error')).to be(false) end it 'identifies socket errors as transient' do - expect(subject.is_permanent_error?('connection refused', 'Errno::ECONNREFUSED')).to be(false) + expect(subject.permanent_error?('connection refused', 'Errno::ECONNREFUSED')).to be(false) end it 'returns false for nil inputs' do - expect(subject.is_permanent_error?(nil, nil)).to be(false) - expect(subject.is_permanent_error?('error', nil)).to be(false) - expect(subject.is_permanent_error?(nil, 'Error')).to be(false) + expect(subject.permanent_error?(nil, nil)).to be(false) + expect(subject.permanent_error?('error', nil)).to be(false) + expect(subject.permanent_error?(nil, 'Error')).to be(false) end end