Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce Unit Tests #1

Open
wants to merge 28 commits into
base: master
Choose a base branch
from
Open

Conversation

michaelahlers
Copy link

This project should have them, and I'll need them for some work I plan on doing. I'll keep this (issue-1) branch around in case you want to merge, but my active development will go elsewhere. The spec demonstrates the tzDate filter works as expected, but Angular's date filter isn't handling the output correctly. Might help you if you want to tackle it.

@michaelahlers
Copy link
Author

A hack that copies properties over to a native Date object fixes the second test without patching AngularJS at all. Not sure how this will perform, but it may be an effective solution. See commit 8dd2a17.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant