created two new User instance methods: has_medication? and add_medication to clean up code in medications_controller#create. Wrote specs to match.
add_medication uses has_medication?as a check before creating theUserMedication` association to avoid duplicates.
created medications#get_symptoms to combine code from SearchResultsFacade and Medications controller into #save_symptoms to lighten up controller.
in medications_controller#create, now using find_or_create_by to make sure we aren't saving duplicates in the medication list.
added sad path test to features/dashboard/medications_spec.rb to check that duplicates aren't added to the med list.
when user tries to add a duplicate medication, they are redirected to medications/new page with a message saying "Sorry, you've already added to your list!"
gabichuelas
commented
4 years ago
Description
Userinstance methods:has_medication?andadd_medicationto clean up code inmedications_controller#create. Wrote specs to match.add_medicationuses has_medication?as a check before creating theUserMedication` association to avoid duplicates.medications#get_symptomsto combine code from SearchResultsFacade and Medications controller into#save_symptomsto lighten up controller.medications_controller#create, now usingfind_or_create_byto make sure we aren't saving duplicates in the medication list.features/dashboard/medications_spec.rbto check that duplicates aren't added to the med list.medications/newpage with a message saying "Sorry, you've already addedCloses issue(s)
Issue #38