From a7303b52db2ae319324ec4f09c470ff1f932cf7b Mon Sep 17 00:00:00 2001 From: Ran Benita Date: Wed, 1 Jul 2020 20:20:11 +0300 Subject: [PATCH] code/source: remove unused method Source.isparseable() --- src/_pytest/_code/source.py | 15 --------------- testing/code/test_source.py | 10 ---------- 2 files changed, 25 deletions(-) diff --git a/src/_pytest/_code/source.py b/src/_pytest/_code/source.py index f7dcdeff9..6c9aaa7e6 100644 --- a/src/_pytest/_code/source.py +++ b/src/_pytest/_code/source.py @@ -132,21 +132,6 @@ class Source: newsource.lines[:] = deindent(self.lines) return newsource - def isparseable(self, deindent: bool = True) -> bool: - """ return True if source is parseable, heuristically - deindenting it by default. - """ - if deindent: - source = str(self.deindent()) - else: - source = str(self) - try: - ast.parse(source) - except (SyntaxError, ValueError, TypeError): - return False - else: - return True - def __str__(self) -> str: return "\n".join(self.lines) diff --git a/testing/code/test_source.py b/testing/code/test_source.py index 8616b2f25..0bf8c0b17 100644 --- a/testing/code/test_source.py +++ b/testing/code/test_source.py @@ -125,15 +125,6 @@ def test_syntaxerror_rerepresentation() -> None: assert ex.value.text.rstrip("\n") == "xyz xyz" -def test_isparseable() -> None: - assert Source("hello").isparseable() - assert Source("if 1:\n pass").isparseable() - assert Source(" \nif 1:\n pass").isparseable() - assert not Source("if 1:\n").isparseable() - assert not Source(" \nif 1:\npass").isparseable() - assert not Source(chr(0)).isparseable() - - class TestAccesses: def setup_class(self) -> None: self.source = Source( @@ -147,7 +138,6 @@ class TestAccesses: def test_getrange(self) -> None: x = self.source[0:2] - assert x.isparseable() assert len(x.lines) == 2 assert str(x) == "def f(x):\n pass"