Skip to content

encoding test is OS-specific #916

Closed
Closed

Description

@jacalata

Describe the bug
This test appears to be OS-dependant
FAIL: test_filter_sort_legacy (test.test_request_option.RequestOptionTests)

Traceback (most recent call last):
File "c:\dev\tsc\test\test_request_option.py", line 177, in test_filter_sort_legacy
self.assertTrue(re.search('filter=tags:in:%2bstocks,market%2d', resp.request.query))
AssertionError: None is not true

Versions
Fails on windows, if I adjust the expected value to match the encoding there it fails on linux.

Activity

  1. added a commit that references this issue on Oct 9, 2021
  2. t8y8 commented on Oct 12, 2021

    @t8y8
    Contributor

    That might be related to the version of requests or urllib3 being run by the test, and not necessarily the OS... that's what I kind of remember from before

  3. added
    infraInfrastructure issues (GitHub, etc)
    on Nov 8, 2021
  4. jacalata commented on Jun 18, 2026

    @jacalata
    ContributorAuthor

    The failing assertion (re.search('filter=tags:in:%2bstocks,market%2d', ...)) was checking for platform-specific URL encoding. The test has since been rewritten to use parse_qs which handles encoding normalization correctly on all platforms. Passes on Windows and Linux. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    buginfraInfrastructure issues (GitHub, etc)

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions