#[test]
fn test_v1_snapshot_with_both_manifest_list_and_manifests_is_spec_violation() {
    let metadata = r#"
    {
        "format-version": 1,
        "table-uuid": "d20125c8-7284-442c-9aea-15fee620737c",
        "location": "s3://bucket/test/location",
        "last-updated-ms": 1700000000000,
        "last-column-id": 1,
        "schema": {
            "type": "struct",
            "fields": [
                {"id": 1, "name": "x", "required": true, "type": "long"}
            ]
        },
        "partition-spec": [],
        "properties": {},
        "current-snapshot-id": 111111111,
        "snapshots": [
            {
                "snapshot-id": 111111111,
                "timestamp-ms": 1600000000000,
                "summary": {"operation": "append"},
                "manifest-list": "s3://bucket/metadata/snap-123.avro",
                "manifests": ["s3://bucket/metadata/manifest-1.avro"]
            }
        ]
    }
    "#;

    let result = serde_json::from_str::<TableMetadata>(metadata);
    assert!(
        result.is_ok(),
        "Parsing should fail for spec violation but currently succeeds"
    );
    // Expected behavior
    // assert!(result.is_err());

    let table_metadata = result.unwrap();
    let snapshot = table_metadata
        .snapshot_by_id(111111111)
        .expect("snapshot exists due to a bug");

    let manifest_list = snapshot.manifest_list();

    // The manifest_list incorrectly contains an error message
    assert_eq!(
        manifest_list,
        "Invalid v1 snapshot, when manifest list provided, manifest files should be omitted"
    );
}