Apply suggestions from code review

Co-authored-by: Mihail Stoykov <[email protected]>

js/modules/k6/experimental/fs/cache.go

@@ -61,7 +61,7 @@ func (fr *cache) open(filename string, fromFs fsext.Fs) (data []byte, err error)
 
 	// The underlying fsext.Fs.Open method will cache the file content during this
 	// operation. Which will lead to effectively holding the content of the file in memory twice.
-	// However, as per #1079, we plan to eventually reduce our dependency on fsext, and
+	// However, as per #1079, we plan to eventually reduce our dependency on afero, and
 	// expect this issue to be resolved at that point.
 	// TODO: re-evaluate opening from the FS this once #1079 is resolved.
 	f, err := fromFs.Open(filename)

js/modules/k6/experimental/fs/file.go

@@ -26,7 +26,7 @@ type file struct {
 // Stat returns a FileInfo describing the named file.
 func (f *file) stat() *FileInfo {
 	filename := filepath.Base(f.path)
-	return &FileInfo{Name: filename, Size: len(f.data)}
+	return &FileInfo{Name: filename, Size: f.size()}
 }
 
 // FileInfo holds information about a file.

js/modules/k6/experimental/fs/module.go

@@ -109,7 +109,7 @@ func (mi *ModuleInstance) openImpl(path string) (*File, error) {
 
 	fs, ok := initEnv.FileSystems["file"]
 	if !ok {
-		common.Throw(mi.vu.Runtime(), errors.New("open() failed; reason: unable to access the file system"))
+		return nil,  errors.New("open() failed; reason: unable to access the file system")
 	}
 
 	if exists, err := fsext.Exists(fs, path); err != nil {