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

Backend [Integration Test] metrics.go handlers MetricsBountiesProviders #1592

Closed
2 tasks
elraphty opened this issue Mar 6, 2024 · 6 comments
Closed
2 tasks
Assignees
Labels
backend bounties go Pull requests that update Go code Posted testing
Milestone

Comments

@elraphty
Copy link
Contributor

elraphty commented Mar 6, 2024

Context

We are trying to increase the test coverage for our Golang backend.

File to to test

handlers/metrics.go

Design

Write tests for the metrics handlers, the test should:

  • Should test that a 401 error is returned if a user is unauthorized
  • Should test that a 406 error is returned if wrong data is passed
  • Should test that it returns the bounty providers and a 200 status code if there is no error.

Metrics Handlers To Test

MetricsBountiesProviders

Add tests to metrics_test.go

Acceptance Criteria

  • I have rebased and tested locally before submitting my PR
  • I can submit a PR within 2 days of taking the bounty

References

Here is an example backend test

@elraphty elraphty added backend bounties go Pull requests that update Go code testing labels Mar 6, 2024
@elraphty elraphty added this to the UCI Tests milestone Mar 6, 2024
@MahtabBukhari
Copy link
Contributor

MahtabBukhari commented Mar 6, 2024

@ecurrencyhodler could you assign me?

@aliraza556
Copy link
Contributor

@elraphty, @ecurrencyhodler
assign me?

@saithsab877
Copy link
Contributor

Hi @ecurrencyhodler,
assign me?

@AbdulWahab3181
Copy link
Contributor

@ecurrencyhodler Could you please assign me?

@ecurrencyhodler ecurrencyhodler moved this from 🆕 New to 🧨 To Do in Bounties Platform Mar 6, 2024
@ecurrencyhodler
Copy link
Contributor

@saithsab877 it's yours.

@ecurrencyhodler
Copy link
Contributor

Paid!

@github-project-automation github-project-automation bot moved this from 🧨 To Do to ✅ Done in Bounties Platform Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend bounties go Pull requests that update Go code Posted testing
Projects
Status: ✅ Done
Development

No branches or pull requests

6 participants