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

Test fixes #7

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Test fixes #7

wants to merge 11 commits into from

Conversation

moconnell
Copy link

No description provided.

moconnell and others added 9 commits April 3, 2022 22:38
adding CI build
update triggers, try wildcard "**"
- add `npm ci`
- drop support for node 12, 14
merging GitHub Actions CI
`npx hardhat coverage` fails, removing for now
@moconnell moconnell mentioned this pull request Apr 4, 2022
3 tasks
@heueristik
Copy link
Owner

Hey, thanks for making a PR @moconnell.
Did you notice and take care that some tests were intentionally placed on the end of the year (the 31st of December) to account for edge cases?
Using the current date might render some tests useless, as the edge cases don't occur. Could you please check this?

@moconnell
Copy link
Author

Hey @Michael-A-Heuer - the problem was that hard-coding the date in most of the tests to Dec 31st of the current year caused many of them to now fail due to the varying nature of which day of the week that turns out to be - in 2022, Dec 31 is a Saturday, which collides with how you setup the calendars!

What is the specific edge case you were trying to test?

@heueristik
Copy link
Owner

Oh, I see the problem. If I remember correctly, it tests meeting times that span from one year to the other (spanning two different months/years).

@moconnell
Copy link
Author

Ok, I can add a test for that. Out of interest, was it that you actually saw problems with that case, or were you just being thorough? :)

@moconnell
Copy link
Author

Ok, I basically reverted the two tests you are talking about. Turns out they weren't failing anyway, so good catch - thanks!

@moconnell
Copy link
Author

Btw, are you active on the Raid Guild Discord still? Wondering where is best to continue discussions, if you have time/are still interested. I am part of the Season 4 cohort and have picked up this project as a first task.

Seems like there is not much left to do apart from actually deploy it and get the front-end talking to the back-end - correct?

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.

2 participants