Haraka 3.1.3 → 3.1.5

package/test/outbound/index.js

@@ -1,8 +1,9 @@
 'use strict'
 
+const { describe, it, beforeEach, afterEach } = require('node:test')
 const assert = require('node:assert')
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 
 const constants = require('haraka-constants')
 const logger = require('../../logger')
@@ -22,56 +23,51 @@ describe('outbound', () => {
             let contents = lines.join(ending)
             let result = ''
 
-            // Set data_lines to lines in contents
             let match
             const re = /^([^\n]*\n?)/
             while ((match = re.exec(contents))) {
                 let line = match[1]
-                line = line.replace(/\r?\n?$/, '\r\n') // assure \r\n ending
+                line = line.replace(/\r?\n?$/, '\r\n')
                 result += line
-                contents = contents.substr(match[1].length)
-                if (contents.length === 0) {
-                    break
-                }
+                contents = contents.substring(match[1].length)
+                if (contents.length === 0) break
             }
 
             assert.deepEqual(lines.join('\r\n'), result)
         }
     })
 
-    it('log_methods added', () => {
+    it('log_methods added to HMailItem prototype', () => {
         const levels = ['DATA', 'PROTOCOL', 'DEBUG', 'INFO', 'NOTICE', 'WARN', 'ERROR', 'CRIT', 'ALERT', 'EMERG']
-
-        const HMailItem = require('../../outbound/hmail')
-
+        // Load via outbound/index to avoid circular-dep boot-order issue
+        const HMailItem = require('../../outbound').HMailItem
         for (const level of levels) {
-            assert.ok(HMailItem.prototype[`log${level.toLowerCase()}`], `Log method for level: ${level}`)
+            assert.ok(HMailItem.prototype[`log${level.toLowerCase()}`], `log method for ${level}`)
         }
     })
 
     it('set_temp_fail_intervals coverage', () => {
         const config = require('../../outbound/config')
-        // Test default configuration
         assert.deepEqual(
             config.cfg.temp_fail_intervals,
             [64, 128, 256, 512, 1024, 2048, 4096, 8192, 16384, 32768, 65536, 131072],
         )
-        // Test a simple configuration
+
         config.cfg.temp_fail_intervals = '10s, 1m*2'
         config.set_temp_fail_intervals()
         assert.deepEqual(config.cfg.temp_fail_intervals, [10, 60, 60])
-        // Test a complex configuration
+
         config.cfg.temp_fail_intervals = '30s, 1m, 5m, 9m, 15m*3, 30m*2, 1h*3, 2h*3, 1d'
         config.set_temp_fail_intervals()
         assert.deepEqual(
             config.cfg.temp_fail_intervals,
             [30, 60, 300, 540, 900, 900, 900, 1800, 1800, 3600, 3600, 3600, 7200, 7200, 7200, 86400],
         )
-        // Test the "none" configuration
+
         config.cfg.temp_fail_intervals = 'none'
         config.set_temp_fail_intervals()
         assert.deepEqual(config.cfg.temp_fail_intervals, [])
-        // Test bad config (should revert to default)
+
         config.cfg.temp_fail_intervals = '60 min'
         config.set_temp_fail_intervals()
         assert.deepEqual(
@@ -81,29 +77,26 @@ describe('outbound', () => {
     })
 
     describe('get_tls_options', () => {
-        beforeEach((done) => {
+        let outbound, obtls
+
+        beforeEach(async () => {
             process.env.HARAKA_TEST_DIR = path.resolve('test')
-            this.outbound = require('../../outbound')
-            this.obtls = require('../../outbound/tls')
+            outbound = require('../../outbound')
+            obtls = require('../../outbound/tls')
             const tls_socket = require('../../tls_socket')
 
-            // reset config to load from tests directory
             const testDir = path.resolve('test')
-            this.outbound.config = this.outbound.config.module_config(testDir)
-            this.obtls.test_config(tls_socket.config.module_config(testDir), this.outbound.config)
-            this.obtls.init(done)
+            outbound.config = outbound.config.module_config(testDir)
+            obtls.test_config(tls_socket.config.module_config(testDir), outbound.config)
+            await new Promise((resolve) => obtls.init(resolve))
         })
 
-        afterEach((done) => {
+        afterEach(() => {
             delete process.env.HARAKA_TEST_DIR
-            done()
         })
 
         it('gets TLS properties from tls.ini.outbound', () => {
-            const tls_config = this.obtls.get_tls_options({
-                exchange: 'mail.example.com',
-            })
-
+            const tls_config = obtls.get_tls_options({ exchange: 'mail.example.com' })
             assert.deepEqual(tls_config, {
                 servername: 'mail.example.com',
                 key: fs.readFileSync(path.resolve('test', 'config', 'outbound_tls_key.pem')),
@@ -122,116 +115,83 @@ describe('outbound', () => {
     })
 
     describe('build_todo', () => {
-        beforeEach((done) => {
-            this.outbound = require('../../outbound')
+        let outbound
+
+        beforeEach(() => {
+            outbound = require('../../outbound')
             try {
                 fs.unlinkSync('test/queue/multibyte')
                 fs.unlinkSync('test/queue/plain')
             } catch (ignore) {}
-            done()
         })
 
-        it('saves a file', () => {
+        it('saves a plain queue file', () => {
             const todo = JSON.parse(
                 '{"queue_time":1507509981169,"domain":"redacteed.com","rcpt_to":[{"original":"<postmaster@redacteed.com>","original_host":"redacteed.com","host":"redacteed.com","user":"postmaster"}],"mail_from":{"original":"<matt@tnpi.net>","original_host":"tnpi.net","host":"tnpi.net","user":"matt"},"notes":{"authentication_results":["spf=pass smtp.mailfrom=tnpi.net"],"spf_mail_result":"Pass","spf_mail_record":"v=spf1 a mx include:mx.theartfarm.com ?include:forwards._spf.tnpi.net include:lists._spf.tnpi.net -all","attachment_count":0,"attachments":[{"ctype":"application/pdf","filename":"FileWithoutAccent Chars.pdf","extension":".pdf","md5":"6c1d5f5c047cff3f6320b1210970bdf6"}],"attachment_ctypes":["application/pdf","multipart/mixed","text/plain","application/pdf"],"attachment_files":["FileWithoutaccent Chars.pdf"],"attachment_archive_files":[]},"uuid":"1D5483B0-3E00-4280-A961-3AFD2017B4FC.1"}',
             )
             const fd = fs.openSync('test/queue/plain', 'w')
-            const ws = new fs.createWriteStream('test/queue/plain', {
-                fd,
-                flags: constants.WRITE_EXCL,
-            })
-            ws.on('close', () => {
-                // console.log(arguments);
-                assert.ok(1)
-            })
-            ws.on('error', (e) => {
-                console.error(e)
-            })
-            this.outbound.build_todo(todo, ws, () => {
+            const ws = new fs.createWriteStream('test/queue/plain', { fd, flags: constants.WRITE_EXCL })
+            ws.on('error', (e) => console.error(e))
+            outbound.build_todo(todo, ws, () => {
                 ws.write(Buffer.from('This is the message body'))
-                fs.fsync(fd, () => {
-                    ws.close()
-                })
+                fs.fsync(fd, () => ws.close())
             })
+            assert.ok(true)
         })
 
-        it('saves a file with multibyte chars', () => {
+        it('saves a queue file with multibyte chars', () => {
             const todo = JSON.parse(
-                '{"queue_time":1507509981169,"domain":"redacteed.com","rcpt_to":[{"original":"<postmaster@redacteed.com>","original_host":"redacteed.com","host":"redacteed.com","user":"postmaster"}],"mail_from":{"original":"<matt@tnpi.net>","original_host":"tnpi.net","host":"tnpi.net","user":"matt"},"notes":{"authentication_results":["spf=pass smtp.mailfrom=tnpi.net"],"spf_mail_result":"Pass","spf_mail_record":"v=spf1 a mx include:mx.theartfarm.com ?include:forwards._spf.tnpi.net include:lists._spf.tnpi.net -all","attachment_count":0,"attachments":[{"ctype":"application/pdf","filename":"FileWîthÁccent Chars.pdf","extension":".pdf","md5":"6c1d5f5c047cff3f6320b1210970bdf6"}],"attachment_ctypes":["application/pdf","multipart/mixed","text/plain","application/pdf"],"attachment_files":["FileWîthÁccent Chars.pdf"],"attachment_archive_files":[]},"uuid":"1D5483B0-3E00-4280-A961-3AFD2017B4FC.1"}',
+                '{"queue_time":1507509981169,"domain":"redacteed.com","rcpt_to":[{"original":"<postmaster@redacteed.com>","original_host":"redacteed.com","host":"redacteed.com","user":"postmaster"}],"mail_from":{"original":"<matt@tnpi.net>","original_host":"tnpi.net","host":"tnpi.net","user":"matt"},"notes":{"authentication_results":["spf=pass smtp.mailfrom=tnpi.net"],"spf_mail_result":"Pass","spf_mail_record":"v=spf1 a mx include:mx.theartfarm.com ?include:forwards._spf.tnpi.net include:lists._spf.tnpi.net -all","attachment_count":0,"attachments":[{"ctype":"application/pdf","filename":"FileW\\u00eeth\\u00c1ccent Chars.pdf","extension":".pdf","md5":"6c1d5f5c047cff3f6320b1210970bdf6"}],"attachment_ctypes":["application/pdf","multipart/mixed","text/plain","application/pdf"],"attachment_files":["FileW\\u00eeth\\u00c1ccent Chars.pdf"],"attachment_archive_files":[]},"uuid":"1D5483B0-3E00-4280-A961-3AFD2017B4FC.1"}',
             )
             const fd = fs.openSync('test/queue/multibyte', 'w')
-            const ws = new fs.WriteStream('test/queue/multibyte', {
-                fd,
-                flags: constants.WRITE_EXCL,
-            })
-            ws.on('close', () => {
-                assert.ok(1)
-            })
-            ws.on('error', (e) => {
-                console.error(e)
-            })
-            this.outbound.build_todo(todo, ws, () => {
+            const ws = new fs.WriteStream('test/queue/multibyte', { fd, flags: constants.WRITE_EXCL })
+            ws.on('error', (e) => console.error(e))
+            outbound.build_todo(todo, ws, () => {
                 ws.write(Buffer.from('This is the message body'))
-                fs.fsync(fd, () => {
-                    ws.close()
-                })
+                fs.fsync(fd, () => ws.close())
             })
+            assert.ok(true)
         })
     })
 
     describe('timer_queue', () => {
-        beforeEach((done) => {
+        let outbound, ob_timer_queue
+
+        beforeEach(() => {
             process.env.HARAKA_TEST_DIR = path.resolve('test')
-            this.outbound = require('../../outbound')
+            outbound = require('../../outbound')
             const TimerQueue = require('../../outbound/timer_queue')
-            this.ob_timer_queue = new TimerQueue(500)
-            done()
+            ob_timer_queue = new TimerQueue(500)
         })
 
-        afterEach((done) => {
+        afterEach(() => {
             delete process.env.HARAKA_TEST_DIR
-            this.ob_timer_queue.shutdown()
-            done()
+            ob_timer_queue.shutdown()
         })
 
         it('has initial length of 0', () => {
-            assert.equal(this.ob_timer_queue.length(), 0)
+            assert.equal(ob_timer_queue.length(), 0)
         })
 
         it('can add items', () => {
-            this.ob_timer_queue.add('1', 1000)
-            this.ob_timer_queue.add('2', 2000)
-
-            assert.equal(this.ob_timer_queue.length(), 2)
+            ob_timer_queue.add('1', 1000)
+            ob_timer_queue.add('2', 2000)
+            assert.equal(ob_timer_queue.length(), 2)
         })
 
         it('can drain items', () => {
-            this.ob_timer_queue.add('1', 1000)
-            this.ob_timer_queue.add('2', 2000)
-
-            let tq_length = this.ob_timer_queue.length()
-
-            assert.equal(tq_length, 2)
-
-            this.ob_timer_queue.drain()
-            tq_length = this.ob_timer_queue.length()
-
-            assert.equal(tq_length, 0)
+            ob_timer_queue.add('1', 1000)
+            ob_timer_queue.add('2', 2000)
+            ob_timer_queue.drain()
+            assert.equal(ob_timer_queue.length(), 0)
         })
 
         it('can discard items by id', () => {
-            this.ob_timer_queue.add('1', 1000)
-            this.ob_timer_queue.add('2', 2000)
-
-            let tq_length = this.ob_timer_queue.length()
-
-            assert.equal(tq_length, 2)
-
-            this.ob_timer_queue.discard('2')
-            tq_length = this.ob_timer_queue.length()
-
-            assert.equal(tq_length, 1)
-            assert.equal(this.ob_timer_queue.queue[0].id, '1')
+            ob_timer_queue.add('1', 1000)
+            ob_timer_queue.add('2', 2000)
+            ob_timer_queue.discard('2')
+            assert.equal(ob_timer_queue.length(), 1)
+            assert.equal(ob_timer_queue.queue[0].id, '1')
         })
     })
 })

package/test/outbound/qfile.js

@@ -1,15 +1,15 @@
+'use strict'
+
+const { describe, it, beforeEach } = require('node:test')
 const assert = require('node:assert')
 const os = require('node:os')
 
-describe('qfile', () => {
-    describe('qfile', () => {
-        beforeEach((done) => {
-            this.qfile = require('../../outbound/qfile')
-            done()
-        })
+const qfile = require('../../outbound/qfile')
 
+describe('outbound/qfile', () => {
+    describe('name', () => {
         it('name() basic functions', () => {
-            const name = this.qfile.name()
+            const name = qfile.name()
             const split = name.split('_')
             assert.equal(split.length, 7)
             assert.equal(split[2], 0)
@@ -25,7 +25,7 @@ describe('qfile', () => {
                 uid: 'XXYYZZ',
                 host: os.hostname(),
             }
-            const name = this.qfile.name(overrides)
+            const name = qfile.name(overrides)
             const split = name.split('_')
             assert.equal(split.length, 7)
             assert.equal(split[0], overrides.arrival)
@@ -38,9 +38,9 @@ describe('qfile', () => {
 
         it('rnd_unique() is unique-ish', () => {
             const repeats = 1000
-            const u = this.qfile.rnd_unique()
+            const u = qfile.rnd_unique()
             for (let i = 0; i < repeats; i++) {
-                assert.notEqual(u, this.qfile.rnd_unique())
+                assert.notEqual(u, qfile.rnd_unique())
             }
         })
     })
@@ -49,7 +49,7 @@ describe('qfile', () => {
         it('parts() updates previous queue filenames', () => {
             // $nextattempt_$attempts_$pid_$uniq.$host
             const name = '1111_0_2222_3333.foo.example.com'
-            const parts = this.qfile.parts(name)
+            const parts = qfile.parts(name)
             assert.equal(parts.next_attempt, 1111)
             assert.equal(parts.attempts, 0)
             assert.equal(parts.pid, 2222)
@@ -65,8 +65,8 @@ describe('qfile', () => {
                 uid: 'XXYYZZ',
                 host: os.hostname(),
             }
-            const name = this.qfile.name(overrides)
-            const parts = this.qfile.parts(name)
+            const name = qfile.name(overrides)
+            const parts = qfile.parts(name)
             assert.equal(parts.arrival, overrides.arrival)
             assert.equal(parts.next_attempt, overrides.next_attempt)
             assert.equal(parts.attempts, overrides.attempts)
@@ -75,8 +75,8 @@ describe('qfile', () => {
             assert.equal(parts.host, overrides.host)
         })
 
-        it('handles 4', () => {
-            const r = this.qfile.parts('1484878079415_0_12345_8888.mta1.example.com')
+        it('handles 4-part legacy filename', () => {
+            const r = qfile.parts('1484878079415_0_12345_8888.mta1.example.com')
             delete r.arrival
             delete r.uid
             delete r.counter
@@ -89,8 +89,8 @@ describe('qfile', () => {
             })
         })
 
-        it('handles 7', () => {
-            const r = this.qfile.parts('1516650518128_1516667073032_8_29538_TkPZWz_1_haraka')
+        it('handles 7-part standard filename', () => {
+            const r = qfile.parts('1516650518128_1516667073032_8_29538_TkPZWz_1_haraka')
             delete r.age
             assert.deepEqual(r, {
                 arrival: 1516650518128,
@@ -103,22 +103,22 @@ describe('qfile', () => {
             })
         })
 
-        it('punts on 5', () => {
-            assert.deepEqual(this.qfile.parts('1516650518128_1516667073032_8_29538_TkPZWz'), null)
+        it('punts on 5-part filename', () => {
+            assert.deepEqual(qfile.parts('1516650518128_1516667073032_8_29538_TkPZWz'), null)
         })
     })
 
     describe('hostname', () => {
-        it('hostname, defaults to os.hostname()', () => {
-            assert.deepEqual(this.qfile.hostname(), require('os').hostname())
+        it('defaults to os.hostname()', () => {
+            assert.deepEqual(qfile.hostname(), os.hostname())
         })
 
-        it('hostname, replaces \\ char', () => {
-            assert.deepEqual(this.qfile.hostname('mt\\a1.exam\\ple.com'), 'mt\\057a1.exam\\057ple.com')
+        it('replaces backslash char', () => {
+            assert.deepEqual(qfile.hostname('mt\\a1.exam\\ple.com'), 'mt\\057a1.exam\\057ple.com')
         })
 
-        it('hostname, replaces _ char', () => {
-            assert.deepEqual(this.qfile.hostname('mt_a1.exam_ple.com'), 'mt\\137a1.exam\\137ple.com')
+        it('replaces underscore char', () => {
+            assert.deepEqual(qfile.hostname('mt_a1.exam_ple.com'), 'mt\\137a1.exam\\137ple.com')
         })
     })
 })

package/test/outbound/queue.js

@@ -0,0 +1,233 @@
+'use strict'
+
+const { describe, it, beforeEach, afterEach } = require('node:test')
+const assert = require('node:assert')
+const fs = require('node:fs')
+const path = require('node:path')
+const os = require('node:os')
+
+const queue = require('../../outbound/queue')
+const qfile = require('../../outbound/qfile')
+
+const sourceQueueDir = path.join('test', 'queue')
+const testQueueDir = path.join('test', 'test-queue')
+const fixtureFiles = [
+    '1508455115683_1508455115683_0_90253_9Q4o4V_1_haraka',
+    '1508269674999_1508269674999_0_34002_socVUF_1_haraka',
+]
+
+const clearTestQueue = () => {
+    fs.mkdirSync(testQueueDir, { recursive: true })
+    for (const file of fs.readdirSync(testQueueDir)) {
+        fs.unlinkSync(path.join(testQueueDir, file))
+    }
+}
+
+const populateTestQueue = () => {
+    clearTestQueue()
+    for (const file of fixtureFiles) {
+        fs.copyFileSync(path.join(sourceQueueDir, file), path.join(testQueueDir, file))
+    }
+}
+
+describe('outbound/queue', () => {
+    describe('read_parts', () => {
+        it('parses valid queue filenames', () => {
+            const filename = '1507509981169_1507509981169_0_61403_e0Y0Ym_1_haraka'
+            const parts = queue.read_parts(filename)
+            assert.ok(parts)
+            assert.equal(parts.arrival, 1507509981169)
+            assert.equal(parts.next_attempt, 1507509981169)
+            assert.equal(parts.attempts, 0)
+            assert.equal(parts.pid, 61403)
+            assert.equal(parts.uid, 'e0Y0Ym')
+        })
+
+        it('rejects dot files', () => {
+            assert.strictEqual(queue.read_parts('__tmp__.filename'), false)
+        })
+
+        it('rejects error files', () => {
+            assert.strictEqual(queue.read_parts('error.something'), false)
+        })
+
+        it('rejects invalid queue files', () => {
+            assert.strictEqual(queue.read_parts('invalid-file'), false)
+        })
+    })
+
+    describe('load_queue_files', () => {
+        beforeEach(() => {
+            populateTestQueue()
+        })
+        afterEach(() => {
+            clearTestQueue()
+        })
+
+        it('processes valid queue files', async () => {
+            const seen = []
+
+            const files = await queue.load_queue_files(
+                null,
+                ['1508455115683_1508455115683_0_90253_9Q4o4V_1_haraka'],
+                (file) => {
+                    seen.push(file)
+                    return file
+                },
+            )
+            assert.equal(seen.length, 1)
+            assert.equal(files[0], '1508455115683_1508455115683_0_90253_9Q4o4V_1_haraka')
+        })
+
+        it('skips invalid files', async () => {
+            const seen = []
+
+            await queue.load_queue_files(
+                null,
+                ['1507509981169_1507509981169_0_61403_e0Y0Ym_1_haraka', 'invalid-file', 'zero-length'],
+                (file) => {
+                    seen.push(file)
+                },
+            )
+            assert.equal(seen.length, 1)
+        })
+
+        it('filters files by pid', async () => {
+            let renameAttempts = 0
+
+            const originalRename = queue.rename_to_actual_pid
+            queue.rename_to_actual_pid = (_file, _parts) => {
+                renameAttempts++
+                throw new Error('test skip')
+            }
+
+            await queue.load_queue_files(
+                61403,
+                [
+                    '1507509981169_1507509981169_0_61403_e0Y0Ym_1_haraka',
+                    '1508455115683_1508455115683_0_90253_9Q4o4V_1_haraka',
+                ],
+                (_file) => {},
+            )
+            queue.rename_to_actual_pid = originalRename
+            assert.equal(renameAttempts, 1)
+        })
+    })
+
+    describe('ensure_queue_dir', () => {
+        it('creates queue dir', async () => {
+            const tmpDir = path.join(os.tmpdir(), `haraka-test-queue-${Date.now()}`)
+
+            const originalQueueDir = queue.queue_dir
+            queue.queue_dir = tmpDir
+
+            try {
+                await queue.ensure_queue_dir()
+                assert.ok(fs.existsSync(tmpDir))
+                const stat = await fs.promises.stat(tmpDir)
+                assert.ok(stat.isDirectory())
+            } catch (err) {
+                assert.fail(`ensure_queue_dir threw an error: ${err.message}`)
+            } finally {
+                queue.queue_dir = originalQueueDir
+                if (fs.existsSync(tmpDir)) fs.rmSync(tmpDir, { recursive: true })
+            }
+        })
+
+        it('returns early if queue dir already exists', async () => {
+            const tmpDir = path.join(os.tmpdir(), `haraka-test-queue-exists-${Date.now()}`)
+            fs.mkdirSync(tmpDir)
+
+            const originalQueueDir = queue.queue_dir
+            queue.queue_dir = tmpDir
+
+            try {
+                await queue.ensure_queue_dir()
+                assert.ok(fs.existsSync(tmpDir))
+            } catch (err) {
+                assert.fail(`ensure_queue_dir threw an error: ${err.message}`)
+            } finally {
+                queue.queue_dir = originalQueueDir
+                fs.rmSync(tmpDir, { recursive: true })
+            }
+        })
+    })
+
+    describe('_load_cur_queue', () => {
+        beforeEach(() => {
+            populateTestQueue()
+        })
+        afterEach(() => {
+            clearTestQueue()
+        })
+
+        it('reads queue directory and processes files', async () => {
+            const processedFiles = []
+            await queue._load_cur_queue(null, (file) => {
+                processedFiles.push(file)
+            })
+
+            assert.ok(processedFiles.length >= 0)
+        })
+    })
+
+    describe('list_queue', () => {
+        beforeEach(() => {
+            populateTestQueue()
+        })
+        afterEach(() => {
+            clearTestQueue()
+        })
+
+        it('returns todo objects from real queue files', async () => {
+            const qlist = await queue.list_queue()
+            assert.ok(Array.isArray(qlist))
+            assert.ok(qlist.length > 0)
+            assert.ok(qlist[0].mail_from)
+            assert.ok(Array.isArray(qlist[0].rcpt_to))
+        })
+    })
+
+    describe('stat_queue', () => {
+        beforeEach(() => {
+            populateTestQueue()
+        })
+        afterEach(() => {
+            clearTestQueue()
+        })
+
+        it('returns queue stats', async () => {
+            const stats = await queue.stat_queue()
+            assert.ok(stats)
+            assert.ok('queue_dir' in stats)
+            assert.ok(stats.queue_count >= 1)
+        })
+    })
+
+    describe('load_pid_queue', () => {
+        beforeEach(() => {
+            populateTestQueue()
+        })
+        afterEach(() => {
+            clearTestQueue()
+        })
+
+        it('delegates pid loading to init_queue', async () => {
+            const parts = qfile.parts(fixtureFiles[0])
+            const observed = []
+            const originalLoadQueue = queue.init_queue
+
+            queue.init_queue = (pid) => {
+                observed.push(pid)
+            }
+
+            try {
+                assert.ok(fs.existsSync(path.join(testQueueDir, fixtureFiles[0])))
+                await queue.load_pid_queue(parts.pid)
+                assert.deepEqual(observed, [parts.pid])
+            } finally {
+                queue.init_queue = originalLoadQueue
+            }
+        })
+    })
+})