Skip to content

Commit

Permalink
Merge pull request #70 from Swirrl/opaque-base-uris
Browse files Browse the repository at this point in the history
Opaque base uris
  • Loading branch information
lkitching authored Feb 17, 2022
2 parents 769efe3 + 2490c27 commit f43f76a
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/csv2rdf/util.clj
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
(defn set-fragment
"Sets the fragment on a URI to the given value."
[^URI uri fragment]
(URI. (.getScheme uri) (.getUserInfo uri) (.getHost uri) (.getPort uri) (.getPath uri) (.getQuery uri) fragment))
(URI. (.getScheme uri) (.getSchemeSpecificPart uri) fragment))

(defn remove-fragment
"Sets the fragment on a URI to nil."
Expand Down
11 changes: 10 additions & 1 deletion test/csv2rdf/util_test.clj
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,15 @@
[clojure.test.check.clojure-test :refer [defspec]])
(:import [java.net URI]))


(deftest set-fragment-test
(testing "transparent uri"
(is (= (URI. "http:8080//example.org/path?param=bar#new-fragment")
(set-fragment (URI. "http:8080//example.org/path?param=bar#fragment") "new-fragment"))))
(testing "opaque uri"
(is (= (URI. "jar:file:///some/path!/inside/zip/path#new-fragment")
(set-fragment (URI. "jar:file:///some/path!/inside/zip/path#fragment") "new-fragment")))))

(defspec filter-values-test 100
(prop/for-all
[m (gen/map gen/keyword gen/int)]
Expand Down Expand Up @@ -54,4 +63,4 @@
(let [[expected-prefix expected-suffix] (split-at n xs)
[prefix suffix] (eager-split-at n xs)]
(and (= expected-prefix prefix)
(= (seq expected-suffix) (seq suffix))))))
(= (seq expected-suffix) (seq suffix))))))

0 comments on commit f43f76a

Please sign in to comment.