doc: make the code a little more accessible with some code comments.

- Clarify the units of rateInterval - Leave a hint for developers wondering how the test suite supresses actually sending email.
nodemailer · Oct 14, 2020 · 0f132fa · 0f132fa
1 parent 1815bad
commit 0f132fa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/ses-transport/index.js b/lib/ses-transport/index.js
@@ -38,7 +38,7 @@ class SESTransport extends EventEmitter {
         // max messages per second
         this.sendingRate = Number(this.options.sendingRate) || Infinity;
         this.sendingRateTTL = null;
-        this.rateInterval = 1000;
+        this.rateInterval = 1000; // milliseconds
         this.rateMessages = [];
 
         this.pending = [];

diff --git a/test/ses-transport/ses-transport-test.js b/test/ses-transport/ses-transport-test.js
@@ -30,6 +30,7 @@ describe('SES Transport Tests', function() {
                 config: {
                     region: 'eu-west-1'
                 },
+                // Prevent tests from actually sending mail by mocking sendRawEmail
                 sendRawEmail: (message, cb) => {
                     setImmediate(() => {
                         cb(null, {