Commit 74b18ce330c07deadaa810cf1dab820a5b47d361
Exists in
colab
and in
4 other branches
Merge pull request #155 from beatrizrezener/issue43
Removing useless code from Processing #43
Showing
2 changed files
with
0 additions
and
39 deletions
Show diff stats
app/models/processing.rb
| 1 | class Processing < KalibroGatekeeperClient::Entities::Processing | 1 | class Processing < KalibroGatekeeperClient::Entities::Processing |
| 2 | include KalibroRecord | 2 | include KalibroRecord |
| 3 | - | ||
| 4 | - def ready? | ||
| 5 | - @state == "READY" | ||
| 6 | - end | ||
| 7 | - | ||
| 8 | - def root_module_result | ||
| 9 | - ModuleResult.find(@results_root_id) | ||
| 10 | - end | ||
| 11 | end | 3 | end |
spec/models/processing_spec.rb
| @@ -1,31 +0,0 @@ | @@ -1,31 +0,0 @@ | ||
| 1 | -require 'rails_helper' | ||
| 2 | - | ||
| 3 | -describe Processing, :type => :model do | ||
| 4 | - describe 'methods' do | ||
| 5 | - subject { FactoryGirl.build(:processing) } | ||
| 6 | - | ||
| 7 | - describe 'ready?' do | ||
| 8 | - context 'with a READY processing' do | ||
| 9 | - it 'should return true' do | ||
| 10 | - expect(subject.ready?).to be_truthy | ||
| 11 | - end | ||
| 12 | - end | ||
| 13 | - | ||
| 14 | - context 'without a READY processing' do | ||
| 15 | - subject { FactoryGirl.build(:processing, state: 'COLLECTING') } | ||
| 16 | - | ||
| 17 | - it 'should return false' do | ||
| 18 | - expect(subject.ready?).to be_falsey | ||
| 19 | - end | ||
| 20 | - end | ||
| 21 | - end | ||
| 22 | - | ||
| 23 | - describe 'root_module_result' do | ||
| 24 | - it 'should call the root_module_result method' do | ||
| 25 | - ModuleResult.expects(:find).with(subject.results_root_id).returns(FactoryGirl.build(:module_result)) | ||
| 26 | - | ||
| 27 | - subject.root_module_result | ||
| 28 | - end | ||
| 29 | - end | ||
| 30 | - end | ||
| 31 | -end | ||
| 32 | \ No newline at end of file | 0 | \ No newline at end of file |