kalibro_client 2.0.0 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3a5bcfbb03d6e6c0d2bf2c3b4d671daf6fe2dcc6
-  data.tar.gz: 968a58a0f973981f2909bdb0ef6969a0c19cce24
+  metadata.gz: 8dbf2ac51375c86cb7e1f7cf8a3aab2a28c530b4
+  data.tar.gz: fdb92022806ccb9a35b1e0b3b2124282494e2fac
 SHA512:
-  metadata.gz: 815e31061ec7660983846ffed1052d3d2bcfaee8c373b287757ca2ba192f3943939fc292687738504a964f664a1dda6d5a4534512e808653407ce8fe9989022b
-  data.tar.gz: cec3da37bf2df5bd5926679ddbda0aa7fc76f5c276f309acc46daf732e75d4953b0f3e25408f59ce0c471618a545d40ccee990cf7772c3c4926889959935d754
+  metadata.gz: 3066c33f39ddd5712c30dfa85b08270cc08a18e65646d97ce6c69c5c22cb9bf9106078099c22aa23a4516bb3f150a24d94da1e9c98afe180f80cc21fac0ea216
+  data.tar.gz: b468950f31c0d32770e3198dc4f878e3e337875f05796df9c25c654895073474eb92a27f389a82a73167d7a196aae5d9aeb1735645eba32a2ea4f57819cb5067

data/.travis.yml

@@ -5,11 +5,11 @@ rvm:
 addons:
   postgresql: "9.3"
 
-before_script: 
+before_script:
   # Remove bugged libzmq3 package, see https://github.com/travis-ci/travis-ci/issues/982 and https://github.com/travis-ci/travis-ci/issues/1715 for details
   - sudo apt-get remove libzmq3
-  - git clone https://github.com/mezuro/kalibro_install.git -b v3.8 kalibro_install
-  - KALIBRO_PROCESSOR_VERSION='v1.0.0' KALIBRO_CONFIGURATIONS_VERSION='v1.1.1' ./kalibro_install/install.sh
+  - git clone https://github.com/mezuro/kalibro_install.git -b v4.0 kalibro_install
+  - KALIBRO_PROCESSOR_VERSION='v1.1.1' KALIBRO_CONFIGURATIONS_VERSION='v1.2.0' ./kalibro_install/install.sh
   - cp features/support/kalibro_cucumber_helpers.yml.sample features/support/kalibro_cucumber_helpers.yml
   - export BUNDLE_GEMFILE=$PWD/Gemfile
   - export CODECLIMATE_REPO_TOKEN=46cbb96b053b03ad66b0355bd96d0787f56fc5a4fc171b8d6eb30c421c5e6777

data/features/kalibro_configuration/hotspot_metric_configurations.feature

@@ -0,0 +1,11 @@
+Feature: HotspotMetricConfiguration retrieval
+  In order to be able to list just with the HotspotMetricConfigurations
+  As a developer
+  I want to get all HotspotMetricConfigurations of a given KalibroConfiguration
+
+  @kalibro_configuration_restart
+  Scenario: get a list of all hotspot metric configurations of some kalibro configuration
+    Given I have a kalibro configuration with name "Kalibro for Java"
+    And I have a hotspot metric configuration within the given kalibro configuration
+    When I request for hotspot_metric_configurations of the given kalibro configuration
+    Then I should get a list with the given HotspotMetricConfiguration

data/features/kalibro_configuration/tree_metric_configurations.feature

@@ -0,0 +1,12 @@
+Feature: TreeMetricConfiguration retrieval
+  In order to be able to list just with the TreeMetricConfigurations
+  As a developer
+  I want to get all TreeMetricConfigurations of a given KalibroConfiguration
+
+  @kalibro_configuration_restart
+  Scenario: get a list of all tree metric configurations of some kalibro configuration
+    Given I have a kalibro configuration with name "Kalibro for Java"
+    And I have a reading group with name "Group"
+    And I have a tree metric configuration within the given kalibro configuration
+    When I request for tree_metric_configurations of the given kalibro configuration
+    Then I should get a list with the given TreeMetricConfiguration

data/features/metric_result/module_results.feature

@@ -0,0 +1,20 @@
+Feature: Module Results
+  In order to be able to get the module result associated with this metric result
+  As a developer
+  I want to get the module result of the given metric result
+
+  @kalibro_configuration_restart @kalibro_processor_restart
+  Scenario: when there is a metric result
+    Given I have a project with name "Kalibro"
+    And I have a kalibro configuration with name "Java"
+    And I have a reading group with name "Group"
+    And I have a loc configuration within the given kalibro configuration
+    And the given project has the following Repositories:
+      |   name    | scm_type |                   address                        |
+      |  Kalibro  |    GIT   | https://github.com/rafamanzo/runge-kutta-vtk.git |
+    And I call the process method for the given repository
+    And I wait up for a ready processing
+    And I call the first_processing method for the given repository
+    When I get the first metric result of the given processing
+    And I ask for the module result of the given metric result
+    Then I should get a module result

data/features/step_definitions/metric_configuration_steps.rb

@@ -18,6 +18,17 @@ Given(/^I have a loc configuration within the given kalibro configuration$/) do
                                               kalibro_configuration_id: @kalibro_configuration.id})
 end
 
+Given(/^I have a hotspot metric configuration within the given kalibro configuration$/) do
+  @hotspot_metric_configuration = FactoryGirl.create(:metric_configuration,
+                                                     {metric: FactoryGirl.build(:hotspot_metric),
+                                                      kalibro_configuration_id: @kalibro_configuration.id})
+end
+
+Given(/^I have a tree metric configuration within the given kalibro configuration$/) do
+  step "I have a loc configuration within the given kalibro configuration"
+  @tree_metric_configuration = @metric_configuration
+end
+
 When(/^I search a metric configuration with the same id of the given metric configuration$/) do
   @found_metric_configuration = KalibroClient::Entities::Configurations::MetricConfiguration.find(@metric_configuration.id)
 end
@@ -49,6 +60,14 @@ When(/^I have a flay configuration within the given kalibro configuration$/) do
                                               kalibro_configuration_id: @kalibro_configuration.id})
 end
 
+When(/^I request for hotspot_metric_configurations of the given kalibro configuration$/) do
+  @hotspot_metric_configurations = @kalibro_configuration.hotspot_metric_configurations
+end
+
+When(/^I request for tree_metric_configurations of the given kalibro configuration$/) do
+  @tree_metric_configurations = @kalibro_configuration.tree_metric_configurations
+end
+
 Then(/^it should return the same metric configuration as the given one$/) do
   expect(@found_metric_configuration).to eq(@metric_configuration)
 end
@@ -73,3 +92,11 @@ end
 Then(/^its metric should be Hotspot one$/) do
   expect(@found_metric_configuration.metric).to be_a(KalibroClient::Entities::Miscellaneous::HotspotMetric)
 end
+
+Then(/^I should get a list with the given HotspotMetricConfiguration$/) do
+  expect(@hotspot_metric_configurations).to include(@hotspot_metric_configuration)
+end
+
+Then(/^I should get a list with the given TreeMetricConfiguration$/) do
+  expect(@tree_metric_configurations).to include(@tree_metric_configuration)
+end

data/features/step_definitions/metric_result_steps.rb

