Create Plant Type Tests

StevonStevoff / ProjectParsnip

A IoT Project for Managing At-Home Smart Farming

3 stars 0 forks source link

Create Plant Type Tests #47

Closed StevonStevoff closed 1 year ago

StevonStevoff commented 1 year ago

Primary changes:

Unit tests for plant type api routes
Small fixes to plant route due to found bugs

Additional changes:

Removed unused arguments from get_objects/get_all_objects and some other places
Fixed Poetry lock not being called in pre-commit

Note: These tests in particular are heavily dependant on the test_plant_profiles tests unforunately and so this test set in particular will need some changes to become independant. I am suggesting holding off this work until after we have synced with dev branch.

sc19sjjd commented 1 year ago

Note: These tests in particular are heavily dependant on the test_plant_profiles tests unforunately and so this test set in particular will need some changes to become independant. I am suggesting holding off this work until after we have synced with dev branch.

Sounds good, don't think we have too much more tests to write so I agree it will probably be easier to just get those finished first. Also think it won't be too complicated to make them independent.

StevonStevoff commented 1 year ago

Sounds good, don't think we have too much more tests to write so I agree it will probably be easier to just get those finished first. Also think it won't be too complicated to make them independent.

Yeah agreed, it will mainly just be changing the content/IDs of accessed objects once we create them for those specific tests. Just want to do it all at once so we don't have a cascade of failures.