Skip to content

Fix Travis failing tests #998

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

Closed
javiereguiluz opened this issue Jun 10, 2019 · 2 comments · Fixed by #1001
Closed

Fix Travis failing tests #998

javiereguiluz opened this issue Jun 10, 2019 · 2 comments · Fixed by #1001

Comments

@javiereguiluz
Copy link
Member

This is our Travis CI test suite now:

image

The failing tests look like this:

image

Full details: https://travis-ci.org/symfony/demo/jobs/543634635

This is NOT related to #997. It was failing way before. Anyone can help us fix this? Thanks!

@dmaicher
Copy link
Contributor

Currently looking into this. Can reproduce it locally using php 7.1.30

@dmaicher
Copy link
Contributor

@javiereguiluz fixed in #1001

javiereguiluz added a commit that referenced this issue Jun 11, 2019
This PR was merged into the master branch.

Discussion
----------

revert fixtures db change

This fixes #998

The issue was that the fixture db data contained user passwords using `argon2` encoder algo which is not implemented in php 7.1 afaik. So authentication failed in the tests.

So this reverts the db changes done in 6c32238.

Build is green now 😉

Commits
-------

a3fb81b revert fixtures db change
@javiereguiluz javiereguiluz unpinned this issue Jun 11, 2019
sayjun0505 added a commit to sayjun0505/sym_proj that referenced this issue Apr 16, 2023
This PR was merged into the master branch.

Discussion
----------

revert fixtures db change

This fixes symfony/demo#998

The issue was that the fixture db data contained user passwords using `argon2` encoder algo which is not implemented in php 7.1 afaik. So authentication failed in the tests.

So this reverts the db changes done in symfony/demo@6c32238.

Build is green now 😉

Commits
-------

a3fb81b revert fixtures db change
spider-yamet added a commit to spider-yamet/sym_proj that referenced this issue Apr 16, 2023
This PR was merged into the master branch.

Discussion
----------

revert fixtures db change

This fixes symfony/demo#998

The issue was that the fixture db data contained user passwords using `argon2` encoder algo which is not implemented in php 7.1 afaik. So authentication failed in the tests.

So this reverts the db changes done in symfony/demo@6c32238.

Build is green now 😉

Commits
-------

a3fb81b revert fixtures db change
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 a pull request may close this issue.

2 participants