@@ -13,6 +13,15 @@ When(/^I call the history of method with the metric name and the results root id
   @response = KalibroClient::Entities::Processor::TreeMetricResult.history_of(@metric.name, @response.root_module_result_id, @repository.id)
 end
 
+When(/^I get the first metric result of the given processing$/) do
+  @metric_result = KalibroClient::Entities::Processor::ModuleResult.find(@response.root_module_result_id)
+                   .tree_metric_results.first
+end
+
+When(/^I ask for the module result of the given metric result$/) do
+  @module_result = @metric_result.module_result
+end
+
 Then (/^I should get a Float list$/) do
   expect(@response).to be_a(Array)
   expect(@response.first).to be_a(Float)
@@ -31,3 +40,7 @@ end
 Then(/^the first metric result should have a metric configuration$/) do
   expect(@response.first.metric_configuration).to be_a(KalibroClient::Entities::Configurations::MetricConfiguration)
 end
+
+Then(/^I should get a module result$/) do
+  expect(@module_result).to be_a(KalibroClient::Entities::Processor::ModuleResult)
+end

data/lib/kalibro_client/entities/configurations/kalibro_configuration.rb

@@ -29,7 +29,19 @@ module KalibroClient
         end
 
         def metric_configurations
-         KalibroClient::Entities::Configurations::MetricConfiguration.create_objects_array_from_hash(self.class.request(':id/metric_configurations', {id: id}, :get))
+          KalibroClient::Entities::Configurations::MetricConfiguration.create_objects_array_from_hash(self.class.request(':id/metric_configurations', {id: id}, :get))
+        end
+
+        def hotspot_metric_configurations
+          hotspot_metric_configurations_hash = self.class.request(':id/hotspot_metric_configurations', {id: id}, :get)
+          KalibroClient::Entities::Configurations::MetricConfiguration.create_objects_array_from_hash(
+            {'metric_configurations' => hotspot_metric_configurations_hash['hotspot_metric_configurations']})
+        end
+
+        def tree_metric_configurations
+          tree_metric_configurations_hash = self.class.request(':id/tree_metric_configurations', {id: id}, :get)
+          KalibroClient::Entities::Configurations::MetricConfiguration.create_objects_array_from_hash(
+            {'metric_configurations' => tree_metric_configurations_hash['tree_metric_configurations']})
         end
       end
     end

data/lib/kalibro_client/entities/processor/kalibro_module.rb

@@ -5,7 +5,7 @@
 # it under the terms of the GNU General Public License as published by
 # the Free Software Foundation, either version 3 of the License, or
 # (at your option) any later version.
-# 
+#
 # This program is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ -18,9 +18,7 @@ module KalibroClient
   module Entities
     module Processor
       class KalibroModule < KalibroClient::Entities::Processor::Base
-
-        attr_accessor :granlrty, :id, :long_name, :module_result_id
-        alias_method :granularity, :granlrty
+        attr_accessor :granularity, :id, :long_name, :module_result_id
 
         def name=(value)
           @long_name = (value.is_a?(Array) ? value.join('.') : value)

data/lib/kalibro_client/entities/processor/metric_result.rb

@@ -18,10 +18,9 @@ module KalibroClient
   module Entities
     module Processor
       class MetricResult < KalibroClient::Entities::Processor::Base
-
         # TODO: related_hotspot_metric_results_id should be enclosed to Processor which does not return this id into the response
         #       there should be a controller that returns all the HotspotMetricResults associated through RelatedHotspotMetricResult
-        attr_accessor :id, :value, :module_result_id, :metric_configuration_id,
+        attr_accessor :id, :value, :metric_configuration_id, :module_result_id,
                       :line_number, :message, :related_hotspot_metric_results_id
         attr_reader :metric_configuration
 
@@ -50,6 +49,13 @@ module KalibroClient
         def value=(value)
           @value = value.to_f
         end
+
+        def module_result
+          return @module_result unless @module_result.nil? || @module_result_id != @module_result.id
+          module_result_hash = KalibroClient::Entities::Processor::MetricResult
+                               .request(':id/module_result', { id: @module_result_id }, :get)['module_result']
+          @module_result = KalibroClient::Entities::Processor::ModuleResult.new module_result_hash
+        end
       end
     end
   end

data/lib/kalibro_client/version.rb

@@ -15,5 +15,5 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 module KalibroClient
-  VERSION = "2.0.0"
+  VERSION = "2.1.0"
 end

data/spec/entities/configurations/kalibro_configuration_spec.rb

@@ -33,7 +33,7 @@ describe KalibroClient::Entities::Configurations::KalibroConfiguration do
           returns({'kalibro_configurations' => []})
       end
 
-      it 'should return nil' do
+      it 'is expected to return nil' do
         expect(KalibroClient::Entities::Configurations::KalibroConfiguration.all).to be_empty
       end
     end
@@ -49,7 +49,7 @@ describe KalibroClient::Entities::Configurations::KalibroConfiguration do
           returns({'kalibro_configurations' => [kalibro_configuration.to_hash, another_kalibro_configuration.to_hash]})
       end
 
-      it 'should return the two elements' do
+      it 'is expected to return the two elements' do
         kalibro_configurations = KalibroClient::Entities::Configurations::KalibroConfiguration.all
 
         expect(kalibro_configurations.size).to eq(2)
@@ -68,7 +68,7 @@ describe KalibroClient::Entities::Configurations::KalibroConfiguration do
           returns({'metric_configurations' => []})
       end
 
-      it 'should return an empty array' do
+      it 'is expected to return an empty array' do
         expect(subject.metric_configurations).to be_empty
       end
     end
@@ -83,9 +83,71 @@ describe KalibroClient::Entities::Configurations::KalibroConfiguration do
           returns({'metric_configurations' => [metric_configuration_1.to_hash, metric_configuration_2.to_hash]})
       end
 
-      it 'should return an empty array' do
+      it 'is expected to return an array with the given metric configurations' do
         expect(subject.metric_configurations).to eq([metric_configuration_1, metric_configuration_2])
       end
     end
   end
+
+  describe 'hotspot_metric_configurations' do
+    context 'with no hotspot metric configurations' do
+      before :each do
+        KalibroClient::Entities::Configurations::KalibroConfiguration.
+          expects(:request).
+          with(':id/hotspot_metric_configurations', {id: subject.id}, :get).
+          returns({'hotspot_metric_configurations' => []})
+      end
+
+      it 'is expected to return an empty array' do
+        expect(subject.hotspot_metric_configurations).to be_empty
+      end
+    end
+
+    context 'with hotspot metric configurations' do
+      let(:metric_configuration_1) { FactoryGirl.build(:metric_configuration, kalibro_configuration_id: subject.id) }
+      let(:metric_configuration_2) { FactoryGirl.build(:metric_configuration, kalibro_configuration_id: subject.id) }
+
+      before :each do
+        KalibroClient::Entities::Configurations::KalibroConfiguration.
+          expects(:request).
+          with(':id/hotspot_metric_configurations', {id: subject.id}, :get).
+          returns({'hotspot_metric_configurations' => [metric_configuration_1.to_hash, metric_configuration_2.to_hash]})
+      end
+
+      it 'is expected to return an array with the hotspot metric configurations' do
+        expect(subject.hotspot_metric_configurations).to eq([metric_configuration_1, metric_configuration_2])
+      end
+    end
+  end
+
+  describe 'tree_metric_configurations' do
+    context 'with no tree metric configurations' do
+      before :each do
+        KalibroClient::Entities::Configurations::KalibroConfiguration.
+          expects(:request).
+          with(':id/tree_metric_configurations', {id: subject.id}, :get).
+          returns({'tree_metric_configurations' => []})
+      end
+
+      it 'is expected to return an empty array' do
+        expect(subject.tree_metric_configurations).to be_empty
+      end
+    end
+
+    context 'with tree metric configurations' do
+      let(:metric_configuration_1) { FactoryGirl.build(:metric_configuration, kalibro_configuration_id: subject.id) }
+      let(:metric_configuration_2) { FactoryGirl.build(:metric_configuration, kalibro_configuration_id: subject.id) }
+
+      before :each do
+        KalibroClient::Entities::Configurations::KalibroConfiguration.
+          expects(:request).
+          with(':id/tree_metric_configurations', {id: subject.id}, :get).
+          returns({'tree_metric_configurations' => [metric_configuration_1.to_hash, metric_configuration_2.to_hash]})
+      end
+
+      it 'is expected to return an array with the tree metric configurations' do
+        expect(subject.tree_metric_configurations).to eq([metric_configuration_1, metric_configuration_2])
+      end
+    end
+  end
 end

data/spec/entities/processor/metric_result_spec.rb

@@ -93,4 +93,46 @@ describe KalibroClient::Entities::Processor::MetricResult do
       expect(subject.value).to eq(42)
     end
   end
+
+  describe 'module_result' do
+    let(:module_result) { FactoryGirl.build(:module_result, :with_id) }
+    subject { FactoryGirl.build(:metric_result, module_result_id: module_result.id) }
+
+    context 'when module_result is nil' do
+      before do
+        described_class.expects(:request).with(':id/module_result', { id: module_result.id }, :get)
+          .returns('module_result' => module_result.to_hash)
+      end
+
+      it 'is expected to request the module result and set it' do
+        expect(subject.module_result).to eq(module_result)
+      end
+    end
+
+    context 'when the module_result_id is different than the module_result\'s id' do
+      let(:different_id) { module_result.id + 1 }
+      let(:different_module_result) { FactoryGirl.build(:module_result, id: different_id) }
+
+      before do
+        described_class.expects(:request).with(':id/module_result', { id: different_id }, :get)
+          .returns('module_result' => different_module_result.to_hash)
+      end
+
+      it 'is expected to set the module_result to the one matched by the id' do
+        subject.module_result_id = different_module_result.id
+        expect(subject.module_result).to eq(different_module_result)
+      end
+    end
+
+    context 'when the module_result_id is the same as the module_result\'s id' do
+      before do
+        described_class.expects(:request).with(':id/module_result', { id: module_result.id }, :get)
+          .once.returns('module_result' => module_result.to_hash)
+      end
+
+      it 'is expected to not change the module_result' do
+        expect(subject.module_result).to eq(module_result)
+      end
+    end
+  end
 end

data/spec/factories/kalibro_module.rb

@@ -5,7 +5,7 @@
 # it under the terms of the GNU General Public License as published by
 # the Free Software Foundation, either version 3 of the License, or
 # (at your option) any later version.
-# 
+#
 # This program is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ -16,13 +16,13 @@
 
 FactoryGirl.define do
   factory :kalibro_module, class: KalibroClient::Entities::Processor::KalibroModule do
-    name  'Qt-Calculator'
-    granlrty 'APPLICATION'
+    name 'Qt-Calculator'
+    granularity 'APPLICATION'
 
     trait :with_id do
       id 1
     end
 
     factory :kalibro_module_with_id, traits: [:with_id]
-  end  
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kalibro_client
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.1.0
 platform: ruby
 authors:
 - Daniel Quadros Miranda
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-30 00:00:00.000000000 Z
+date: 2015-10-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -193,7 +193,9 @@ files:
 - features/kalibro_configuration/all.feature
 - features/kalibro_configuration/creation.feature
 - features/kalibro_configuration/destroy.feature
+- features/kalibro_configuration/hotspot_metric_configurations.feature
 - features/kalibro_configuration/metric_configurations.feature
+- features/kalibro_configuration/tree_metric_configurations.feature
 - features/kalibro_range/destroy.feature
 - features/kalibro_range/exists.feature
 - features/kalibro_range/find.feature
@@ -208,6 +210,7 @@ files:
 - features/metric_configuration/metric_configurations_of.feature
 - features/metric_result/hotspot_related_results.feature
 - features/metric_result/metric_results_of.feature
+- features/metric_result/module_results.feature
 - features/metric_result/tree/descendant_values.feature
 - features/metric_result/tree/history_of.feature
 - features/module_result/children.feature
@@ -403,7 +406,9 @@ test_files:
 - features/kalibro_configuration/all.feature
 - features/kalibro_configuration/creation.feature
 - features/kalibro_configuration/destroy.feature
+- features/kalibro_configuration/hotspot_metric_configurations.feature
 - features/kalibro_configuration/metric_configurations.feature
+- features/kalibro_configuration/tree_metric_configurations.feature
 - features/kalibro_range/destroy.feature
 - features/kalibro_range/exists.feature
 - features/kalibro_range/find.feature
@@ -418,6 +423,7 @@ test_files:
 - features/metric_configuration/metric_configurations_of.feature
 - features/metric_result/hotspot_related_results.feature
 - features/metric_result/metric_results_of.feature
+- features/metric_result/module_results.feature
 - features/metric_result/tree/descendant_values.feature
 - features/metric_result/tree/history_of.feature
 - features/module_result/children.feature