Haraka 3.0.3 → 3.0.5

package/test/outbound/hmail.js

@@ -0,0 +1,141 @@
+
+const assert = require('node:assert')
+const fs   = require('node:fs')
+const path = require('node:path')
+
+const Hmail = require('../../outbound/hmail');
+const outbound = require('../../outbound/index');
+
+
+describe('outbound/hmail', () => {
+    beforeEach((done) => {
+        this.hmail = new Hmail('1508455115683_1508455115683_0_90253_9Q4o4V_1_haraka', 'test/queue/1508455115683_1508455115683_0_90253_9Q4o4V_1_haraka', {});
+        done()
+    })
+
+    it('sort_mx', (done) => {
+        const sorted = this.hmail.sort_mx([
+            { exchange: 'mx2.example.com', priority: 5 },
+            { exchange: 'mx1.example.com', priority: 6 },
+        ])
+        assert.equal(sorted[0].exchange, 'mx2.example.com')
+        done()
+    })
+    it('sort_mx, shuffled', (done) => {
+        const sorted = this.hmail.sort_mx([
+            { exchange: 'mx2.example.com', priority: 5 },
+            { exchange: 'mx1.example.com', priority: 6 },
+            { exchange: 'mx3.example.com', priority: 6 },
+        ])
+        assert.equal(sorted[0].exchange, 'mx2.example.com')
+        assert.ok(sorted[1].exchange == 'mx3.example.com' || sorted[1].exchange == 'mx1.example.com')
+        done()
+    })
+    it('force_tls', (done) => {
+        this.hmail.todo = { domain: 'miss.example.com' }
+        this.hmail.obtls.cfg = { force_tls_hosts: ['1.2.3.4', 'hit.example.com'] }
+        assert.equal(this.hmail.get_force_tls({ exchange: '1.2.3.4' }), true)
+        assert.equal(this.hmail.get_force_tls({ exchange: '1.2.3.5' }), false)
+        this.hmail.todo = { domain: 'hit.example.com' }
+        assert.equal(this.hmail.get_force_tls({ exchange: '1.2.3.5' }), true)
+        done()
+    })
+})
+
+describe('outbound/hmail.HMailItem', () => {
+    it('normal queue file', (done) => {
+        this.hmail = new Hmail('1508455115683_1508455115683_0_90253_9Q4o4V_1_haraka', 'test/queue/1508455115683_1508455115683_0_90253_9Q4o4V_1_haraka', {});
+        this.hmail.on('ready', () => {
+            // console.log(this.hmail);
+            assert.ok(this.hmail)
+            done()
+        })
+        this.hmail.on('error', err => {
+            console.log(err)
+            assert.equal(err, undefined)
+            done()
+        })
+    })
+    it('normal TODO w/multibyte chars loads w/o error', (done) => {
+        this.hmail = new Hmail('1507509981169_1507509981169_0_61403_e0Y0Ym_1_qfile', 'test/fixtures/todo_qfile.txt', {});
+        this.hmail.on('ready', () => {
+            // console.log(this.hmail);
+            assert.ok(this.hmail)
+            done()
+        })
+        this.hmail.on('error', err => {
+            console.log(err)
+            assert.equal(err, undefined)
+            done()
+        })
+    })
+    it('too short TODO length declared', (done) => {
+        this.hmail = new Hmail('1507509981169_1507509981169_0_61403_e0Y0Ym_1_haraka', 'test/queue/1507509981169_1507509981169_0_61403_e0Y0Ym_1_haraka', {});
+        this.hmail.on('ready', () => {
+            // console.log(this.hmail);
+            assert.ok(this.hmail)
+            done();
+        })
+        this.hmail.on('error', (err) => {
+            console.log(err);
+            assert.ok(err);
+            done();
+        })
+    })
+    it('too long TODO length declared', (done) => {
+        this.hmail = new Hmail('1508269674999_1508269674999_0_34002_socVUF_1_haraka', 'test/queue/1508269674999_1508269674999_0_34002_socVUF_1_haraka', {});
+        this.hmail.on('ready', () => {
+            // console.log(this.hmail);
+            assert.ok(this.hmail)
+            done();
+        })
+        this.hmail.on('error', (err) => {
+            console.log(err);
+            assert.ok(err);
+            done();
+        })
+    })
+    it('zero-length file load skip w/o crash', (done) => {
+        this.hmail = new Hmail('1507509981169_1507509981169_0_61403_e0Y0Ym_2_zero', 'test/queue/zero-length', {});
+        this.hmail.on('ready', () => {
+            assert.ok(this.hmail)
+            done();
+        })
+        this.hmail.on('error', (err) => {
+            console.error(err);
+            assert.ok(err);
+            done();
+        })
+    })
+    it('lifecycle, reads and writes a haraka queue file', (done) => {
+
+        this.hmail = new Hmail('1507509981169_1507509981169_0_61403_e0Y0Ym_2_qfile', 'test/fixtures/todo_qfile.txt', {});
+
+        this.hmail.on('error', (err) => {
+            // console.log(err);
+            assert.equals(err, undefined);
+            done();
+        })
+
+        this.hmail.on('ready', () => {
+
+            const tmpfile = path.resolve('test', 'test-queue', 'delete-me');
+            const ws = new fs.createWriteStream(tmpfile)
+
+            outbound.build_todo(this.hmail.todo, ws, () => {
+                // console.log('returned from build_todo, piping')
+                // console.log(this.hmail.todo)
+                // assert.equals(this.hmail.todo.message_stream.headers.length, 22);
+
+                const ds = this.hmail.data_stream()
+                ds.pipe(ws, { dot_stuffing: true });
+
+                ws.on('close', () => {
+                    // console.log(this.hmail.todo)
+                    assert.equal(fs.statSync(tmpfile).size, 4204);
+                    done();
+                })
+            })
+        })
+    })
+})

