feat: add new tags (#16)

tests/application_test.py

@@ -42,7 +42,7 @@ def test_abort(test_application, mocker, capsys) -> None:
     assert "Done!" in captured.out
 
 
-def test_add_metadata_frontmatter_success(test_application, mocker, capsys) -> None:
+def test_add_metadata_frontmatter(test_application, mocker, capsys) -> None:
     """Test adding new metadata to the vault."""
     app = test_application
     app._load_vault()
@@ -69,7 +69,7 @@ def test_add_metadata_frontmatter_success(test_application, mocker, capsys) -> N
     assert captured.out == Regex(r"SUCCESS +\| Added metadata to.*\d+.*notes", re.DOTALL)
 
 
-def test_add_metadata_inline_success(test_application, mocker, capsys) -> None:
+def test_add_metadata_inline(test_application, mocker, capsys) -> None:
     """Test adding new metadata to the vault."""
     app = test_application
     app._load_vault()
@@ -96,6 +96,29 @@ def test_add_metadata_inline_success(test_application, mocker, capsys) -> None:
     assert captured.out == Regex(r"SUCCESS +\| Added metadata to.*\d+.*notes", re.DOTALL)
 
 
+def test_add_metadata_tag(test_application, mocker, capsys) -> None:
+    """Test adding new metadata to the vault."""
+    app = test_application
+    app._load_vault()
+    mocker.patch(
+        "obsidian_metadata.models.application.Questions.ask_application_main",
+        side_effect=["add_metadata", KeyError],
+    )
+    mocker.patch(
+        "obsidian_metadata.models.application.Questions.ask_area",
+        return_value=MetadataType.TAGS,
+    )
+    mocker.patch(
+        "obsidian_metadata.models.application.Questions.ask_new_tag",
+        return_value="new_tag",
+    )
+
+    with pytest.raises(KeyError):
+        app.application_main()
+    captured = capsys.readouterr()
+    assert captured.out == Regex(r"SUCCESS +\| Added metadata to.*\d+.*notes", re.DOTALL)
+
+
 def test_delete_inline_tag(test_application, mocker, capsys) -> None:
     """Test renaming an inline tag."""
     app = test_application

tests/metadata_test.py

@@ -69,6 +69,455 @@ repeated_key:: repeated_key_value2
 """
 
 
+def test_frontmatter_create() -> None:
+    """Test frontmatter creation."""
+    frontmatter = Frontmatter(INLINE_CONTENT)
+    assert frontmatter.dict == {}
+
+    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
+    assert frontmatter.dict == {
+        "frontmatter_Key1": ["frontmatter_Key1_value"],
+        "frontmatter_Key2": ["article", "note"],
+        "shared_key1": ["shared_key1_value"],
+        "tags": ["tag_1", "tag_2", "📅/tag_3"],
+    }
+    assert frontmatter.dict_original == {
+        "frontmatter_Key1": ["frontmatter_Key1_value"],
+        "frontmatter_Key2": ["article", "note"],
+        "shared_key1": ["shared_key1_value"],
+        "tags": ["tag_1", "tag_2", "📅/tag_3"],
+    }
+
+
+def test_frontmatter_contains() -> None:
+    """Test frontmatter contains."""
+    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
+
+    assert frontmatter.contains("frontmatter_Key1") is True
+    assert frontmatter.contains("frontmatter_Key2", "article") is True
+    assert frontmatter.contains("frontmatter_Key3") is False
+    assert frontmatter.contains("frontmatter_Key2", "no value") is False
+
+    assert frontmatter.contains(r"\d$", is_regex=True) is True
+    assert frontmatter.contains(r"^\d", is_regex=True) is False
+    assert frontmatter.contains("key", r"_\d", is_regex=True) is False
+    assert frontmatter.contains("key", r"\w\d_", is_regex=True) is True
+
+
+def test_frontmatter_add() -> None:
+    """Test frontmatter add."""
+    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
+
+    assert frontmatter.add("frontmatter_Key1") is False
+    assert frontmatter.add("added_key") is True
+    assert frontmatter.dict == {
+        "added_key": [],
+        "frontmatter_Key1": ["frontmatter_Key1_value"],
+        "frontmatter_Key2": ["article", "note"],
+        "shared_key1": ["shared_key1_value"],
+        "tags": ["tag_1", "tag_2", "📅/tag_3"],
+    }
+
+    assert frontmatter.add("added_key", "added_value") is True
+    assert frontmatter.dict == {
+        "added_key": ["added_value"],
+        "frontmatter_Key1": ["frontmatter_Key1_value"],
+        "frontmatter_Key2": ["article", "note"],
+        "shared_key1": ["shared_key1_value"],
+        "tags": ["tag_1", "tag_2", "📅/tag_3"],
+    }
+
+    assert frontmatter.add("added_key", "added_value_2") is True
+    assert frontmatter.dict == {
+        "added_key": ["added_value", "added_value_2"],
+        "frontmatter_Key1": ["frontmatter_Key1_value"],
+        "frontmatter_Key2": ["article", "note"],
+        "shared_key1": ["shared_key1_value"],
+        "tags": ["tag_1", "tag_2", "📅/tag_3"],
+    }
+
+    assert frontmatter.add("added_key", ["added_value_3", "added_value_4"]) is True
+    assert frontmatter.dict == {
+        "added_key": ["added_value", "added_value_2", "added_value_3", "added_value_4"],
+        "frontmatter_Key1": ["frontmatter_Key1_value"],
+        "frontmatter_Key2": ["article", "note"],
+        "shared_key1": ["shared_key1_value"],
+        "tags": ["tag_1", "tag_2", "📅/tag_3"],
+    }
+
+    assert frontmatter.add("added_key2", ["added_value_1", "added_value_2"]) is True
+    assert frontmatter.dict == {
+        "added_key": ["added_value", "added_value_2", "added_value_3", "added_value_4"],
+        "added_key2": ["added_value_1", "added_value_2"],
+        "frontmatter_Key1": ["frontmatter_Key1_value"],
+        "frontmatter_Key2": ["article", "note"],
+        "shared_key1": ["shared_key1_value"],
+        "tags": ["tag_1", "tag_2", "📅/tag_3"],
+    }
+
+    assert frontmatter.add("added_key3", "added_value_1") is True
+    assert frontmatter.dict == {
+        "added_key": ["added_value", "added_value_2", "added_value_3", "added_value_4"],
+        "added_key2": ["added_value_1", "added_value_2"],
+        "added_key3": ["added_value_1"],
+        "frontmatter_Key1": ["frontmatter_Key1_value"],
+        "frontmatter_Key2": ["article", "note"],
+        "shared_key1": ["shared_key1_value"],
+        "tags": ["tag_1", "tag_2", "📅/tag_3"],
+    }
+
+    assert frontmatter.add("added_key3", "added_value_1") is False
+
+
+def test_frontmatter_rename() -> None:
+    """Test frontmatter rename."""
+    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
+    assert frontmatter.dict == {
+        "frontmatter_Key1": ["frontmatter_Key1_value"],
+        "frontmatter_Key2": ["article", "note"],
+        "shared_key1": ["shared_key1_value"],
+        "tags": ["tag_1", "tag_2", "📅/tag_3"],
+    }
+
+    assert frontmatter.rename("no key", "new key") is False
+    assert frontmatter.rename("tags", "no tag", "new key") is False
+
+    assert frontmatter.has_changes() is False
+    assert frontmatter.rename("tags", "tag_2", "new tag") is True
+
+    assert frontmatter.dict["tags"] == ["new tag", "tag_1", "📅/tag_3"]
+    assert frontmatter.rename("tags", "old_tags") is True
+    assert frontmatter.dict["old_tags"] == ["new tag", "tag_1", "📅/tag_3"]
+    assert "tags" not in frontmatter.dict
+
+    assert frontmatter.has_changes() is True
+
+
+def test_frontmatter_delete() -> None:
+    """Test Frontmatter delete method."""
+    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
+    assert frontmatter.dict == {
+        "frontmatter_Key1": ["frontmatter_Key1_value"],
+        "frontmatter_Key2": ["article", "note"],
+        "shared_key1": ["shared_key1_value"],
+        "tags": ["tag_1", "tag_2", "📅/tag_3"],
+    }
+
+    assert frontmatter.delete("no key") is False
+    assert frontmatter.delete("tags", "no value") is False
+    assert frontmatter.delete(r"\d{3}") is False
+    assert frontmatter.has_changes() is False
+    assert frontmatter.delete("tags", "tag_2") is True
+    assert frontmatter.dict["tags"] == ["tag_1", "📅/tag_3"]
+    assert frontmatter.delete("tags") is True
+    assert "tags" not in frontmatter.dict
+    assert frontmatter.has_changes() is True
+    assert frontmatter.delete("shared_key1", r"\w+") is True
+    assert frontmatter.dict["shared_key1"] == []
+    assert frontmatter.delete(r"\w.tter") is True
+    assert frontmatter.dict == {"shared_key1": []}
+
+
+def test_frontmatter_yaml_conversion():
+    """Test Frontmatter to_yaml method."""
+    new_frontmatter: str = """\
+tags:
+  - tag_1
+  - tag_2
+  - 📅/tag_3
+frontmatter_Key1: frontmatter_Key1_value
+frontmatter_Key2:
+  - article
+  - note
+shared_key1: shared_key1_value
+"""
+    new_frontmatter_sorted: str = """\
+frontmatter_Key1: frontmatter_Key1_value
+frontmatter_Key2:
+  - article
+  - note
+shared_key1: shared_key1_value
+tags:
+  - tag_1
+  - tag_2
+  - 📅/tag_3
+"""
+    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
+    assert frontmatter.to_yaml() == new_frontmatter
+    assert frontmatter.to_yaml(sort_keys=True) == new_frontmatter_sorted
+
+
+def test_inline_metadata_add() -> None:
+    """Test inline add."""
+    inline = InlineMetadata(INLINE_CONTENT)
+
+    assert inline.add("bold_key1") is False
+    assert inline.add("bold_key1", "bold_key1_value") is False
+    assert inline.add("added_key") is True
+    assert inline.dict == {
+        "added_key": [],
+        "bold_key1": ["bold_key1_value"],
+        "bold_key2": ["bold_key2_value"],
+        "emoji_📅_key": ["emoji_📅_key_value"],
+        "in_text_key1": ["in_text_key1_value"],
+        "in_text_key2": ["in_text_key2_value"],
+        "link_key": ["link_key_value"],
+        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
+        "tag_key": ["tag_key_value"],
+    }
+
+    assert inline.add("added_key1", "added_value") is True
+    assert inline.dict == {
+        "added_key": [],
+        "added_key1": ["added_value"],
+        "bold_key1": ["bold_key1_value"],
+        "bold_key2": ["bold_key2_value"],
+        "emoji_📅_key": ["emoji_📅_key_value"],
+        "in_text_key1": ["in_text_key1_value"],
+        "in_text_key2": ["in_text_key2_value"],
+        "link_key": ["link_key_value"],
+        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
+        "tag_key": ["tag_key_value"],
+    }
+
+    with pytest.raises(ValueError):
+        assert inline.add("added_key1", "added_value_2") is True
+
+    assert inline.dict == {
+        "added_key": [],
+        "added_key1": ["added_value"],
+        "bold_key1": ["bold_key1_value"],
+        "bold_key2": ["bold_key2_value"],
+        "emoji_📅_key": ["emoji_📅_key_value"],
+        "in_text_key1": ["in_text_key1_value"],
+        "in_text_key2": ["in_text_key2_value"],
+        "link_key": ["link_key_value"],
+        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
+        "tag_key": ["tag_key_value"],
+    }
+
+    assert inline.add("added_key", "added_value")
+    assert inline.dict == {
+        "added_key": ["added_value"],
+        "added_key1": ["added_value"],
+        "bold_key1": ["bold_key1_value"],
+        "bold_key2": ["bold_key2_value"],
+        "emoji_📅_key": ["emoji_📅_key_value"],
+        "in_text_key1": ["in_text_key1_value"],
+        "in_text_key2": ["in_text_key2_value"],
+        "link_key": ["link_key_value"],
+        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
+        "tag_key": ["tag_key_value"],
+    }
+
+
+def test_inline_metadata_contains() -> None:
+    """Test inline metadata contains method."""
+    inline = InlineMetadata(INLINE_CONTENT)
+
+    assert inline.contains("bold_key1") is True
+    assert inline.contains("bold_key2", "bold_key2_value") is True
+    assert inline.contains("bold_key3") is False
+    assert inline.contains("bold_key2", "no value") is False
+
+    assert inline.contains(r"\w{4}_key", is_regex=True) is True
+    assert inline.contains(r"^\d", is_regex=True) is False
+    assert inline.contains("1$", r"\d_value", is_regex=True) is True
+    assert inline.contains("key", r"^\d_value", is_regex=True) is False
+
+
+def test_inline_metadata_create() -> None:
+    """Test inline metadata creation."""
+    inline = InlineMetadata(FRONTMATTER_CONTENT)
+    assert inline.dict == {}
+    inline = InlineMetadata(INLINE_CONTENT)
+    assert inline.dict == {
+        "bold_key1": ["bold_key1_value"],
+        "bold_key2": ["bold_key2_value"],
+        "emoji_📅_key": ["emoji_📅_key_value"],
+        "in_text_key1": ["in_text_key1_value"],
+        "in_text_key2": ["in_text_key2_value"],
+        "link_key": ["link_key_value"],
+        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
+        "tag_key": ["tag_key_value"],
+    }
+    assert inline.dict_original == {
+        "bold_key1": ["bold_key1_value"],
+        "bold_key2": ["bold_key2_value"],
+        "emoji_📅_key": ["emoji_📅_key_value"],
+        "in_text_key1": ["in_text_key1_value"],
+        "in_text_key2": ["in_text_key2_value"],
+        "link_key": ["link_key_value"],
+        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
+        "tag_key": ["tag_key_value"],
+    }
+
+
+def test_inline_metadata_delete() -> None:
+    """Test inline metadata delete."""
+    inline = InlineMetadata(INLINE_CONTENT)
+    assert inline.dict == {
+        "bold_key1": ["bold_key1_value"],
+        "bold_key2": ["bold_key2_value"],
+        "emoji_📅_key": ["emoji_📅_key_value"],
+        "in_text_key1": ["in_text_key1_value"],
+        "in_text_key2": ["in_text_key2_value"],
+        "link_key": ["link_key_value"],
+        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
+        "tag_key": ["tag_key_value"],
+    }
+
+    assert inline.delete("no key") is False
+    assert inline.delete("repeated_key", "no value") is False
+    assert inline.has_changes() is False
+    assert inline.delete("repeated_key", "repeated_key_value1") is True
+    assert inline.dict["repeated_key"] == ["repeated_key_value2"]
+    assert inline.delete("repeated_key") is True
+    assert "repeated_key" not in inline.dict
+    assert inline.has_changes() is True
+    assert inline.delete(r"\d{3}") is False
+    assert inline.delete(r"bold_key\d") is True
+    assert inline.dict == {
+        "emoji_📅_key": ["emoji_📅_key_value"],
+        "in_text_key1": ["in_text_key1_value"],
+        "in_text_key2": ["in_text_key2_value"],
+        "link_key": ["link_key_value"],
+        "tag_key": ["tag_key_value"],
+    }
+    assert inline.delete("emoji_📅_key", ".*📅.*") is True
+    assert inline.dict == {
+        "emoji_📅_key": [],
+        "in_text_key1": ["in_text_key1_value"],
+        "in_text_key2": ["in_text_key2_value"],
+        "link_key": ["link_key_value"],
+        "tag_key": ["tag_key_value"],
+    }
+
+
+def test_inline_metadata_rename() -> None:
+    """Test inline metadata rename."""
+    inline = InlineMetadata(INLINE_CONTENT)
+    assert inline.dict == {
+        "bold_key1": ["bold_key1_value"],
+        "bold_key2": ["bold_key2_value"],
+        "emoji_📅_key": ["emoji_📅_key_value"],
+        "in_text_key1": ["in_text_key1_value"],
+        "in_text_key2": ["in_text_key2_value"],
+        "link_key": ["link_key_value"],
+        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
+        "tag_key": ["tag_key_value"],
+    }
+
+    assert inline.rename("no key", "new key") is False
+    assert inline.rename("repeated_key", "no value", "new key") is False
+    assert inline.has_changes() is False
+    assert inline.rename("repeated_key", "repeated_key_value1", "new value") is True
+    assert inline.dict["repeated_key"] == ["new value", "repeated_key_value2"]
+    assert inline.rename("repeated_key", "old_key") is True
+    assert inline.dict["old_key"] == ["new value", "repeated_key_value2"]
+    assert "repeated_key" not in inline.dict
+    assert inline.has_changes() is True
+
+
+def test_inline_tags_add() -> None:
+    """Test inline tags add."""
+    tags = InlineTags(INLINE_CONTENT)
+
+    assert tags.add("bold_tag") is False
+    assert tags.add("new_tag") is True
+    assert tags.list == [
+        "bold_tag",
+        "in_text_tag",
+        "inline_tag_top1",
+        "inline_tag_top2",
+        "new_tag",
+        "tag_key_value",
+    ]
+
+
+def test_inline_tags_contains() -> None:
+    """Test inline tags contains."""
+    tags = InlineTags(INLINE_CONTENT)
+    assert tags.contains("bold_tag") is True
+    assert tags.contains("no tag") is False
+
+    assert tags.contains(r"\w_\w", is_regex=True) is True
+    assert tags.contains(r"\d_\d", is_regex=True) is False
+
+
+def test_inline_tags_create() -> None:
+    """Test inline tags creation."""
+    tags = InlineTags(FRONTMATTER_CONTENT)
+    tags.metadata_key
+    assert tags.list == []
+
+    tags = InlineTags(INLINE_CONTENT)
+    assert tags.list == [
+        "bold_tag",
+        "in_text_tag",
+        "inline_tag_top1",
+        "inline_tag_top2",
+        "tag_key_value",
+    ]
+    assert tags.list_original == [
+        "bold_tag",
+        "in_text_tag",
+        "inline_tag_top1",
+        "inline_tag_top2",
+        "tag_key_value",
+    ]
+
+
+def test_inline_tags_delete() -> None:
+    """Test inline tags delete."""
+    tags = InlineTags(INLINE_CONTENT)
+    assert tags.list == [
+        "bold_tag",
+        "in_text_tag",
+        "inline_tag_top1",
+        "inline_tag_top2",
+        "tag_key_value",
+    ]
+
+    assert tags.delete("no tag") is False
+    assert tags.has_changes() is False
+    assert tags.delete("bold_tag") is True
+    assert tags.list == [
+        "in_text_tag",
+        "inline_tag_top1",
+        "inline_tag_top2",
+        "tag_key_value",
+    ]
+    assert tags.has_changes() is True
+    assert tags.delete(r"\d{3}") is False
+    assert tags.delete(r"inline_tag_top\d") is True
+    assert tags.list == ["in_text_tag", "tag_key_value"]
+
+
+def test_inline_tags_rename() -> None:
+    """Test inline tags rename."""
+    tags = InlineTags(INLINE_CONTENT)
+    assert tags.list == [
+        "bold_tag",
+        "in_text_tag",
+        "inline_tag_top1",
+        "inline_tag_top2",
+        "tag_key_value",
+    ]
+
+    assert tags.rename("no tag", "new tag") is False
+    assert tags.has_changes() is False
+    assert tags.rename("bold_tag", "new tag") is True
+    assert tags.list == [
+        "in_text_tag",
+        "inline_tag_top1",
+        "inline_tag_top2",
+        "new tag",
+        "tag_key_value",
+    ]
+    assert tags.has_changes() is True
+
+
 def test_vault_metadata() -> None:
     """Test VaultMetadata class."""
     vm = VaultMetadata()
@@ -295,452 +744,3 @@ def test_vault_metadata_rename() -> None:
     assert vm.rename("tags", "old_tags") is True
     assert vm.dict["old_tags"] == ["new tag", "tag 1", "tag 3"]
     assert "tags" not in vm.dict
-
-
-def test_frontmatter_create() -> None:
-    """Test frontmatter creation."""
-    frontmatter = Frontmatter(INLINE_CONTENT)
-    assert frontmatter.dict == {}
-
-    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
-    assert frontmatter.dict == {
-        "frontmatter_Key1": ["frontmatter_Key1_value"],
-        "frontmatter_Key2": ["article", "note"],
-        "shared_key1": ["shared_key1_value"],
-        "tags": ["tag_1", "tag_2", "📅/tag_3"],
-    }
-    assert frontmatter.dict_original == {
-        "frontmatter_Key1": ["frontmatter_Key1_value"],
-        "frontmatter_Key2": ["article", "note"],
-        "shared_key1": ["shared_key1_value"],
-        "tags": ["tag_1", "tag_2", "📅/tag_3"],
-    }
-
-
-def test_frontmatter_contains() -> None:
-    """Test frontmatter contains."""
-    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
-
-    assert frontmatter.contains("frontmatter_Key1") is True
-    assert frontmatter.contains("frontmatter_Key2", "article") is True
-    assert frontmatter.contains("frontmatter_Key3") is False
-    assert frontmatter.contains("frontmatter_Key2", "no value") is False
-
-    assert frontmatter.contains(r"\d$", is_regex=True) is True
-    assert frontmatter.contains(r"^\d", is_regex=True) is False
-    assert frontmatter.contains("key", r"_\d", is_regex=True) is False
-    assert frontmatter.contains("key", r"\w\d_", is_regex=True) is True
-
-
-def test_frontmatter_add() -> None:
-    """Test frontmatter add."""
-    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
-
-    assert frontmatter.add("frontmatter_Key1") is False
-    assert frontmatter.add("added_key") is True
-    assert frontmatter.dict == {
-        "added_key": [],
-        "frontmatter_Key1": ["frontmatter_Key1_value"],
-        "frontmatter_Key2": ["article", "note"],
-        "shared_key1": ["shared_key1_value"],
-        "tags": ["tag_1", "tag_2", "📅/tag_3"],
-    }
-
-    assert frontmatter.add("added_key", "added_value") is True
-    assert frontmatter.dict == {
-        "added_key": ["added_value"],
-        "frontmatter_Key1": ["frontmatter_Key1_value"],
-        "frontmatter_Key2": ["article", "note"],
-        "shared_key1": ["shared_key1_value"],
-        "tags": ["tag_1", "tag_2", "📅/tag_3"],
-    }
-
-    assert frontmatter.add("added_key", "added_value_2") is True
-    assert frontmatter.dict == {
-        "added_key": ["added_value", "added_value_2"],
-        "frontmatter_Key1": ["frontmatter_Key1_value"],
-        "frontmatter_Key2": ["article", "note"],
-        "shared_key1": ["shared_key1_value"],
-        "tags": ["tag_1", "tag_2", "📅/tag_3"],
-    }
-
-    assert frontmatter.add("added_key", ["added_value_3", "added_value_4"]) is True
-    assert frontmatter.dict == {
-        "added_key": ["added_value", "added_value_2", "added_value_3", "added_value_4"],
-        "frontmatter_Key1": ["frontmatter_Key1_value"],
-        "frontmatter_Key2": ["article", "note"],
-        "shared_key1": ["shared_key1_value"],
-        "tags": ["tag_1", "tag_2", "📅/tag_3"],
-    }
-
-    assert frontmatter.add("added_key2", ["added_value_1", "added_value_2"]) is True
-    assert frontmatter.dict == {
-        "added_key": ["added_value", "added_value_2", "added_value_3", "added_value_4"],
-        "added_key2": ["added_value_1", "added_value_2"],
-        "frontmatter_Key1": ["frontmatter_Key1_value"],
-        "frontmatter_Key2": ["article", "note"],
-        "shared_key1": ["shared_key1_value"],
-        "tags": ["tag_1", "tag_2", "📅/tag_3"],
-    }
-
-    assert frontmatter.add("added_key3", "added_value_1") is True
-    assert frontmatter.dict == {
-        "added_key": ["added_value", "added_value_2", "added_value_3", "added_value_4"],
-        "added_key2": ["added_value_1", "added_value_2"],
-        "added_key3": ["added_value_1"],
-        "frontmatter_Key1": ["frontmatter_Key1_value"],
-        "frontmatter_Key2": ["article", "note"],
-        "shared_key1": ["shared_key1_value"],
-        "tags": ["tag_1", "tag_2", "📅/tag_3"],
-    }
-
-    assert frontmatter.add("added_key3", "added_value_1") is False
-
-
-def test_frontmatter_rename() -> None:
-    """Test frontmatter rename."""
-    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
-    assert frontmatter.dict == {
-        "frontmatter_Key1": ["frontmatter_Key1_value"],
-        "frontmatter_Key2": ["article", "note"],
-        "shared_key1": ["shared_key1_value"],
-        "tags": ["tag_1", "tag_2", "📅/tag_3"],
-    }
-
-    assert frontmatter.rename("no key", "new key") is False
-    assert frontmatter.rename("tags", "no tag", "new key") is False
-
-    assert frontmatter.has_changes() is False
-    assert frontmatter.rename("tags", "tag_2", "new tag") is True
-
-    assert frontmatter.dict["tags"] == ["new tag", "tag_1", "📅/tag_3"]
-    assert frontmatter.rename("tags", "old_tags") is True
-    assert frontmatter.dict["old_tags"] == ["new tag", "tag_1", "📅/tag_3"]
-    assert "tags" not in frontmatter.dict
-
-    assert frontmatter.has_changes() is True
-
-
-def test_frontmatter_delete() -> None:
-    """Test Frontmatter delete method."""
-    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
-    assert frontmatter.dict == {
-        "frontmatter_Key1": ["frontmatter_Key1_value"],
-        "frontmatter_Key2": ["article", "note"],
-        "shared_key1": ["shared_key1_value"],
-        "tags": ["tag_1", "tag_2", "📅/tag_3"],
-    }
-
-    assert frontmatter.delete("no key") is False
-    assert frontmatter.delete("tags", "no value") is False
-    assert frontmatter.delete(r"\d{3}") is False
-    assert frontmatter.has_changes() is False
-    assert frontmatter.delete("tags", "tag_2") is True
-    assert frontmatter.dict["tags"] == ["tag_1", "📅/tag_3"]
-    assert frontmatter.delete("tags") is True
-    assert "tags" not in frontmatter.dict
-    assert frontmatter.has_changes() is True
-    assert frontmatter.delete("shared_key1", r"\w+") is True
-    assert frontmatter.dict["shared_key1"] == []
-    assert frontmatter.delete(r"\w.tter") is True
-    assert frontmatter.dict == {"shared_key1": []}
-
-
-def test_frontmatter_yaml_conversion():
-    """Test Frontmatter to_yaml method."""
-    new_frontmatter: str = """\
-tags:
-  - tag_1
-  - tag_2
-  - 📅/tag_3
-frontmatter_Key1: frontmatter_Key1_value
-frontmatter_Key2:
-  - article
-  - note
-shared_key1: shared_key1_value
-"""
-    new_frontmatter_sorted: str = """\
-frontmatter_Key1: frontmatter_Key1_value
-frontmatter_Key2:
-  - article
-  - note
-shared_key1: shared_key1_value
-tags:
-  - tag_1
-  - tag_2
-  - 📅/tag_3
-"""
-    frontmatter = Frontmatter(FRONTMATTER_CONTENT)
-    assert frontmatter.to_yaml() == new_frontmatter
-    assert frontmatter.to_yaml(sort_keys=True) == new_frontmatter_sorted
-
-
-def test_inline_metadata_create() -> None:
-    """Test inline metadata creation."""
-    inline = InlineMetadata(FRONTMATTER_CONTENT)
-    assert inline.dict == {}
-    inline = InlineMetadata(INLINE_CONTENT)
-    assert inline.dict == {
-        "bold_key1": ["bold_key1_value"],
-        "bold_key2": ["bold_key2_value"],
-        "emoji_📅_key": ["emoji_📅_key_value"],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
-        "tag_key": ["tag_key_value"],
-    }
-    assert inline.dict_original == {
-        "bold_key1": ["bold_key1_value"],
-        "bold_key2": ["bold_key2_value"],
-        "emoji_📅_key": ["emoji_📅_key_value"],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
-        "tag_key": ["tag_key_value"],
-    }
-
-
-def test_inline_contains() -> None:
-    """Test inline metadata contains method."""
-    inline = InlineMetadata(INLINE_CONTENT)
-
-    assert inline.contains("bold_key1") is True
-    assert inline.contains("bold_key2", "bold_key2_value") is True
-    assert inline.contains("bold_key3") is False
-    assert inline.contains("bold_key2", "no value") is False
-
-    assert inline.contains(r"\w{4}_key", is_regex=True) is True
-    assert inline.contains(r"^\d", is_regex=True) is False
-    assert inline.contains("1$", r"\d_value", is_regex=True) is True
-    assert inline.contains("key", r"^\d_value", is_regex=True) is False
-
-
-def test_inline_add() -> None:
-    """Test inline add."""
-    inline = InlineMetadata(INLINE_CONTENT)
-
-    assert inline.add("bold_key1") is False
-    assert inline.add("bold_key1", "bold_key1_value") is False
-    assert inline.add("added_key") is True
-    assert inline.dict == {
-        "added_key": [],
-        "bold_key1": ["bold_key1_value"],
-        "bold_key2": ["bold_key2_value"],
-        "emoji_📅_key": ["emoji_📅_key_value"],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
-        "tag_key": ["tag_key_value"],
-    }
-
-    assert inline.add("added_key1", "added_value") is True
-    assert inline.dict == {
-        "added_key": [],
-        "added_key1": ["added_value"],
-        "bold_key1": ["bold_key1_value"],
-        "bold_key2": ["bold_key2_value"],
-        "emoji_📅_key": ["emoji_📅_key_value"],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
-        "tag_key": ["tag_key_value"],
-    }
-
-    with pytest.raises(ValueError):
-        assert inline.add("added_key1", "added_value_2") is True
-
-    assert inline.dict == {
-        "added_key": [],
-        "added_key1": ["added_value"],
-        "bold_key1": ["bold_key1_value"],
-        "bold_key2": ["bold_key2_value"],
-        "emoji_📅_key": ["emoji_📅_key_value"],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
-        "tag_key": ["tag_key_value"],
-    }
-
-    assert inline.add("added_key2", ["added_value_1", "added_value_2"]) is True
-    assert inline.dict == {
-        "added_key": [],
-        "added_key1": ["added_value"],
-        "added_key2": ["added_value_1"],
-        "bold_key1": ["bold_key1_value"],
-        "bold_key2": ["bold_key2_value"],
-        "emoji_📅_key": ["emoji_📅_key_value"],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
-        "tag_key": ["tag_key_value"],
-    }
-
-    assert inline.add("added_key", "added_value")
-    assert inline.dict == {
-        "added_key": ["added_value"],
-        "added_key1": ["added_value"],
-        "added_key2": ["added_value_1"],
-        "bold_key1": ["bold_key1_value"],
-        "bold_key2": ["bold_key2_value"],
-        "emoji_📅_key": ["emoji_📅_key_value"],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
-        "tag_key": ["tag_key_value"],
-    }
-
-
-def test_inline_metadata_rename() -> None:
-    """Test inline metadata rename."""
-    inline = InlineMetadata(INLINE_CONTENT)
-    assert inline.dict == {
-        "bold_key1": ["bold_key1_value"],
-        "bold_key2": ["bold_key2_value"],
-        "emoji_📅_key": ["emoji_📅_key_value"],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
-        "tag_key": ["tag_key_value"],
-    }
-
-    assert inline.rename("no key", "new key") is False
-    assert inline.rename("repeated_key", "no value", "new key") is False
-    assert inline.has_changes() is False
-    assert inline.rename("repeated_key", "repeated_key_value1", "new value") is True
-    assert inline.dict["repeated_key"] == ["new value", "repeated_key_value2"]
-    assert inline.rename("repeated_key", "old_key") is True
-    assert inline.dict["old_key"] == ["new value", "repeated_key_value2"]
-    assert "repeated_key" not in inline.dict
-    assert inline.has_changes() is True
-
-
-def test_inline_metadata_delete() -> None:
-    """Test inline metadata delete."""
-    inline = InlineMetadata(INLINE_CONTENT)
-    assert inline.dict == {
-        "bold_key1": ["bold_key1_value"],
-        "bold_key2": ["bold_key2_value"],
-        "emoji_📅_key": ["emoji_📅_key_value"],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "repeated_key": ["repeated_key_value1", "repeated_key_value2"],
-        "tag_key": ["tag_key_value"],
-    }
-
-    assert inline.delete("no key") is False
-    assert inline.delete("repeated_key", "no value") is False
-    assert inline.has_changes() is False
-    assert inline.delete("repeated_key", "repeated_key_value1") is True
-    assert inline.dict["repeated_key"] == ["repeated_key_value2"]
-    assert inline.delete("repeated_key") is True
-    assert "repeated_key" not in inline.dict
-    assert inline.has_changes() is True
-    assert inline.delete(r"\d{3}") is False
-    assert inline.delete(r"bold_key\d") is True
-    assert inline.dict == {
-        "emoji_📅_key": ["emoji_📅_key_value"],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "tag_key": ["tag_key_value"],
-    }
-    assert inline.delete("emoji_📅_key", ".*📅.*") is True
-    assert inline.dict == {
-        "emoji_📅_key": [],
-        "in_text_key1": ["in_text_key1_value"],
-        "in_text_key2": ["in_text_key2_value"],
-        "link_key": ["link_key_value"],
-        "tag_key": ["tag_key_value"],
-    }
-
-
-def test_inline_tags_create() -> None:
-    """Test inline tags creation."""
-    tags = InlineTags(FRONTMATTER_CONTENT)
-    tags.metadata_key
-    assert tags.list == []
-
-    tags = InlineTags(INLINE_CONTENT)
-    assert tags.list == [
-        "bold_tag",
-        "in_text_tag",
-        "inline_tag_top1",
-        "inline_tag_top2",
-        "tag_key_value",
-    ]
-    assert tags.list_original == [
-        "bold_tag",
-        "in_text_tag",
-        "inline_tag_top1",
-        "inline_tag_top2",
-        "tag_key_value",
-    ]
-
-
-def test_inline_tags_contains() -> None:
-    """Test inline tags contains."""
-    tags = InlineTags(INLINE_CONTENT)
-    assert tags.contains("bold_tag") is True
-    assert tags.contains("no tag") is False
-
-    assert tags.contains(r"\w_\w", is_regex=True) is True
-    assert tags.contains(r"\d_\d", is_regex=True) is False
-
-
-def test_inline_tags_rename() -> None:
-    """Test inline tags rename."""
-    tags = InlineTags(INLINE_CONTENT)
-    assert tags.list == [
-        "bold_tag",
-        "in_text_tag",
-        "inline_tag_top1",
-        "inline_tag_top2",
-        "tag_key_value",
-    ]
-
-    assert tags.rename("no tag", "new tag") is False
-    assert tags.has_changes() is False
-    assert tags.rename("bold_tag", "new tag") is True
-    assert tags.list == [
-        "in_text_tag",
-        "inline_tag_top1",
-        "inline_tag_top2",
-        "new tag",
-        "tag_key_value",
-    ]
-    assert tags.has_changes() is True
-
-
-def test_inline_tags_delete() -> None:
-    """Test inline tags delete."""
-    tags = InlineTags(INLINE_CONTENT)
-    assert tags.list == [
-        "bold_tag",
-        "in_text_tag",
-        "inline_tag_top1",
-        "inline_tag_top2",
-        "tag_key_value",
-    ]
-
-    assert tags.delete("no tag") is False
-    assert tags.has_changes() is False
-    assert tags.delete("bold_tag") is True
-    assert tags.list == [
-        "in_text_tag",
-        "inline_tag_top1",
-        "inline_tag_top2",
-        "tag_key_value",
-    ]
-    assert tags.has_changes() is True
-    assert tags.delete(r"\d{3}") is False
-    assert tags.delete(r"inline_tag_top\d") is True
-    assert tags.list == ["in_text_tag", "tag_key_value"]

tests/notes_test.py

@@ -103,6 +103,7 @@ def test_add_metadata_inline(short_note) -> None:
 def test_add_metadata_frontmatter(sample_note) -> None:
     """Test adding metadata."""
     note = Note(note_path=sample_note)
+
     assert note.add_metadata(MetadataType.FRONTMATTER, "frontmatter_Key1") is False
     assert note.add_metadata(MetadataType.FRONTMATTER, "shared_key1", "shared_key1_value") is False
     assert note.add_metadata(MetadataType.FRONTMATTER, "new_key1") is True
@@ -159,6 +160,30 @@ def test_add_metadata_frontmatter(sample_note) -> None:
     }
 
 
+def test_add_metadata_tag(sample_note) -> None:
+    """Test adding inline tags."""
+    note = Note(note_path=sample_note)
+
+    assert (
+        note.add_metadata(MetadataType.TAGS, value="shared_tag", location=InsertLocation.TOP)
+        is False
+    )
+    assert (
+        note.add_metadata(MetadataType.TAGS, value="a_new_tag", location=InsertLocation.TOP) is True
+    )
+    assert note.inline_tags.list == [
+        "a_new_tag",
+        "inline_tag_bottom1",
+        "inline_tag_bottom2",
+        "inline_tag_top1",
+        "inline_tag_top2",
+        "intext_tag1",
+        "intext_tag2",
+        "shared_tag",
+    ]
+    assert "#a_new_tag" in note.file_content
+
+
 def test_contains_inline_tag(sample_note) -> None:
     """Test contains inline tag."""
     note = Note(note_path=sample_note)