https://github.com/puppetlabs/facter-ng/tree/master/lib/facts/fedora/disk
Currently Fedora has hardcoded facts for disks sda and sr0. If the user has a disk with a different name, the facts will not work.
Also, the fact should be diskS, not disk.