CSC/ECE 517 Spring 2018 E1812 - Revision history

Mpan2 at 05:04, 6 April 2018

2018-04-06T05:04:34Z

← Older revision		Revision as of 05:04, 6 April 2018
Line 100:		Line 100:
	We used the Ubuntu-Expertiza image to setup the environment. We forked the master from Expertiza, cloned that and then run through the command terminal.		We used the Ubuntu-Expertiza image to setup the environment. We forked the master from Expertiza, cloned that and then run through the command terminal.

	cd/~~expetiza~~/app/models		cd/expertiza/app/models

	The on_the_fly_calc.rb model file is present in this directory.		The on_the_fly_calc.rb model file is present in this directory.

Mpan2: /* Conclusions */

2018-04-03T02:01:54Z

Conclusions

← Older revision		Revision as of 02:01, 3 April 2018
Line 132:		Line 132:
	For the last method score, the majority of the method is included in private methods. For the varying_rubrics_by_round case, we tested the built-in calculate_score method of Assignment.		For the last method score, the majority of the method is included in private methods. For the varying_rubrics_by_round case, we tested the built-in calculate_score method of Assignment.

	==~~Conclusions~~==		==Conclusion and Learning Outcomes==
			After writing the unit test we managed to increase the coverage of our rails application.
			The difficulty of the test is that it involves lots of different classes and methods from different classes. A four-fold hash is used to denote the score. It is not straightforward to create assignments with all necessary attributes and other related objects.

	==Github Page==		==Github Page==
	https://github.com/DileepBadveli/expertiza/blob/master/spec/models/on_the_fly_calc_spec.rb		https://github.com/DileepBadveli/expertiza/blob/master/spec/models/on_the_fly_calc_spec.rb

Mpan2: /* Implementation Steps */

2018-04-03T01:56:22Z

Implementation Steps

← Older revision		Revision as of 01:56, 3 April 2018
Line 115:		Line 115:

	The last method score calculates scores of a question. It calls private methods score_assignment and participant_score to obtain hashes of teams and participants. Like previous methods, it deals with two cases. If varying_rubrics_by_round? is true, it calls another three private methods to calculate the number of rounds, the average, and range of assessments and scores. If it does not vary, assessments and scores are obtained with the built-in method get_assessments_for of ReviewResponseMap and compute_scores of Answer. That is the general function of the OnTheFlyCalc module.		The last method score calculates scores of a question. It calls private methods score_assignment and participant_score to obtain hashes of teams and participants. Like previous methods, it deals with two cases. If varying_rubrics_by_round? is true, it calls another three private methods to calculate the number of rounds, the average, and range of assessments and scores. If it does not vary, assessments and scores are obtained with the built-in method get_assessments_for of ReviewResponseMap and compute_scores of Answer. That is the general function of the OnTheFlyCalc module.

			=== Test Entries Creation ===

			Mock/dummy objects are needed to be created for any unit testing criteria.These objects are loaded freshly and deleted after every testing condition.

			For most tests, there is no difference between build and create. Usually build is faster because it stores objects in the memory. However, in this case, create is preferred because we need to create the object in the database. During the test, sometimes we cannot find the object in the database if we use build rather than create.
			We create two assignments whose assignment_questionnaires have been used in 1 and 2 rounds. That is for different cases of varying_rubrics_by_round?.

			=== Testing Conditions ===
			The varying_rubrics_by_round? has been called on self for each method except compute_total_score. Through the pattern of varying_rubrics_by_round? being used as an if statement, test cases have been made for the behaviors if it returns true and false.

			For the compute_reviews_hash method located in the model,varying_rubrics_by_round? method called on an Assignment object will return true when multiple review phases with different review rubrics exist. Creating test examples which return true and false for the method call varying_rubrics_by_round? let the right behavior be tested. When it returns true scores_varying_rubrics behavior would have been accepted, while when return false scores_non_varying_rubrics behavior will occur.

			For the compute_avg_and_ranges_hash method, the get_assessments_for method of ReviewResponseMap and compute_scores of Answer are tested for both cases. The assignment_teams are also tested. For varying_rubrics_by_round case, the review_questionnaire_id method of Assignment is tested.

			For the last method score, the majority of the method is included in private methods. For the varying_rubrics_by_round case, we tested the built-in calculate_score method of Assignment.

	==Conclusions==		==Conclusions==

Mpan2: /* Implementation Steps */

2018-04-03T01:52:47Z

Implementation Steps

@@ Line 102: / Line 102: @@
 cd/expetiza/app/models
-The on_the_fly_calc.rb model file is present in this directory
+The on_the_fly_calc.rb model file is present in this directory.
 ==Conclusions==

Mpan2: /* Implementation Steps */

2018-04-03T01:49:16Z

Implementation Steps

← Older revision		Revision as of 01:49, 3 April 2018
Line 96:		Line 96:

	==Implementation Steps==		==Implementation Steps==
			===Expertiza Environment Setup===

			We used the Ubuntu-Expertiza image to setup the environment. We forked the master from Expertiza, cloned that and then run through the command terminal.

			cd/expetiza/app/models

			The on_the_fly_calc.rb model file is present in this directory

	==Conclusions==		==Conclusions==
	==Github Page==		==Github Page==

Mpan2: /* Plan of Work */

2018-04-03T01:26:31Z

Plan of Work

@@ Line 94: / Line 94: @@
 Objects were created using structures left in the factory file when needed. An example of this is  let(:questionnaire) { create(:questionnaire, id: 1)}.
 After developing different objects by this method, the objects get used in test cases to either avoid the implementation of the methods specified in the model or make it return a value which can be used to test if conditions.
 ==Implementation Steps==

Mpan2: /* Problem Statement */

