Skip to content
This repository has been archived by the owner on Nov 7, 2021. It is now read-only.

add command - test automatic date detection #6

Closed
matt-allan opened this issue May 6, 2019 · 0 comments · Fixed by #32
Closed

add command - test automatic date detection #6

matt-allan opened this issue May 6, 2019 · 0 comments · Fixed by #32
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@matt-allan
Copy link
Owner

The add command will default to today if you don't specify a date. For example:

$ php hours add blog --from '11:00 am' --to '1:00 pm'
Added frame for blog from May 6, 2019 11:00 am to May 6, 2019 1:00 pm (2 hours).

This is a great feature but we don't currently have a test that guarantees this continues to work.

It would be helpful if we had a feature test that tested the aforementioned scenario.

To write the test you can add a new method to the AddCommandTest.
@matt-allan matt-allan added enhancement New feature or request good first issue Good for newcomers labels May 6, 2019
Douglasdc3 added a commit to Douglasdc3/hours that referenced this issue Oct 5, 2019
@Douglasdc3
Add automatic date detection test
f209fa2 
Resolves matt-allan#6
@Douglasdc3 Douglasdc3 mentioned this issue Oct 5, 2019
Merged
matt-allan pushed a commit that referenced this issue Oct 8, 2019
@Douglasdc3 @matt-allan
Add automatic date detection test
dfc6294 
Resolves #6
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add automatic date detection test Douglasdc3/hours
1 participant
@matt-allan