Commit 47588095 authored by Chris Lamb's avatar Chris Lamb 👀
Browse files

Use a simpler, previous fix to support gnumeric 1.12.43.

This reverts both 344ae4d6 and
ef5ae496. The mess was caused by me not
actually pushing my change causing duplicated work by Mattia. Mea culpa...
parent 344ae4d6
Loading
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -61,4 +61,4 @@ class GnumericFile(File):
        ))

        with open(t.name) as f:
            return f.read()
            return f.read().strip()
+4 −20
Original line number Diff line number Diff line
@@ -18,27 +18,17 @@
# along with diffoscope.  If not, see <https://www.gnu.org/licenses/>.

import pytest
import subprocess

from diffoscope.comparators.gnumeric import GnumericFile

from ..utils.data import load_fixture, get_data
from ..utils.tools import skip_unless_tools_exist
from ..utils.nonexisting import assert_non_existing
from ..utils.tools import (
    skip_unless_tools_exist,
    skip_unless_tool_is_at_least,
    skip_unless_tool_is_at_most,
)

gnumeric1 = load_fixture('test1.gnumeric')
gnumeric2 = load_fixture('test2.gnumeric')


def ssconvert_version():
    out = subprocess.check_output(('ssconvert', '--version'))
    return out.decode('utf-8').split()[2].replace("'", '')


def test_identification(gnumeric1):
    assert isinstance(gnumeric1, GnumericFile)

@@ -53,15 +43,9 @@ def differences(gnumeric1, gnumeric2):
    return gnumeric1.compare(gnumeric2).details


@skip_unless_tool_is_at_most('ssconvert', ssconvert_version, '1.12.42')
def test_diff_old(differences):
    expected_diff = get_data('gnumeric_1.12.42_expected_diff')
    assert differences[0].unified_diff == expected_diff


@skip_unless_tool_is_at_least('ssconvert', ssconvert_version, '1.12.43')
def test_diff_new(differences):
    expected_diff = get_data('gnumeric_1.12.43_expected_diff')
@skip_unless_tools_exist('ssconvert')
def test_diff(differences):
    expected_diff = get_data('gnumeric_expected_diff')
    assert differences[0].unified_diff == expected_diff


+0 −5
Original line number Diff line number Diff line
@@ -1,3 +1,3 @@
-foo
+bar