Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add analyzer specs #13

Merged
merged 8 commits into from
Dec 27, 2013
Merged

Add analyzer specs #13

merged 8 commits into from
Dec 27, 2013

Conversation

dresselm
Copy link
Owner

Issue #5

Summary

  • Decoupled ObjectSpace from Analyzer via Memtf::Analyzer::Memory
  • Added specs for the public Analyzer methods

Specs

Finished in 1.32 seconds
35 examples, 0 failures

Documentation

Files:           9
Modules:         3 (    0 undocumented)
Classes:         5 (    0 undocumented)
Constants:       7 (    0 undocumented)
Methods:        18 (    0 undocumented)
 100.00% documented

@ghost ghost assigned dresselm Dec 27, 2013
dresselm added a commit that referenced this pull request Dec 27, 2013
Issue 5
Milestone v0.0.3

* origin/5_add_analyzer_specs:
  Replaced mock() with double()
  Decoupled ObjectSpace and Analyzer; added specs for Analyzer#analyze
  Added Analyser.analyze specs
  Added Analyzer.analyze_group specs
  Silence stdout in specs
  Added newlines at end of files
  Added pending Analyzer specs
  Added analyzer spec placeholoders
@dresselm dresselm merged commit cd3b4e9 into master Dec 27, 2013
@dresselm dresselm deleted the 5_add_analyzer_specs branch December 27, 2013 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant