Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

protein-translation: update tests to v1.1.0 #1329

Merged
merged 4 commits into from
Feb 21, 2018

Conversation

mrcfps
Copy link

@mrcfps mrcfps commented Feb 18, 2018

Closes #1245.

Since canonical data was only updated for new input policy, which has nothing to do with test file in Python, so update of version number suffices.

@mrcfps
Copy link
Author

mrcfps commented Feb 19, 2018

Actually the issue I've referenced should be #440.

@cmccandless
Copy link
Contributor

@mrcfps you can modify commit messages

git commit --amend (or git rebase -i master)
git push -f

@mrcfps mrcfps force-pushed the protein-translation-1245 branch from 51969b5 to 6071a7c Compare February 20, 2018 00:11

def test_stops_translation_if_stop_codon_in_middle_of_six_codon_sequence(
self):
strand = 'UGGUGUUAUUAAUGGUUU'
expected = ['Tryptophan', 'Cysteine', 'Tyrosine']
self.assertEqual(expected, proteins(strand))
self.assertEqual(proteins(strand), expected)

# Utility functions
Copy link
Contributor

Choose a reason for hiding this comment

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

Since this exercise no longer has tests that expected exceptions, I think it's safe to remove the Utility functions block

@cmccandless cmccandless merged commit 6560eaa into exercism:master Feb 21, 2018
@mrcfps mrcfps deleted the protein-translation-1245 branch February 26, 2018 12:59
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