Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix fatal syntax error "unexpected '-', expecting '{'" #3217

Merged
merged 1 commit into from Aug 7, 2018
Merged

Fix fatal syntax error "unexpected '-', expecting '{'" #3217

merged 1 commit into from Aug 7, 2018

Conversation

Hecke29
Copy link
Contributor

@Hecke29 Hecke29 commented Jul 18, 2018

caused by special chars in wsdl file name.

This is the fix, you can find the tests here: sebastianbergmann/phpunit-mock-objects#424
Im sorry but the fix might be best placed here while the tests for it are in another project :(

@codecov-io
Copy link

codecov-io commented Jul 18, 2018

Codecov Report

Merging #3217 into 6.5 will decrease coverage by 0.01%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##                6.5    #3217      +/-   ##
============================================
- Coverage     80.17%   80.16%   -0.02%     
  Complexity     2850     2850              
============================================
  Files           105      105              
  Lines          7520     7521       +1     
============================================
  Hits           6029     6029              
- Misses         1491     1492       +1
Impacted Files Coverage Δ Complexity Δ
src/Framework/TestCase.php 66.31% <0%> (-0.08%) 319 <0> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36101dc...0d5debc. Read the comment docs.

Hecke29 added a commit to webfoersterei/domain-bestellsystem-api-client that referenced this pull request Jul 18, 2018
@sebastianbergmann
Copy link
Owner

The tests are in this project, just in a different branch. phpunit-mock-objects was merged into this project starting with PHPUnit 7.2.

@Hecke29
Copy link
Contributor Author

Hecke29 commented Jul 19, 2018

will fix this in phpunit 7.2 (mock-object project was merged into main project):
#3219

@Hecke29 Hecke29 closed this Jul 19, 2018
@sebastianbergmann sebastianbergmann merged commit d64306d into sebastianbergmann:6.5 Aug 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants