Skip to content

[Apple] nodeobs_settings test fix#1652

Merged
sandboxcoder merged 1 commit intostagingfrom
rno/osn-encoder-tests
Mar 27, 2026
Merged

[Apple] nodeobs_settings test fix#1652
sandboxcoder merged 1 commit intostagingfrom
rno/osn-encoder-tests

Conversation

@sandboxcoder
Copy link
Copy Markdown
Contributor

@sandboxcoder sandboxcoder commented Mar 27, 2026

Description

  • Update test_nodeobs_settings.ts to use correct encoder name for OS as used in other tests (x264 for Windows / obs_x264 for Mac)
  • Update error message to include input name (tiny QoL improvement; noticed this while looking at logs)

Motivation and Context

Fix tests to pass on Mac

How Has This Been Tested?

Verified tests pass on Mac locally

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

* use correct encoder for the OS x264/obs_x264
* osn-input: add name of Named input to debug message
@sandboxcoder sandboxcoder force-pushed the rno/osn-encoder-tests branch from da363d1 to 2dc92b9 Compare March 27, 2026 19:53
@sandboxcoder sandboxcoder merged commit 90f44a4 into staging Mar 27, 2026
16 checks passed
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