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

Merge WB develop branch and fix behat tests #132

Open
dasistwas opened this issue Jun 15, 2020 · 4 comments
Open

Merge WB develop branch and fix behat tests #132

dasistwas opened this issue Jun 15, 2020 · 4 comments
Assignees

Comments

@dasistwas
Copy link
Member

I have done some modification in travis.yml. Behat now breaks. Could you merge the branch and try to fix the behat tests?

@michaelpollak
Copy link
Collaborator

PHP7.2 seems to fail because of errors in the moodle ci build matrix, out of our control as far as I was able to determine. I created a test array that currently stays with PHP7.1 but to test mdl3.9 we have to find a solution that works with >= PHP 7.2

MDL3.6 also fails but that is known for a while.

MDL3.8 is throwning errors that were not there last time I ran the tests, will look into this.

@dasistwas
Copy link
Member Author

OK, let's focus on the tests where behat actually runs:
https://travis-ci.org/github/Wunderbyte-GmbH/moodle-mod_datalynx/jobs/698603920

For Moodle 35 I think we can not test with PHP7.1 so maybe we replace the test with php7.2 and M35

@dasistwas
Copy link
Member Author

PHP 7.4 works with M3.8 and M3.9

@michaelpollak
Copy link
Collaborator

Travis just is no fun.
So our tests work, I went back to the trusty VM because on bionic somehow 3.7 and 3.8 failed.
3.5 works with PHP7.1 even under bionic. Will test some more, it's mainly pushing and a lot of waiting...

https://travis-ci.org/github/michaelpollak/moodle-mod_datalynx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants