Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pager tests failing due to non-called pager? #495

Open
sebix opened this issue Jan 25, 2025 · 0 comments
Open

pager tests failing due to non-called pager? #495

sebix opened this issue Jan 25, 2025 · 0 comments

Comments

@sebix
Copy link

sebix commented Jan 25, 2025

There are 5 test function which are causing some trouble in the build environment of the iredis package for openSUSE:
test_using_pager_when_rows_too_high, test_using_pager_works_for_help, test_pager_works_for_peek, test_using_pager_from_config, test_using_pager_from_config_when_env_config_both_set
All of them fail in the child.expect(TEST_PAGER_BOUNDARY_NUMBER) / child.expect(TEST_PAGER_BOUNDARY) step.

To investigate this a bit, I redirected the output of the process to stdout to see what iredis did output (with child.logfile = sys.stdout.buffer):

iredis  1.15.0 (Python 3.11.11)              
redis-server  7.4.2                                       
Home:   https://iredis.xbin.io/                                                                                                                                                                                                              Issues: https://github.com/laixintao/iredis/issues                                                                                                                                                                                           
127.0.0.1:6379[15]> lrange long-list 0 -1
127.0.0.1:6379[15]> lrange long-list 0 -1
  1) "value-99"
[...]
 23) "value-77"

There is indeed no ---boundary--- in the output.

the pager variable is correctly set to /usr/bin/python3.11 /home/abuild/rpmbuild/BUILD/iredis-1.15.0/tests/cli_tests/wrappager.py ---boundary--- (and executing it also works).
I also checked if setting the default pager in the client.py works, which is the case (branch Default pager found in PAGER environment variable).

But it still appears the pager isn't called as expected.
Do you have an idea, what could be the cause for this?

Using iredis version 1.15.0 and pexect version 4.9.0.
The full build log is here: iredis_build.log

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant