expertiza / expertiza / 11595

37%

main: 47%

push

travis-ci-com

E2115: Mentor Management (#1957)

* Fix Markdown formatting in design_document.md

* Add DUTY_MENTOR constant

* Implement select_mentor and helper methods in new class MentorManagement

* Fix select mentor to return the selected mentor id or nil

Currently, it would return a tuple of [id, team_count] which is a detail callers don't need to know about.

* Replace `first(1)` with `first` so unpacking works as expected

* Add trigger to invitation.rb

* Implement update_mentor_state

* wip mentor added by id

Nice peer programming session guys!

Co-Authored-By: AvlWx2014 <16199909+AvlWx2014@users.noreply.github.com>
Co-Authored-By: Hatsukikeika <43552712+Hatsukikeika@users.noreply.github.com>
Co-Authored-By: Bahati Wanza <bahati2000@users.noreply.github.com>

* mentor views and instructor team adding trigger

another bites the dust!

Co-Authored-By: AvlWx2014 <16199909+AvlWx2014@users.noreply.github.com>
Co-Authored-By: Hatsukikeika <43552712+Hatsukikeika@users.noreply.github.com>

* missed null check

Co-Authored-By: AvlWx2014 <16199909+AvlWx2014@users.noreply.github.com>
Co-Authored-By: Hatsukikeika <43552712+Hatsukikeika@users.noreply.github.com>

* Add auto_assign_mentor column to assignments

* Start adding mentor checkbox to assignment form

* rename file to match class name

* rename the checkbox element

* Match jQuery selector to checkbox id

* auto_assign_mentor flag logic

wip

Co-Authored-By: AvlWx2014 <16199909+AvlWx2014@users.noreply.github.com>
Co-Authored-By: Hatsukikeika <43552712+Hatsukikeika@users.noreply.github.com>

* added notify email, and feature toggle for mentor auto assign

Co-Authored-By: Bahati Wanza <bahati2000@users.noreply.github.com>
Co-Authored-By: AvlWx2014 <16199909+AvlWx2014@users.noreply.github.com>
Co-Authored-By: Hatsukikeika <43552712+Hatsukikeika@users.noreply.github.com>

* added 5 tests for the MentorManagement methods

* Add some more method-level c... (continued)

47 of 47 new or added lines in 6 files covered. (100.0%)

7196 of 13783 relevant lines covered (52.21%)

53.17 hits per line