Swapped values in conn_default configuration of test_prefetch01.py

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Prefetch
    • None
    • Storage Engines - Persistence
    • SE Persistence backlog
    • 1

      Issue Summary

      The test_prefetch01.py test suite contains swapped values in the conn_default configuration for Python scenarios. Specifically, the names 'default-off' and 'default-on' are assigned to the wrong configuration values, resulting in misleading output names.

      Context

      The following code snippet illustrates the issue:

      Unable to find source-code formatter for language: diff. Available languages are: actionscript, ada, applescript, bash, c, c#, c++, cpp, css, erlang, go, groovy, haskell, html, java, javascript, js, json, lua, none, nyan, objc, perl, php, python, r, rainbow, ruby, scala, sh, sql, swift, visualbasic, xml, yaml
      @@ -42,8 +42,8 @@ class test_prefetch01(wttest.WiredTigerTestCase):
          conn_default = [
      -        ('default-off', dict(default=True)),
      -        ('default-on', dict(default=False)),
      +        ('default-off', dict(default=False)),
      +        ('default-on', dict(default=True)),
          ]
      

      Although this does not impact the test execution, it causes the output to display incorrect names for the configurations, which can be confusing for users reviewing test results.

      Proposed Solution

      • Update the conn_default configuration in test_prefetch01.py so that 'default-off' is assigned dict(default=False) and 'default-on' is assigned dict(default=True).
      • Ensure that output names accurately reflect the configuration values to prevent confusion.

      Original Slack thread: Slack Thread
      This ticket was generated by AI from a Slack thread.

            Assignee:
            Dylan Liang
            Reporter:
            Memento Slack Bot
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: