notid/integreat
1
0
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases Wiki Activity

added validation

Browse Source
This commit is contained in:
Bryce Covert
2019-05-12 09:27:03 -07:00
parent 2146073df8
commit 51691fa553
2 changed files with 62 additions and 47 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/clj/auto_ap/graphql/transaction_rules.clj
View File

@@ -3,6 +3,7 @@
[auto-ap.datomic.transactions :as t] [auto-ap.datomic.transactions :as t]
[datomic.api :as d] [datomic.api :as d]
[auto-ap.datomic :refer [remove-nils uri merge-query]] [auto-ap.datomic :refer [remove-nils uri merge-query]]
[auto-ap.utils :refer [dollars=]]
[auto-ap.graphql.utils :refer [->graphql <-graphql limited-clients assert-admin result->page]] [auto-ap.graphql.utils :refer [->graphql <-graphql limited-clients assert-admin result->page]]
[clj-time.coerce :as c] [clj-time.coerce :as c]
[clojure.set :as set]) [clojure.set :as set])
@@ -33,6 +34,10 @@
#_(assert-admin (:id context)) #_(assert-admin (:id context))
(let [existing-transaction (tr/get-by-id id) (let [existing-transaction (tr/get-by-id id)
deleted (deleted-accounts existing-transaction accounts) deleted (deleted-accounts existing-transaction accounts)
account-total (reduce + 0 (map (fn [x] (:percentage x)) accounts))
_ (when-not (dollars= 1.0 account-total)
(let [error (str "Account total (" account-total ") does not reach 100%")]
(throw (ex-info error {:validation-error error}))))
transaction [(remove-nils #:transaction-rule {:db/id (if id transaction [(remove-nils #:transaction-rule {:db/id (if id
id id
"transaction-rule") "transaction-rule")

18
test/clj/auto_ap/graphql.clj
View File

@@ -32,7 +32,6 @@
(is (seqable? (:transaction-rules result)))))) (is (seqable? (:transaction-rules result))))))
(deftest upsert-transaction-rule (deftest upsert-transaction-rule
(testing "it should add rules"
(let [{:strs [vendor-id account-id]} (-> (d/connect uri) (let [{:strs [vendor-id account-id]} (-> (d/connect uri)
(d/transact (d/transact
[{:vendor/name "Bryce's Meat Co" [{:vendor/name "Bryce's Meat Co"
@@ -40,8 +39,19 @@
{:account/name "hello" {:account/name "hello"
:db/id "account-id"}]) :db/id "account-id"}])
deref deref
:tempids) :tempids)]
q (v/graphql-query {:venia/operation {:operation/type :mutation (testing "it should reject rules that don't add up to 100%"
(let [q (v/graphql-query {:venia/operation {:operation/type :mutation
:operation/name "UpsertTransactionRule"}
:venia/queries [{:query/data (sut/->graphql [:upsert-transaction-rule
{:transaction-rule {:accounts [{:account-id account-id
:percentage "0.25"
:location "B"}]}}
[:id ]])}]})
]
(is (thrown? clojure.lang.ExceptionInfo (sut/query nil q)))))
(testing "it should add rules"
(let [q (v/graphql-query {:venia/operation {:operation/type :mutation
:operation/name "UpsertTransactionRule"} :operation/name "UpsertTransactionRule"}
:venia/queries [{:query/data (sut/->graphql [:upsert-transaction-rule :venia/queries [{:query/data (sut/->graphql [:upsert-transaction-rule
{:transaction-rule {:description "123" {:transaction-rule {:description "123"
@@ -79,7 +89,7 @@
:data :data
:upsert-transaction-rule)] :upsert-transaction-rule)]
(is (= 1 (count (:accounts result))))))))) (is (= 1 (count (:accounts result))))))))))
(deftest test-transaction-rule (deftest test-transaction-rule
(testing "it should match rules" (testing "it should match rules"