justicenation / slashclock

/clock concept as a new Slack app
GNU General Public License v3.0
0 stars 0 forks source link

Clock in and out yesterday #28

Closed martyychang closed 6 years ago

martyychang commented 6 years ago

This pr resolves #18 by adding support for an optional "yesterday" parameter that would come before the time for the /clock in and /clock out commands.

Also included are overdue Apex tests in SlashclockInCommandTest and SlashclockInCommandTest to cover expected functionality of the two augmented commands. These tests exposed a need to set up mocks for the Slack API, which were implemented using a new SmartHttpCalloutMockSuite framework. In this framework, each external API service gets at least its own mock suite (e.g., SlackApiServiceMockSuite), where each nested class inside represents a single response scenario.