lovell
diff --git a/‎src/pipeline.cc‎
Lines changed: 2 additions & 2 deletions b/‎src/pipeline.cc‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎test/fixtures/expected/fast-shrink-on-load-false.png‎
-270 Bytes b/‎test/fixtures/expected/fast-shrink-on-load-false.png‎
-270 Bytes
diff --git a/‎…es/expected/fast-shrink-on-load-true.png‎ ‎…ixtures/expected/fast-shrink-on-load.png‎test/fixtures/expected/fast-shrink-on-load-true.png renamed to test/fixtures/expected/fast-shrink-on-load.png b/‎…es/expected/fast-shrink-on-load-true.png‎ ‎…ixtures/expected/fast-shrink-on-load.png‎test/fixtures/expected/fast-shrink-on-load-true.png renamed to test/fixtures/expected/fast-shrink-on-load.png
diff --git a/‎test/unit/resize.js‎
Lines changed: 15 additions & 22 deletions b/‎test/unit/resize.js‎
Lines changed: 15 additions & 22 deletions
@@ -184,7 +184,7 @@ class PipelineWorker : public Napi::AsyncWorker {
 
         if (inputImageType == sharp::ImageType::JPEG) {
           // Leave at least a factor of two for the final resize step, when fastShrinkOnLoad: false
-          // for more consistent results and avoid occasional small image shifting
+          // for more consistent results and to avoid extra sharpness to the image
           int factor = baton->fastShrinkOnLoad ? 1 : 2;
           if (shrink >= 8 * factor) {
             jpegShrinkOnLoad = 8;
@@ -193,7 +193,7 @@ class PipelineWorker : public Napi::AsyncWorker {
           } else if (shrink >= 2 * factor) {
             jpegShrinkOnLoad = 2;
           }
-          // Skip shrink-on-load for known libjpeg rounding errors
+          // Lower shrink-on-load for known libjpeg rounding errors
           if (jpegShrinkOnLoad > 1 && static_cast<int>(shrink) == jpegShrinkOnLoad) {
             jpegShrinkOnLoad /= 2;
           }
 
@@ -637,28 +637,21 @@ describe('Resize dimensions', function () {
       });
   });
 
-  it('fastShrinkOnLoad: false ensures image is not shifted', function (done) {
-    return sharp(fixtures.inputJpgCenteredImage)
-      .resize(9, 8, { fastShrinkOnLoad: false })
-      .png()
-      .toBuffer(function (err, data, info) {
-        if (err) throw err;
-        assert.strictEqual(9, info.width);
-        assert.strictEqual(8, info.height);
-        fixtures.assertSimilar(fixtures.expected('fast-shrink-on-load-false.png'), data, done);
-      });
-  });
-
-  it('fastShrinkOnLoad: true (default) might result in shifted image', function (done) {
-    return sharp(fixtures.inputJpgCenteredImage)
-      .resize(9, 8)
-      .png()
-      .toBuffer(function (err, data, info) {
-        if (err) throw err;
-        assert.strictEqual(9, info.width);
-        assert.strictEqual(8, info.height);
-        fixtures.assertSimilar(fixtures.expected('fast-shrink-on-load-true.png'), data, done);
-      });
+  [
+    true,
+    false
+  ].forEach(function (value) {
+    it(`fastShrinkOnLoad: ${value} does not causes image shifts`, function (done) {
+      sharp(fixtures.inputJpgCenteredImage)
+        .resize(9, 8, { fastShrinkOnLoad: value })
+        .png()
+        .toBuffer(function (err, data, info) {
+          if (err) throw err;
+          assert.strictEqual(9, info.width);
+          assert.strictEqual(8, info.height);
+          fixtures.assertSimilar(fixtures.expected('fast-shrink-on-load.png'), data, done);
+        });
+    });
   });
 
   [