backdrop-contrib / search_api

Provides a generic API for modules offering search capabilities
GNU General Public License v2.0
0 stars 6 forks source link

Automated test issues #65

Open argiepiano opened 11 months ago

argiepiano commented 11 months ago

When opening the "Testing" page after enabling Testing, I'm seeing:

File modules/search_api/contrib/search_api_views/search_api_views.tests.info contains no testing classes. File modules/search_api/contrib/search_api_facetapi/search_api_facetapi.tests.info contains no testing classes. File modules/search_api/tests/search_api_test.tests.info contains no testing classes. File modules/search_api/tests/search_api_test_2.tests.info contains no testing classes.

PHP 8.1

laryn commented 11 months ago

@argiepiano I think automated tests are in bad shape on this module at the moment. I haven't had a chance to dig much deeper but these relate:

argiepiano commented 11 months ago

Thanks, @laryn. Those two links are different modules, though. It'd be good to try to fix the tests for this module. I'll try to take a look in the future.

indigoxela commented 23 hours ago

Problem still exists.

Contents of file search_api_test.tests.info in a released module version:

; Added by Backdrop CMS packaging script on 2024-09-14
project = search_api
version = 1.x-1.29.2
timestamp = 1726367023

Not sure, the file seems to confuse the packaging script, but apparently it does.

Because the actual content of the file should be: https://github.com/backdrop-contrib/search_api/blob/1.x-1.x/search_api.tests.info

Edit: Oh, wait, it's the wrong file name. Should be search_api.tests.info instead. Hopefully then the packager doesn't mangle the contents.

laryn commented 23 hours ago

It looks like there's a few empty files in the /tests folder from the initial port, I'm not sure why. We definitely need some cleanup in the testing department here.