bmchristensen/tier_list

CS362 Project: League of Legends Tier List

tier_list's Issues

Every champion in a list has a unique role

Users do not pick a tier for a champion. Users pick a champion and champions know what their tier is.

Clean up list.rb syntax

Each Champion in a List should have a different Role

A List has a max of 5 Champions, one of each type (role).

Redundant syntax in list.rb

Use of explicit returns and redundant 'self' declarations. Could also improve readability with Ruby style guard clauses.

Refactor syntax and conventions

Inconsistent formatting.

Workflow Test Failing

Failing at line 37: Champion string is not parsing correctly.

System tests failing after custom validator enforces list size

Following the implementation of a customer Application Record validator to enforce the size of a champion list, the system tests began to fail. The method cannot find a valid 'id.'

Refactor List Unit Tests

Refactor unit tests for List so that they use less factories

View to support editing a list

Delete list and replace a champion in a list should be supported by view.

Ability to edit a list

A user needs to be able to delete any list that they have previously created.

New View for List Creation

Need to change the view so that each champion is entered in its own box instead of in one big describe block that is then parsed. Maximum of 5 champions and needs exactly 5 champions per list, so we need 5 boxes.

Add ability to remove an existing list

A user should be able to not only edit a list, but completely remove a list of champions.