Skip to content

Commit

Permalink
qa: add another false positive to Psalm baseline
Browse files Browse the repository at this point in the history
Several tests have ".dist" config files that a setUp process copies to a path omitting the suffix.
That file is then _included_ by the test in order to do assertions.
Psalm was flagging the include as invalid due to the file not existing in a clean setup (which it will not).

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
  • Loading branch information
weierophinney committed Jul 12, 2023
1 parent 765cb25 commit a56d200
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions psalm-baseline.xml
Original file line number Diff line number Diff line change
Expand Up @@ -5134,6 +5134,9 @@
<DocblockTypeContradiction occurrences="1">
<code>empty($this-&gt;modulePathSpec)</code>
</DocblockTypeContradiction>
<MissingFile occurrences="1">
<code>include $path . '/config/module.config.php'</code>
</MissingFile>
<MixedArgument occurrences="25">
<code>$configData-&gt;config_file</code>
<code>$configData-&gt;config_file</code>
Expand Down

0 comments on commit a56d200

Please sign in to comment.