Review Request 116810: Code cleaning (and modularization) of reports_spec.rb

yash ladia yashladia1 at gmail.com
Sat Mar 15 23:49:33 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116810/
-----------------------------------------------------------

(Updated March 15, 2014, 11:49 p.m.)


Review request for KDE Websites and Ben Cooksley.


Changes
-------

fixed the issue pointed out by Ahmed


Repository: reports-kde-org


Description
-------

I have split the reports_spec.rb (530 lines long) file into two files:
  1. news_reports_spec.rb (152 lines)
  2. project_reports_spec.rb (266 lines)
I have used the rspec feature of "shared examples" to reduce code duplication.
Also, I have removed some trailing whitespaces, changed few 'string' to "string" and removed some blank lines.

This patch has virtually no effect (except creating a few more users during testing) on the working of the code. It is just a code cleaning patch.


Diffs (updated)
-----

  spec/requests/news_reports_spec.rb PRE-CREATION 
  spec/requests/project_reports_spec.rb PRE-CREATION 
  spec/requests/reports_spec.rb 9cd9b4f 

Diff: https://git.reviewboard.kde.org/r/116810/diff/


Testing
-------

I have tested by running "bundle exec rspec" in both with and without this patch. The same results (87 examples, 70 failures, 16 pending) are obtained in both.


Thanks,

yash ladia

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kde-www/attachments/20140315/8d0102e2/attachment-0001.html>


More information about the kde-www mailing list