Make slic3rutils tests more hermetic (low priority) #1705

New Issue

MrUnknownDE · 2026-04-05T19:18:05+02:00

MrUnknownDE commented

2026-04-05 19:18:05 +02:00

Originally created by @coryrc on 11/4/2025

Is there an existing issue for this feature request?

I have searched the existing issues

I am porting in Catch2 v3 in #10906 (not uploaded yet) then others can help work on this if desired.

Tests are under tests in the repo.

The slic3erutils tests check whether HTTP basic and digest auth work by hitting some external web server. It recently caused us some annoyance with intermittent 503.

Maybe these tests are dumb and should be deleted. Maybe they should be a true "unit" test and mock curl and/or just check some fields were set -- which sounds like a "change detector" test which is dumb. Or stand up apache locally and confirm basic and digest work? Would need to ensure apache is installed in GH Actions and ideally don't fail if it's missing so it's not annoying for such an unimportant test. I tried looking for a smaller webserver that supports Basic and Digest auth OOTB but didn't find any (i.e. caddy webserver says "digest is irretrievably insecure so we won't support it" so it relies on random other implementation).

Which printers will be beneficial to this feature?

All

Describe the solution you'd like

The two auth tests don't rely on external services, if they even still exist.

Describe alternatives you've considered

No response

Additional context

No response

*Originally created by @coryrc on 11/4/2025* ### Is there an existing issue for this feature request? - [x] I have searched the existing issues ### Is your feature request related to a problem? I am porting in Catch2 v3 in #10906 (not uploaded yet) then others can help work on this if desired. Tests are under `tests` in the repo. The slic3erutils tests check whether HTTP basic and digest auth work by hitting some external web server. It recently caused us some annoyance with intermittent 503. Maybe these tests are dumb and should be deleted. Maybe they should be a true "unit" test and mock curl and/or just check some fields were set -- which sounds like a "change detector" test which is dumb. Or stand up apache locally and confirm basic and digest work? Would need to ensure apache is installed in GH Actions and ideally don't fail if it's missing so it's not annoying for such an unimportant test. I tried looking for a smaller webserver that supports Basic and Digest auth OOTB but didn't find any (i.e. caddy webserver says "digest is irretrievably insecure so we won't support it" so it relies on random other implementation). ### Which printers will be beneficial to this feature? All ### Describe the solution you'd like The two auth tests don't rely on external services, if they even still exist. ### Describe alternatives you've considered _No response_ ### Additional context _No response_

Sign in to join this conversation.

No Label enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement enhancement

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/OrcaSlicer#1705