Skip to content

roman-numerals: rename tests for clarity #2321

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.

Sign up for GitHub

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

Merged
BethanyG merged 1 commit into from
Feb 5, 2021
Merged

roman-numerals: rename tests for clarity #2321

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 19 additions & 19 deletions exercises/practice/roman-numerals/roman_numerals_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,61 +6,61 @@

# Tests adapted from `problem-specifications//canonical-data.json`
class RomanNumeralsTest(unittest.TestCase):
def test_1_is_a_single_i(self):
def test_1_is_i(self):
self.assertEqual(roman(1), "I")

def test_2_is_two_i_s(self):
def test_2_is_ii(self):
self.assertEqual(roman(2), "II")

def test_3_is_three_i_s(self):
def test_3_is_iii(self):
self.assertEqual(roman(3), "III")

def test_4_being_5_1_is_iv(self):
def test_4_is_iv(self):
self.assertEqual(roman(4), "IV")

def test_5_is_a_single_v(self):
def test_5_is_v(self):
self.assertEqual(roman(5), "V")

def test_6_being_5_1_is_vi(self):
def test_6_is_vi(self):
self.assertEqual(roman(6), "VI")

def test_9_being_10_1_is_ix(self):
def test_9_is_ix(self):
self.assertEqual(roman(9), "IX")

def test_20_is_two_x_s(self):
def test_27_is_xxvii(self):
self.assertEqual(roman(27), "XXVII")

def test_48_is_not_50_2_but_rather_40_8(self):
def test_48_is_xlviii(self):
self.assertEqual(roman(48), "XLVIII")

def test_49_is_not_40_5_4_but_rather_50_10_10_1(self):
def test_49_is_xlix(self):
self.assertEqual(roman(49), "XLIX")

def test_50_is_a_single_l(self):
def test_59_is_lix(self):
self.assertEqual(roman(59), "LIX")

def test_90_being_100_10_is_xc(self):
def test_93_is_xciii(self):
self.assertEqual(roman(93), "XCIII")

def test_100_is_a_single_c(self):
def test_141_is_cxli(self):
self.assertEqual(roman(141), "CXLI")

def test_60_being_50_10_is_lx(self):
def test_163_is_clxiii(self):
self.assertEqual(roman(163), "CLXIII")

def test_400_being_500_100_is_cd(self):
def test_402_is_cdii(self):
self.assertEqual(roman(402), "CDII")

def test_500_is_a_single_d(self):
def test_575_is_dlxxv(self):
self.assertEqual(roman(575), "DLXXV")

def test_900_being_1000_100_is_cm(self):
def test_911_is_cmxi(self):
self.assertEqual(roman(911), "CMXI")

def test_1000_is_a_single_m(self):
def test_1024_is_mxxiv(self):
self.assertEqual(roman(1024), "MXXIV")

def test_3000_is_three_m_s(self):
def test_3000_is_mmm(self):
self.assertEqual(roman(3000), "MMM")


Expand Down