-
Notifications
You must be signed in to change notification settings - Fork 1
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
#-187431151 implementing jest test in sign up #15
Conversation
2068e0f
to
c25b0bb
Compare
8d66432
to
e74adb3
Compare
if (req.body) { | ||
const { error } = userValidate(req.body); | ||
if (error) { | ||
return res |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Identical blocks of code found in 2 locations. Consider refactoring.
); | ||
} | ||
if (error) | ||
return res |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Identical blocks of code found in 2 locations. Consider refactoring.
e74adb3
to
1531571
Compare
); | ||
|
||
export default passport; | ||
import { Request } from "express"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parsing error: 'import' and 'export' may appear only with 'sourceType: module'
@@ -0,0 +1,23 @@ | |||
import { hashPassword } from "../utils/password"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parsing error: 'import' and 'export' may appear only with 'sourceType: module'
@@ -0,0 +1,15 @@ | |||
|
|||
export const deleteTableData = async (Model: any, tableName: string) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parsing error: 'import' and 'export' may appear only with 'sourceType: module'
d5cb000
to
0fcd223
Compare
0fcd223
to
01f270c
Compare
password: "<Password@345>", | ||
}; | ||
|
||
export const login_user = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parsing error: 'import' and 'export' may appear only with 'sourceType: module'
01f270c
to
f617513
Compare
|
||
it("should create a new user", async () => { | ||
// Your test implementation goes here | ||
it("it should register a user and return 201", async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar blocks of code found in 2 locations. Consider refactoring.
Code Climate has analyzed commit f617513 and detected 4 issues on this pull request. Here's the issue category breakdown:
The test coverage on the diff in this pull request is 88.0% (60% is the threshold). This pull request will bring the total coverage in the repository to 87.5% (2.0% change). View more on Code Climate. |
#-187431151 implementing jest test in sign up
What does this PR do?
This pr is responsible for testing sign up end points