<EdwinGrubbs> allenap: I'm surprised that you added a line to tests/externalbugtracker.py to set the batch_size to BATCH_SIZE_UNLIMITED, but it didn't change the results of any of the tests.
<allenap> EdwinGrubbs: It kind of does the same as lines 153 to 157 used to do.
<allenap> EdwinGrubbs: ... because most calls into updateBugWatches() and _getRemoteIdsToCheck() didn't specify batch_size.
<EdwinGrubbs> allenap: ok, that makes sense. Looks good.
Conversation re. additional changes from IRC:
<EdwinGrubbs> allenap: I'm surprised that you added a line to tests/externalb ugtracker. py to set the batch_size to BATCH_SIZE_ UNLIMITED, but it didn't change the results of any of the tests. Check() didn't specify batch_size.
<allenap> EdwinGrubbs: It kind of does the same as lines 153 to 157 used to do.
<allenap> EdwinGrubbs: ... because most calls into updateBugWatches() and _getRemoteIdsTo
<EdwinGrubbs> allenap: ok, that makes sense. Looks good.