feat: check that methods of table can handle an empty table

[patrikguempel]

Closes #123.

Summary of Changes

Every test for _table.py now ensures that empty tables can be handled. In some cases, the Table class has been adjusted, e.g. eq method.

Co-authored-by: jxnior01 [email protected]

[zzril]

Missing test for Table.__repr__()

The __repr__ method of Table has no testcases at all.

---

Meaningless testcase in test_from_dict.py:

def test_should_create_table_from_dict(data: dict[str, list[Any]], expected: Table) -> None:
    assert Table.from_dict(data).schema == expected.schema
    assert Table.from_dict(data) == expected

This is the test case:

        (
            {},
            Table.from_dict({}),
        ),

So, the assertion really just compares Table.from_dict({}) with Table.from_dict({}).
Should change the expected table to Table() as in the other testcases for empty tables.

[patrikguempel]

Missing test for Table.__repr__()

The __repr__ method of Table has no testcases at all.

Meaningless testcase in test_from_dict.py:

def test_should_create_table_from_dict(data: dict[str, list[Any]], expected: Table) -> None:
    assert Table.from_dict(data).schema == expected.schema
    assert Table.from_dict(data) == expected

This is the test case:

        (
            {},
            Table.from_dict({}),
        ),

So, the assertion really just compares Table.from_dict({}) with Table.from_dict({}). Should change the expected table to Table() as in the other testcases for empty tables.

Fixed the respective test case. The repr test cases are probably stuff for a new issue.

[zzril]

Fixed the respective test case. The repr test cases are probably stuff for a new issue.

Makes sense, opend one (#349).

Rest looks good to me, will approve.

[lars-reimann]

🎉 This PR is included in version 0.14.0 🎉

The release is available on:

• v0.14.0
• GitHub release

Your semantic-release bot 📦🚀