hdo-storting-importer 0.1.4 → 0.1.5

data/.gitignore

@@ -14,4 +14,4 @@ spec/reports
 test/tmp
 test/version_tmp
 tmp
-.tm_properties
+.tm_properties

data/features/support/env.rb

@@ -1,6 +1,10 @@
 require 'aruba'
 require 'aruba/cucumber'
 
+require 'simplecov'
+SimpleCov.start
+SimpleCov.command_name 'cucumber'
+
 Before do
   @aruba_timeout_seconds = 10
   ENV['VOTE_COUNT'] = '5'

data/hdo-storting-importer.gemspec

@@ -29,4 +29,5 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency 'guard'
   gem.add_development_dependency 'guard-rspec'
   gem.add_development_dependency 'rspec'
+  gem.add_development_dependency 'simplecov'
 end

data/lib/hdo/storting_importer/promise.rb

@@ -16,45 +16,21 @@ module Hdo
       schema_path StortingImporter.lib.join("hdo/storting_importer/schema/promise.json").to_s
 
       def self.example
-        pr = new("H", "Stille strengere krav til orden og oppførsel for å hindre at uro ødelegger undervisningen.", true, ["GRUNNSKOLE"], "PP", 8)
-        pr.external_id = "1"
-
-        pr
+        new(
+          "1",
+          "H",
+          "Stille strengere krav til orden og oppførsel for å hindre at uro ødelegger undervisningen.",
+          true,
+          ["GRUNNSKOLE"],
+          "PP",
+          8
+        )
       end
 
       def self.json_example
         Util.json_pretty example
       end
 
-      def self.from_csv(str)
-        # cleanup
-        str.gsub!(/\bFrp\b/, "FrP")
-        str.gsub!(/\bKrf\b/, "KrF")
-        str.gsub!(/\bSP\b/, "Sp")
-
-        rows = CSV.parse(
-          str,
-          headers: [:party, :body, :general, :categories, :source, :page],
-          col_sep: ";",
-          skip_blanks: true,
-          return_headers: false,
-        )
-
-        rows.map do |e|
-          pr = e.to_hash
-          next if pr[:party].to_s.strip == "Parti" # header row
-
-          new(
-            pr[:party].to_s.strip,
-            pr[:body].to_s.strip,
-            pr[:general].to_s.downcase == 'ja',
-            pr[:categories].to_s.split(","),
-            pr[:source].to_s.strip,
-            pr[:page].to_s.strip
-            )
-        end.compact
-      end
-
       # doesn't really belong here - data source?
       def self.from_fusion_table(table_id, api_key)
         table = FusionTable.new(api_key)
@@ -75,32 +51,31 @@ module Hdo
       end
 
       def self.from_hash(hash)
-        pr = new hash['party'],
+        pr = new hash['externalId'],
+                 hash['party'],
                  hash['body'],
                  hash['general'],
                  hash['categories'],
                  hash['source'],
                  hash['page']
 
-        pr.external_id = hash['externalId']
-
         pr
       end
 
-      def initialize(party, body, general, categories, source, page)
+      def initialize(external_id, party, body, general, categories, source, page)
+        @external_id = external_id
         @party       = strip_if_string(party)
         @body        = strip_if_string(body)
         @general     = general
         @categories  = clean_categories(categories)
         @source      = strip_if_string(source)
         @page        = page
-
-        @external_id = nil
       end
 
       def to_hash
         h = {
           'kind'       => self.class.kind,
+          'externalId' => @external_id,
           'party'      => @party,
           'general'    => @general,
           'categories' => @categories,
@@ -109,8 +84,6 @@ module Hdo
           'body'       => @body
         }
 
-        h["externalId"] = @external_id if @external_id
-
         h
       end
 

data/lib/hdo/storting_importer/schema/promise.json

