Permalink
Commits on Oct 10, 2015
  1. Refactor `Yardstick::RuleConfig` to use concord

    backus committed with backus Oct 4, 2015
  2. Refactor `Yardstick::Document` to use concord

    backus committed with backus Oct 4, 2015
  3. Add dependency on `concord` (~> 0.1.x)

    backus committed with backus Oct 4, 2015
Commits on Oct 7, 2015
  1. Merge pull request #39 from abinoam/syntax_highlight_readme

    Add language identifiers to code blocks on README.md for syntax highl…
    backus committed Oct 7, 2015
Commits on Sep 29, 2015
  1. Merge pull request #31 from dkubb/fix/master/require-tempfile

    Add explicit require for Tempfile to fix failing travis build
    backus committed Sep 29, 2015
  2. Add explicit require for Tempfile to fix failing travis build

    committed Sep 29, 2015
  3. Merge pull request #28 from backus/feature/kill-new-mutations

    Improve mutation coverage
    committed Sep 29, 2015
  4. Fix race condition causing neutral mutant failure

    I think there was a race condition under mutant when
    `Yardstick::Rake::Measurement#define` is being mutated. The race
    condition worked like so:
    
    **Thread 1** writes to measurements/report.txt
    **Thread 2** deletes to measurements/report.txt
    **Thread 1** asserts the value of report.txt
    
    The spec in question now writes to `tmp/measurements-$pid/report.txt`
    which eliminates the issue
    backus committed Sep 28, 2015
  5. Fix maximum line length rule

    backus committed Sep 28, 2015
  6. Remove `ReportOutput#initialize` spec

    Increases mutation coverage for `ReportOutput` as a whole
    backus committed Sep 28, 2015
  7. Kill `Parser.method_objects` filtering mutation

    All method objects which have a `file` attribute should have
    a non-nil `line` attribute
    backus committed Sep 28, 2015
  8. Kill `Parser.parse_paths` mutation

    - `YARD::Logger::ERROR` is equal to `Logger::ERROR`
    - `YARD::Logger` is autoloaded so, if `YARD::Logger` has not been
      previously referenced, the logger library will not be required and
      tests will fail
    - I tried using `hide_const` but `hide_const` continues to hide
      the `Logger` constant *after* it has been required by YARD
    - I tried `Object.send(:remove_const, 'Logger')` with a conditional
      but this causes an order dependent issue because the second call
      to `require 'logger'` returns false
    - I ended up requiring `yard/logging` to force `Logger` and
      `YARD::Logger` to load
    backus committed Sep 27, 2015
  9. Kill `OrderedSet#initialize` mutation

    `merge` in `#initialize` can default to `self` because this
    results in the same noop iteration within `#merge`
    backus committed Sep 27, 2015
  10. Kill `OrderedSet#<<` mutations

    backus committed Sep 27, 2015
  11. Change RSpec example wording to not use 'should'

    Good RSpec style usually discourages using the word 'should' in
    an example's wording
    backus committed Sep 27, 2015
  12. Change devtools timeout from `0.1` to `1.0`

    Occasionally I get a mutant failure because a test runs longer than 0.1
    seconds so I'm bumping up the timeout to 1 second
    backus committed Sep 27, 2015
  13. Remove `RuleConfig#initialize` spec

    Increases mutation coverage of `RuleConfig` as a whole
    backus committed Sep 27, 2015
  14. Kill `MeasurementSet#puts` mutation

    The `#puts` method was not specifying the percentage rounding behavior
    so I added an additional measurement mock to the spec to give it a
    success/fail ratio that would test the rounding.
    backus committed Sep 27, 2015
  15. Remove `Measurement#initialize` spec

    This specific spec file does not specify much by itself.  Mutant only
    selects the tests in this file since its description is
    `Measurement#initialize`.  Removing this spec killed 16 mutations
    backus committed Sep 27, 2015