Function test_hash_registry_remove_by_hash

tests/test_remove.py:224–253

Issue #58 / Bug 1: `remove_by_hash` is the hash-keyed sibling of `remove_by_doc_name`. It exists so callers that already have the file_hash in hand (e.g. the CLI after `_resolve_doc_identifier`) don't need to round-trip through the `doc_name` slug — that round trip is what silently n

(tmp_path)

def test_hash_registry_remove_by_hash(tmp_path):
    """Issue #58 / Bug 1: `remove_by_hash` is the hash-keyed sibling of
    `remove_by_doc_name`. It exists so callers that already have the
    file_hash in hand (e.g. the CLI after `_resolve_doc_identifier`)
    don't need to round-trip through the `doc_name` slug — that round
    trip is what silently no-ops for legacy registry entries lacking a
    `doc_name` key (ingested before commit c504e26).
    """
    path = tmp_path / "hashes.json"
    path.write_text(
        json.dumps(
            {
                "h_modern": {"name": "a.pdf", "doc_name": "a-h_modern", "type": "short"},
                "h_legacy": {"name": "b.pdf", "type": "short"},  # no doc_name
            }
        )
    )

    reg = HashRegistry(path)

    # Modern entry: removable by hash.
    assert reg.remove_by_hash("h_modern") is True
    assert reg.remove_by_hash("h_modern") is False  # idempotent
    # Legacy entry (no doc_name): removable by hash too — that's the point.
    assert reg.remove_by_hash("h_legacy") is True
    # Unknown hash: returns False, doesn't raise.
    assert reg.remove_by_hash("never-existed") is False

    assert reg.all_entries() == {}
    assert json.loads(path.read_text()) == {}


# ---------------------------------------------------------------------------