tennis 0.0.4 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5bd5759605bb85cc0627c04bcbcee66593b513a8
-  data.tar.gz: 0fa38355782c45b1fe77347a51da96657b0a2831
+  metadata.gz: 51fd014a2c833965e4531fe2161bb9e3f332ee39
+  data.tar.gz: 9c56807c77df6b4f088670d6405b03ebe79109ab
 SHA512:
-  metadata.gz: 10074593f568f7929736fa2258a01f91fbe8761f955abdf01696b4a6498c546749cb85c2dd7764552fb9029bab95a414bdeb9c2c468cf513ad672cb2849efe12
-  data.tar.gz: 152ca820399495c84fbc3dc12413e07d0004d8908f53c72334e371b3eecea6d06d7b83a5966fd3eeb04c77afbb98174973c1587c692bbab5741daecf06915828
+  metadata.gz: 493ff2541f5bd5549e3e64d93a15718acbbc3640d0728f33334b62f39bb3d43391fd69570c9225d75940576297107ecc0070e19a1ca4fe0a28c26ec1281f4902
+  data.tar.gz: 6c8adb590d4b2c802b478a0142825086f09ab74a861368ef29ce7ce8ca836b95ed2e1e4631efe536bd32a2c57794e524d631d55e5887c803037dc57fe3a6e65b

data/lib/tennis/version.rb

@@ -1,3 +1,3 @@
 class Tennis
-	VERSION = "0.0.4"
+	VERSION = "0.1.0"
 end

data/lib/tennis.rb

@@ -1,101 +1,107 @@
 require "tennis/version"
 
 class Tennis
-	def initialize(scores)
-		@scores = scores != 'default-1' && scores != 'default-2' ? scores.split(/[-,,]/).map(&:to_i) : scores
-		@result = (1 if scores == 'default-1') || (2 if scores == 'default-2') || :default
-		if @result == :default
-			# to check if score for only 1 set has been input
-			validation_1 = @scores.length == 2
-			# to check if any input > 7
-			validation_2 = @scores.any? { |score| score > 7 }
-			@result = :error if validation_1 || validation_2
-		end
-	end
+  def initialize(scores)
+    @scores = scores != 'default-1' && scores != 'default-2' ? scores.split(/[-,,]/).map(&:to_i) : scores
+    @result = (1 if scores == 'default-1') || (2 if scores == 'default-2') || :default
+    if @result == :default
+      # to check if score for only 1 set has been input
+      validation_1 = @scores.length == 2
+      # to check if any input > 7
+      validation_2 = @scores.any? { |score| score > 7 }
+      # to check if one of the input is 7 and the other is not 6 
+      # bad tie break input
+      validation_3 = false 
+      @scores.each_slice(2).each {|r| validation_3 = true if r.any? {|score| score == 7} && !r.any? {|score| score == 6} }
+      @result = :error if validation_1 || validation_2 || validation_3
+      # if set score is not complete eg: 4-6,7-6,4-1
+      @scores.each_slice(2).each {|r| @result = :incomplete_match if r[0] < 6 && r[1] < 6 }
+    end
+  end
 
-	# returns who won the match
-	# :incomplete_match (bad input/incomplete match)
-	# :error (bad input for sure)
-	# 1 (player-1 won)
-	# 2 (player-2 won)
-	def result
-		return @result if @result != :default
-		return @result = (@scores.length == 4) ? two_sets : three_sets
-	end
+  # returns who won the match
+  # :incomplete_match (bad input/incomplete match)
+  # :error (bad input for sure)
+  # 1 (player-1 won)
+  # 2 (player-2 won)
+  def result
+    return @result if @result != :default
+    return @result = (@scores.length == 4) ? two_sets : three_sets
+  end
 
-	# returns an array of points
-	# returns (points_player_1 , points_player_2)
-	# returns (0,0) for bad input
-	def points
-		@result = self.result
-		(return [0, 0]) if @result == :error
-		return (complete_match_points if @result == 1 || @result == 2) || incomplete_match_points
-	end
+  # returns an array of points
+  # returns (points_player_1 , points_player_2)
+  # returns (0,0) for bad input
+  def points
+    @result = self.result
+    (return [0, 0]) if @result == :error
+    return (complete_match_points if @result == 1 || @result == 2) || incomplete_match_points
+  end
 
-	private
+  private
 
