Windows: mounted is always true, even when no disk on the CD-ROM

[jacargentina]

This is shown for a Windows 10 VirtualBox.

> console.log('%o', x.filesystemSync())
{ 'C:':
   { id: 'C:',
     node: 'C:',
     whole: true,
     parent: 'C:',
     name: 'C:',
     size: 125332606976,
     description: 'Disco fijo local',
     protocol: null,
     blockSize: null,
     readOnly: null,
     removable: null,
     volumes:
      [ { id: 'C:',
          node: 'C:',
          whole: false,
          parent: 'C:',
          name: null,
          description: null,
          blockSize: null,
          blocks: null,
          readOnly: null,
          mounted: true,
          mountPoint: 'C:',
          partitionType: null,
          fs: 'NTFS',
          space:
           { total: 125332606976,
             available: 15318355968,
             used: 110014251008 } },
        [length]: 1 ] },
  undefined:
   { id: 'D:',
     node: 'D:',
     whole: true,
     parent: 'D:',
     name: undefined,
     size: NaN,
     description: 'Disco CD-ROM',
     protocol: null,
     blockSize: null,
     readOnly: null,
     removable: null,
     volumes:
      [ { id: 'D:',
          node: 'D:',
          whole: false,
          parent: 'D:',
          name: null,
          description: null,
          blockSize: null,
          blocks: null,
          readOnly: null,
          mounted: true,
          mountPoint: undefined,
          partitionType: null,
          fs: 'D:',
          space: { total: NaN, available: NaN, used: NaN } },
        [length]: 1 ] },
  'E:':
   { id: 'E:',
     node: 'E:',
     whole: true,
     parent: 'E:',
     name: 'E:',
     size: 286298181632,
     description: 'Conexión de red',
     protocol: null,
     blockSize: null,
     readOnly: null,
     removable: null,
     volumes:
      [ { id: 'E:',
          node: 'E:',
          whole: false,
          parent: 'E:',
          name: 'VBOX_Dropbox',
          description: null,
          blockSize: null,
          blocks: null,
          readOnly: null,
          mounted: true,
          mountPoint: 'E:',
          partitionType: null,
          fs: 'VBoxSharedFolderFS',
          space:
           { total: 286298181632,
             available: 140859002880,
             used: 145439178752 } },
        [length]: 1 ] } }

Notes

• Second drive is undefined key; may be it is related to not having any disk on it? Maybe would be nice for it to be D: even it doesnt have a disk mounted.
• mounted: true, but there is no CD on the drive

[jacargentina]

@philippefutureboy what do you think?

[p-hebert]

Hm, that's really interesting. I'm pretty sure it's a bug due to the wmic output leaving the name field blank. Then again, I've never tested on W10. I only tested on a W7 VM.

Have you tried debugging it using the Node Debugger protocol to see what is the input data?

[pdirksen]

> wmic logicaldisk get access,DeviceID,Description,FileSystem

Access  Description         DeviceID  FileSystem
1       CD                  E:        CDFS

Access is set to 1 when having a CD in the drive. The 'Access' column should also be parsed just for this case.

[jacargentina]

@philippefutureboy is not related to Windows version; code simply puts it with true always:

// src/windows/windows.js:24
   volume.mounted = true;

I'll implement the access column parsing for that as @pdirksen pointed.

[jacargentina]

Mmm, testing the Access column. https://msdn.microsoft.com/en-us/library/aa394173(v=vs.85).aspx#properties

On my W10 virtual box, I've got Access = 0 for the C: (windows) drive... Unknown access?

😳🤨

[jacargentina]

FileSystem may be the real thing for this?

[pdirksen]

Might be. I would test it with different media like CDs, DVD, Bluray and blanks as well.

[p-hebert]

That's the big problem with wmic: It's not that useful.
If I remember properly, when I wrote this, W7 and below did not have any other non-elevated-privilege filesystem utilities. Would you know any other native utilities that could help?