Skip to content

Commit a91c226

Browse files
committed
update tests
1 parent 1e12fec commit a91c226

File tree

3 files changed

+25
-16
lines changed

3 files changed

+25
-16
lines changed

test_unstructured_inference/inference/test_layout.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
EmbeddedTextRegion,
1414
ImageTextRegion,
1515
)
16+
from unstructured_inference.constants import IsExtracted
1617
from unstructured_inference.models.unstructuredmodel import (
1718
UnstructuredElementExtractionModel,
1819
UnstructuredObjectDetectionModel,
@@ -34,7 +35,7 @@ def mock_initial_layout():
3435
6,
3536
8,
3637
text="A very repetitive narrative. " * 10,
37-
is_extracted=True,
38+
is_extracted=IsExtracted.TRUE,
3839
)
3940

4041
title_block = EmbeddedTextRegion.from_coords(
@@ -43,7 +44,7 @@ def mock_initial_layout():
4344
3,
4445
4,
4546
text="A Catchy Title",
46-
is_extracted=True,
47+
is_extracted=IsExtracted.TRUE,
4748
)
4849

4950
return [text_block, title_block]

test_unstructured_inference/inference/test_layout_element.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
from unstructured_inference.inference.layoutelement import LayoutElement, TextRegion
2-
from unstructured_inference.constants import Source
2+
from unstructured_inference.constants import IsExtracted, Source
33

44

55
def test_layout_element_to_dict(mock_layout_element):
@@ -28,20 +28,22 @@ def test_layoutelement_inheritance_works_correctly():
2828

2929
# Create a TextRegion with both source and text_source
3030
region = TextRegion.from_coords(
31-
0, 0, 10, 10, text="test", source=Source.YOLOX, is_extracted=True
31+
0, 0, 10, 10, text="test", source=Source.YOLOX, is_extracted=IsExtracted.TRUE
3232
)
3333

3434
# Convert to LayoutElement
3535
element = LayoutElement.from_region(region)
3636

3737
# Check that both properties are preserved
3838
assert element.source == Source.YOLOX, "LayoutElement should inherit source from TextRegion"
39-
assert element.is_extracted, "LayoutElement should inherit is_extracted from TextRegion"
39+
assert (
40+
element.is_extracted == IsExtracted.TRUE
41+
), "LayoutElement should inherit is_extracted from TextRegion"
4042

4143
# Check that to_dict() works correctly
4244
d = element.to_dict()
4345
assert d["source"] == Source.YOLOX
44-
assert d["is_extracted"]
46+
assert d["is_extracted"] == IsExtracted.TRUE
4547

4648
# Check that we can set source directly on LayoutElement
4749
element.source = Source.DETECTRON2_ONNX

test_unstructured_inference/test_elements.py

Lines changed: 16 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
import numpy as np
66
import pytest
77

8-
from unstructured_inference.constants import Source
8+
from unstructured_inference.constants import IsExtracted, Source
99
from unstructured_inference.inference import elements
1010
from unstructured_inference.inference.elements import (
1111
Rectangle,
@@ -456,8 +456,8 @@ def test_layoutelements_concatenate():
456456
]
457457
),
458458
texts=np.array(["0", "1", "2", "3", "4"]),
459-
is_extracted_array=np.array([True] * 5),
460-
is_extracted=True,
459+
is_extracted_array=np.array([IsExtracted.TRUE] * 5),
460+
is_extracted=IsExtracted.TRUE,
461461
),
462462
LayoutElements(
463463
element_coords=np.array(
@@ -473,7 +473,7 @@ def test_layoutelements_concatenate():
473473
sources=np.array([Source.YOLOX] * 5),
474474
source=Source.YOLOX,
475475
is_extracted_array=np.array([] * 5),
476-
is_extracted=True,
476+
is_extracted=IsExtracted.TRUE,
477477
element_probs=np.array([0.0, 0.1, 0.2, 0.3, 0.4]),
478478
),
479479
],
@@ -495,15 +495,17 @@ def test_textregions_from_list_collects_sources():
495495
from unstructured_inference.inference.elements import TextRegion
496496

497497
regions = [
498-
TextRegion.from_coords(0, 0, 10, 10, text="first", source=Source.YOLOX, is_extracted=True),
498+
TextRegion.from_coords(
499+
0, 0, 10, 10, text="first", source=Source.YOLOX, is_extracted=IsExtracted.TRUE
500+
),
499501
TextRegion.from_coords(
500502
10,
501503
10,
502504
20,
503505
20,
504506
text="second",
505507
source=Source.DETECTRON2_ONNX,
506-
is_extracted=True,
508+
is_extracted=IsExtracted.TRUE,
507509
),
508510
]
509511

@@ -529,7 +531,9 @@ def test_textregions_iter_elements_preserves_source():
529531
from unstructured_inference.inference.elements import TextRegion
530532

531533
regions = [
532-
TextRegion.from_coords(0, 0, 10, 10, text="first", source=Source.YOLOX, is_extracted=True),
534+
TextRegion.from_coords(
535+
0, 0, 10, 10, text="first", source=Source.YOLOX, is_extracted=IsExtracted.TRUE
536+
),
533537
]
534538
text_regions = TextRegions.from_list(regions)
535539

@@ -544,15 +548,17 @@ def test_textregions_slice_preserves_sources():
544548
from unstructured_inference.inference.elements import TextRegion
545549

546550
regions = [
547-
TextRegion.from_coords(0, 0, 10, 10, text="first", source=Source.YOLOX, is_extracted=True),
551+
TextRegion.from_coords(
552+
0, 0, 10, 10, text="first", source=Source.YOLOX, is_extracted=IsExtracted.TRUE
553+
),
548554
TextRegion.from_coords(
549555
10,
550556
10,
551557
20,
552558
20,
553559
text="second",
554560
source=Source.DETECTRON2_ONNX,
555-
is_extracted=True,
561+
is_extracted=IsExtracted.TRUE,
556562
),
557563
]
558564
text_regions = TextRegions.from_list(regions)
@@ -581,7 +587,7 @@ def test_textregions_from_coords_accepts_source():
581587
"""Test that TextRegion.from_coords() accepts source parameter"""
582588
# This should fail because from_coords() doesn't accept source parameter
583589
region = TextRegion.from_coords(
584-
0, 0, 10, 10, text="test", source=Source.YOLOX, is_extracted=True
590+
0, 0, 10, 10, text="test", source=Source.YOLOX, is_extracted=IsExtracted.TRUE
585591
)
586592

587593
assert region.source == Source.YOLOX

0 commit comments

Comments
 (0)