janhoy commented on code in PR #4430:
URL: https://github.com/apache/solr/pull/4430#discussion_r3250567642
##########
solr/packaging/test/test_post.bats:
##########
@@ -138,18 +138,9 @@ teardown() {
assert_output --partial '"numFound":45'
}
-# this test doesn't complete due to issues in posting to the /extract handler
@test "crawling a web site" {
Review Comment:
@epugh You wrote this BATS test case years ago (2023), but today it started
causing issues for me, both in github CI and locally, in that the following two
tests would start failing due to the missing `tikaserver.url` param.
Not sure why it started failing now and not earlier.
From your comment it seems you planned for it to fail/not complete. But do
you see any problem in my fix here, not adding the handler at all? It would
still print the expected output, but would fail crawling in a different way
than before.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
