Skip to content

fix: rename stem_and_suffix to stem_and_affix in tests#6

Open
bitflicker64 wants to merge 1 commit into
SakanaAI:mainfrom
bitflicker64:fix/stem-and-affix-rename
Open

fix: rename stem_and_suffix to stem_and_affix in tests#6
bitflicker64 wants to merge 1 commit into
SakanaAI:mainfrom
bitflicker64:fix/stem-and-affix-rename

Conversation

@bitflicker64

Copy link
Copy Markdown

Description:

stem_and_suffix does not exist anywhere in the codebase. The correct function name is stem_and_affix, defined in core/forms/utils.py.

Problem

  • tests/test_adoption.py fails at import time with ImportError: cannot import name 'stem_and_suffix' from 'core.forms.utils'
  • tests/test_agent.py, tests/test_compression.py, and tests/test_induction.py fail at runtime with AttributeError: module 'core.forms.morphology' has no attribute 'stem_and_suffix'

Fix

Rename all 9 occurrences of stem_and_suffixstem_and_affix across the four affected test files. No production code is changed. The function signature is identical — (form) -> (stem, affixes).

Files changed

File Changes
tests/test_adoption.py Fix import + 3 call sites
tests/test_agent.py Fix 1 call site
tests/test_compression.py Fix 2 call sites
tests/test_induction.py Fix 2 call sites
.gitignore Add .idea/

stem_and_suffix does not exist in core/forms/utils.py or anywhere
else in the codebase. The correct function name is stem_and_affix.

Affected files:
- tests/test_adoption.py: fix import and 3 call sites
- tests/test_agent.py: fix 1 call site
- tests/test_compression.py: fix 2 call sites
- tests/test_induction.py: fix 2 call sites

Signed-off-by: Himanshu Verma <himnshuverma10152006@gmail.com>
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.

stem_and_suffix does not exist — tests fail with ImportError/AttributeError

1 participant