Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-9619

Test passes wrong argument to assert.soon

    • Type: Icon: Bug Bug
    • Resolution: Done
    • Priority: Icon: Minor - P4 Minor - P4
    • 2.5.0
    • Affects Version/s: 2.4.3
    • Component/s: None
    • Labels:
      None
    • Fully Compatible
    • ALL

      jstests/sharding/mrShardedOutput.js passes a function and a timeout to assert.soon on lines 111 to 114.

          assert.soon(function(){
              return hasChunkOnShard("test.foo", shards[0]._id) && 
                     hasChunkOnShard("test.foo", shards[1]._id);
          }, 2 * 60 * 1000);
      
      

      This does not match the function signature for assert.soon:

      assert.soon = function(f, msg, timeout /*ms*/, interval) {
      

      The timeout will not be used, but will be displayed as the error message if the default 30 second timeout occurs.

            Assignee:
            tad Tad Marshall
            Reporter:
            tad Tad Marshall
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: