Skip to content

Conversation

@remicollet
Copy link
Contributor

@remicollet remicollet commented Jul 21, 2025

Followup of #154

@sonarqubecloud
Copy link

@CloCkWeRX
Copy link
Member

CloCkWeRX commented Jul 21, 2025

Could we (maybe in a separate PR if they are incidental) update the test expectations, ie:

001+ $pearweb->addRESTConfig("https://pecl.php.net/rest/r/test/allreleases.xml", false, false);
002+ No REST config for URL https://pecl.php.net/rest/r/test/allreleases.xml
========DONE========

(Or rebase if this was taken care of recently)

Edit: majority of test failures are not the above, turns out I clicked on the one lucky test result

@remicollet
Copy link
Contributor Author

I understand now, the message displayed (addRESTConfig) is what should be used in the test,
so tests need to be updated

@remicollet remicollet closed this Jul 21, 2025
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 this pull request may close these issues.

2 participants