Permalink
Browse files

We actually count css selectors, not rules in compass stats

  • Loading branch information...
1 parent b4d6550 commit 9888efdbc22f67c9ed6d54cd09a10a7c7d094f73 @chriseppstein chriseppstein committed Nov 4, 2011
Showing with 22 additions and 22 deletions.
  1. +15 −15 features/command_line.feature
  2. +7 −7 lib/compass/commands/project_stats.rb
@@ -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
@@ -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] == '_'

0 comments on commit 9888efd

Please sign in to comment.