Continuing the Package about forecast

[Craytor]

/cc @joecohens

[joecohens]

For now these tasks excellent!

[GrahamCampbell]

All but tests are done know BTW.

[Craytor]

Thanks a ton @GrahamCampbell. I'm currently writing up some more content in the readme, and I will have a look at tests tomorrow. Am I able to test the app now since it has been rewritten, or will guzzle still be the death of tests?

[joecohens]

Still missing a response object IMO since all the responses have the same output.

[Craytor]

How do they have the same output? ->currently() and ->daily() (for example) will return different things, unless you're referring to something else.

[joecohens]

@Craytor You can mock the client to test it :). I mean same keys

summary: "No precipitation throughout the week, with temperatures peaking at 70°F on Monday.",
icon: "clear-day",
data: [
{
time: 1423728000,
summary: "Partly cloudy throughout the day.",
icon: "partly-cloudy-day",
sunriseTime: 1423753469,
sunsetTime: 1423791970,
moonPhase: 0.78,
precipIntensity: 0,
precipIntensityMax: 0,
precipProbability: 0,
temperatureMin: 53.2,
temperatureMinTime: 1423749600,
temperatureMax: 66.93,
temperatureMaxTime: 1423785600,
apparentTemperatureMin: 53.2,
apparentTemperatureMinTime: 1423749600,
apparentTemperatureMax: 66.93,
apparentTemperatureMaxTime: 1423785600,
dewPoint: 51.14,
humidity: 0.76,
windSpeed: 3.78,
windBearing: 39,
visibility: 8.33,
cloudCover: 0.44,
pressure: 1021.24,
ozone: 275.16
}
]

[Craytor]

So, I would like to finish this up - and add tests. Any suggestions?

/cc @joecohens @GrahamCampbell