rnhmjoj/qutebrowser
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Update tests to use search.ignore_case

Browse Source
This commit is contained in:
Justin Partain 2017-10-27 08:49:22 -04:00
parent 9f511fe18c
commit bcd9d13684
5 changed files with 31 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
tests/end2end/features/qutescheme.feature
View File

@ -99,19 +99,19 @@ Feature: Special qute:// pages
# qute://settings # qute://settings
Scenario: Focusing input fields in qute://settings and entering valid value Scenario: Focusing input fields in qute://settings and entering valid value
When I set ignore_case to never When I set search.ignore_case to never
And I open qute://settings And I open qute://settings
# scroll to the right - the table does not fit in the default screen # scroll to the right - the table does not fit in the default screen
And I run :scroll-to-perc -x 100 And I run :scroll-to-perc -x 100
And I run :jseval document.getElementById('input-ignore_case').value = '' And I run :jseval document.getElementById('input-search.ignore_case').value = ''
And I run :click-element id input-ignore_case And I run :click-element id input-search.ignore_case
And I wait for "Entering mode KeyMode.insert *" in the log And I wait for "Entering mode KeyMode.insert *" in the log
And I press the keys "always" And I press the keys "always"
And I press the key "<Escape>" And I press the key "<Escape>"
# an explicit Tab to unfocus the input field seems to stabilize the tests # an explicit Tab to unfocus the input field seems to stabilize the tests
And I press the key "<Tab>" And I press the key "<Tab>"
And I wait for "Config option changed: ignore_case *" in the log And I wait for "Config option changed: search.ignore_case *" in the log
Then the option ignore_case should be set to always Then the option search.ignore_case should be set to always
# Sometimes, an unrelated value gets set # Sometimes, an unrelated value gets set
@flaky @flaky
@ -119,8 +119,8 @@ Feature: Special qute:// pages
When I open qute://settings When I open qute://settings
# scroll to the right - the table does not fit in the default screen # scroll to the right - the table does not fit in the default screen
And I run :scroll-to-perc -x 100 And I run :scroll-to-perc -x 100
And I run :jseval document.getElementById('input-ignore_case').value = '' And I run :jseval document.getElementById('input-search.ignore_case').value = ''
And I run :click-element id input-ignore_case And I run :click-element id input-search.ignore_case
And I wait for "Entering mode KeyMode.insert *" in the log And I wait for "Entering mode KeyMode.insert *" in the log
And I press the keys "foo" And I press the keys "foo"
And I press the key "<Escape>" And I press the key "<Escape>"

34
tests/end2end/features/search.feature
View File

@ -22,7 +22,7 @@ Feature: Searching on a page
Then "Bar" should be found Then "Bar" should be found
Scenario: Searching with --reverse Scenario: Searching with --reverse
When I set ignore_case to always When I set search.ignore_case to always
And I run :search -r foo And I run :search -r foo
And I wait for "search found foo with flags FindBackward" in the log And I wait for "search found foo with flags FindBackward" in the log
Then "Foo" should be found Then "Foo" should be found
@ -52,28 +52,28 @@ Feature: Searching on a page
And I wait for "search didn't find blub" in the log And I wait for "search didn't find blub" in the log
Then the warning "Text 'blub' not found on page!" should be shown Then the warning "Text 'blub' not found on page!" should be shown
## ignore_case ## search.ignore_case
Scenario: Searching text with ignore_case = always Scenario: Searching text with search.ignore_case = always
When I set ignore_case to always When I set search.ignore_case to always
And I run :search bar And I run :search bar
And I wait for "search found bar" in the log And I wait for "search found bar" in the log
Then "Bar" should be found Then "Bar" should be found
Scenario: Searching text with ignore_case = never Scenario: Searching text with search.ignore_case = never
When I set ignore_case to never When I set search.ignore_case to never
And I run :search bar And I run :search bar
And I wait for "search found bar with flags FindCaseSensitively" in the log And I wait for "search found bar with flags FindCaseSensitively" in the log
Then "bar" should be found Then "bar" should be found
Scenario: Searching text with ignore_case = smart (lower-case) Scenario: Searching text with search.ignore_case = smart (lower-case)
When I set ignore_case to smart When I set search.ignore_case to smart
And I run :search bar And I run :search bar
And I wait for "search found bar" in the log And I wait for "search found bar" in the log
Then "Bar" should be found Then "Bar" should be found
Scenario: Searching text with ignore_case = smart (upper-case) Scenario: Searching text with search.ignore_case = smart (upper-case)
When I set ignore_case to smart When I set search.ignore_case to smart
And I run :search Foo And I run :search Foo
And I wait for "search found Foo with flags FindCaseSensitively" in the log And I wait for "search found Foo with flags FindCaseSensitively" in the log
Then "Foo" should be found # even though foo was first Then "Foo" should be found # even though foo was first
@ -81,7 +81,7 @@ Feature: Searching on a page
## :search-next ## :search-next
Scenario: Jumping to next match Scenario: Jumping to next match
When I set ignore_case to always When I set search.ignore_case to always
And I run :search foo And I run :search foo
And I wait for "search found foo" in the log And I wait for "search found foo" in the log
And I run :search-next And I run :search-next
@ -89,7 +89,7 @@ Feature: Searching on a page
Then "Foo" should be found Then "Foo" should be found
Scenario: Jumping to next match with count Scenario: Jumping to next match with count
When I set ignore_case to always When I set search.ignore_case to always
And I run :search baz And I run :search baz
And I wait for "search found baz" in the log And I wait for "search found baz" in the log
And I run :search-next with count 2 And I run :search-next with count 2
@ -97,7 +97,7 @@ Feature: Searching on a page
Then "BAZ" should be found Then "BAZ" should be found
Scenario: Jumping to next match with --reverse Scenario: Jumping to next match with --reverse
When I set ignore_case to always When I set search.ignore_case to always
And I run :search --reverse foo And I run :search --reverse foo
And I wait for "search found foo with flags FindBackward" in the log And I wait for "search found foo with flags FindBackward" in the log
And I run :search-next And I run :search-next
@ -121,7 +121,7 @@ Feature: Searching on a page
# https://github.com/qutebrowser/qutebrowser/issues/2438 # https://github.com/qutebrowser/qutebrowser/issues/2438
Scenario: Jumping to next match after clearing Scenario: Jumping to next match after clearing
When I set ignore_case to always When I set search.ignore_case to always
And I run :search foo And I run :search foo
And I wait for "search found foo" in the log And I wait for "search found foo" in the log
And I run :search And I run :search
@ -132,7 +132,7 @@ Feature: Searching on a page
## :search-prev ## :search-prev
Scenario: Jumping to previous match Scenario: Jumping to previous match
When I set ignore_case to always When I set search.ignore_case to always
And I run :search foo And I run :search foo
And I wait for "search found foo" in the log And I wait for "search found foo" in the log
And I run :search-next And I run :search-next
@ -142,7 +142,7 @@ Feature: Searching on a page
Then "foo" should be found Then "foo" should be found
Scenario: Jumping to previous match with count Scenario: Jumping to previous match with count
When I set ignore_case to always When I set search.ignore_case to always
And I run :search baz And I run :search baz
And I wait for "search found baz" in the log And I wait for "search found baz" in the log
And I run :search-next And I run :search-next
@ -154,7 +154,7 @@ Feature: Searching on a page
Then "baz" should be found Then "baz" should be found
Scenario: Jumping to previous match with --reverse Scenario: Jumping to previous match with --reverse
When I set ignore_case to always When I set search.ignore_case to always
And I run :search --reverse foo And I run :search --reverse foo
And I wait for "search found foo with flags FindBackward" in the log And I wait for "search found foo with flags FindBackward" in the log
And I run :search-next And I run :search-next

