Python3 compatibility fixes

tests/test_webpack_manifest.py

@@ -58,7 +58,7 @@ def test_errors_handled(self):
  self.assertFalse('should not reach this')
  except webpack_manifest.WebpackError as e:
  self.assertEqual(
- e.message,
+ e.args[0],
  'Webpack errors: \n\nerror 1\n\nerror 2',
  )
 
@@ -71,7 +71,7 @@ def test_status_handled(self):
  self.assertFalse('should not reach this')
  except webpack_manifest.WebpackError as e:
  self.assertEqual(
- e.message,
+ e.args[0],
  'Webpack errors: \n\nerror 1\n\nerror 2',
  )
 
@@ -87,7 +87,7 @@ def test_handles_timeouts_in_debug(self):
  self.assertFalse('should not reach this')
  except webpack_manifest.WebpackManifestBuildingStatusTimeout as e:
  self.assertEqual(
- e.message,
+ e.args[0],
  'Timed out reading the webpack manifest at "{}"'.format(path),
  )
 
@@ -101,6 +101,6 @@ def test_handles_unknown_statuses(self):
  self.assertFalse('should not reach this')
  except webpack_manifest.WebpackManifestStatusError as e:
  self.assertEqual(
- e.message,
+ e.args[0],
  'Unknown webpack manifest status: "unknown status"',
  )

webpack_manifest/webpack_manifest.py

@@ -186,7 +186,7 @@ def read(path, read_retry):
  raise WebpackManifestFileError('Path "{}" is not an absolute path to a file'.format(path))
 
  with open(path, 'r') as manifest_file:
- content = manifest_file.read().encode('utf-8')
+ content = manifest_file.read()
 
  # In certain conditions, the file's contents evaluate to an empty string, so
  # we provide a hook to perform a single retry after a delay.