Skip to content

Commit 2fc1ca9

Browse files
committed
Squiz/EmbeddedPhp: make test cases more descriptive
... to allow for easier debugging.
1 parent 6875b9e commit 2fc1ca9

File tree

2 files changed

+32
-32
lines changed

2 files changed

+32
-32
lines changed

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.inc

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -15,30 +15,30 @@
1515
<?php
1616
echo $moreBody;
1717
?>
18-
<?php echo 'hi'; ?>
19-
<?php echo 'hi'; ?>
20-
<?php echo 'hi;' ?>
21-
<?php echo 'hi'; echo 'hi;'; ?>
22-
<?php echo 'hi'; echo 'hi;'; ?>
18+
<?php echo 'one'; ?>
19+
<?php echo 'two'; ?>
20+
<?php echo 'three;' ?>
21+
<?php echo 'fourA'; echo 'fourB;'; ?>
22+
<?php echo 'fiveA'; echo 'fiveB;'; ?>
2323

2424
<?php
2525
?>
2626
<?php ?>
2727

2828
<?php
2929

30-
echo $moreBody;
30+
echo $start - $indent + $end;
3131

3232
?>
3333
<?php
3434

35-
echo $moreBody;
35+
echo $blankLines;
3636

3737
?>
3838

3939
<?php
40-
echo $moreBody; ?>
41-
<?php echo $moreBody;
40+
echo $closerNeedsOwnLine; ?>
41+
<?php echo $openerNeedsOwnLine;
4242
?>
4343

4444
<?php
@@ -108,12 +108,12 @@ function foo()
108108
<?php if ($foo) { ?>
109109
<?php } ?>
110110

111-
<?php echo 'oops'; // Something. ?>
112-
<?php echo 'oops'; // Something. ?>
113-
<?php echo 'oops'; // Something.?>
111+
<?php echo 'okay'; // Something. ?>
112+
<?php echo 'too much space before close'; // Something. ?>
113+
<?php echo 'no space before close'; // Something.?>
114114

115-
<?php /* translators: My sites label */ ?>
116-
<?php /* translators: My sites label */?>
117-
<?php /* translators: My sites label */ ?>
115+
<?php /* translators: okay */ ?>
116+
<?php /* translators: no space before close */?>
117+
<?php /* translators: too much space before close */ ?>
118118

119-
<?php echo 'oops'; // phpcs:ignore Standard.Category.Sniff -- reason. ?>
119+
<?php echo 'okay'; // phpcs:ignore Standard.Category.Sniff -- reason. ?>

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.inc.fixed

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -15,25 +15,25 @@
1515
<?php
1616
echo $moreBody;
1717
?>
18-
<?php echo 'hi'; ?>
19-
<?php echo 'hi'; ?>
20-
<?php echo 'hi;'; ?>
21-
<?php echo 'hi'; echo 'hi;'; ?>
22-
<?php echo 'hi'; echo 'hi;'; ?>
18+
<?php echo 'one'; ?>
19+
<?php echo 'two'; ?>
20+
<?php echo 'three;'; ?>
21+
<?php echo 'fourA'; echo 'fourB;'; ?>
22+
<?php echo 'fiveA'; echo 'fiveB;'; ?>
2323

2424

2525
<?php
26-
echo $moreBody;
26+
echo $start - $indent + $end;
2727
?>
2828
<?php
29-
echo $moreBody;
29+
echo $blankLines;
3030
?>
3131

3232
<?php
33-
echo $moreBody;
33+
echo $closerNeedsOwnLine;
3434
?>
3535
<?php
36-
echo $moreBody;
36+
echo $openerNeedsOwnLine;
3737
?>
3838

3939
<?php
@@ -108,12 +108,12 @@ function foo()
108108
<?php if ($foo) { ?>
109109
<?php } ?>
110110

111-
<?php echo 'oops'; // Something. ?>
112-
<?php echo 'oops'; // Something. ?>
113-
<?php echo 'oops'; // Something. ?>
111+
<?php echo 'okay'; // Something. ?>
112+
<?php echo 'too much space before close'; // Something. ?>
113+
<?php echo 'no space before close'; // Something. ?>
114114

115-
<?php /* translators: My sites label */ ?>
116-
<?php /* translators: My sites label */ ?>
117-
<?php /* translators: My sites label */ ?>
115+
<?php /* translators: okay */ ?>
116+
<?php /* translators: no space before close */ ?>
117+
<?php /* translators: too much space before close */ ?>
118118

119-
<?php echo 'oops'; // phpcs:ignore Standard.Category.Sniff -- reason. ?>
119+
<?php echo 'okay'; // phpcs:ignore Standard.Category.Sniff -- reason. ?>

0 commit comments

Comments
 (0)