Skip to content

test: remove duplicate plugin extension regression methods - #40

Merged
bewaterhere-coder merged 3 commits into
mainfrom
test/dedupe-plugin-extension-regressions
Nov 1, 2026
Merged

bewaterhere-coder merged 3 commits into
mainfrom
test/dedupe-plugin-extension-regressions

Conversation

@bewaterhere-coder

Copy link
Copy Markdown
Owner

Summary

  • Remove the duplicated tail block in tests/test_plugin_extension.py that redefined seven existing test methods.
  • Move the Windows drive-prefix path rejection case into its own independent test method.
  • Keep the symlink test limited to symlink behavior.

Why

Python class bodies silently replace earlier methods when a later method has the same name. The duplicate definitions meant the earlier test methods were not independently represented in the unittest suite. The drive-prefix case was also nested inside the symlink test, so it did not have its own test identity and could be skipped if symlink setup failed.

Scope

Tests only. No runtime behavior, API, plugin contract or documentation changes.

Verification

Checked the updated file for duplicate test_* method names and confirmed the drive-prefix case is an independent method.

@bewaterhere-coder
bewaterhere-coder merged commit 7edb21e into main Nov 1, 2026
6 checks passed
@bewaterhere-coder
bewaterhere-coder deleted the test/dedupe-plugin-extension-regressions branch November 1, 2026 12:11
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.

1 participant