package/test/outbound/index.js

@@ -0,0 +1,220 @@
+'use strict';
+
+const assert = require('node:assert')
+const fs   = require('fs');
+const path = require('path');
+
+const constants = require('haraka-constants');
+const logger = require('../../logger');
+
+const lines = [
+    'From: John Johnson <john@example.com>',
+    'To: Jane Johnson <jane@example.com>',
+    "Subject: What's for dinner?",
+    '',
+    "I'm hungry.",
+    '',
+];
+
+describe('outbound', () => {
+
+    it('converts \\n and \\r\\n line endings to \\r\\n' , () => {
+
+        for (const ending of ['\n', '\r\n']) {
+            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
+                result += line;
+                contents = contents.substr(match[1].length);
+                if (contents.length === 0) {
+                    break;
+                }
+            }
+
+            assert.deepEqual(lines.join('\r\n'), result);
+        }
+    })
+
+    it('log_methods added', () => {
+        const levels = ['DATA','PROTOCOL','DEBUG','INFO','NOTICE','WARN','ERROR','CRIT','ALERT','EMERG']
+
+        const HMailItem = require('../../outbound/hmail');
+
+        for (const level of levels) {
+            assert.ok(HMailItem.prototype[`log${level.toLowerCase()}`], `Log method for level: ${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(config.cfg.temp_fail_intervals, [64,128,256,512,1024,2048,4096,8192,16384,32768,65536,131072]);
+    })
+
+    describe('get_tls_options', () => {
+        beforeEach((done) => {
+            process.env.HARAKA_TEST_DIR=path.resolve('test');
+            this.outbound = require('../../outbound');
+            this.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)
+        })
+
+        afterEach((done) => {
+            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'}
+            );
+
+            assert.deepEqual(tls_config, {
+                servername: 'mail.example.com',
+                key: fs.readFileSync(path.resolve('test','config','outbound_tls_key.pem')),
+                cert: fs.readFileSync(path.resolve('test','config','outbound_tls_cert.pem')),
+                dhparam: fs.readFileSync(path.resolve('test','config','dhparams.pem')),
+                ciphers: 'ECDHE-RSA-AES256-GCM-SHA384',
+                minVersion: 'TLSv1',
+                rejectUnauthorized: false,
+                requestCert: false,
+                honorCipherOrder: false,
+                redis: { disable_for_failed_hosts: false },
+                no_tls_hosts: ['127.0.0.2', '192.168.31.1/24'],
+                force_tls_hosts: ['first.example.com', 'second.example.net']
+            })
+        })
+    })
+
+    describe('build_todo', () => {
+        beforeEach((done) => {
+            this.outbound = require('../../outbound');
+            try {
+                fs.unlinkSync('test/queue/multibyte');
+                fs.unlinkSync('test/queue/plain');
+            }
+            catch (ignore) {}
+            done();
+        })
+
+        it('saves a 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, () => {
+                ws.write(Buffer.from('This is the message body'));
+                fs.fsync(fd, () => { ws.close(); })
+            })
+        })
+
+        it('saves a 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"}');
+            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, () => {
+                ws.write(Buffer.from('This is the message body'));
+                fs.fsync(fd, () => { ws.close(); })
+            })
+        })
+    })
+
+    describe('timer_queue', () => {
+        beforeEach((done) => {
+            process.env.HARAKA_TEST_DIR=path.resolve('test');
+            this.outbound = require('../../outbound');
+            const TimerQueue = require('../../outbound/timer_queue');
+            this.ob_timer_queue = new TimerQueue(500);
+            done();
+        })
+
+        afterEach((done) => {
+            delete process.env.HARAKA_TEST_DIR;
+            this.ob_timer_queue.shutdown();
+            done()
+        })
+
+        it('has initial length of 0', () => {
+            assert.equal(this.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);
+        })
+
+        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);
+        })
+
+        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");
+        })
+    })
+})

package/test/outbound/qfile.js

@@ -0,0 +1,126 @@
+const assert = require('node:assert')
+const os   = require('node:os');
+
+describe('qfile', () => {
+
+    describe('qfile', () => {
+        beforeEach((done) => {
+            this.qfile = require('../../outbound/qfile')
+            done();
+        })
+
+        it('name() basic functions', () => {
+            const name = this.qfile.name();
+            const split = name.split('_');
+            assert.equal(split.length, 7);
+            assert.equal(split[2], 0);
+            assert.equal(split[3], process.pid);
+        })
+
+        it('name() with overrides', () => {
+            const overrides = {
+                arrival : 12345,
+                next_attempt : 12345,
+                attempts : 15,
+                pid : process.pid,
+                uid : 'XXYYZZ',
+                host : os.hostname(),
+            };
+            const name = this.qfile.name(overrides);
+            const split = name.split('_');
+            assert.equal(split.length, 7);
+            assert.equal(split[0], overrides.arrival);
+            assert.equal(split[1], overrides.next_attempt);
+            assert.equal(split[2], overrides.attempts);
+            assert.equal(split[3], overrides.pid);
+            assert.equal(split[4], overrides.uid);
+            assert.equal(split[6], overrides.host);
+        })
+
+        it('rnd_unique() is unique-ish', () => {
+            const repeats = 1000;
+            const u = this.qfile.rnd_unique();
+            for (let i = 0; i < repeats; i++){
+                assert.notEqual(u, this.qfile.rnd_unique());
+            }
+        })
+    })
+
+    describe('parts', () => {
+
+        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);
+            assert.equal(parts.next_attempt, 1111);
+            assert.equal(parts.attempts, 0);
+            assert.equal(parts.pid, 2222);
+            assert.equal(parts.host, 'foo.example.com');
+        })
+
+        it('parts() handles standard queue filenames', () => {
+            const overrides = {
+                arrival : 12345,
+                next_attempt : 12345,
+                attempts : 15,
+                pid : process.pid,
+                uid : 'XXYYZZ',
+                host : os.hostname(),
+            };
+            const name = this.qfile.name(overrides);
+            const parts = this.qfile.parts(name);
+            assert.equal(parts.arrival, overrides.arrival);
+            assert.equal(parts.next_attempt, overrides.next_attempt);
+            assert.equal(parts.attempts, overrides.attempts);
+            assert.equal(parts.pid, overrides.pid);
+            assert.equal(parts.uid, overrides.uid);
+            assert.equal(parts.host, overrides.host);
+        })
+
+        it('handles 4', () => {
+            const r = this.qfile.parts('1484878079415_0_12345_8888.mta1.example.com')
+            delete r.arrival
+            delete r.uid
+            delete r.counter
+            assert.deepEqual(r, {
+                next_attempt: 1484878079415,
+                attempts: 0,
+                pid: 12345,
+                host: 'mta1.example.com',
+                age: 0
+            })
+        })
+
+        it('handles 7', () => {
+            const r = this.qfile.parts('1516650518128_1516667073032_8_29538_TkPZWz_1_haraka')
+            delete r.age;
+            assert.deepEqual(r, {
+                arrival: 1516650518128,
+                next_attempt: 1516667073032,
+                attempts: 8,
+                pid: 29538,
+                uid: 'TkPZWz',
+                counter: 1,
+                host: 'haraka',
+            })
+        })
+
+        it('punts on 5', () => {
+            assert.deepEqual(this.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('hostname, replaces \\ char', () => {
+            assert.deepEqual(this.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')
+        })
+    })
+})

package/test/outbound_bounce_net_errors.js

@@ -0,0 +1,142 @@
+'use strict';
+
+// Testing bounce email contents related to errors occuring during SMTP dialog
+
+// About running the tests:
+// - Making a folder for queuing files
+// - Creating a HMailItem instance using fixtures/util_hmailitem
+// - Talk some STMP in the playbook
+// - Test the outcome by replacing trigger functions with our testing code (outbound.send_email, HMailItem.temp_fail, ...)
+
+const assert = require('node:assert')
+const dns    = require('node:dns');
+const fs     = require('node:fs');
+const path   = require('node:path');
+
+const constants   = require('haraka-constants');
+const util_hmailitem = require('./fixtures/util_hmailitem');
+const TODOItem       = require('../outbound/todo');
+const HMailItem      = require('../outbound/hmail');
+const outbound       = require('../outbound');
+
+const outbound_context = {
+    TODOItem,
+    exports: outbound
+}
+
+const queue_dir = path.resolve(__dirname, 'test-queue');
+
+describe('outbound_bounce_net_errors', () => {
+    beforeEach((done) => {
+        fs.exists(queue_dir, exists => {
+            if (exists) {
+                done();
+            }
+            else {
+                fs.mkdir(queue_dir, done)
+            }
+        });
+    })
+
+    afterEach((done) => {
+        fs.exists(queue_dir, (exists) => {
+            if (exists) {
+                for (const file of fs.readdirSync(queue_dir)) {
+                    const curPath = path.resolve(queue_dir, file);
+                    if (fs.lstatSync(curPath).isDirectory()) {
+                        console.error(`did not expect an sub folder here ("${curPath}")! cancel`)
+                    }
+                    fs.unlinkSync(curPath);
+                }
+            }
+            done()
+        })
+    })
+
+    it('test get-mx-deny triggers bounce(...)', (done) => {
+        util_hmailitem.newMockHMailItem(outbound_context, done, {}, mock_hmail => {
+            const orig_bounce = HMailItem.prototype.bounce;
+            HMailItem.prototype.bounce = function (err, opts) {
+                assert.ok(true, 'get_mx=DENY: bounce function called');
+                /* dsn_code: 550,
+                 dsn_status: '5.1.2',
+                 dsn_action: 'failed' */
+                assert.equal('5.1.2', this.todo.rcpt_to[0].dsn_status, 'get_mx=DENY dsn status = 5.1.2');
+                done()
+            };
+            mock_hmail.domain = mock_hmail.todo.domain;
+            HMailItem.prototype.get_mx_respond.apply(mock_hmail, [constants.deny, {}]);
+            HMailItem.prototype.bounce = orig_bounce;
+        })
+    })
+
+    it('test get-mx-denysoft triggers temp_fail(...)', (done) => {
+        util_hmailitem.newMockHMailItem(outbound_context, done, {}, mock_hmail => {
+            const orig_temp_fail = HMailItem.prototype.temp_fail;
+            HMailItem.prototype.temp_fail = function (err, opts) {
+                assert.ok(true, 'get_mx-DENYSOFT: temp_fail function called');
+                /*dsn_code: 450,
+                 dsn_status: '4.1.2',
+                 dsn_action: 'delayed' */
+                assert.equal('4.1.2', this.todo.rcpt_to[0].dsn_status, 'get_mx=DENYSOFT dsn status = 4.1.2');
+                done()
+            };
+            mock_hmail.domain = mock_hmail.todo.domain;
+            HMailItem.prototype.get_mx_respond.apply(mock_hmail, [constants.denysoft, {}]);
+            HMailItem.prototype.temp_fail = orig_temp_fail;
+        })
+    })
+
+    it('test found_mx({code:dns.NXDOMAIN}) triggers bounce(...)', (done) => {
+        util_hmailitem.newMockHMailItem(outbound_context, done, {}, mock_hmail => {
+            const orig_bounce = HMailItem.prototype.bounce;
+            HMailItem.prototype.bounce = function (err, opts) {
+                assert.ok(true, 'get_mx_error({code: dns.NXDOMAIN}): bounce function called');
+                assert.equal('5.1.2', this.todo.rcpt_to[0].dsn_status, 'get_mx_error({code: dns.NXDOMAIN}: dsn status = 5.1.2');
+                done()
+            };
+            HMailItem.prototype.get_mx_error.apply(mock_hmail, [{code: dns.NXDOMAIN}]);
+            HMailItem.prototype.bounce = orig_bounce;
+        });
+    })
+
+    it('test get_mx_error({code:\'SOME-OTHER-ERR\'}) triggers temp_fail(...)', (done) => {
+        util_hmailitem.newMockHMailItem(outbound_context, done, {}, mock_hmail => {
+            const orig_temp_fail = HMailItem.prototype.temp_fail;
+            HMailItem.prototype.temp_fail = function (err, opts) {
+                assert.ok(true, 'get_mx_error({code: "SOME-OTHER-ERR"}): temp_fail function called');
+                assert.equal('4.1.0', this.todo.rcpt_to[0].dsn_status, 'get_mx_error({code: "SOME-OTHER-ERR"}: dsn status = 4.1.0');
+                done()
+            };
+            HMailItem.prototype.get_mx_error.apply(mock_hmail, [{code: 'SOME-OTHER-ERR'}, {}]);
+            HMailItem.prototype.temp_fail = orig_temp_fail;
+        });
+    })
+
+    it('test found_mx(null, [{priority:0,exchange:\'\'}]) triggers bounce(...)', (done) => {
+        util_hmailitem.newMockHMailItem(outbound_context, done, {}, mock_hmail => {
+            const orig_bounce = HMailItem.prototype.bounce;
+            HMailItem.prototype.bounce = function (err, opts) {
+                assert.ok(true, 'found_mx(null, [{priority:0,exchange:""}]): bounce function called');
+                assert.equal('5.1.2', this.todo.rcpt_to[0].dsn_status, 'found_mx(null, [{priority:0,exchange:""}]): dsn status = 5.1.2');
+                done()
+            };
+            HMailItem.prototype.found_mx.apply(mock_hmail, [[{priority:0,exchange:''}]]);
+            HMailItem.prototype.bounce = orig_bounce;
+        });
+    })
+
+    it('test try_deliver while hmail.mxlist=[] triggers bounce(...)', (done) => {
+        util_hmailitem.newMockHMailItem(outbound_context, done, {}, mock_hmail => {
+            mock_hmail.mxlist = [];
+            const orig_temp_fail = HMailItem.prototype.temp_fail;
+            HMailItem.prototype.temp_fail = function (err, opts) {
+                assert.ok(true, 'try_deliver while hmail.mxlist=[]: temp_fail function called');
+                assert.equal('5.1.2', this.todo.rcpt_to[0].dsn_status, 'try_deliver while hmail.mxlist=[]: dsn status = 5.1.2');
+                done()
+            };
+            HMailItem.prototype.try_deliver.apply(mock_hmail, []);
+            HMailItem.prototype.temp_fail = orig_temp_fail;
+        });
+    })
+})