Skip to content

Commit

Permalink
We actually count css selectors, not rules in compass stats
Browse files Browse the repository at this point in the history
  • Loading branch information
@chriseppstein
chriseppstein committed Nov 7, 2011
1 parent b4d6550 commit 9888efd
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 22 deletions.
30 changes: 15 additions & 15 deletions features/command_line.feature
View file
Expand Up @@ -277,21 +277,21 @@ Feature: Command Line
Given I am using the existing project in test/fixtures/stylesheets/compass
When I run: compass stats
Then I am told statistics for each file:
| Filename | Rules | Properties | Mixins Defs | Mixins Used | CSS Rules | CSS Properties |
| sass/border_radius.scss | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/box.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/fonts.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/gradients.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/image_size.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/images.scss | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/layout.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/legacy_clearfix.scss | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/lists.scss | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/print.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/reset.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/utilities.scss | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| ------------------------- | ----- | ---------- | -------------- | ----------- | --------- | -------------- |
| Total.* | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| Filename | Rules | Properties | Mixins Defs | Mixins Used | CSS Selectors | CSS Properties |
| sass/border_radius.scss | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/box.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/fonts.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/gradients.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/image_size.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/images.scss | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/layout.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/legacy_clearfix.scss | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/lists.scss | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/print.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/reset.sass | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| sass/utilities.scss | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |
| ------------------------- | ----- | ---------- | -------------- | ----------- | ------------- | -------------- |
| Total.* | \d+ | \d+ | \d+ | \d+ | \d+ | \d+ |

@listframeworks
Scenario: List frameworks registered with compass
Expand Down
14 changes: 7 additions & 7 deletions lib/compass/commands/project_stats.rb
View file
Expand Up @@ -34,13 +34,13 @@ def perform
compiler = new_compiler_instance
sass_files = sorted_sass_files(compiler)
total_label = "Total (#{sass_files.size} files):"
rows = [[ :-, :-, :-, :-, :-, :-, :- ],
[ 'Filename', 'Rules', 'Properties', 'Mixins Defs', 'Mixins Used', 'CSS Rules', 'CSS Properties' ],
[ :-, :-, :-, :-, :-, :-, :- ]]
maximums = [ total_label.length, 5, 10, 14, 11, 9, 14 ]
alignments = [ :left, :right, :right, :right, :right, :right, :right ]
delimiters = [ ['| ', ' |'], [' ', ' |'], [' ', ' |'], [' ', ' |'], [' ', ' |'], [' ', ' |'], [' ', ' |'] ]
totals = [ total_label, 0, 0, 0, 0, 0, 0 ]
rows = [[ :-, :-, :-, :-, :-, :-, :- ],
[ 'Filename', 'Rules', 'Properties', 'Mixins Defs', 'Mixins Used', 'CSS Selectors', 'CSS Properties' ],
[ :-, :-, :-, :-, :-, :-, :- ]]
maximums = [ total_label.length, 5, 10, 14, 11, 13, 14 ]
alignments = [ :left, :right, :right, :right, :right, :right, :right ]
delimiters = [ ['| ', ' |'], [' ', ' |'], [' ', ' |'], [' ', ' |'], [' ', ' |'], [' ', ' |'], [' ', ' |'] ]
totals = [ total_label, 0, 0, 0, 0, 0, 0 ]

sass_files.each do |sass_file|
css_file = compiler.corresponding_css_file(sass_file) unless sass_file[0..0] == '_'
Expand Down

0 comments on commit 9888efd

Please sign in to comment.