6
tests/end2end/test_invocations.py
View File

@ -359,14 +359,14 @@ def test_qute_settings_persistence(short_tmpdir, request, quteproc_new):
args = _base_args(request.config) + ['--basedir', str(short_tmpdir)] args = _base_args(request.config) + ['--basedir', str(short_tmpdir)]
quteproc_new.start(args) quteproc_new.start(args)
quteproc_new.open_path( quteproc_new.open_path(
'qute://settings/set?option=ignore_case&value=always') 'qute://settings/set?option=search.ignore_case&value=always')
assert quteproc_new.get_setting('ignore_case') == 'always' assert quteproc_new.get_setting('search.ignore_case') == 'always'
quteproc_new.send_cmd(':quit') quteproc_new.send_cmd(':quit')
quteproc_new.wait_for_quit() quteproc_new.wait_for_quit()
quteproc_new.start(args) quteproc_new.start(args)
assert quteproc_new.get_setting('ignore_case') == 'always' assert quteproc_new.get_setting('search.ignore_case') == 'always'
@pytest.mark.no_xvfb @pytest.mark.no_xvfb

5
tests/unit/config/test_configcommands.py
View File

@ -281,7 +281,7 @@ class TestSource:
def test_config_source(self, tmpdir, commands, config_stub, config_tmpdir, def test_config_source(self, tmpdir, commands, config_stub, config_tmpdir,
use_default_dir, clear): use_default_dir, clear):
assert config_stub.val.content.javascript.enabled assert config_stub.val.content.javascript.enabled
config_stub.val.ignore_case = 'always' config_stub.val.search.ignore_case = 'always'
if use_default_dir: if use_default_dir:
pyfile = config_tmpdir / 'config.py' pyfile = config_tmpdir / 'config.py'
@ -295,7 +295,8 @@ class TestSource:
commands.config_source(arg, clear=clear) commands.config_source(arg, clear=clear)
assert not config_stub.val.content.javascript.enabled assert not config_stub.val.content.javascript.enabled
assert config_stub.val.ignore_case == ('smart' if clear else 'always') ignore_case = config_stub.val.search.ignore_case
assert ignore_case == ('smart' if clear else 'always')
def test_errors(self, commands, config_tmpdir): def test_errors(self, commands, config_tmpdir):
pyfile = config_tmpdir / 'config.py' pyfile = config_tmpdir / 'config.py'

2
tests/unit/config/test_configdata.py
View File

@ -34,7 +34,7 @@ def test_init(config_stub):
# configdata.init() is called by config_stub # configdata.init() is called by config_stub
config_stub.val.aliases = {} config_stub.val.aliases = {}
assert isinstance(configdata.DATA, dict) assert isinstance(configdata.DATA, dict)
assert 'ignore_case' in configdata.DATA assert 'search.ignore_case' in configdata.DATA
def test_data(config_stub): def test_data(config_stub):