Rename history.exclude to completion.web_history.exclude

If the blacklist is only valid for the completion, the setting should also be
under completion.

This also un-renames history.gap_interval and renames
completion.web_history_max_items.

1 files changed, 9 insertions, 4 deletions

diff --git a/tests/unit/browser/test_history.py b/tests/unit/browser/test_history.py
index d0b1134c8..7c6d4af45 100644
--- a/tests/unit/browser/test_history.py
+++ b/tests/unit/browser/test_history.py
@@ -226,7 +226,8 @@ class TestAdd:
 
     def test_exclude(self, hist, config_stub):
         """Excluded URLs should be in the history but not completion."""
-        config_stub.set_obj('history.exclude', ['*.example.org'])
+        config_stub.set_obj('completion.web_history.exclude',
+                            ['*.example.org'])
         url = QUrl('http://www.example.org/')
         hist.add_from_tab(url, url, 'title')
         assert list(hist)
@@ -484,8 +485,12 @@ class TestRebuild:
         assert not hist3.metainfo['force_rebuild']
 
     def test_exclude(self, config_stub, hist):
-        """Ensure that patterns in history.exclude are ignored."""
-        config_stub.set_obj('history.exclude', ['*.example.org'])
+        """Ensure that patterns in completion.web_history.exclude are ignored.
+
+        This setting should only be used for the completion.
+        """
+        config_stub.set_obj('completion.web_history.exclude',
+                            ['*.example.org'])
         assert hist.metainfo['force_rebuild']
 
         hist.add_url(QUrl('http://example.com'), redirect=False, atime=1)
@@ -495,7 +500,7 @@ class TestRebuild:
         assert list(hist2.completion) == [('http://example.com', '', 1)]
 
     def test_unrelated_config_change(self, config_stub, hist):
-        config_stub.set_obj('history.gap_interval', 1234)
+        config_stub.set_obj('history_gap_interval', 1234)
         assert not hist.metainfo['force_rebuild']