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

Automated test issues #65

Open
argiepiano opened this issue Dec 13, 2023 · 4 comments
Open

Automated test issues #65

argiepiano opened this issue Dec 13, 2023 · 4 comments

Comments

@argiepiano
Copy link
Contributor

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
Copy link
Member

laryn commented Dec 13, 2023

@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
Copy link
Contributor Author

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
Copy link
Member

indigoxela commented Dec 4, 2024

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
Copy link
Member

laryn commented Dec 4, 2024

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.

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

No branches or pull requests

3 participants