Remove js.Bundle.IsExecutable() because it's not needed anymore

js/bundle.go

@@ -296,13 +296,6 @@ func (b *Bundle) Instantiate(
 	return bi, instErr
 }
 
-// IsExecutable returns whether the given name is an exported and
-// executable function in the script.
-func (b *Bundle) IsExecutable(name string) bool {
-	_, exists := b.exports[name]
-	return exists
-}
-
 // Instantiates the bundle into an existing runtime. Not public because it also messes with a bunch
 // of other things, will potentially thrash data and makes a mess in it if the operation fails.
 func (b *Bundle) instantiate(logger logrus.FieldLogger, rt *goja.Runtime, init *InitContext, vuID uint64) error {

js/runner.go

@@ -364,10 +364,9 @@ func (r *Runner) GetOptions() lib.Options {
 
 // IsExecutable returns whether the given name is an exported and
 // executable function in the script.
-//
-// TODO: completely remove this?
 func (r *Runner) IsExecutable(name string) bool {
-	return r.Bundle.IsExecutable(name)
+	_, exists := r.Bundle.exports[name]
+	return exists
 }
 
 // HandleSummary calls the specified summary callback, if supplied.