Skip to content

guarantor and view loan#20

Open
bosstha wants to merge 1 commit into
masterfrom
guarantor_view_loans
Open

guarantor and view loan#20
bosstha wants to merge 1 commit into
masterfrom
guarantor_view_loans

Conversation

@bosstha

@bosstha bosstha commented Jul 3, 2018

Copy link
Copy Markdown
Collaborator

TDD for guarantor approve and user view single and multiple loans.

@bosstha bosstha requested a review from wilforlan July 3, 2018 13:24

@wilforlan wilforlan left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, make sure that your file is properly indented.

const joi = frisby.joi

it ('POST should return a status of 201 Created', function () {
frisby.globalSetup({

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incorrect Flow, you should be getting the token from the login endpoint. So I can test automatically.

password: "miracle"
})
.expect('status', 201);
.get('http://localhost:3000/loans/approve/,5b3b249b0d216645b40447aa')

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the comma here before the id ?

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