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] bots.go handlers GetBotByUniqueName, GetBotsByOwner, & SearchBots #1566

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

Comments

@elraphty
Copy link
Contributor

elraphty commented Feb 27, 2024

Context

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

File to to test

handlers/bots.go

Design

Write tests for the bots handlers, the test should:

  • GetBotByUniqueName: Should test that a bot can be gotten by its unique name bot//{name}
  • GetBotsByOwner: Should test that all bots by an owner can be listed
  • GetBotsByOwner: Should test that it returns an empty list is returned when a user has no bots
  • SearchBots: Should test that a bot returns data when the search query matches some bots
  • SearchBots: Should test that a bot returns no data when the search query does not match any bot
  • DeleteBot: This should test that a bot can be deleted by the creator of the bot.

Bots Handlers To Test

DeleteBot
GetBotByUniqueName
GetBotsByOwne
SearchBots

Add tests to bots_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 Feb 27, 2024
@elraphty elraphty added this to the UCI Tests milestone Feb 27, 2024
@Jagmohan2k
Copy link

@ecurrencyhodler can I take this?

@AbdulWahab3181
Copy link
Contributor

@ecurrencyhodler could you please assign me?

@MirzaHanan
Copy link
Contributor

@ecurrencyhodler,
Please assign me?

@MahtabBukhari
Copy link
Contributor

@ecurrencyhodler I would like to work on this task

@ecurrencyhodler
Copy link
Contributor

Let's get a PR up in 2 days!

@ecurrencyhodler
Copy link
Contributor

Paid 200k sats! https://community.sphinx.chat/bounty/1588

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