API tests

[LiorRabin]

Why

In order to make the development process faster and more reliable by making sure API is not broken when developing new features and/or squashing some bugs.

What

The README provides some basic data how to start with the Inventory Manager.

The GETTING-STARTED provider some basic info on how to use the Inventory Manager as well as the API this app provides.

The API should be tested 😊

Any testing framework is welcomed as long as you can provide an explanation on why it was chosen.

Remarks

• It will be nice to be able to run API tests separately from the current tests in the app and/or along with them.

[gitcoinbot]

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

---

This issue now has a funding of 13000.0 CLN (189.54 USD @ $0.01/CLN) attached to it.

• If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
• Want to chip in? Add your own contribution here.
• Questions? Checkout Gitcoin Help or the Gitcoin Slack
• $40,566.93 more funded OSS Work available on the Gitcoin Issue Explorer

[gitcoinbot]

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

---

Work has been started.

These users each claimed they can complete the work by 4 weeks, 1 day from now.
Please review their action plans below:

1) squanto has been approved to start work.

I will use the mocha test framework, as it's already a truffle requirement.
I will use the chai assertion library for unit and integration testing of controller methods as included already.
I'll use supertest for easily-integrated and terse http acceptance tests.
In order to just run api tests, i'd create separate npm tasks for all and api tests.
I'd also create isolated mongo + ganache test instances for runs that reset on tear down.

Learn more on the Gitcoin Issue Details page.

[LiorRabin]

@squanto - thanks, you're approved as you were the first to offer :)
@preciousgeorge - thank you too for the interest, see you next time :)

[squanto]

Hey @LiorRabin , I will not be able to complete the bounty at this time and would like to free it so others can work on it. Thanks 👍.

[LiorRabin]

Hi @preciousgeorge, still interested? Thanks

[preciousgeorge]

Hi @LiorRabin , yes, i would not mind...

[LiorRabin]

Cool @preciousgeorge, so can you apply to work and I will approve you on gitcoin?

[gitcoinbot]

Issue Status: 1. Open 2. Cancelled

---

Work has been started.

These users each claimed they can complete the work by 3 years, 2 months ago.
Please review their action plans below:

1) preciousgeorge has been approved to start work.

I Would be using SuperTest because it helps in high-level abstraction for testing HTTP and lower level api. For testing functions, i would employ mocha since it offer more flexibility.

Learn more on the Gitcoin Issue Details page.

[LiorRabin]

@preciousgeorge hello, how's it going? have you had any progress?

[preciousgeorge]

@LiorRabin Hi, still familiarizing myself with the codebase, was perusing it online but i have cloned now and would reach with questions soon then i can start work fully, sorry this took some time.

[LiorRabin]

@preciousgeorge that's ok :) just wanted to make sure you're still here...anyway, i've just made some updates to the repo so please make sure you're up to date. Thanks

[preciousgeorge]

@LiorRabin of course i am still here, no problems would make sure i update...

[gitcoinbot]

@preciousgeorge Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

• reminder (3 days)
• escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

[gitcoinbot]

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

---

@preciousgeorge due to inactivity, we have escalated this issue to Gitcoin's moderation team. Let us know if you believe this has been done in error!

• reminder (3 days)
• escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

[rmshea]

hey @preciousgeorge, what's the update on this one? Ryan from Gitcoin checking in here 😄