Skip to content

Fixed testWordExtraction not to depend on installed modules#197

Merged
Holzhaus merged 1 commit intojasperproject:masterfrom
alexsiri7:test_fix
Sep 25, 2014
Merged

Fixed testWordExtraction not to depend on installed modules#197
Holzhaus merged 1 commit intojasperproject:masterfrom
alexsiri7:test_fix

Conversation

@alexsiri7
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread client/test.py Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Instead, can we do something like:

words = [
    'MUSIC', 'SPOTIFY'
]

mock_module = Mock()
mock_module.WORDS = [
    'MOCK'
]
words.extend(mock_module.WORDS)

This avoids inconsistencies between WORDS and mock_module.WORDS.

@alexsiri7 alexsiri7 force-pushed the test_fix branch 2 times, most recently from 33e744b to 421f331 Compare September 25, 2014 05:22
Holzhaus added a commit that referenced this pull request Sep 25, 2014
Fixed testWordExtraction not to depend on installed modules
@Holzhaus Holzhaus merged commit 35cc0d5 into jasperproject:master Sep 25, 2014
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.

3 participants