transfermarkt 0.0.3 → 0.0.5

data/spec/units/club_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+describe Transfermarkt::Club do
+  before(:all) do
+    #FakeWeb.register_uri(:get, "www.transfermarkt.co.uk/maccabi-haifa/startseite/verein/1064", body: File.read(File.join("spec", "static_htmls", "maccabi_haifa_page.html")), status: ["200", "OK"])
+    @haifa = Transfermarkt::Club.fetch_by_club_uri("/maccabi-haifa/startseite/verein/1064")
+  end
+
+  it "should be the right team" do
+    expect(@haifa.name).to eq("Maccabi Haifa")
+  end
+
+  it "should find 25 players" do
+    expect(@haifa.player_uris.size).to eq(25)
+  end
+
+  it "should be in Israel" do
+    expect(@haifa.country).to eq("Israel")
+  end
+end

data/spec/units/league_spec.rb

@@ -0,0 +1,54 @@
+require 'spec_helper'
+
+describe Transfermarkt::League do
+
+  describe "Real League" do
+    before(:all) do
+      FakeWeb.register_uri(:get, "http://www.transfermarkt.co.uk/jumplist/startseite/wettbewerb/GB1", body: File.read(File.join("spec", "static_htmls", "premier_league_html.html")), status: ["200", "OK"])
+      @permier_league = Transfermarkt::League.fetch_clubs_and_uris_by_league_uri("/jumplist/startseite/wettbewerb/GB1")
+    end
+
+    describe "#valid_league?" do
+
+      it "should return true for a league page" do
+        expect(@permier_league.valid_league?).to eq(true)
+      end
+
+    end
+
+    describe '#name' do
+      it "should be premier league" do
+        expect(@permier_league.name).to eq("Premier League")
+      end
+    end
+
+    describe '#country' do
+      it "should be england" do
+        expect(@permier_league.country).to eq("England")
+      end
+    end
+
+    describe "#fetch_clubs_and_uris_by_league_uri" do
+      it "should fetch 20 teams" do
+        expect(@permier_league.clubs_index.count).to eq(20)
+      end
+    end
+  end
+
+  describe "A Cup" do
+    before(:all) do
+      FakeWeb.register_uri(:get, "http://www.transfermarkt.co.uk/mtn8/startseite/pokalwettbewerb/SFAL", body: File.read(File.join("spec", "static_htmls", "cup.html")), status: ["200", "OK"])
+      @cup = Transfermarkt::League.fetch_clubs_and_uris_by_league_uri("/mtn8/startseite/pokalwettbewerb/SFAL")
+    end
+
+    describe "#valid_league?" do
+
+      it "should return false for a cup page" do
+        expect(@cup.valid_league?).to eq(false)
+      end
+
+    end
+
+  end
+
+end

data/spec/units/player_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+describe Transfermarkt::Player do
+  describe "Player with good html" do
+    before(:all) do
+      FakeWeb.register_uri(:get, "http://www.transfermarkt.co.uk/lionel-messi/profil/spieler/28003", body: File.read(File.join("spec", "static_htmls", "messi_player_page.html")), status: ["200", "OK"])
+      @messi = Transfermarkt::Player.fetch_by_profile_uri("/lionel-messi/profil/spieler/28003")
+    end
+
+    it "should fetch a player details successfully" do
+
+      expect(@messi.full_name).to eq("Lionel Messi")
+      expect(@messi.name_in_native_country).to eq("Lionel Andrés Messi Cuccitini")
+      expect(@messi.nationality).to eq(["Argentina", "Spain"])
+      expect(@messi.date_of_birth).to eq("Jun 24, 1987")
+      expect(@messi.age).to eq(27)
+      expect(@messi.foot).to eq("left")
+      expect(@messi.height).to eq(169)
+      expect(@messi.club).to eq("FC Barcelona")
+      expect(@messi.market_value).to eq(105_600_000)
+      expect(@messi.position).to eq("Striker - Centre Forward")
+    end
+
+    it "should return player uri is valid" do
+      expect(@messi.valid_player?).to eq(true)
+    end
+  end
+
+  describe "Player with invalid html" do
+    before(:all) do
+      FakeWeb.register_uri(:get, "http://www.transfermarkt.co.uk/jose-antonio-reyes/profil/spieler/7717", body: File.read(File.join("spec", "static_htmls", "jose_antonio_reyes_player_page.html")), status: ["200", "OK"])
+      @jose = Transfermarkt::Player.fetch_by_profile_uri("/jose-antonio-reyes/profil/spieler/7717")
+    end
+
+    it "should return player is invalid" do
+      expect(@jose.valid_player?).to eq(false)
+    end
+  end
+end

data/spec/units/transfermarkt_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe Transfermarkt do
+  it "should respond to base_uri" do
+    expect(Transfermarkt.respond_to?(:base_uri)).to be_truthy
+  end
+
+  it "should have a user agent by default" do
+    expect(Transfermarkt::USER_AGENT).to eq("Firefox")
+  end
+end

data/transfermarkt.gemspec

@@ -20,6 +20,11 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"
+  spec.add_development_dependency "rspec"
+  spec.add_development_dependency 'fakeweb'
+  spec.add_development_dependency 'byebug'
+  spec.add_dependency 'activesupport'
   spec.add_dependency "httparty"
   spec.add_dependency "nokogiri"
+  spec.add_dependency "useragents"
 end

metadata

@@ -1,69 +1,139 @@
 --- !ruby/object:Gem::Specification
 name: transfermarkt
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.5
 platform: ruby
 authors:
 - Elad Meidar
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-11-14 00:00:00.000000000 Z
+date: 2016-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: fakeweb
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: httparty
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: useragents
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Integrate with Transfermarkt
@@ -73,7 +143,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".byebug_history"
+- ".gitignore"
+- ".rspec"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -82,8 +154,19 @@ files:
 - lib/transfermarkt/club.rb
 - lib/transfermarkt/entity_base.rb
 - lib/transfermarkt/league.rb
+- lib/transfermarkt/live_game.rb
 - lib/transfermarkt/player.rb
 - lib/transfermarkt/version.rb
+- spec/spec_helper.rb
+- spec/static_htmls/cup.html
+- spec/static_htmls/jose_antonio_reyes_player_page.html
+- spec/static_htmls/maccabi_haifa_page.html
+- spec/static_htmls/messi_player_page.html
+- spec/static_htmls/premier_league_html.html
+- spec/units/club_spec.rb
+- spec/units/league_spec.rb
+- spec/units/player_spec.rb
+- spec/units/transfermarkt_spec.rb
 - transfermarkt.gemspec
 homepage: ''
 licenses:
@@ -95,18 +178,28 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.6
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Allows access to transfermarkt data
-test_files: []
+test_files:
+- spec/spec_helper.rb
+- spec/static_htmls/cup.html
+- spec/static_htmls/jose_antonio_reyes_player_page.html
+- spec/static_htmls/maccabi_haifa_page.html
+- spec/static_htmls/messi_player_page.html
+- spec/static_htmls/premier_league_html.html
+- spec/units/club_spec.rb
+- spec/units/league_spec.rb
+- spec/units/player_spec.rb
+- spec/units/transfermarkt_spec.rb