@@ -10,7 +10,9 @@
       "required": true
     },
     "externalId": {
-      "type": "string"
+      "type": "string",
+      "description": "A unique identifier for this promise.",
+      "required": true
     },
     "party": {
       "type": "string",

data/lib/hdo/storting_importer/version.rb

@@ -1,5 +1,5 @@
 module Hdo
   module StortingImporter
-    VERSION = "0.1.4"
+    VERSION = "0.1.5"
   end
 end

data/spec/hdo/storting_importer/promise_spec.rb

@@ -8,34 +8,6 @@ module Hdo
       it_behaves_like 'type with JSON schema'
       it_behaves_like 'type with #short_inspect'
 
-      it 'builds promises from CSV' do
-        csv = <<-CSV
-        Parti;Løftetekst;Generell;Emner;Kilde;Side
-        A;Ha et inntektspolitisk samarbeid og en økonomisk politikk som sikrer arbeid til alle. ;Ja;LØNN , SYSSELSETTING;PP;10
-        A;Følge handlingsregelen for en forsvarlig bruk av oljepenger.;Ja;FINANSER;PP;10
-        A;Bruke penger slik at det bidrar til å utjevne svingninger i økonomien.;Ja;FINANSER;PP;10
-        A;Sette ned et utvalg som skal gjennomgå årsakene til finanskrisen.;Nei;FINANSER;PP;10
-        A;Gjennomføre de tiltak som er nødvendig for å sikre et stabilt og fungerende finansmarked.;Ja;FINANSER;PP;10
-        A;Stille etiske krav til forvaltning av statlig kapital. ;Ja;FINANSER;PP;10
-        A;Holde de samlede skattene og avgiftene på samme nivå som i dag.;Nei;SKATTER;PP;10
-        A;Ha en grundig evaluering av skattereformen av 2006, med særlig blikk på fordelingsvirkningene. ;Ja;SKATTER;PP;10
-        A;Opprettholde formueskatten, men gjøre den mer rettferdig.;Ja;SKATTER;PP;10
-        CSV
-
-
-        promises = Promise.from_csv(csv)
-
-        promises.size.should == 9
-
-        prom = promises.first
-        prom.party.should == 'A'
-        prom.body.should == 'Ha et inntektspolitisk samarbeid og en økonomisk politikk som sikrer arbeid til alle.'
-        prom.general.should be_true
-        prom.categories.should == %w[LØNN SYSSELSETTING]
-        prom.source.should == "PP"
-        prom.page.should == '10'
-      end
-
       it 'serializes to JSON' do
         Promise.example.to_json.should be_json <<-JSON
         {
@@ -51,25 +23,18 @@ module Hdo
         JSON
       end
 
-      it 'deserializes from JSON with externalId missing' do
-        pr = Promise.example
-        pr.external_id = nil
-
-        Promise.from_json(pr.to_json).should be_kind_of(Promise)
-      end
-
       it 'strips trailing space from the body' do
-        promise = Promise.new("Party", "Body   ", true, ["æøå"], "PP", 8)
+        promise = Promise.new("1", "Party", "Body   ", true, ["æøå"], "PP", 8)
         promise.body.should == "Body"
       end
 
       it 'correctly upcases non-ASCII category names' do
-        promise = Promise.new("Party", "Body", true, ["æøå"], "PP", 8)
+        promise = Promise.new("1", "Party", "Body", true, ["æøå"], "PP", 8)
         promise.categories.should == ["ÆØÅ"]
       end
 
       it 'ignores empty categories' do
-        promise = Promise.new("Party", "Body", true, ["FOO", ""], "PP", 8)
+        promise = Promise.new("1", "Party", "Body", true, ["FOO", ""], "PP", 8)
         promise.categories.should == ["FOO"]
       end
 

data/spec/spec_helper.rb

@@ -1,8 +1,12 @@
 require 'rspec'
-require 'hdo/storting_importer'
 require 'pry'
 require 'pp'
 
+require 'simplecov'
+SimpleCov.start
+SimpleCov.command_name 'spec'
+
+require 'hdo/storting_importer'
 Dir[File.expand_path("../support/**/*.rb", __FILE__)].each do |f|
   require f
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hdo-storting-importer
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-08-11 00:00:00.000000000 Z
+date: 2012-08-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: builder
@@ -219,6 +219,22 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Gem to process data from data.stortinget.no
 email:
 - jari.bakken@gmail.com