Open adrianlindell opened 2 years ago
Merging #531 (888351b) into master (eb31fec) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #531 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 3 3
Lines 68 68
Branches 4 4
=========================================
Hits 68 68
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
In this PR we test that instructors can download a student roster properly as CSV and JSON files. Tests file download and compares file headers to hardcoded expected values. All tests in
test/system/courses_test.rb
pass.Suggested testing:
rails test test/system/courses_test.rb
Note: Running all system tests using
rails test:system
is expected to fail due to existing bugs in files that were not edited in this PR. I created issue #528 to address this.Closes #525