search

OXIDprojects / oxid_modules_config

OXID Module Configuration Im-/Exporter module by OXID Professional Services
GNU General Public License v3.0
3 stars 9 forks source link

execute tests in CI #34

Open alfredbez opened 4 years ago

alfredbez commented 4 years ago

We have a few unit tests, but we need to execute them in CI. There was already one attempt, but it looks like it is not working at the moment.

@keywan-ghadami-oxid Do you want to take a look into this or should I tackle this?

alfredbez commented 4 years ago

I talked to @keywan-ghadami-oxid and I will take a look at this. We're not sure if we want to make the old unit-tests work or write them from scratch.

We will also remove the UI in the admin area in the next major version, to make it easier to maintain. We recommend using only the oxid-console as an interface to this module.

olivereanderson commented 4 years ago

I have worked on making the old unit tests work: I managed to get the old tests working in my oxid 6.1 shop with this branch https://github.com/OXIDprojects/oxid_modules_config/tree/fix_tests_for_oxid_6 . Note that the linked branch is based on https://github.com/OXIDprojects/oxid_modules_config/commit/d01f87afe0965252ae5b3ce0e845f8784fabc1f0 . I have also merged these changes in with the current master branch here: https://github.com/OXIDprojects/oxid_modules_config/tree/make-tests-run-on-oxid-6. Hope this can be of help.