Add singleton support for Logger class

[tsingh2k15]

My attempt to contribute and help for #63

• Modify Logger class such that a single instance can be reused across app.
• Update unit tests impacted by this change.
• Fix failing tests due to incorrect protobuf objects used in source code. {Ignore this, seems already fixed in master when I rebased)

Note: I did not freeze Logger instance because it is required to be
spied in jest unit tests. Freezing it would have not let it be spied by
jest mocks on log function. This change should allow reuse of a single
logger instance but improper use of logger inside source code can still
allow the verbose setting to be misused.

[codecov]

Codecov Report

Merging #106 into master will increase coverage by 0.20%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
+ Coverage   70.94%   71.15%   +0.20%     
==========================================
  Files          21       21              
  Lines         716      721       +5     
  Branches       94       95       +1     
==========================================
+ Hits          508      513       +5     
  Misses        184      184              
  Partials       24       24              

Impacted Files	Coverage Δ
src/syft.js	15.09% <ø> (ø)
src/_helpers.js	95.45% <100.00%> (+0.21%)	⬆️
src/logger.js	100.00% <100.00%> (ø)
src/sockets.js	100.00% <100.00%> (ø)
src/types/message.js	83.60% <100.00%> (+0.27%)	⬆️
src/webrtc.js	90.66% <100.00%> (ø)

---

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 0d68026...7e161a8. Read the comment docs.

[cereallarceny]

@tsingh2k15 Does this preserve the verbose setting originally set in the main Syft class? You should be able to set verbose: true and get logs or verbose: false and not get them... all within the main new syft() call. Can you confirm this and I'll merge?

[tsingh2k15]

@tsingh2k15 Does this preserve the verbose setting originally set in the main Syft class? You should be able to set verbose: true and get logs or verbose: false and not get them... all within the main new syft() call. Can you confirm this and I'll merge?

@cereallarceny Sure, I will look into it and get back to you.

[tsingh2k15]

@tsingh2k15 Does this preserve the verbose setting originally set in the main Syft class? You should be able to set verbose: true and get logs or verbose: false and not get them... all within the main new syft() call. Can you confirm this and I'll merge?

@cereallarceny Sure, I will look into it and get back to you.

@cereallarceny I added unit test to validate singleton behavior. It works as expected. Let me know your thoughts. 7925f0f

[cereallarceny]

I'm good to merge this once @vvmnnnkv approves.

[tsingh2k15]

I'm good to merge this once @vvmnnnkv approves.

@vvmnnnkv Can you please review ? There were some merge conflicts I have resolved, but please let me know if I may have missed something while resolving merge conflicts. Thanks!

[tsingh2k15]

I'm good to merge this once @vvmnnnkv approves.

@vvmnnnkv Can you please review ? There were some merge conflicts I have resolved, but please let me know if I may have missed something while resolving merge conflicts. Thanks!

@vvmnnnkv I am following up on this, please let me know if you have questions.

[vvmnnnkv]

Looks good, thanks! :)