Skip to content

Commit

Permalink
Small improvements (#951)
Browse files Browse the repository at this point in the history
  • Loading branch information
solar05 committed Jun 22, 2022
1 parent f60c34c commit 1b34b48
Show file tree
Hide file tree
Showing 6 changed files with 15 additions and 14 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ generate: clean
clean:
rm -rf issues/*

format:
fix-format:
clojure -M:cljfmt-fix

checks: check-format check-namespaces check-style check-kondo
Expand Down
2 changes: 1 addition & 1 deletion deps.edn
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
faker/faker {:mvn/version "0.3.2"}
kovacnica/clojure.network.ip {:mvn/version "0.1.3"}}
:aliases {:test {:extra-paths ["test"]
:extra-deps {lambdaisland/kaocha {:mvn/version "1.67.1055"}}
:extra-deps {lambdaisland/kaocha {:mvn/version "1.68.1059"}}
:main-opts ["-m" "kaocha.runner"]}
:eastwood {:main-opts ["-m"
"eastwood.lint"
Expand Down
7 changes: 2 additions & 5 deletions src/battle_asserts/core.clj
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,7 @@
(let [generator (build-generator)
size (count samples)
coll (gen/sample generator (- max-asserts size))
generated (map #(hash-map :expected (apply solution %) :arguments %)
coll)]
generated (map #(hash-map :expected (apply solution %) :arguments %) coll)]
(reduce
(fn [acc task] (conj acc (into (sorted-map) task)))
generated
Expand Down Expand Up @@ -51,9 +50,7 @@
(defn generate-issues
[issue-ns-name]
(require [issue-ns-name])
(let [issue-name (s/replace (last (s/split (str issue-ns-name) #"\."))
#"-"
"_")
(let [issue-name (s/replace (last (s/split (str issue-ns-name) #"\.")) #"-" "_")
build-generator (ns-resolve issue-ns-name 'arguments-generator)
solution (ns-resolve issue-ns-name 'solution)
disabled (ns-resolve issue-ns-name 'disabled)
Expand Down
4 changes: 2 additions & 2 deletions src/battle_asserts/issues/students_discipline_count.clj
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
(:require [clojure.test.check.generators :as gen]
[clojure.string :as s]
[faker.generate :as faker]
[faker.name :as f]))
[battle-asserts.utility :as util]))

(def level :elementary)

Expand All @@ -21,7 +21,7 @@
:output {:type {:name "hash" :nested {:name "integer"}}}})

(defn arguments-generator []
(letfn [(student-name [] (s/join " " [(f/first-name) (f/last-name)]))
(letfn [(student-name [] (util/gen-name))
(student-names [] (repeatedly 30 student-name))
(student-disciplines [] (conj (map s/capitalize (repeatedly 3 faker/word)) "Chemistry" "Jedi" "Magic" "Programming" "Dance" "Biology"))
(student-info-gen [] (gen/generate (gen/hash-map :student-name (gen/elements (student-names)) :discipline (gen/elements (student-disciplines)))))]
Expand Down
4 changes: 2 additions & 2 deletions src/battle_asserts/issues/students_name_slugify.clj
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
(:require [clojure.test.check.generators :as gen]
[clojure.string :as s]
[faker.generate :as faker]
[faker.name :as f]))
[battle-asserts.utility :as util]))

(def level :elementary)

Expand All @@ -19,7 +19,7 @@
:output {:type {:name "array" :nested {:name "string"}}}})

(defn arguments-generator []
(letfn [(student-name [] (s/join " " [(f/first-name) (f/last-name)]))
(letfn [(student-name [] (util/gen-name))
(student-names [] (repeatedly 40 student-name))
(student-disciplines [] (conj (map s/capitalize (repeatedly 10 faker/word)) "Chemistry" "Jedi" "Magic" "Programming" "Dance" "Biology"))
(student-info-gen [] (gen/generate (gen/hash-map :student-name (gen/elements (student-names)) :discipline (gen/elements (student-disciplines)))))]
Expand Down
10 changes: 7 additions & 3 deletions src/battle_asserts/utility_tools.clj
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,7 @@
nsf/find-namespaces-in-dir))

(defn- prepare-namespace-name [namespace]
(s/replace (last (s/split (str namespace) #"\."))
#"-"
"_"))
(s/replace (last (s/split (str namespace) #"\.")) #"-" "_"))

(defn- present-untraslated-tasks [task-list level]
(if (empty? task-list)
Expand Down Expand Up @@ -129,3 +127,9 @@
(if (empty? @disabled-list)
(println "There is no disabled tasks! Yaay!")
(println (str "Disabled tasks list:\n" (s/join ", " @disabled-list) "\nTotal number of disabled tasks is: " (count @disabled-list) "!")))))

; TODO implement tool for issue generating
(defn generate-isssue-template
"Tool that generates issue template"
[& _args]
(println "Not implemented yet!"))

0 comments on commit 1b34b48

Please sign in to comment.