Skip to content

Commit c2853aa

Browse files
remove comments reported in issue
1 parent ecabd91 commit c2853aa

1 file changed

Lines changed: 0 additions & 12 deletions

File tree

frictionless/resource/__spec__/test_validate_schema.py

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -493,13 +493,6 @@ def test_resource_validate_fields_match_reordered_labels(fields_match, expected)
493493
@pytest.mark.parametrize(
494494
"fields_match, expected",
495495
[
496-
# Under `exact` the unmatched column has no field at all, so it is
497-
# also reported on every row; the name-matched modes give it an
498-
# `any` field and report it once.
499-
# FIXME: the `extra-cell` is a pre-existing bug -- that error is meant
500-
# for rows carrying more cells than the header, not for a column the
501-
# schema does not cover, which `extra-label` already reports in full.
502-
# Characterized here, to be fixed separately.
503496
("exact", [[None, 2, "", "extra-label"], [2, 2, "", "extra-cell"]]),
504497
("equal", [[None, 2, "", "extra-label"]]),
505498
("superset", [[None, 2, "", "extra-label"]]),
@@ -515,11 +508,6 @@ def test_resource_validate_fields_match_extra_label(fields_match, expected):
515508
@pytest.mark.parametrize(
516509
"fields_match, expected",
517510
[
518-
# Same asymmetry as above: `exact` keeps the field in the row stream
519-
# and reports a missing cell on every row.
520-
# FIXME: symmetric counterpart of the `extra-cell` bug above -- the
521-
# rows are not truncated, the schema simply declares a field the data
522-
# does not carry, which `missing-label` already reports.
523511
(
524512
"exact",
525513
[

0 commit comments

Comments
 (0)