Skip to content

Commit 2da0d46

Browse files
committed
Comment out goto_construct tests
These tests are no longer appropriate given that this deprecation is being upgraded to an exception. For the time being, I'm simply commenting out the tests rather than deleting them, in the hope that we can replace them with tests of some other deprecation-related warning (which I have yet to locate).
1 parent 544e4b4 commit 2da0d46

File tree

1 file changed

+53
-53
lines changed

1 file changed

+53
-53
lines changed

t/porting/deprecation.t

Lines changed: 53 additions & 53 deletions
Original file line numberDiff line numberDiff line change
@@ -90,56 +90,56 @@ if (-e ".git") {
9090
"There should not be any new files which mention WARN_DEPRECATED");
9191
}
9292

93-
# Test that deprecation warnings are produced under "use warnings"
94-
# (set above)
95-
{
96-
my $warning = "nada";
97-
local $SIG{__WARN__} = sub { $warning = $_[0] };
98-
my $count = 0;
99-
while ($count<1) {
100-
LABEL: $count++;
101-
goto DONE if $count>1;
102-
}
103-
goto LABEL;
104-
DONE:
105-
like($warning,
106-
qr/Use of "goto" to jump into a construct is deprecated, and will become fatal in Perl 5\.42/,
107-
"Got expected deprecation warning");
108-
}
109-
# Test that we can silence deprecation warnings with "no warnings 'deprecated'"
110-
# as we used to.
111-
{
112-
no warnings 'deprecated';
113-
my $warning = "nada";
114-
local $SIG{__WARN__} = sub { $warning = $_[0] };
115-
my $count = 0;
116-
while ($count<1) {
117-
LABEL: $count++;
118-
goto DONE if $count>1;
119-
}
120-
goto LABEL;
121-
DONE:
122-
like($warning, qr/nada/,
123-
"no warnings 'deprecated'; silenced deprecation warning as expected");
124-
}
125-
126-
# Test that we can silence a specific deprecation warnings with "no warnings 'deprecated::$subcategory'"
127-
# and that by doing so we don't silence any other deprecation warnings.
128-
{
129-
no warnings 'deprecated::goto_construct';
130-
my $warning = "nada";
131-
local $SIG{__WARN__} = sub { $warning = $_[0] };
132-
my $count = 0;
133-
while ($count<1) {
134-
LABEL: $count++;
135-
goto DONE if $count>1;
136-
}
137-
goto LABEL;
138-
DONE:
139-
like($warning, qr/nada/,
140-
"no warnings 'deprecated::goto_construct'; silenced deprecation warning as expected");
141-
@INC = ();
142-
do "regen.pl"; # this should produce a deprecation warning
143-
like($warning, qr/is no longer in \@INC/,
144-
"no warnings 'deprecated::goto_construct'; did not silence deprecated::dot_in_inc warnings");
145-
}
93+
## Test that deprecation warnings are produced under "use warnings"
94+
## (set above)
95+
#{
96+
# my $warning = "nada";
97+
# local $SIG{__WARN__} = sub { $warning = $_[0] };
98+
# my $count = 0;
99+
# while ($count<1) {
100+
# LABEL: $count++;
101+
# goto DONE if $count>1;
102+
# }
103+
# goto LABEL;
104+
# DONE:
105+
# like($warning,
106+
# qr/Use of "goto" to jump into a construct is deprecated, and will become fatal in Perl 5\.42/,
107+
# "Got expected deprecation warning");
108+
#}
109+
## Test that we can silence deprecation warnings with "no warnings 'deprecated'"
110+
## as we used to.
111+
#{
112+
# no warnings 'deprecated';
113+
# my $warning = "nada";
114+
# local $SIG{__WARN__} = sub { $warning = $_[0] };
115+
# my $count = 0;
116+
# while ($count<1) {
117+
# LABEL: $count++;
118+
# goto DONE if $count>1;
119+
# }
120+
# goto LABEL;
121+
# DONE:
122+
# like($warning, qr/nada/,
123+
# "no warnings 'deprecated'; silenced deprecation warning as expected");
124+
#}
125+
#
126+
## Test that we can silence a specific deprecation warnings with "no warnings 'deprecated::$subcategory'"
127+
## and that by doing so we don't silence any other deprecation warnings.
128+
#{
129+
# no warnings 'deprecated::goto_construct';
130+
# my $warning = "nada";
131+
# local $SIG{__WARN__} = sub { $warning = $_[0] };
132+
# my $count = 0;
133+
# while ($count<1) {
134+
# LABEL: $count++;
135+
# goto DONE if $count>1;
136+
# }
137+
# goto LABEL;
138+
# DONE:
139+
# like($warning, qr/nada/,
140+
# "no warnings 'deprecated::goto_construct'; silenced deprecation warning as expected");
141+
# @INC = ();
142+
# do "regen.pl"; # this should produce a deprecation warning
143+
# like($warning, qr/is no longer in \@INC/,
144+
# "no warnings 'deprecated::goto_construct'; did not silence deprecated::dot_in_inc warnings");
145+
#}

0 commit comments

Comments
 (0)