Skip to content

Commit ad73380

Browse files
seancorfieldCopilot
andcommitted
clean up weird merge code in test; add 1.13 to tests
Co-authored-by: Copilot <copilot@github.com> Signed-off-by: Sean Corfield <sean@corfield.org>
1 parent 865e988 commit ad73380

4 files changed

Lines changed: 9 additions & 10 deletions

File tree

‎.gitignore‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,3 +18,4 @@
1818
bin/
1919
obj/
2020
target
21+
outdated.txt

‎deps.edn‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@
33
:1.9 {:override-deps {org.clojure/clojure {:mvn/version "1.9.0"}}}
44
:1.10 {:override-deps {org.clojure/clojure {:mvn/version "1.10.3"}}}
55
:1.11 {:override-deps {org.clojure/clojure {:mvn/version "1.11.4"}}}
6-
:1.12 {:override-deps {org.clojure/clojure {:mvn/version "1.12.4"}}}
6+
:1.12 {:override-deps {org.clojure/clojure {:mvn/version "1.12.6"}}}
7+
:1.13 {:override-deps {org.clojure/clojure {:mvn/version "1.13.0-alpha7"}}}
78
:runner
89
{:extra-deps {io.github.cognitect-labs/test-runner
910
{:git/tag "v0.5.1" :git/sha "dfb30dd"}}

‎run-tests.sh‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#!/bin/sh
22

3-
for v in 9 10 11 12
3+
for v in 9 10 11 12 13
44
do
55
echo ""
66
echo "Running tests for Clojure 1.$v..."

‎src/test/clojure/clojure/tools/cli_legacy_test.cljc‎

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -67,14 +67,11 @@
6767
(testing "merging args"
6868
(deftest should-merge-identical-arguments
6969
(let [assoc-fn (fn [previous key val]
70-
(assoc previous key
71-
(if-let [oldval (get previous key)]
72-
(merge oldval val)
73-
(hash-set val))))
74-
[options args _] (cli ["-p" "1" "--port" "2"]
75-
["-p" "--port" "description"
76-
:assoc-fn assoc-fn
77-
:parse-fn #(parse-int %)])]
70+
(update previous key (fnil conj #{}) val))
71+
[options _ _] (cli ["-p" "1" "--port" "2"]
72+
["-p" "--port" "description"
73+
:assoc-fn assoc-fn
74+
:parse-fn #(parse-int %)])]
7875
(is (= {:port #{1 2}} options)))))
7976

8077
(testing "extra arguments"

0 commit comments

Comments
 (0)