From 39b6ea23d9dede37dc2e2fb14c64704f5747f47f Mon Sep 17 00:00:00 2001 From: Marc van der Wal Date: Mon, 3 Jun 2024 15:04:33 +0200 Subject: [PATCH] t/translator.t: use instance() method, not new() Pull request zonemaster-engine#1347 (https://github.com/zonemaster/zonemaster-engine/pull/1347) has been merged, which fixes Zonemaster::Backend::Translator->instance(). So we can remove one use of the deprecated Zonemaster::Backend::Translator->new() in the unit tests. --- t/translator.t | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/t/translator.t b/t/translator.t index a7f78168..1e0b9690 100644 --- a/t/translator.t +++ b/t/translator.t @@ -16,12 +16,9 @@ isa_ok 'Zonemaster::Backend::Translator', 'Zonemaster::Engine::Translator'; my $translator; -# FIXME: the new() method is deprecated but the instance() method does not -# work; the following should be changed once Zonemaster::Engine::Translator -# (Engine, not Backend) is fixed. -$translator = Zonemaster::Backend::Translator->new(); +$translator = Zonemaster::Backend::Translator->instance(); isa_ok $translator, 'Zonemaster::Backend::Translator', - "Zonemaster::Backend::Translator->new()"; + "Zonemaster::Backend::Translator->instance()"; ### ### Change locale