Commit 40e5a5f1 authored by Chris Lamb's avatar Chris Lamb 💬

Drop "DOS/MBR" source string test.

This always seemed heavy-handed to me, both technically (there should be real
tests for any incorrect identification of file format, etc.) but moreover from
a community/collaborative point of view.
parent 2693ad67
Pipeline #38084 passed with stage
in 15 minutes and 12 seconds
......@@ -18,7 +18,6 @@
# along with diffoscope. If not, see <https://www.gnu.org/licenses/>.
import os
import glob
import diffoscope
import subprocess
......@@ -27,12 +26,6 @@ from .utils.tools import skip_unless_tools_exist
BASE_DIR = os.path.dirname(os.path.abspath(diffoscope.__file__))
def test_dos_mbr():
for x in glob.iglob(os.path.join(BASE_DIR, '**', '*.py'), recursive=True):
with open(x, 'rb') as f:
assert b'DOS/MBR' not in f.read()
@skip_unless_tools_exist('black')
def test_code_is_black_clean():
output = subprocess.check_output(
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment