Closed ccooper1 closed 9 years ago
I merged this but your comment says admin and card controllers. dashboard-ctrl and card-ctrl are added. Do you have admin-ctrl.spec.js to add?
----- Original Message ----- From: "ccooper1" notifications@github.com To: "Apereo-Learning-Analytics-Initiative/OpenDashboard" OpenDashboard@noreply.github.com Sent: Friday, October 16, 2015 12:38:24 PM Subject: [OpenDashboard] Add jasmine tests for admin and card controllers (#16)
You can view, comment on, or merge this pull request online at:
https://github.com/Apereo-Learning-Analytics-Initiative/OpenDashboard/pull/16 Commit Summary
* Add jasmine tests for admin and card controllers
File Changes
* M pom.xml (2) * M src/test/resources/js/specs/card-ctrl.spec.js (463) * M src/test/resources/js/specs/dashboard-ctrl.spec.js (196)
Patch Links:
* https://github.com/Apereo-Learning-Analytics-Initiative/OpenDashboard/pull/16.patch * https://github.com/Apereo-Learning-Analytics-Initiative/OpenDashboard/pull/16.diff
— Reply to this email directly or view it on GitHub .
I merged this but your comment says admin and card controllers. dashboard-ctrl and card-ctrl are added. Do you have admin-ctrl.spec.js to add?
----- Original Message ----- From: "ccooper1" notifications@github.com To: "Apereo-Learning-Analytics-Initiative/OpenDashboard" OpenDashboard@noreply.github.com Sent: Friday, October 16, 2015 12:38:24 PM Subject: [OpenDashboard] Add jasmine tests for admin and card controllers (#16)
You can view, comment on, or merge this pull request online at:
https://github.com/Apereo-Learning-Analytics-Initiative/OpenDashboard/pull/16 Commit Summary
File Changes
Patch Links:
— Reply to this email directly or view it on GitHub .