| @@ -22,7 +22,7 class ReportsController < ApplicationController | |||||
| 22 | def issue_report |
|
22 | def issue_report | |
| 23 | @trackers = @project.trackers |
|
23 | @trackers = @project.trackers | |
| 24 | @versions = @project.shared_versions.sort |
|
24 | @versions = @project.shared_versions.sort | |
| 25 | @priorities = IssuePriority.all |
|
25 | @priorities = IssuePriority.all.reverse | |
| 26 | @categories = @project.issue_categories |
|
26 | @categories = @project.issue_categories | |
| 27 | @assignees = (Setting.issue_group_assignment? ? @project.principals : @project.users).sort |
|
27 | @assignees = (Setting.issue_group_assignment? ? @project.principals : @project.users).sort | |
| 28 | @authors = @project.users.sort |
|
28 | @authors = @project.users.sort | |
| @@ -53,7 +53,7 class ReportsController < ApplicationController | |||||
| 53 | @report_title = l(:field_version) |
|
53 | @report_title = l(:field_version) | |
| 54 | when "priority" |
|
54 | when "priority" | |
| 55 | @field = "priority_id" |
|
55 | @field = "priority_id" | |
| 56 | @rows = IssuePriority.all |
|
56 | @rows = IssuePriority.all.reverse | |
| 57 | @data = Issue.by_priority(@project) |
|
57 | @data = Issue.by_priority(@project) | |
| 58 | @report_title = l(:field_priority) |
|
58 | @report_title = l(:field_priority) | |
| 59 | when "category" |
|
59 | when "category" | |
| @@ -38,9 +38,11 class ReportsControllerTest < ActionController::TestCase | |||||
| 38 | assert_template 'issue_report' |
|
38 | assert_template 'issue_report' | |
| 39 |
|
39 | |||
| 40 | [:issues_by_tracker, :issues_by_version, :issues_by_category, :issues_by_assigned_to, |
|
40 | [:issues_by_tracker, :issues_by_version, :issues_by_category, :issues_by_assigned_to, | |
| 41 | :issues_by_author, :issues_by_subproject].each do |ivar| |
|
41 | :issues_by_author, :issues_by_subproject, :issues_by_priority].each do |ivar| | |
| 42 | assert_not_nil assigns(ivar) |
|
42 | assert_not_nil assigns(ivar) | |
| 43 | end |
|
43 | end | |
|
|
44 | ||||
|
|
45 | assert_equal IssuePriority.all.reverse, assigns(:priorities) | |||
| 44 | end |
|
46 | end | |
| 45 |
|
47 | |||
| 46 | def test_get_issue_report_details |
|
48 | def test_get_issue_report_details | |
| @@ -56,6 +58,11 class ReportsControllerTest < ActionController::TestCase | |||||
| 56 | end |
|
58 | end | |
| 57 | end |
|
59 | end | |
| 58 |
|
60 | |||
|
|
61 | def test_get_issue_report_details_by_priority | |||
|
|
62 | get :issue_report_details, :id => 1, :detail => 'priority' | |||
|
|
63 | assert_equal IssuePriority.all.reverse, assigns(:rows) | |||
|
|
64 | end | |||
|
|
65 | ||||
| 59 | def test_get_issue_report_details_with_an_invalid_detail |
|
66 | def test_get_issue_report_details_with_an_invalid_detail | |
| 60 | get :issue_report_details, :id => 1, :detail => 'invalid' |
|
67 | get :issue_report_details, :id => 1, :detail => 'invalid' | |
| 61 |
|
68 | |||
General Comments 0
You need to be logged in to leave comments.
Login now