Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-100378

Split out QNetworkReply tests that can be run locally

    XMLWordPrintable

Details

    • Task
    • Resolution: Unresolved
    • P3: Somewhat important
    • None
    • 6.4
    • Network
    • All

    Description

      The current work-around is to comment out the network-server testing from tests and running individual test-cases.

      We currently have a bunch of tests which require a network testing server or the docker setup. But we also have a lot of tests that only use local resources. They're currently sharing the same source-files (e.g. tst_qnetworkreply.cpp), and preferably the tests would be split so that anyone can run the networking tests that only use local resources without needing workarounds. This will let external developers at least run some tests locally, before pushing. Not having to rely on the integration system. They can then also work on and iterate on their own local tests without going through the hoop mentioned at the start or asking for CI pre-checks.

      There is now a tst_qnetworkreply_local that was initially started partially for this purpose, but also to be able to test that local-socket support works correctly in QNetworkAccessManager.
      It has a "MiniHttpServerV2", similar to the MiniHttpServer in tst_qnetworkreply, but with support for local-sockets and a handful of other differences.

      The goal for this task would be to move tests that are using MiniHttpServer (without redirecting to or otherwise using the network-testing server) to tst_qnetworkreply_local, making it work with local-sockets and using MiniHttpServerV2

      This will break historical data of flaky tests and so on.

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          People

            cnn Qt Core & Network
            manordheim MÃ¥rten Nordheim
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:

              Gerrit Reviews

                There are no open Gerrit changes