Skip to content

Commit

Permalink
t/translator.t: use instance() method, not new()
Browse files Browse the repository at this point in the history
Pull request zonemaster-engine#1347
(zonemaster/zonemaster-engine#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.
  • Loading branch information
marc-vanderwal committed Jun 3, 2024
1 parent 2e0e172 commit 39b6ea2
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions t/translator.t
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 39b6ea2

Please sign in to comment.