Separate WMI Query Handling from Util

[retomerz]

No description provided.

[codecov-io]

Codecov Report

Merging #728 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #728   +/-   ##
=========================================
  Coverage     82.64%   82.64%           
  Complexity      198      198           
=========================================
  Files            28       28           
  Lines          1527     1527           
  Branches        224      224           
=========================================
  Hits           1262     1262           
  Misses          127      127           
  Partials        138      138

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1588e9b...8f2fb48. Read the comment docs.

[coveralls]

Coverage remained the same at 90.504% when pulling 8f2fb48 on retomerz:issue716 into 1588e9b on oshi:master.

[dbwiddis]

An initial run-through. Key initial comment is that we should move the instantiation of these new classes out of the top-level SystemInfo class so we don't have to pass arguments around everywhere.

[dbwiddis]

I went ahead and made most of the changes that I suggested in the code review:

• Instead of passing the WmiQueryHandler around everywhere as an argument, I made it a singleton.
• I removed the caching classes you put in -- a lot of bloat for simple string maps. I did update those classes to remove the static initializers though.