iq_triplestorage 0.1.1 → 0.1.2

data/.gitignore

@@ -0,0 +1 @@
+/coverage

data/Gemfile

@@ -7,4 +7,5 @@ gem "rake"
 group :test do
   gem "minitest"
   gem "webmock"
+  gem "simplecov", :require => false
 end

data/Rakefile

@@ -6,3 +6,13 @@ end
 
 desc "Run tests"
 task :default => :test
+
+desc "Build gem"
+task :build do |t|
+  `rm *.gem; gem build *.gemspec`
+end
+
+desc "Release gem"
+task :release => :build do |t|
+  `gem push *.gem`
+end

data/lib/iq_triplestorage.rb

@@ -1,3 +1,3 @@
 module IqTriplestorage
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

data/lib/iq_triplestorage/virtuoso_adaptor.rb

@@ -1,11 +1,14 @@
-require 'net/http'
-require 'base64'
-require 'typhoeus'
+require "net/http"
+require "base64"
+require "typhoeus"
 
 module IqTriplestorage
   class VirtuosoAdaptor
 
     def initialize(host, port, username, password)
+      # validate to avoid nasty errors
+      raise(ArgumentError, "username must not be nil") if username.nil?
+
       @host = host
       @port = port
       @username = username
@@ -59,7 +62,7 @@ module IqTriplestorage
         "Authorization" => "Basic #{auth}", # XXX: seems like this should be built into Typhoeus!?
         "Content-Type" => content_type
       }
-      res = Typhoeus::Request.put("#{@host}:#{@port}#{path}",
+      res = Typhoeus::Request.put("#{@host}:#{@port}#{path}", # XXX: is PUT correct here?
           :headers => headers, :body => rdf_data)
 
       return res.code == 201
@@ -84,7 +87,7 @@ module IqTriplestorage
         "Authorization" => "Basic #{auth}", # XXX: seems like this should be built into Typhoeus!?
         "Content-Type" => "application/sparql-query"
       }
-      res = Typhoeus::Request.put("#{@host}:#{@port}#{path}",
+      res = Typhoeus::Request.post("#{@host}:#{@port}#{path}",
           :headers => headers, :body => query)
 
       return res.code == 201

data/test/test_helper.rb

@@ -0,0 +1,8 @@
+require "simplecov"
+
+# limit coverage to /lib
+cwd = File.expand_path(File.join(File.dirname(__FILE__), "../"))
+SimpleCov.add_filter do |src_file|
+  !src_file.filename.start_with?(File.join(cwd, "lib").to_s)
+end
+SimpleCov.start

data/test/virtuoso_test.rb

@@ -1,6 +1,8 @@
-require 'base64'
+require File.join(File.expand_path(File.dirname(__FILE__)), "test_helper")
+
+require "base64"
 require "minitest/autorun"
-require 'webmock/test_unit'
+require "webmock/test_unit"
 
 require "iq_triplestorage/virtuoso_adaptor"
 
@@ -13,7 +15,7 @@ class VirtuosoTest < MiniTest::Unit::TestCase
       # not using WebMock's custom assertions as those didn't seem to provide
       # sufficient flexibility
       fn = @observers.shift
-      raise(TypeError, "missing request observer") unless fn
+      raise(TypeError, "missing request observer: #{req.inspect}") unless fn
       fn.call(req)
       true
     end.to_return do |req|
@@ -90,7 +92,9 @@ class VirtuosoTest < MiniTest::Unit::TestCase
     end
     @observers << lambda do |req|
       assert_equal :put, req.method
-      assert req.uri.path.start_with?("/DAV/home/#{@username}/rdf_sink/")
+      path = req.uri.path
+      assert path.start_with?("/DAV/home/#{@username}/rdf_sink/")
+      assert_equal 5, path.count("/")
       assert_equal "application/rdf+xml", req.headers["Content-Type"]
       assert_equal rdf_data, req.body
     end
@@ -104,11 +108,21 @@ class VirtuosoTest < MiniTest::Unit::TestCase
     }
 
     @observers << lambda do |req|
+      assert_equal :post, req.method
+      path = req.uri.path
+      assert path.start_with?("/DAV/home/#{@username}/")
+      assert_equal 4, path.count("/")
+      assert_equal "application/sparql-query", req.headers["Content-Type"]
       data.keys.each do |graph_uri|
         assert req.body.include?("CLEAR GRAPH <#{graph_uri}>")
       end
     end
     @observers << lambda do |req|
+      assert_equal :post, req.method
+      path = req.uri.path
+      assert path.start_with?("/DAV/home/#{@username}/")
+      assert_equal 4, path.count("/")
+      assert_equal "application/sparql-query", req.headers["Content-Type"]
       data.each do |graph_uri, ntriples|
         assert req.body.
             include?(<<-EOS)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: iq_triplestorage
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-23 00:00:00.000000000 Z
+date: 2012-10-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: typhoeus
@@ -33,6 +33,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- .gitignore
 - .travis.yml
 - Gemfile
 - README.md
@@ -40,6 +41,7 @@ files:
 - iq_triplestorage.gemspec
 - lib/iq_triplestorage.rb
 - lib/iq_triplestorage/virtuoso_adaptor.rb
+- test/test_helper.rb
 - test/virtuoso_test.rb
 homepage: http://github.com/innoq/iq_triplestorage
 licenses: []
@@ -53,12 +55,18 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 1344314608360584677
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 1344314608360584677
 requirements: []
 rubyforge_project: iq_triplestorage
 rubygems_version: 1.8.23
@@ -66,4 +74,5 @@ signing_key:
 specification_version: 3
 summary: IqTriplestorage - library for interacting with RDF triplestores / quadstores
 test_files:
+- test/test_helper.rb
 - test/virtuoso_test.rb