OpenAGI icon indicating copy to clipboard operation
OpenAGI copied to clipboard

[Feature] Fix tools and add tests

Open zzfoo opened this issue 1 year ago • 6 comments

It seems that the tools haven't been thoroughly tested; some tools just don't work. So I'm planning to check every tool, add tests, and make sure they all work as expected.

Tools

  • [ ] Arxiv
  • [ ] BingSearch
  • [ ] CurrencyConverterAPI
  • [ ] GooglePlacesAPI
  • [ ] GoogleSearch
  • [ ] MoonPhaseSearch
  • [ ] Wikipedia
  • [ ] WolframAlpha
  • [ ] WordsAPI
  • [x] HotelSearchAPI, discord id: FlynnFlag
  • [x] TopMovieAPI
  • [x] TopSeriesAPI
  • [ ] WeatherAPI
  • [ ] SongAutocompleteAPI
  • [x] AirportSearch, discord id: zhonghuali
  • [ ] HotelLocationSearch
  • [ ] RestaurantLocationSearch

zzfoo avatar May 14 '24 02:05 zzfoo

Hi @zzfoo, I am thinking of assigning these unit tests to a new contributor as an onboarding task. Let's use this issue as a mega-thread for this. Thanks for creating it!

TataKKKL avatar May 14 '24 03:05 TataKKKL

I will proceed with adding the unit tests for the "HotelSearchAPI"

FlynnFlag avatar May 14 '24 04:05 FlynnFlag

I will take the airport search test cases.

Firy01 avatar May 14 '24 19:05 Firy01

I would like to write TopSeriesAPI unit tests.

PetarSPal avatar May 14 '24 19:05 PetarSPal

✅ I will take the Top Movie API Unit Test!

Cyberhoney avatar May 15 '24 01:05 Cyberhoney

I just recently made a pull request that changes a couple of the tools because they weren't working as intended as you all know. Might want to check that out

itsthemoon avatar May 15 '24 19:05 itsthemoon