2022-08-21 18:41:40 +02:00
|
|
|
# Standard Packages
|
|
|
|
import json
|
2023-08-31 21:55:17 +02:00
|
|
|
import os
|
2022-08-21 18:41:40 +02:00
|
|
|
|
|
|
|
# Internal Packages
|
2023-11-22 07:11:32 +01:00
|
|
|
from khoj.processor.content.org_mode.org_to_entries import OrgToEntries
|
2023-11-01 22:51:33 +01:00
|
|
|
from khoj.processor.text_to_entries import TextToEntries
|
2023-02-14 21:50:51 +01:00
|
|
|
from khoj.utils.helpers import is_none_or_empty
|
|
|
|
from khoj.utils.rawconfig import Entry
|
2023-08-31 21:55:17 +02:00
|
|
|
from khoj.utils.fs_syncer import get_org_files
|
|
|
|
from khoj.utils.rawconfig import TextContentConfig
|
2022-08-21 18:41:40 +02:00
|
|
|
|
|
|
|
|
2022-09-11 11:40:58 +02:00
|
|
|
def test_configure_heading_entry_to_jsonl(tmp_path):
|
2023-02-17 17:04:26 +01:00
|
|
|
"""Ensure entries with empty body are ignored, unless explicitly configured to index heading entries.
|
|
|
|
Property drawers not considered Body. Ignore control characters for evaluating if Body empty."""
|
2022-08-21 18:41:40 +02:00
|
|
|
# Arrange
|
2023-02-17 17:04:26 +01:00
|
|
|
entry = f"""*** Heading
|
2022-08-21 18:41:40 +02:00
|
|
|
:PROPERTIES:
|
|
|
|
:ID: 42-42-42
|
|
|
|
:END:
|
2022-09-11 11:40:58 +02:00
|
|
|
\t \r
|
2023-02-17 17:04:26 +01:00
|
|
|
"""
|
2023-08-31 21:55:17 +02:00
|
|
|
|
|
|
|
data = {
|
|
|
|
f"{tmp_path}": entry,
|
|
|
|
}
|
2022-08-21 18:41:40 +02:00
|
|
|
|
2022-09-11 11:40:58 +02:00
|
|
|
for index_heading_entries in [True, False]:
|
|
|
|
# Act
|
|
|
|
# Extract entries into jsonl from specified Org files
|
2023-11-01 22:51:33 +01:00
|
|
|
jsonl_string = OrgToEntries.convert_org_entries_to_jsonl(
|
|
|
|
OrgToEntries.convert_org_nodes_to_entries(
|
|
|
|
*OrgToEntries.extract_org_entries(org_files=data), index_heading_entries=index_heading_entries
|
2023-02-17 17:04:26 +01:00
|
|
|
)
|
|
|
|
)
|
2022-09-11 11:40:58 +02:00
|
|
|
jsonl_data = [json.loads(json_string) for json_string in jsonl_string.splitlines()]
|
|
|
|
|
|
|
|
# Assert
|
|
|
|
if index_heading_entries:
|
|
|
|
# Entry with empty body indexed when index_heading_entries set to True
|
|
|
|
assert len(jsonl_data) == 1
|
|
|
|
else:
|
|
|
|
# Entry with empty body ignored when index_heading_entries set to False
|
|
|
|
assert is_none_or_empty(jsonl_data)
|
2022-08-21 18:41:40 +02:00
|
|
|
|
|
|
|
|
2023-11-16 10:08:51 +01:00
|
|
|
def test_entry_split_when_exceeds_max_words():
|
2022-12-23 19:45:53 +01:00
|
|
|
"Ensure entries with compiled words exceeding max_words are split."
|
|
|
|
# Arrange
|
2023-11-16 10:08:51 +01:00
|
|
|
tmp_path = "/tmp/test.org"
|
2023-02-17 17:04:26 +01:00
|
|
|
entry = f"""*** Heading
|
2022-12-23 19:45:53 +01:00
|
|
|
\t\r
|
2023-03-30 07:38:45 +02:00
|
|
|
Body Line
|
2023-02-17 17:04:26 +01:00
|
|
|
"""
|
2023-08-31 21:55:17 +02:00
|
|
|
data = {
|
|
|
|
f"{tmp_path}": entry,
|
|
|
|
}
|
2023-11-16 10:08:51 +01:00
|
|
|
expected_heading = f"* Path: {tmp_path}\n** Heading"
|
2022-12-23 19:45:53 +01:00
|
|
|
|
|
|
|
# Act
|
|
|
|
# Extract Entries from specified Org files
|
2023-11-01 22:51:33 +01:00
|
|
|
entries, entry_to_file_map = OrgToEntries.extract_org_entries(org_files=data)
|
2022-12-23 19:45:53 +01:00
|
|
|
|
2022-12-26 01:45:40 +01:00
|
|
|
# Split each entry from specified Org files by max words
|
2023-11-01 22:51:33 +01:00
|
|
|
jsonl_string = OrgToEntries.convert_org_entries_to_jsonl(
|
|
|
|
TextToEntries.split_entries_by_max_tokens(
|
|
|
|
OrgToEntries.convert_org_nodes_to_entries(entries, entry_to_file_map), max_tokens=4
|
2022-12-23 19:45:53 +01:00
|
|
|
)
|
2023-02-17 17:04:26 +01:00
|
|
|
)
|
2022-12-23 19:45:53 +01:00
|
|
|
jsonl_data = [json.loads(json_string) for json_string in jsonl_string.splitlines()]
|
|
|
|
|
|
|
|
# Assert
|
|
|
|
assert len(jsonl_data) == 2
|
2023-05-03 11:47:33 +02:00
|
|
|
# Ensure compiled entries split by max_words start with entry heading (for search context)
|
2023-05-03 13:51:25 +02:00
|
|
|
assert all([entry["compiled"].startswith(expected_heading) for entry in jsonl_data])
|
2022-12-23 19:45:53 +01:00
|
|
|
|
|
|
|
|
2023-05-03 11:47:33 +02:00
|
|
|
def test_entry_split_drops_large_words():
|
2023-01-07 20:59:33 +01:00
|
|
|
"Ensure entries drops words larger than specified max word length from compiled version."
|
|
|
|
# Arrange
|
2023-02-17 17:04:26 +01:00
|
|
|
entry_text = f"""*** Heading
|
2023-01-07 20:59:33 +01:00
|
|
|
\t\r
|
|
|
|
Body Line 1
|
2023-02-17 17:04:26 +01:00
|
|
|
"""
|
2023-01-07 20:59:33 +01:00
|
|
|
entry = Entry(raw=entry_text, compiled=entry_text)
|
|
|
|
|
|
|
|
# Act
|
|
|
|
# Split entry by max words and drop words larger than max word length
|
2023-11-01 22:51:33 +01:00
|
|
|
processed_entry = TextToEntries.split_entries_by_max_tokens([entry], max_word_length=5)[0]
|
2023-01-07 20:59:33 +01:00
|
|
|
|
|
|
|
# Assert
|
|
|
|
# "Heading" dropped from compiled version because its over the set max word limit
|
|
|
|
assert len(processed_entry.compiled.split()) == len(entry_text.split()) - 1
|
|
|
|
|
|
|
|
|
2022-08-21 18:41:40 +02:00
|
|
|
def test_entry_with_body_to_jsonl(tmp_path):
|
|
|
|
"Ensure entries with valid body text are loaded."
|
|
|
|
# Arrange
|
2023-02-17 17:04:26 +01:00
|
|
|
entry = f"""*** Heading
|
2022-08-21 18:41:40 +02:00
|
|
|
:PROPERTIES:
|
|
|
|
:ID: 42-42-42
|
|
|
|
:END:
|
2022-09-10 12:11:58 +02:00
|
|
|
\t\r
|
|
|
|
Body Line 1
|
2023-02-17 17:04:26 +01:00
|
|
|
"""
|
2023-08-31 21:55:17 +02:00
|
|
|
data = {
|
|
|
|
f"{tmp_path}": entry,
|
|
|
|
}
|
2022-09-10 12:11:58 +02:00
|
|
|
|
|
|
|
# Act
|
|
|
|
# Extract Entries from specified Org files
|
2023-11-01 22:51:33 +01:00
|
|
|
entries, entry_to_file_map = OrgToEntries.extract_org_entries(org_files=data)
|
2022-09-10 12:11:58 +02:00
|
|
|
|
|
|
|
# Process Each Entry from All Notes Files
|
2023-11-01 22:51:33 +01:00
|
|
|
jsonl_string = OrgToEntries.convert_org_entries_to_jsonl(
|
|
|
|
OrgToEntries.convert_org_nodes_to_entries(entries, entry_to_file_map)
|
2023-02-17 17:04:26 +01:00
|
|
|
)
|
2022-09-10 12:11:58 +02:00
|
|
|
jsonl_data = [json.loads(json_string) for json_string in jsonl_string.splitlines()]
|
|
|
|
|
|
|
|
# Assert
|
|
|
|
assert len(jsonl_data) == 1
|
|
|
|
|
|
|
|
|
2023-03-01 19:11:33 +01:00
|
|
|
def test_file_with_entry_after_intro_text_to_jsonl(tmp_path):
|
|
|
|
"Ensure intro text before any headings is indexed."
|
|
|
|
# Arrange
|
|
|
|
entry = f"""
|
|
|
|
Intro text
|
|
|
|
|
|
|
|
* Entry Heading
|
|
|
|
entry body
|
|
|
|
"""
|
2023-08-31 21:55:17 +02:00
|
|
|
data = {
|
|
|
|
f"{tmp_path}": entry,
|
|
|
|
}
|
2023-03-01 19:11:33 +01:00
|
|
|
|
|
|
|
# Act
|
|
|
|
# Extract Entries from specified Org files
|
2023-11-01 22:51:33 +01:00
|
|
|
entry_nodes, file_to_entries = OrgToEntries.extract_org_entries(org_files=data)
|
2023-03-01 19:11:33 +01:00
|
|
|
|
|
|
|
# Process Each Entry from All Notes Files
|
2023-11-01 22:51:33 +01:00
|
|
|
entries = OrgToEntries.convert_org_nodes_to_entries(entry_nodes, file_to_entries)
|
|
|
|
jsonl_string = OrgToEntries.convert_org_entries_to_jsonl(entries)
|
2023-03-01 19:11:33 +01:00
|
|
|
jsonl_data = [json.loads(json_string) for json_string in jsonl_string.splitlines()]
|
|
|
|
|
|
|
|
# Assert
|
|
|
|
assert len(jsonl_data) == 2
|
|
|
|
|
|
|
|
|
2022-09-10 12:11:58 +02:00
|
|
|
def test_file_with_no_headings_to_jsonl(tmp_path):
|
|
|
|
"Ensure files with no heading, only body text are loaded."
|
|
|
|
# Arrange
|
2023-02-17 17:04:26 +01:00
|
|
|
entry = f"""
|
2022-09-10 12:11:58 +02:00
|
|
|
- Bullet point 1
|
|
|
|
- Bullet point 2
|
2023-02-17 17:04:26 +01:00
|
|
|
"""
|
2023-08-31 21:55:17 +02:00
|
|
|
data = {
|
|
|
|
f"{tmp_path}": entry,
|
|
|
|
}
|
2022-08-21 18:41:40 +02:00
|
|
|
|
|
|
|
# Act
|
|
|
|
# Extract Entries from specified Org files
|
2023-11-01 22:51:33 +01:00
|
|
|
entry_nodes, file_to_entries = OrgToEntries.extract_org_entries(org_files=data)
|
2022-08-21 18:41:40 +02:00
|
|
|
|
|
|
|
# Process Each Entry from All Notes Files
|
2023-11-01 22:51:33 +01:00
|
|
|
entries = OrgToEntries.convert_org_nodes_to_entries(entry_nodes, file_to_entries)
|
|
|
|
jsonl_string = OrgToEntries.convert_org_entries_to_jsonl(entries)
|
2023-02-17 17:04:26 +01:00
|
|
|
jsonl_data = [json.loads(json_string) for json_string in jsonl_string.splitlines()]
|
2022-08-21 18:41:40 +02:00
|
|
|
|
|
|
|
# Assert
|
|
|
|
assert len(jsonl_data) == 1
|
|
|
|
|
|
|
|
|
2022-09-12 09:39:39 +02:00
|
|
|
def test_get_org_files(tmp_path):
|
|
|
|
"Ensure Org files specified via input-filter, input-files extracted"
|
|
|
|
# Arrange
|
|
|
|
# Include via input-filter globs
|
|
|
|
group1_file1 = create_file(tmp_path, filename="group1-file1.org")
|
|
|
|
group1_file2 = create_file(tmp_path, filename="group1-file2.org")
|
|
|
|
group2_file1 = create_file(tmp_path, filename="group2-file1.org")
|
|
|
|
group2_file2 = create_file(tmp_path, filename="group2-file2.org")
|
|
|
|
# Include via input-file field
|
|
|
|
orgfile1 = create_file(tmp_path, filename="orgfile1.org")
|
|
|
|
# Not included by any filter
|
|
|
|
create_file(tmp_path, filename="orgfile2.org")
|
|
|
|
create_file(tmp_path, filename="text1.txt")
|
|
|
|
|
2023-08-31 21:55:17 +02:00
|
|
|
expected_files = set(
|
|
|
|
[
|
|
|
|
os.path.join(tmp_path, file.name)
|
|
|
|
for file in [group1_file1, group1_file2, group2_file1, group2_file2, orgfile1]
|
|
|
|
]
|
|
|
|
)
|
2022-09-12 09:39:39 +02:00
|
|
|
|
|
|
|
# Setup input-files, input-filters
|
2023-02-17 17:04:26 +01:00
|
|
|
input_files = [tmp_path / "orgfile1.org"]
|
|
|
|
input_filter = [tmp_path / "group1*.org", tmp_path / "group2*.org"]
|
2022-09-12 09:39:39 +02:00
|
|
|
|
2023-08-31 21:55:17 +02:00
|
|
|
org_config = TextContentConfig(
|
|
|
|
input_files=input_files,
|
|
|
|
input_filter=[str(filter) for filter in input_filter],
|
|
|
|
compressed_jsonl=tmp_path / "test.jsonl",
|
|
|
|
embeddings_file=tmp_path / "test_embeddings.jsonl",
|
|
|
|
)
|
|
|
|
|
2022-09-12 09:39:39 +02:00
|
|
|
# Act
|
2023-08-31 21:55:17 +02:00
|
|
|
extracted_org_files = get_org_files(org_config)
|
2022-09-12 09:39:39 +02:00
|
|
|
|
|
|
|
# Assert
|
|
|
|
assert len(extracted_org_files) == 5
|
2023-08-31 21:55:17 +02:00
|
|
|
assert set(extracted_org_files.keys()) == expected_files
|
2022-09-12 09:39:39 +02:00
|
|
|
|
|
|
|
|
2023-01-17 16:42:36 +01:00
|
|
|
def test_extract_entries_with_different_level_headings(tmp_path):
|
|
|
|
"Extract org entries with different level headings."
|
|
|
|
# Arrange
|
2023-02-17 17:04:26 +01:00
|
|
|
entry = f"""
|
2023-01-17 16:42:36 +01:00
|
|
|
* Heading 1
|
|
|
|
** Heading 2
|
2023-02-17 17:04:26 +01:00
|
|
|
"""
|
2023-08-31 21:55:17 +02:00
|
|
|
data = {
|
|
|
|
f"{tmp_path}": entry,
|
|
|
|
}
|
2023-01-17 16:42:36 +01:00
|
|
|
|
|
|
|
# Act
|
|
|
|
# Extract Entries from specified Org files
|
2023-11-01 22:51:33 +01:00
|
|
|
entries, _ = OrgToEntries.extract_org_entries(org_files=data)
|
2023-01-17 16:42:36 +01:00
|
|
|
|
|
|
|
# Assert
|
|
|
|
assert len(entries) == 2
|
2023-02-17 17:04:26 +01:00
|
|
|
assert f"{entries[0]}".startswith("* Heading 1")
|
|
|
|
assert f"{entries[1]}".startswith("** Heading 2")
|
2023-01-17 16:42:36 +01:00
|
|
|
|
|
|
|
|
2022-08-21 18:41:40 +02:00
|
|
|
# Helper Functions
|
2022-09-12 09:39:39 +02:00
|
|
|
def create_file(tmp_path, entry=None, filename="test.org"):
|
|
|
|
org_file = tmp_path / filename
|
2022-08-21 18:41:40 +02:00
|
|
|
org_file.touch()
|
2022-09-12 09:39:39 +02:00
|
|
|
if entry:
|
|
|
|
org_file.write_text(entry)
|
|
|
|
return org_file
|