-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
LOWNot important, nor necessary. Probably a QoL improvement.Not important, nor necessary. Probably a QoL improvement.
Description
Low: Add Mock Tests That Don't Require OBS
Priority: LOW
Type: Testing
Location
tests/test.c
Issue
All tests require a running OBS instance:
// All tests assume OBS is running on localhost:4455
obsws_connection_t *conn = obsws_connect(&config);
if (!conn) {
fprintf(stderr, "Failed to connect\n");
return 1;
}Impact
- Can't run tests in CI/CD without OBS
- Can't test error conditions (OBS crash, network failure)
- Slower test execution
- Tests are integration tests, not unit tests
Benefits:
- Fast tests that run anywhere
- Can test error conditions
- Better code coverage
- CI/CD friendly
Metadata
Metadata
Assignees
Labels
LOWNot important, nor necessary. Probably a QoL improvement.Not important, nor necessary. Probably a QoL improvement.