Apply suggestions from code review

Co-authored-by: Manuel Spigolon <[email protected]>
fastify · Sep 22, 2022 · 934be7b · 934be7b
1 parent 5a083e3
commit 934be7b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -97,9 +97,9 @@ module.exports = fp(plugin, {
 ```
 
 #### Encapsulate
-You can optionally keep the plugin encapsulated, which is the default behavior if you weren't using `fastify-plugin`.
+
+By default, `fastify-plugin` breaks the [encapsulation](https://github.com/fastify/fastify/blob/HEAD/docs/Reference/Encapsulation.md) but you can optionally keep the plugin encapsulated.
 This allows you to set the plugin's name and validate its dependencies without making the plugin accessible.
-Read more about encapsulation [here](https://github.com/fastify/fastify/blob/main/docs/Reference/Encapsulation.md).
 ```js
 const fp = require('fastify-plugin')
 

diff --git a/test/test.js b/test/test.js
@@ -266,7 +266,7 @@ test('should accept an option to encapsulate', t => {
   }))
 
   fastify.ready(err => {
-    t.notOk(err)
+    t.error(err)
     t.ok(fastify.hasDecorator('accessible'))
     t.ok(fastify.hasDecorator('alsoAccessible'))
     t.notOk(fastify.hasDecorator('encapsulated'))