Add ability to read, and set the active state of streams on groups

[SteelPhase]

This is exposing the RESTful portion of the Entertainment API

[codecov-io]

Codecov Report

Merging #29 into master will decrease coverage by 0.50%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   67.28%   66.78%   -0.51%     
==========================================
  Files           5        5              
  Lines        1131     1165      +34     
==========================================
+ Hits          761      778      +17     
- Misses        202      214      +12     
- Partials      168      173       +5     

Impacted Files	Coverage Δ
group.go	86.40% <50.00%> (-13.60%)	⬇️

---

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 7b16d16...648a98c. Read the comment docs.

[amimof]

@SteelPhase Thanks for the contribution. I was thinking. Perhaps it’s better to name the functions EnableStreaming() and DisableStreaming() for readability. What do you think?

[SteelPhase]

That's a good Name. I was undecided on what to call it and just picked something. I'll try it out.

[SteelPhase]

Sorry for the delay. I adjust the naming, and tested further. Found a few bugs with how nil booleans were handled in the DisableStream function. Verified it's all working as it should.

[amimof]

Nicely done. Have a look at my comments

[SteelPhase]

Going to need a bit of time to look at the tests, but i'll verify what I can soon

[SteelPhase]

I added significantly more testing around the entertainment api components

[SteelPhase]

Got sick of fighting with the unit tests differences between 1.13 and 1.14, I split them out. I'm waiting for the travis-ci job to complete.

[SteelPhase]

I think this is ready for review again

[amimof]

@SteelPhase You might have already tried it but perhaps you could use the assert? For example:

_, err := b.getAPIPath("/")
assert.NotNil(t, err)

[SteelPhase]

Switched it over to use assert.NotNil

[amimof]

Nice work! Two minor typos. I’ll merge once fixed

[SteelPhase]

Should be fixed. got distracted with GopherCon stuff