Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
C
composer-phpcodesniffer-standards-plugin
Manage
Activity
Members
Labels
Plan
Issues
8
Issue boards
Milestones
Code
Merge requests
2
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Operate
Environments
Monitor
Incidents
Service Desk
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Daniel Hürtgen
composer-phpcodesniffer-standards-plugin
Commits
13927573
There was a problem fetching the pipeline stages.
Commit
13927573
authored
7 years ago
by
Daniel Hürtgen
Browse files
Options
Downloads
Patches
Plain Diff
TASK: Optimize recognizing last test pass status
parent
c16e8bfe
Branches
44-composer-plugin-is-not-always-triggered
1 merge request
!45
WIP: Resolve "Composer plugin is not always triggered"
Pipeline
#397
failed with stages
in 8 minutes and 57 seconds
Changes
1
Pipelines
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
tests/Functional/ComposerTestCase.php
+9
-1
9 additions, 1 deletion
tests/Functional/ComposerTestCase.php
with
9 additions
and
1 deletion
tests/Functional/ComposerTestCase.php
+
9
−
1
View file @
13927573
...
...
@@ -55,13 +55,21 @@ class ComposerTestCase extends TestCase
{
parent
::
tearDown
();
chdir
(
$this
->
oldWorkingDirectory
);
if
(
$this
->
testWorkingDir
&&
!
$this
->
has
Fail
ed
())
{
if
(
$this
->
testWorkingDir
&&
$this
->
has
Pass
ed
())
{
$fs
=
new
Filesystem
();
$fs
->
remove
(
$this
->
testWorkingDir
);
}
unset
(
$this
->
oldWorkingDirectory
,
$this
->
testWorkingDir
);
}
/**
* @return bool
*/
protected
function
hasPassed
()
{
return
$this
->
getStatus
()
===
\PHPUnit_Runner_BaseTestRunner
::
STATUS_PASSED
;
}
/**
* @return string
*/
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment