diff --git a/whipper/test/test_common_accurip.py b/whipper/test/test_common_accurip.py index 316ab93..82461aa 100644 --- a/whipper/test/test_common_accurip.py +++ b/whipper/test/test_common_accurip.py @@ -43,7 +43,7 @@ class TestAccurateRipResponse(TestCase): join(self.cache_dir, self.other_path) ) # ask cache for other entry and assert cached entry equals normal entry - self.assertEquals(self.entry, get_db_entry(self.other_path)) + self.assertEqual(self.entry, get_db_entry(self.other_path)) def test_raises_entrynotfound_for_no_entry(self): with self.assertRaises(EntryNotFound): @@ -58,37 +58,37 @@ class TestAccurateRipResponse(TestCase): def test_retrieves_and_saves_accuraterip_entry(self): # for path, entry in zip(self.paths[0], self.entries): self.assertFalse(exists(join(self.cache_dir, self.path))) - self.assertEquals(get_db_entry(self.path), self.entry) + self.assertEqual(get_db_entry(self.path), self.entry) self.assertTrue(exists(join(self.cache_dir, self.path))) def test_AccurateRipResponse_parses_correctly(self): responses = get_db_entry(self.path) - self.assertEquals(len(responses), 2) + self.assertEqual(len(responses), 2) - self.assertEquals(responses[0].num_tracks, 2) - self.assertEquals(responses[0].discId1, '0000f21c') - self.assertEquals(responses[0].discId2, '00027ef8') - self.assertEquals(responses[0].cddbDiscId, '05021002') - self.assertEquals(responses[0].confidences[0], 12) - self.assertEquals(responses[0].confidences[1], 20) - self.assertEquals(responses[0].checksums[0], '284fc705') - self.assertEquals(responses[0].checksums[1], '9cc1f32e') + self.assertEqual(responses[0].num_tracks, 2) + self.assertEqual(responses[0].discId1, '0000f21c') + self.assertEqual(responses[0].discId2, '00027ef8') + self.assertEqual(responses[0].cddbDiscId, '05021002') + self.assertEqual(responses[0].confidences[0], 12) + self.assertEqual(responses[0].confidences[1], 20) + self.assertEqual(responses[0].checksums[0], '284fc705') + self.assertEqual(responses[0].checksums[1], '9cc1f32e') - self.assertEquals(responses[1].num_tracks, 2) - self.assertEquals(responses[1].discId1, '0000f21c') - self.assertEquals(responses[1].discId2, '00027ef8') - self.assertEquals(responses[1].cddbDiscId, '05021002') - self.assertEquals(responses[1].confidences[0], 4) - self.assertEquals(responses[1].confidences[1], 4) - self.assertEquals(responses[1].checksums[0], 'dc77f9ab') - self.assertEquals(responses[1].checksums[1], 'dd97d2c3') + self.assertEqual(responses[1].num_tracks, 2) + self.assertEqual(responses[1].discId1, '0000f21c') + self.assertEqual(responses[1].discId2, '00027ef8') + self.assertEqual(responses[1].cddbDiscId, '05021002') + self.assertEqual(responses[1].confidences[0], 4) + self.assertEqual(responses[1].confidences[1], 4) + self.assertEqual(responses[1].checksums[0], 'dc77f9ab') + self.assertEqual(responses[1].checksums[1], 'dd97d2c3') # XXX: test arc.py class TestCalculateChecksums(TestCase): def test_returns_none_for_bad_files(self): - self.assertEquals( + self.assertEqual( calculate_checksums(['/does/not/exist']), {'v1': [None], 'v2': [None]} ) @@ -116,26 +116,26 @@ class TestVerifyResult(TestCase): self.result.tracks.append(track) def test_empty_result_returns_false(self): - self.assertEquals( + self.assertEqual( verify_result(RipResult(), self.responses, self.checksums), False ) def test_empty_responses_returns_false(self): - self.assertEquals( + self.assertEqual( verify_result(self.result, [], self.checksums), False ) # XXX: would this happen? def test_empty_checksums_returns_false(self): - self.assertEquals( + self.assertEqual( verify_result(self.result, self.responses, {}), False ) def test_wrong_checksums_returns_false(self): - self.assertEquals( + self.assertEqual( verify_result(self.result, self.responses, { 'v1': ['deadbeef', '89abcdef'], 'v2': ['76543210', '01234567'] @@ -144,21 +144,21 @@ class TestVerifyResult(TestCase): ) def test_incomplete_checksums(self): - self.assertEquals( + self.assertEqual( verify_result(self.result, self.responses, { 'v1': ['284fc705', '9cc1f32e'], 'v2': [None, 'dd97d2c3'], }), True ) - self.assertEquals( + self.assertEqual( verify_result(self.result, self.responses, { 'v1': ['284fc705', None], 'v2': ['dc77f9ab', 'dd97d2c3'], }), True ) - self.assertEquals( + self.assertEqual( verify_result(self.result, self.responses, { 'v1': ['284fc705', None], 'v2': [None, 'dd97d2c3'], @@ -167,13 +167,13 @@ class TestVerifyResult(TestCase): ) def test_matches_only_v1_or_v2_responses(self): - self.assertEquals( + self.assertEqual( verify_result( self.result, [self.responses[0]], self.checksums ), True ) - self.assertEquals( + self.assertEqual( verify_result( self.result, [self.responses[1]], self.checksums ), @@ -184,17 +184,17 @@ class TestVerifyResult(TestCase): htoa = TrackResult() htoa.number = 0 self.result.tracks.append(htoa) - self.assertEquals( + self.assertEqual( verify_result(self.result, self.responses, self.checksums), True ) def test_stores_accuraterip_results_on_result(self): - self.assertEquals( + self.assertEqual( verify_result(self.result, self.responses, self.checksums), True ) - self.assertEquals(self.result.tracks[0].AR, { + self.assertEqual(self.result.tracks[0].AR, { 'v1': { 'CRC': '284fc705', 'DBCRC': '284fc705', @@ -208,7 +208,7 @@ class TestVerifyResult(TestCase): 'DBMaxConfidence': 12, 'DBMaxConfidenceCRC': '284fc705', }) - self.assertEquals(self.result.tracks[1].AR, { + self.assertEqual(self.result.tracks[1].AR, { 'v1': { 'CRC': '9cc1f32e', 'DBCRC': '9cc1f32e', @@ -254,7 +254,7 @@ class TestAccurateRipReport(TestCase): track.number = 1 self.result.tracks[0] = track print_report(self.result) - self.assertEquals( + self.assertEqual( sys.stdout.getvalue(), 'track 1: unknown (error)\n' ) @@ -262,7 +262,7 @@ class TestAccurateRipReport(TestCase): def test_track_not_found(self): self.result.tracks[0].AR['DBMaxConfidence'] = None print_report(self.result) - self.assertEquals( + self.assertEqual( sys.stdout.getvalue(), 'track 1: rip NOT accurate (not found) ' ' v1 [284fc705], v2 [dc77f9ab], DB [notfound]\n' @@ -273,7 +273,7 @@ class TestAccurateRipReport(TestCase): self.result.tracks[0].AR['v1']['CRC'] = None self.result.tracks[0].AR['v2']['CRC'] = None print_report(self.result) - self.assertEquals( + self.assertEqual( sys.stdout.getvalue(), 'track 0: unknown (not tracked)\n' ) @@ -282,7 +282,7 @@ class TestAccurateRipReport(TestCase): self.result.tracks[0].AR['v2']['DBCRC'] = None self.result.tracks[0].AR['v2']['DBConfidence'] = None print_report(self.result) - self.assertEquals( + self.assertEqual( sys.stdout.getvalue(), 'track 1: rip accurate (max confidence 12)' ' v1 [284fc705], v2 [dc77f9ab], DB [284fc705]\n' @@ -292,7 +292,7 @@ class TestAccurateRipReport(TestCase): self.result.tracks[0].AR['v1']['DBCRC'] = None self.result.tracks[0].AR['v1']['DBConfidence'] = None print_report(self.result) - self.assertEquals( + self.assertEqual( sys.stdout.getvalue(), 'track 1: rip accurate (confidence 4 of 12)' ' v1 [284fc705], v2 [dc77f9ab], DB [dc77f9ab]\n' @@ -300,7 +300,7 @@ class TestAccurateRipReport(TestCase): def test_report_v1_and_v2_max_confidence(self): print_report(self.result) - self.assertEquals( + self.assertEqual( sys.stdout.getvalue(), 'track 1: rip accurate (max confidence 12)' ' v1 [284fc705], v2 [dc77f9ab], DB [284fc705, dc77f9ab]\n' @@ -309,7 +309,7 @@ class TestAccurateRipReport(TestCase): def test_report_v1_and_v2(self): self.result.tracks[0].AR['DBMaxConfidence'] = 66 print_report(self.result) - self.assertEquals( + self.assertEqual( sys.stdout.getvalue(), 'track 1: rip accurate (confidence 12 of 66)' ' v1 [284fc705], v2 [dc77f9ab], DB [284fc705, dc77f9ab]\n' diff --git a/whipper/test/test_common_cache.py b/whipper/test/test_common_cache.py index e0a7d23..4422306 100644 --- a/whipper/test/test_common_cache.py +++ b/whipper/test/test_common_cache.py @@ -16,8 +16,8 @@ class ResultCacheTestCase(tcommon.TestCase): def testGetResult(self): result = self.cache.getRipResult('fe105a11') - self.assertEquals(result.object.title, "The Writing's on the Wall") + self.assertEqual(result.object.title, "The Writing's on the Wall") def testGetIds(self): ids = self.cache.getIds() - self.assertEquals(ids, ['fe105a11']) + self.assertEqual(ids, ['fe105a11']) diff --git a/whipper/test/test_common_common.py b/whipper/test/test_common_common.py index a74443d..3ac8fea 100644 --- a/whipper/test/test_common_common.py +++ b/whipper/test/test_common_common.py @@ -21,7 +21,7 @@ class ShrinkTestCase(tcommon.TestCase): 'Are Off Our Lands!".flac') shorter = common.shrinkPath(path) - self.failUnless(os.path.splitext(path)[0].startswith( + self.assertTrue(os.path.splitext(path)[0].startswith( os.path.splitext(shorter)[0])) self.failIfEquals(path, shorter) @@ -29,13 +29,13 @@ class ShrinkTestCase(tcommon.TestCase): class FramesTestCase(tcommon.TestCase): def testFrames(self): - self.assertEquals(common.framesToHMSF(123456), '00:27:26.06') + self.assertEqual(common.framesToHMSF(123456), '00:27:26.06') class FormatTimeTestCase(tcommon.TestCase): def testFormatTime(self): - self.assertEquals(common.formatTime(7202), '02:00:02.000') + self.assertEqual(common.formatTime(7202), '02:00:02.000') class GetRelativePathTestCase(tcommon.TestCase): @@ -45,7 +45,7 @@ class GetRelativePathTestCase(tcommon.TestCase): cue = './' + directory + '/Placebo - Black Market Music (2000)' track = './' + directory + '/01. Placebo - Taste in Men.flac' - self.assertEquals(common.getRelativePath(track, cue), + self.assertEqual(common.getRelativePath(track, cue), '01. Placebo - Taste in Men.flac') @@ -55,12 +55,12 @@ class GetRealPathTestCase(tcommon.TestCase): fd, path = tempfile.mkstemp(suffix=u'back\\slash.flac') refPath = os.path.join(os.path.dirname(path), 'fake.cue') - self.assertEquals(common.getRealPath(refPath, path), + self.assertEqual(common.getRealPath(refPath, path), path) # same path, but with wav extension, will point to flac file wavPath = path[:-4] + 'wav' - self.assertEquals(common.getRealPath(refPath, wavPath), + self.assertEqual(common.getRealPath(refPath, wavPath), path) os.close(fd) diff --git a/whipper/test/test_common_config.py b/whipper/test/test_common_config.py index cdfdbf2..e9d9a31 100644 --- a/whipper/test/test_common_config.py +++ b/whipper/test/test_common_config.py @@ -27,13 +27,13 @@ class ConfigTestCase(tcommon.TestCase): # getting it from memory should work offset = self._config.getReadOffset( 'PLEXTOR ', 'DVDR PX-L890SA', '1.05') - self.assertEquals(offset, 6) + self.assertEqual(offset, 6) # and so should getting it after reading it again self._config.open() offset = self._config.getReadOffset( 'PLEXTOR ', 'DVDR PX-L890SA', '1.05') - self.assertEquals(offset, 6) + self.assertEqual(offset, 6) def testAddReadOffsetSpaced(self): self.assertRaises(KeyError, self._config.getReadOffset, @@ -43,13 +43,13 @@ class ConfigTestCase(tcommon.TestCase): # getting it from memory should work offset = self._config.getReadOffset( 'Slimtype', 'eSAU208 2 ', 'ML03') - self.assertEquals(offset, 6) + self.assertEqual(offset, 6) # and so should getting it after reading it again self._config.open() offset = self._config.getReadOffset( 'Slimtype', 'eSAU208 2 ', 'ML03') - self.assertEquals(offset, 6) + self.assertEqual(offset, 6) def testDefeatsCache(self): self.assertRaises(KeyError, self._config.getDefeatsCache, @@ -59,16 +59,16 @@ class ConfigTestCase(tcommon.TestCase): 'PLEXTOR ', 'DVDR PX-L890SA', '1.05', False) defeats = self._config.getDefeatsCache( 'PLEXTOR ', 'DVDR PX-L890SA', '1.05') - self.assertEquals(defeats, False) + self.assertEqual(defeats, False) self._config.setDefeatsCache( 'PLEXTOR ', 'DVDR PX-L890SA', '1.05', True) defeats = self._config.getDefeatsCache( 'PLEXTOR ', 'DVDR PX-L890SA', '1.05') - self.assertEquals(defeats, True) + self.assertEqual(defeats, True) def test_get_musicbrainz_server(self): - self.assertEquals(self._config.get_musicbrainz_server(), + self.assertEqual(self._config.get_musicbrainz_server(), 'musicbrainz.org', msg='Default value is correct') @@ -77,7 +77,7 @@ class ConfigTestCase(tcommon.TestCase): self._config._parser.set('musicbrainz', 'server', '192.168.2.141:5000') self._config.write() - self.assertEquals(self._config.get_musicbrainz_server(), + self.assertEqual(self._config.get_musicbrainz_server(), '192.168.2.141:5000', msg='Correctly returns user-set value') diff --git a/whipper/test/test_common_directory.py b/whipper/test/test_common_directory.py index 4290489..f9b7c26 100644 --- a/whipper/test/test_common_directory.py +++ b/whipper/test/test_common_directory.py @@ -12,7 +12,7 @@ class DirectoryTestCase(common.TestCase): def testAll(self): path = directory.config_path() - self.failUnless(path.startswith(DirectoryTestCase.HOME_PARENT)) + self.assertTrue(path.startswith(DirectoryTestCase.HOME_PARENT)) path = directory.cache_path() - self.failUnless(path.startswith(DirectoryTestCase.HOME_PARENT)) + self.assertTrue(path.startswith(DirectoryTestCase.HOME_PARENT)) diff --git a/whipper/test/test_common_drive.py b/whipper/test/test_common_drive.py index e3d13c4..f8e9b58 100644 --- a/whipper/test/test_common_drive.py +++ b/whipper/test/test_common_drive.py @@ -9,8 +9,8 @@ class ListifyTestCase(common.TestCase): def testString(self): string = '/dev/sr0' - self.assertEquals(drive._listify(string), [string, ]) + self.assertEqual(drive._listify(string), [string, ]) def testList(self): lst = ['/dev/scd0', '/dev/sr0'] - self.assertEquals(drive._listify(lst), lst) + self.assertEqual(drive._listify(lst), lst) diff --git a/whipper/test/test_common_mbngs.py b/whipper/test/test_common_mbngs.py index a3b2f2a..10beaed 100644 --- a/whipper/test/test_common_mbngs.py +++ b/whipper/test/test_common_mbngs.py @@ -22,7 +22,7 @@ class MetadataTestCase(unittest.TestCase): metadata = mbngs._getMetadata({}, response['release'], discid) - self.failIf(metadata.release) + self.assertFalse(metadata.release) def test2MeterSessies10(self): # various artists, multiple artists per track @@ -35,21 +35,21 @@ class MetadataTestCase(unittest.TestCase): metadata = mbngs._getMetadata({}, response['release'], discid) - self.assertEquals(metadata.artist, u'Various Artists') - self.assertEquals(metadata.release, u'2001-10-15') - self.assertEquals(metadata.mbidArtist, + self.assertEqual(metadata.artist, u'Various Artists') + self.assertEqual(metadata.release, u'2001-10-15') + self.assertEqual(metadata.mbidArtist, u'89ad4ac3-39f7-470e-963a-56509c546377') - self.assertEquals(len(metadata.tracks), 18) + self.assertEqual(len(metadata.tracks), 18) track16 = metadata.tracks[15] - self.assertEquals(track16.artist, 'Tom Jones & Stereophonics') - self.assertEquals(track16.mbidArtist, + self.assertEqual(track16.artist, 'Tom Jones & Stereophonics') + self.assertEqual(track16.mbidArtist, u'57c6f649-6cde-48a7-8114-2a200247601a' ';0bfba3d3-6a04-4779-bb0a-df07df5b0558' ) - self.assertEquals(track16.sortName, + self.assertEqual(track16.sortName, u'Jones, Tom & Stereophonics') def testBalladOfTheBrokenSeas(self): @@ -63,24 +63,24 @@ class MetadataTestCase(unittest.TestCase): metadata = mbngs._getMetadata({}, response['release'], discid) - self.assertEquals(metadata.artist, u'Isobel Campbell & Mark Lanegan') - self.assertEquals(metadata.sortName, + self.assertEqual(metadata.artist, u'Isobel Campbell & Mark Lanegan') + self.assertEqual(metadata.sortName, u'Campbell, Isobel & Lanegan, Mark') - self.assertEquals(metadata.release, u'2006-01-30') - self.assertEquals(metadata.mbidArtist, + self.assertEqual(metadata.release, u'2006-01-30') + self.assertEqual(metadata.mbidArtist, u'd51f3a15-12a2-41a0-acfa-33b5eae71164;' 'a9126556-f555-4920-9617-6e013f8228a7') - self.assertEquals(len(metadata.tracks), 12) + self.assertEqual(len(metadata.tracks), 12) track12 = metadata.tracks[11] - self.assertEquals(track12.artist, u'Isobel Campbell & Mark Lanegan') - self.assertEquals(track12.sortName, + self.assertEqual(track12.artist, u'Isobel Campbell & Mark Lanegan') + self.assertEqual(track12.sortName, u'Campbell, Isobel' ' & Lanegan, Mark' ) - self.assertEquals(track12.mbidArtist, + self.assertEqual(track12.mbidArtist, u'd51f3a15-12a2-41a0-acfa-33b5eae71164;' 'a9126556-f555-4920-9617-6e013f8228a7') @@ -96,20 +96,20 @@ class MetadataTestCase(unittest.TestCase): metadata = mbngs._getMetadata({}, response['release'], discid) - self.assertEquals(metadata.artist, u'Mala') - self.assertEquals(metadata.sortName, u'Mala') - self.assertEquals(metadata.release, u'2012-09-17') - self.assertEquals(metadata.mbidArtist, + self.assertEqual(metadata.artist, u'Mala') + self.assertEqual(metadata.sortName, u'Mala') + self.assertEqual(metadata.release, u'2012-09-17') + self.assertEqual(metadata.mbidArtist, u'09f221eb-c97e-4da5-ac22-d7ab7c555bbb') - self.assertEquals(len(metadata.tracks), 14) + self.assertEqual(len(metadata.tracks), 14) track6 = metadata.tracks[5] - self.assertEquals(track6.artist, u'Mala feat. Dreiser & Sexto Sentido') - self.assertEquals(track6.sortName, + self.assertEqual(track6.artist, u'Mala feat. Dreiser & Sexto Sentido') + self.assertEqual(track6.sortName, u'Mala feat. Dreiser & Sexto Sentido') - self.assertEquals(track6.mbidArtist, + self.assertEqual(track6.mbidArtist, u'09f221eb-c97e-4da5-ac22-d7ab7c555bbb' ';ec07a209-55ff-4084-bc41-9d4d1764e075' ';f626b92e-07b1-4a19-ad13-c09d690db66c' @@ -130,29 +130,29 @@ class MetadataTestCase(unittest.TestCase): discid = "rzGHHqfPWIq1GsOLhhlBcZuqo.I-" metadata = mbngs._getMetadata({}, response['release'], discid) - self.assertEquals(metadata.artist, u'Various Artists') - self.assertEquals(metadata.release, u'2010') - self.assertEquals(metadata.mbidArtist, + self.assertEqual(metadata.artist, u'Various Artists') + self.assertEqual(metadata.release, u'2010') + self.assertEqual(metadata.mbidArtist, u'89ad4ac3-39f7-470e-963a-56509c546377') - self.assertEquals(len(metadata.tracks), 10) + self.assertEqual(len(metadata.tracks), 10) track2 = metadata.tracks[1] - self.assertEquals(track2.artist, u'Twisted Reaction feat. Danielle') - self.assertEquals(track2.sortName, + self.assertEqual(track2.artist, u'Twisted Reaction feat. Danielle') + self.assertEqual(track2.sortName, u'Twisted Reaction feat. [unknown]') - self.assertEquals(track2.mbidArtist, + self.assertEqual(track2.mbidArtist, u'4f69f624-73ea-4a16-b822-bd2ca58032bf' ';125ec42a-7229-4250-afc5-e057484327fe' ) track4 = metadata.tracks[3] - self.assertEquals(track4.artist, u'BioGenesis') - self.assertEquals(track4.sortName, + self.assertEqual(track4.artist, u'BioGenesis') + self.assertEqual(track4.sortName, u'Bio Genesis') - self.assertEquals(track4.mbidArtist, + self.assertEqual(track4.mbidArtist, u'dd61b86c-c015-43e1-9a28-58fceb0975c8' ) @@ -169,26 +169,26 @@ class MetadataTestCase(unittest.TestCase): discid = "X2c2IQ5vUy5x6Jh7Xi_DGHtA1X8-" metadata = mbngs._getMetadata({}, response['release'], discid) - self.assertEquals(metadata.artist, u'Nena & Kim Wilde') - self.assertEquals(metadata.release, u'2003-05-19') - self.assertEquals(metadata.mbidArtist, + self.assertEqual(metadata.artist, u'Nena & Kim Wilde') + self.assertEqual(metadata.release, u'2003-05-19') + self.assertEqual(metadata.mbidArtist, u'38bfaa7f-ee98-48cb-acd0-946d7aeecd76' ';4b462375-c508-432a-8c88-ceeec38b16ae') - self.assertEquals(len(metadata.tracks), 4) + self.assertEqual(len(metadata.tracks), 4) track1 = metadata.tracks[0] - self.assertEquals(track1.artist, u'Nena & Kim Wilde') - self.assertEquals(track1.sortName, u'Nena & Wilde, Kim') - self.assertEquals(track1.mbidArtist, + self.assertEqual(track1.artist, u'Nena & Kim Wilde') + self.assertEqual(track1.sortName, u'Nena & Wilde, Kim') + self.assertEqual(track1.mbidArtist, u'38bfaa7f-ee98-48cb-acd0-946d7aeecd76' ';4b462375-c508-432a-8c88-ceeec38b16ae') track2 = metadata.tracks[1] - self.assertEquals(track2.artist, u'Nena & Kim Wilde') - self.assertEquals(track2.sortName, u'Nena & Wilde, Kim') - self.assertEquals(track2.mbidArtist, + self.assertEqual(track2.artist, u'Nena & Kim Wilde') + self.assertEqual(track2.sortName, u'Nena & Wilde, Kim') + self.assertEqual(track2.mbidArtist, u'38bfaa7f-ee98-48cb-acd0-946d7aeecd76' ';4b462375-c508-432a-8c88-ceeec38b16ae') diff --git a/whipper/test/test_common_path.py b/whipper/test/test_common_path.py index a7e1638..a62a7f3 100644 --- a/whipper/test/test_common_path.py +++ b/whipper/test/test_common_path.py @@ -13,18 +13,18 @@ class FilterTestCase(common.TestCase): def testSlash(self): part = u'A Charm/A Blade' - self.assertEquals(self._filter.filter(part), u'A Charm-A Blade') + self.assertEqual(self._filter.filter(part), u'A Charm-A Blade') def testFat(self): part = u'A Word: F**k you?' - self.assertEquals(self._filter.filter(part), u'A Word - F__k you_') + self.assertEqual(self._filter.filter(part), u'A Word - F__k you_') def testSpecial(self): part = u'<<< $&*!\' "()`{}[]spaceship>>>' - self.assertEquals(self._filter.filter(part), + self.assertEqual(self._filter.filter(part), u'___ _____ ________spaceship___') def testGreatest(self): part = u'Greatest Ever! Soul: The Definitive Collection' - self.assertEquals(self._filter.filter(part), + self.assertEqual(self._filter.filter(part), u'Greatest Ever_ Soul - The Definitive Collection') diff --git a/whipper/test/test_common_program.py b/whipper/test/test_common_program.py index 2ba2a84..895497f 100644 --- a/whipper/test/test_common_program.py +++ b/whipper/test/test_common_program.py @@ -15,7 +15,7 @@ class PathTestCase(unittest.TestCase): path = prog.getPath(u'/tmp', DEFAULT_DISC_TEMPLATE, 'mbdiscid', None) - self.assertEquals(path, + self.assertEqual(path, unicode('/tmp/unknown/Unknown Artist - mbdiscid/' 'Unknown Artist - mbdiscid')) @@ -27,7 +27,7 @@ class PathTestCase(unittest.TestCase): path = prog.getPath(u'/tmp', DEFAULT_DISC_TEMPLATE, 'mbdiscid', md, 0) - self.assertEquals(path, + self.assertEqual(path, unicode('/tmp/unknown/Jeff Buckley - Grace/' 'Jeff Buckley - Grace')) @@ -38,5 +38,5 @@ class PathTestCase(unittest.TestCase): md.title = 'Grace' path = prog.getPath(u'/tmp', u'%A/%d', 'mbdiscid', md, 0) - self.assertEquals(path, + self.assertEqual(path, u'/tmp/Jeff Buckley/Grace') diff --git a/whipper/test/test_common_renamer.py b/whipper/test/test_common_renamer.py index b99f48a..7207ced 100644 --- a/whipper/test/test_common_renamer.py +++ b/whipper/test/test_common_renamer.py @@ -18,7 +18,7 @@ class RenameInFileTestcase(unittest.TestCase): def testVerify(self): o = renamer.RenameInFile(self._path, 'is is a', 'at was some') - self.assertEquals(o.verify(), None) + self.assertEqual(o.verify(), None) os.unlink(self._path) self.assertRaises(AssertionError, o.verify) @@ -26,7 +26,7 @@ class RenameInFileTestcase(unittest.TestCase): o = renamer.RenameInFile(self._path, 'is is a', 'at was some') o.do() output = open(self._path).read() - self.assertEquals(output, 'That was some test\nThat was somenother\n') + self.assertEqual(output, 'That was some test\nThat was somenother\n') os.unlink(self._path) def testSerialize(self): @@ -35,7 +35,7 @@ class RenameInFileTestcase(unittest.TestCase): o2 = renamer.RenameInFile.deserialize(data) o2.do() output = open(self._path).read() - self.assertEquals(output, 'That was some test\nThat was somenother\n') + self.assertEqual(output, 'That was some test\nThat was somenother\n') os.unlink(self._path) @@ -52,14 +52,14 @@ class RenameFileTestcase(unittest.TestCase): self._operation = renamer.RenameFile(self._source, self._destination) def testVerify(self): - self.assertEquals(self._operation.verify(), None) + self.assertEqual(self._operation.verify(), None) handle = open(self._destination, 'w') handle.close() self.assertRaises(AssertionError, self._operation.verify) os.unlink(self._destination) - self.assertEquals(self._operation.verify(), None) + self.assertEqual(self._operation.verify(), None) os.unlink(self._source) self.assertRaises(AssertionError, self._operation.verify) @@ -67,7 +67,7 @@ class RenameFileTestcase(unittest.TestCase): def testDo(self): self._operation.do() output = open(self._destination).read() - self.assertEquals(output, 'This is a test\nThis is another\n') + self.assertEqual(output, 'This is a test\nThis is another\n') os.unlink(self._destination) def testSerialize(self): @@ -75,7 +75,7 @@ class RenameFileTestcase(unittest.TestCase): o = renamer.RenameFile.deserialize(data) o.do() output = open(self._destination).read() - self.assertEquals(output, 'This is a test\nThis is another\n') + self.assertEqual(output, 'This is a test\nThis is another\n') os.unlink(self._destination) @@ -107,48 +107,48 @@ class OperatorTestCase(unittest.TestCase): o = renamer.Operator(self._statePath, 'test') o.load() - self.assertEquals(o._todo, self._operator._todo) - self.assertEquals(o._done, []) + self.assertEqual(o._todo, self._operator._todo) + self.assertEqual(o._done, []) os.unlink(self._source) def testLoadOneDone(self): - self.assertEquals(len(self._operator._done), 0) + self.assertEqual(len(self._operator._done), 0) self._operator.save() self._operator.next() - self.assertEquals(len(self._operator._done), 1) + self.assertEqual(len(self._operator._done), 1) o = renamer.Operator(self._statePath, 'test') o.load() - self.assertEquals(len(o._done), 1) - self.assertEquals(o._todo, self._operator._todo) - self.assertEquals(o._done, self._operator._done) + self.assertEqual(len(o._done), 1) + self.assertEqual(o._todo, self._operator._todo) + self.assertEqual(o._done, self._operator._done) # now continue o.next() - self.assertEquals(len(o._done), 2) + self.assertEqual(len(o._done), 2) os.unlink(self._destination) def testLoadOneInterrupted(self): - self.assertEquals(len(self._operator._done), 0) + self.assertEqual(len(self._operator._done), 0) self._operator.save() # cheat by doing a task without saving self._operator._todo[0].do() - self.assertEquals(len(self._operator._done), 0) + self.assertEqual(len(self._operator._done), 0) o = renamer.Operator(self._statePath, 'test') o.load() - self.assertEquals(len(o._done), 0) - self.assertEquals(o._todo, self._operator._todo) - self.assertEquals(o._done, self._operator._done) + self.assertEqual(len(o._done), 0) + self.assertEqual(o._todo, self._operator._todo) + self.assertEqual(o._done, self._operator._done) # now continue, resuming o.next() - self.assertEquals(len(o._done), 1) + self.assertEqual(len(o._done), 1) o.next() - self.assertEquals(len(o._done), 2) + self.assertEqual(len(o._done), 2) os.unlink(self._destination) diff --git a/whipper/test/test_image_cue.py b/whipper/test/test_image_cue.py index eedffa8..3bdbc66 100644 --- a/whipper/test/test_image_cue.py +++ b/whipper/test/test_image_cue.py @@ -18,14 +18,14 @@ class KingsSingleTestCase(unittest.TestCase): self.cue = cue.CueFile(os.path.join(os.path.dirname(__file__), u'kings-single.cue')) self.cue.parse() - self.assertEquals(len(self.cue.table.tracks), 11) + self.assertEqual(len(self.cue.table.tracks), 11) def testGetTrackLength(self): t = self.cue.table.tracks[0] - self.assertEquals(self.cue.getTrackLength(t), 17811) + self.assertEqual(self.cue.getTrackLength(t), 17811) # last track has unknown length t = self.cue.table.tracks[-1] - self.assertEquals(self.cue.getTrackLength(t), -1) + self.assertEqual(self.cue.getTrackLength(t), -1) class KingsSeparateTestCase(unittest.TestCase): @@ -34,14 +34,14 @@ class KingsSeparateTestCase(unittest.TestCase): self.cue = cue.CueFile(os.path.join(os.path.dirname(__file__), u'kings-separate.cue')) self.cue.parse() - self.assertEquals(len(self.cue.table.tracks), 11) + self.assertEqual(len(self.cue.table.tracks), 11) def testGetTrackLength(self): # all tracks have unknown length t = self.cue.table.tracks[0] - self.assertEquals(self.cue.getTrackLength(t), -1) + self.assertEqual(self.cue.getTrackLength(t), -1) t = self.cue.table.tracks[-1] - self.assertEquals(self.cue.getTrackLength(t), -1) + self.assertEqual(self.cue.getTrackLength(t), -1) class KanyeMixedTestCase(unittest.TestCase): @@ -50,11 +50,11 @@ class KanyeMixedTestCase(unittest.TestCase): self.cue = cue.CueFile(os.path.join(os.path.dirname(__file__), u'kanye.cue')) self.cue.parse() - self.assertEquals(len(self.cue.table.tracks), 13) + self.assertEqual(len(self.cue.table.tracks), 13) def testGetTrackLength(self): t = self.cue.table.tracks[0] - self.assertEquals(self.cue.getTrackLength(t), -1) + self.assertEqual(self.cue.getTrackLength(t), -1) class WriteCueFileTestCase(unittest.TestCase): diff --git a/whipper/test/test_image_table.py b/whipper/test/test_image_table.py index e6b1b6c..53a6760 100644 --- a/whipper/test/test_image_table.py +++ b/whipper/test/test_image_table.py @@ -14,10 +14,10 @@ class TrackTestCase(tcommon.TestCase): def testRepr(self): track = table.Track(1) - self.assertEquals(repr(track), "") + self.assertEqual(repr(track), "") track.index(1, 100) - self.failUnless(repr(track.indexes[1]).startswith('