acidhelm / challonge_mgr / 94

Builds Branch Commit Type Ran Committer Via Coverage
94 master Whitespace and style fixups push acidhelm travis-ci-com pending completion   set done
93 master Add a test that updates the scores and winner of a match push acidhelm travis-ci-com pending completion   set done
92 use-webmock Add a test that updates the scores and winner of a match push acidhelm travis-ci-com pending completion   set done
91 use-webmock Add a test that updates the score of a match and simulates an API failure push acidhelm travis-ci-com pending completion   set done
90 use-webmock Add a test for another code path in TournamentsController#refresh_all This test is for the case where tournaments were deleted from Challonge since the last refresh. That code path also runs when a tournament has been finalized since the last ref... push acidhelm travis-ci-com pending completion   set done
89 use-webmock Add Match tests for the team_prereq_match_id and is_prereq_match_loser fields push acidhelm travis-ci-com pending completion   set done
88 use-webmock Write UsersController tests that test creating a demo tournament push acidhelm travis-ci-com pending completion   set done
87 use-webmock Add tests that exercise finalizing a tournament Use WebMock to fake responses from Challonge APIs. I can now write tests that finalize a tournament, and test our response to an API failure. I also changed the owner of `tournament_3` to `user_wil... push acidhelm travis-ci-com pending completion   set done
86 master Add a fixture that's a completed tournament, and associated tests This fixture lets tests exercise a couple more lines in contollers. I'll be adding more tests that use this tournament later. push acidhelm travis-ci-com pending completion   set done
85 master Simplify get_api_url call sites All API URLs start with "tournaments", so don't require the caller to pass that component in its string. push acidhelm travis-ci-com pending completion   set done
84 move-calls-to-models Simplify get_api_url call sites All API URLs start with "tournaments", so don't require the caller to pass that component in its string. push acidhelm travis-ci-com pending completion   set done
83 move-calls-to-models Move some of the code that calls Challonge APIs to ChallongeHelper Continuing the effort to make the code organized better, I created `ChallongeHelper`. Classes that call Challonge can now do `include ChallongeHelper` and call its methods directl... push acidhelm travis-ci-com pending completion   set done
82 move-calls-to-models Add Match#update_scores This replaces direct calls to `ApplicationHelper.update_match`. And I was even able to hide the complexity of making the scores_csv string to pass to the API. That let me make `Match#make_scores_csv` protected. push acidhelm travis-ci-com pending completion   set done
81 master Use api_failed? in UsersController#demo Oopth I forgot about that function. push acidhelm travis-ci-com pending completion   set done
80 master Remove an explicit "attribute" line It was required by older versions of attr_encrypted, but it isn't necessary anymore. push acidhelm travis-ci-com pending completion   set done