-	# helper method: called by RESULT method for valid matches with 2 sets
-	def two_sets
-		set_results = []
-		[0, 2].each do |i|
-			# tie breaker (assuming a 7 point tie breaker) or a 7-5 scores
-			if @scores[i] == 7 || @scores[i+1] == 7
-				set_results << (@scores[i] == 7 ? 1 : 2)
-				# regular set victory - 6 games with a margin of 2
-			else
-				return :incomplete_match if ( @scores[i] - @scores[i + 1] ).abs < 2
-				set_results << (@scores[i] == 6 ? 1 : 2)
-			end
-		end
-		# incomplete match e.g: 6-4,5-3
-		return (set_results[0] if set_results[0] == set_results[1]) || :incomplete_match
-	end
+  # helper method: called by RESULT method for valid matches with 2 sets
+  def two_sets
+    set_results = []
+    [0, 2].each do |i|
+      # tie breaker (assuming a 7 point tie breaker) or a 7-5 scores
+      if @scores[i] == 7 || @scores[i+1] == 7
+        set_results << (@scores[i] == 7 ? 1 : 2)
+        # regular set victory - 6 games with a margin of 2
+      else
+        return :incomplete_match if ( @scores[i] - @scores[i + 1] ).abs < 2
+        set_results << (@scores[i] == 6 ? 1 : 2)
+      end
+    end
+    # incomplete match e.g: 6-4,5-3
+    return (set_results[0] if set_results[0] == set_results[1]) || :incomplete_match
+  end
 
-	# helper method: called by RESULT method for valid matches with 3 sets
-	def three_sets
-		set_results = []
-		[0, 2, 4].each do |i|
-			# tie breaker (assuming a 7 point tie breaker) or a 7-5 score
-			if @scores[i] == 7 || @scores[i + 1] == 7
-				set_results << (@scores[i] == 7 ? 1 : 2)
-				# regular set victory - 6 games with a margin of 2
-			else
-				return :incomplete_match if (@scores[i] - @scores[i + 1]).abs < 2
-				set_results << (@scores[i] == 6 ? 1 : 2)
-			end
-		end
-		# checks if the result has been decided in the first 2 sets
-		# but the 3rd set is also present in the input
-		return :error if set_results[0] == set_results[1]
-		return  set_results.count(1) ? 1 : 2
-	end
+  # helper method: called by RESULT method for valid matches with 3 sets
+  def three_sets
+    set_results = []
+    [0, 2, 4].each do |i|
+      # tie breaker (assuming a 7 point tie breaker) or a 7-5 score
+      if @scores[i] == 7 || @scores[i + 1] == 7
+        set_results << (@scores[i] == 7 ? 1 : 2)
+        # regular set victory - 6 games with a margin of 2
+      else
+        return :incomplete_match if (@scores[i] - @scores[i + 1]).abs < 2
+        set_results << (@scores[i] == 6 ? 1 : 2)
+      end
+    end
+    # checks if the result has been decided in the first 2 sets
+    # but the 3rd set is also present in the input
+    return :error if set_results[0] == set_results[1]
+    return  set_results.count(1) == 2 ? 1 : 2
+  end
 
-	# helper method: called by POINTS for complete matches
-	def complete_match_points
-		points = [0, 0]
-		@result = self.result
-		points[@result - 1] = (@scores.length == 6) ? 12 : 14
-		runner_up = (@result == 1) ? 2 : 1
-		runner_up_points = player_points(runner_up)
-		points[runner_up - 1] = runner_up_points < 8 ? runner_up_points : 8
-		return points
-	end
+  # helper method: called by POINTS for complete matches
+  def complete_match_points
+    points = [0, 0]
+    @result = self.result
+    points[@result - 1] = (@scores.length == 6) ? 12 : 14
+    runner_up = (@result == 1) ? 2 : 1
+    runner_up_points = player_points(runner_up)
+    points[runner_up - 1] = runner_up_points < 8 ? runner_up_points : 8
+    return points
+  end
 
-	# helper method: called by POINTS for incomplete matches
-	def incomplete_match_points
-		points = [0, 0]
-		player_1_points = player_points(1)
-		player_2_points = player_points(2)
-		points[0] = player_1_points < 10 ? player_1_points : 10
-		points[1] = player_2_points < 10 ? player_2_points : 10
-		return points
-	end
+  # helper method: called by POINTS for incomplete matches
+  def incomplete_match_points
+    points = [0, 0]
+    player_1_points = player_points(1)
+    player_2_points = player_points(2)
+    points[0] = player_1_points < 10 ? player_1_points : 10
+    points[1] = player_2_points < 10 ? player_2_points : 10
+    return points
+  end
 
