feat(expose, hide): allow any number of strings as argument

Author: solkimicreb

README.md

@@ -76,28 +76,28 @@ context.item = {name: 'item name'}
 result = expression(context) // result is 'item name'
 ```
 
-### compiler.expose(String)
+### compiler.expose(String, String, String, ...)
 
 Use this method to expose globals to the compiler. Non of the globals are exposed by default.
 
 ```js
-const code = compiler.compileCode('console.log(message)')
-compiler.expose('console')
-code({message: 'Hello World'}) // logs 'Hello World' to the console
+const code = compiler.compileCode('console.log(Math.round(num))')
+compiler.expose('console', 'Math')
+code({num: 1.8}) // logs 2 to the console
 ```
 
 Context variables are always favored over global ones, when both are present (with the same name).
 
-### compiler.hide(String)
+### compiler.hide(String, String, String, ...)
 
 Use this method to hide exposed globals from the compiler.
 
 ```js
-const code = compiler.compileCode('console.log(message)')
-compiler.expose('console')
-code({message: 'Hello World'}) // logs 'Hello World' to the console
-compiler.hide('console')
-code({message: 'Hello World'}) // throws an error, console is undefined
+const code = compiler.compileCode('console.log(Math.round(num))')
+compiler.expose('console', 'Math')
+code({num: 1.8}) // logs 2 to the console
+compiler.hide('console', 'Math')
+code({num: 1.8}) // throws an error, console and Math are undefined
 ```
 
 Context variables are always favored over global ones, when both are present (with the same name).

compiler.js

@@ -56,18 +56,16 @@ function compileCode (src) {
   return code
 }
 
-function expose (globalName) {
-  if (typeof globalName !== 'string') {
-    throw new TypeError('first argument must be a string')
+function expose (...globalNames) {
+  for (let globalName of globalNames) {
+    globals.add(globalName)
   }
-  globals.add(globalName)
 }
 
-function hide (globalName) {
-  if (typeof globalName !== 'string') {
-    throw new TypeError('first argument must be a string')
+function hide (...globalNames) {
+  for (let globalName of globalNames) {
+    globals.delete(globalName)
   }
-  globals.delete(globalName)
 }
 
 function toSandbox (obj) {

compiler.test.js

@@ -81,16 +81,10 @@ describe('nx-compile', () => {
   })
 
   describe('expose', () => {
-    it('should throw a TypeError on non string source argument', () => {
-      expect(() => compiler.expose({})).to.throw(TypeError)
-      expect(() => compiler.expose(undefined)).to.throw(TypeError)
-      expect(() => compiler.expose(12)).to.throw(TypeError)
-    })
-
     it('should expose the passed global', () => {
       const expression = compiler.compileExpression('console')
       expect(expression({})).to.be.undefined
-      compiler.expose('console')
+      compiler.expose('Math', 'console')
       expect(expression({})).to.equal(console)
     })
 
@@ -101,16 +95,10 @@ describe('nx-compile', () => {
   })
 
   describe('hide', () => {
-    it('should throw a TypeError on non string source argument', () => {
-      expect(() => compiler.hide({})).to.throw(TypeError)
-      expect(() => compiler.hide(undefined)).to.throw(TypeError)
-      expect(() => compiler.hide(12)).to.throw(TypeError)
-    })
-
     it('should hide exposed globals', () => {
       const expression = compiler.compileExpression('console')
       expect(expression({})).to.equal(console)
-      compiler.hide('console')
+      compiler.hide('Math', 'console')
       expect(expression({})).to.be.undefined
     })
   })