2018-04-03T01:26:11Z

Problem Statement

@@ Line 77: / Line 77: @@
 ===Team Members===
 . Dileep Badveli (dbadvel@ncsu.edu)
 . Minghao Pan (mpan2@ncsu.edu)
 ==Plan of Work==

Mpan2: /* Testing plan */

2018-04-03T01:05:19Z

Testing plan

← Older revision		Revision as of 01:05, 3 April 2018
Line 91:		Line 91:
	The last method score calculates scores of a question. It calls private methods score_assignment and participant_score to obtain hashes of teams and participants. Like previous methods, it deals with two cases. If varying_rubrics_by_round? is true, it calls another three private methods to calculate the number of rounds, the average, and range of assessments and scores. If it does not vary, assessments and scores are obtained with the built-in method get_assessments_for of ReviewResponseMap and compute_scores of Answer. That is the general function of the OnTheFlyCalc module.		The last method score calculates scores of a question. It calls private methods score_assignment and participant_score to obtain hashes of teams and participants. Like previous methods, it deals with two cases. If varying_rubrics_by_round? is true, it calls another three private methods to calculate the number of rounds, the average, and range of assessments and scores. If it does not vary, assessments and scores are obtained with the built-in method get_assessments_for of ReviewResponseMap and compute_scores of Answer. That is the general function of the OnTheFlyCalc module.

	==~~Testing plan~~==		==Plan of Work==
			The task is to write test cases for testing the on_the_fly_calc model file. There was no Rspec file for the corresponding model exists so we need to create a new file and build tests from scratch. For this purpose different sub-tasks involved

			1. Setting up the Expertiza environment;

			2. Understand the functionality of model file in on_the_fly_calc.rb;

			3. Understand the linked data attributes and methods being used;

			4. Creating stub entries for testing different methods;

			5. Writing testing conditions for different methods and compare with the expected outputs.
	Objects were created using structures left in the factory file when needed. An example of this is let(:questionnaire) { create(:questionnaire, id: 1)}.		Objects were created using structures left in the factory file when needed. An example of this is let(:questionnaire) { create(:questionnaire, id: 1)}.
	After developing different objects by this method, the objects get used in test cases to either avoid the implementation of the methods specified in the model or make it return a value which can be used to test if conditions.		After developing different objects by this method, the objects get used in test cases to either avoid the implementation of the methods specified in the model or make it return a value which can be used to test if conditions.


	For most tests, there is no difference between build and create. Usually build is faster because it stores objects in the memory. However, in this case, create is preferred because we need to create the object in the database. During the test, sometimes we cannot find the object in the database if we use build rather than create.		For most tests, there is no difference between build and create. Usually build is faster because it stores objects in the memory. However, in this case, create is preferred because we need to create the object in the database. During the test, sometimes we cannot find the object in the database if we use build rather than create.
Line 106:		Line 118:

	For the last method score, the majority of the method is included in private methods. For the varying_rubrics_by_round case, we tested the built-in calculate_score method of Assignment.		For the last method score, the majority of the method is included in private methods. For the varying_rubrics_by_round case, we tested the built-in calculate_score method of Assignment.

	==Implementation Steps==		==Implementation Steps==
	==Conclusions==		==Conclusions==

Mpan2: /* Problem Statement */

2018-04-03T01:00:26Z

Problem Statement

← Older revision		Revision as of 01:00, 3 April 2018
Line 75:		Line 75:

	2. spec/models/on_the_fly_calc_spec.rb		2. spec/models/on_the_fly_calc_spec.rb
	===Team Members		===Team Members===
	1. Dileep Badveli (dbadvel@ncsu.edu)		1. Dileep Badveli (dbadvel@ncsu.edu)
	2. Minghao Pan (mpan2@ncsu.edu)		2. Minghao Pan (mpan2@ncsu.edu)

Mpan2 at 00:58, 3 April 2018

2018-04-03T00:58:38Z

← Older revision		Revision as of 00:58, 3 April 2018
Line 68:		Line 68:
	When software is developed using a test-driven approach, the combination of writing the unit test to specify the interface plus the refactoring activities performed after the test is passing, may take the place of formal design. Each unit test can be seen as a design element specifying classes, methods, and observable behavior.		When software is developed using a test-driven approach, the combination of writing the unit test to specify the interface plus the refactoring activities performed after the test is passing, may take the place of formal design. Each unit test can be seen as a design element specifying classes, methods, and observable behavior.
	==Problem Statement==		==Problem Statement==
	~~===Tasks to be completed===~~		There is no corresponding unit test for on_the_fly_calc.rb. The OnTheFlyCal module provides 4 publish methods for assignments: compute_total_score, compute_reviews_hash, compute_avg_and_ranges_hash and scores. We need to test all those methods. Privately methods are tested when testing public methods instead of tested directly.
	The OnTheFlyCal module provides 4 publish methods for assignments: compute_total_score, compute_reviews_hash, compute_avg_and_ranges_hash and scores. We need to test all those methods. Privately methods are tested when testing public methods instead of tested directly.		===Files Involved===
			The files to be understood and created are:

			1. app/models/on_the_fly_calc.rb

			2. spec/models/on_the_fly_calc_spec.rb
			===Team Members
			1. Dileep Badveli (dbadvel@ncsu.edu)
			2. Minghao Pan (mpan2@ncsu.edu)

	===Description of the the module===		===Description of the the module===
	The compute_total_score method is to compute the total score of a given assignment. The method collects all questionnaires of the assignment and calls the get_weighted_score method of Questionnaire Class to calculate the total score.		The compute_total_score method is to compute the total score of a given assignment. The method collects all questionnaires of the assignment and calls the get_weighted_score method of Questionnaire Class to calculate the total score.