-	# helper method: returns the POINTS of a player given the player number
-	def player_points(player)
-		player_scores = []
-		@scores.each_with_index { |score, index| (player_scores << score; player +=2) if index == (player - 1) }
-		player_scores = player_scores.sort! { |x, y| y <=> x }
-		return player_scores[0] + player_scores[1]
-	end
+  # helper method: returns the POINTS of a player given the player number
+  def player_points(player)
+    player_scores = []
+    @scores.each_with_index { |score, index| (player_scores << score; player +=2) if index == (player - 1) }
+    player_scores = player_scores.sort! { |x, y| y <=> x }
+    return player_scores[0] + player_scores[1]
+  end
 end

data/spec/tennis_spec.rb

@@ -11,7 +11,88 @@ describe Tennis, "#scores" do
     score = Tennis.new("6-4, 6-4")
     expect(score.result).to eq 1
   end
+
+  it "finds the winner properly in three sets" do
+    score = Tennis.new("4-6, 6-2, 3-6")
+    expect(score.result).to eq 2
+  end
+
+  it "finds the winner properly in two sets with tie break" do
+    score = Tennis.new("6-4, 7-6")
+    expect(score.result).to eq 1
+  end
+
+  it "finds the winner properly in three sets with tie break" do
+    score = Tennis.new("6-4, 6-7, 7-6")
+    expect(score.result).to eq 1
+  end
+
+  it "reports incomplete match score (set 1-1)" do
+    score = Tennis.new("6-4,4-6")
+    expect(score.result).to eq :incomplete_match
+  end
+
+  it "reports incomplete match score (set incomplete)" do
+    score = Tennis.new("6-4,4-5")
+    expect(score.result).to eq :incomplete_match
+  end
+
+  it "checks invalid score: difference in games won < 2" do
+    score = Tennis.new("6-5,4-6,7-6")
+    expect(score.result).to eq :incomplete_match
+  end
+
+  it "checks invalid score: only 1 set input" do
+    score = Tennis.new("6-4")
+    expect(score.result).to eq :error
+  end
+
+  it "checks invalid score: result decided in first 2 sets but 3rd set input" do
+    score = Tennis.new("6-4,6-4,4-6")
+    expect(score.result).to eq :error
+  end
+
+  it "checks invalid score: bad input for tie break" do
+    score = Tennis.new("7-0,4-6,6-2")
+    expect(score.result).to eq :error
+  end
+
+  it "checks invalid score: no score > 7" do
+    score = Tennis.new("8-4,2-6,6-1")
+    expect(score.result).to eq :error
+  end
+
 end
 
 describe Tennis, "#points" do
+  it "returns 12 points in a three set win in a complete match" do
+    score = Tennis.new("6-4, 4-6, 6-4")
+    expect(score.points).to eq [12,8]
+  end
+
+  it "returns 14 points in two set win in a complete match" do
+    score = Tennis.new("4-6, 4-6")
+    expect(score.points).to eq [8,14]
+  end
+
+  it "returns a max of 8 points for the runners up in a complete match" do
+    score = Tennis.new("4-6, 6-2, 3-6")
+    expect(score.points).to eq [8,12]
+  end
+
+  it "return a max of 10 points for each player in an incomplete match" do
+    score = Tennis.new("7-6,6-7")
+    expect(score.points).to eq [10,10]
+  end
+
+  it "return a max of 10 points for each player in an incomplete match" do
+    score = Tennis.new("7-6,4-6,4-1")
+    expect(score.points).to eq [10,10]
+  end
+
+  it "return [0,0] for bad input" do
+    score = Tennis.new("8-1")
+    expect(score.points).to eq [0,0]
+  end
+
 end

data/tennis.gemspec

@@ -6,7 +6,7 @@ require 'tennis/version'
 Gem::Specification.new do |spec|
   spec.name          = "tennis"
   spec.version       = Tennis::VERSION
-  spec.authors       = ["Carlos Puchol"]
+  spec.authors       = ["Rohan Katyal", "Carlos Puchol"]
   spec.email         = ["cpg+git@amahi.org"]
   spec.summary       = %q{A gem to manage tennis scores.}
   spec.description   = %q{A gem to manage and validate tennis scores.}

metadata

@@ -1,9 +1,10 @@
 --- !ruby/object:Gem::Specification
 name: tennis
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.1.0
 platform: ruby
 authors:
+- Rohan Katyal
 - Carlos Puchol
 autorequire: 
 bindir: bin