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

[feature] Wrote tests for router function #9

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

Joe-Degs
Copy link

[feature] Made couple of changes to the controller function, so it will be easily testable.
Wrote some tests for the function and it is working well.
check to see that all test are passing and emails are delivering.
Also i commented out the debug email part of the mailingController functions, they were breaking the server.
I also took out body parser because express ships with body parser.

Joe-Degs added 5 commits June 10, 2020 09:00
…mely easy and clear to use.

So you just import the mail_validator.js file, which returns an object with two things, a validator function and a schema table.
It is extremely easy to use and structured in a way you dont have to think about the implementation to use it.
The validator is promised based, so if you call it, it resolves a `value` or rejects an `error`.
slackName: Joe.Attah
Team Fierce.
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