Fix movie audio profile test by JonnyWong16 · Pull Request #1580 · pushingkarmaorg/python-plexapi

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@JonnyWong16

Copy link Copy Markdown

Collaborator

Description

Fix movie audio profile test.

Missed tests in #1579.

Type of change

Please delete options that are not relevant.

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

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated the docstring for new or existing methods
  • I have added tests when applicable

Copilot AI review requested due to automatic review settings

January 30, 2026 19:27
Copy link Copy Markdown

Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes movie audio profile test assertions to correctly handle cases where the audio profile attribute may be None in addition to the expected "lc" (low complexity) value. These test updates were missed in PR #1579.

Changes:

  • Updated three test assertions from strict equality checks to membership checks that allow None or "lc" values

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JonnyWong16 JonnyWong16 merged commit a20a7ac into pushingkarmaorg:master

Jan 30, 2026

11 checks passed

@JonnyWong16 JonnyWong16 deleted the hotfix/movie_audio_profile_test branch

January 30, 2026 19:34

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JonnyWong16