From 5ac8f118532e88daea374afb4c16a467aeec47ec Mon Sep 17 00:00:00 2001 From: Simon Michael Date: Thu, 14 Jul 2022 19:00:49 +0100 Subject: [PATCH] dev: errors: update tests & readme, now including timeclock test --- hledger/test/errors/README.md | 23 ++++++++++++++++++- hledger/test/errors/accounts.test | 2 +- hledger/test/errors/assertions.test | 6 ++--- hledger/test/errors/balanced.test | 2 +- .../test/errors/balancednoautoconversion.test | 5 ++-- hledger/test/errors/commodities.test | 2 +- hledger/test/errors/ordereddates.test | 5 ++-- hledger/test/errors/parseable-dates.test | 2 +- hledger/test/errors/parseable-regexps.test | 2 +- hledger/test/errors/parseable.test | 2 +- hledger/test/errors/payees.test | 2 +- hledger/test/errors/tcclockouttime.test | 9 ++++++++ hledger/test/errors/tcclockouttime.timeclock | 2 +- hledger/test/errors/tcorderedactions.test | 8 +++++++ hledger/test/errors/uniqueleafnames.test | 5 ++-- 15 files changed, 56 insertions(+), 21 deletions(-) create mode 100644 hledger/test/errors/tcclockouttime.test create mode 100644 hledger/test/errors/tcorderedactions.test diff --git a/hledger/test/errors/README.md b/hledger/test/errors/README.md index 68acf0965..6fc739605 100644 --- a/hledger/test/errors/README.md +++ b/hledger/test/errors/README.md @@ -109,7 +109,7 @@ Here are some current limitations of hledger's error messages: -hledger 1.26.99-gc22e9f6cc-20220713 error messages: +hledger 1.26.99-g32c7f6300-20220714 error messages: ### accounts ``` @@ -274,3 +274,24 @@ b:c Consider changing these account names so their last parts are different. ``` + +### tcclockouttime +``` +hledger: Error: /Users/simon/src/hledger/hledger/test/errors/./tcclockouttime.timeclock:5:1: + | i 2022-01-01 00:01:00 +5 | o 2022-01-01 00:00:00 + +This clockout time (2022-01-01 00:00:00) is earlier than the previous clockin. +Please adjust it to be later than 2022-01-01 00:01:00. +``` + + +### tcorderedactions +``` +hledger: Error: /Users/simon/src/hledger/hledger/test/errors/./tcorderedactions.timeclock:8:1: +8 | i 2022-01-01 00:01:00 + +Expected timeclock o entry but got i. +Only one session may be clocked in at a time, so please alternate i and o. +``` + diff --git a/hledger/test/errors/accounts.test b/hledger/test/errors/accounts.test index 491793f78..b31160c65 100644 --- a/hledger/test/errors/accounts.test +++ b/hledger/test/errors/accounts.test @@ -12,4 +12,4 @@ account a account a ; type:A ; \(L,E,R,X,C,V\) / ->>>= 1 \ No newline at end of file +>>>= 1 diff --git a/hledger/test/errors/assertions.test b/hledger/test/errors/assertions.test index 28ddcf7c1..986869c04 100644 --- a/hledger/test/errors/assertions.test +++ b/hledger/test/errors/assertions.test @@ -10,6 +10,6 @@ and commodity: this balance was asserted: 1 but the actual balance is: 0 a difference of: 1 -.* -/ ->>>= 1 \ No newline at end of file + +Consider viewing this account'/ +>>>= 1 diff --git a/hledger/test/errors/balanced.test b/hledger/test/errors/balanced.test index dbd9c341e..468b6868e 100644 --- a/hledger/test/errors/balanced.test +++ b/hledger/test/errors/balanced.test @@ -7,4 +7,4 @@ This transaction is unbalanced. The real postings' sum should be 0 but is: 1 Consider adjusting this entry's amounts, or adding missing postings. / ->>>= 1 \ No newline at end of file +>>>= 1 diff --git a/hledger/test/errors/balancednoautoconversion.test b/hledger/test/errors/balancednoautoconversion.test index e0741e3e1..96bd50591 100644 --- a/hledger/test/errors/balancednoautoconversion.test +++ b/hledger/test/errors/balancednoautoconversion.test @@ -7,6 +7,5 @@ $$$ hledger check balancednoautoconversion -f balancednoautoconversion.j This multi-commodity transaction is unbalanced. Automatic commodity conversion is not enabled. The real postings' sum should be 0 but is: 1 A, -1 B -.* -/ ->>>= 1 \ No newline at end of file +Consider adjusting this entry's/ +>>>= 1 diff --git a/hledger/test/errors/commodities.test b/hledger/test/errors/commodities.test index a382a75ee..aa038c444 100644 --- a/hledger/test/errors/commodities.test +++ b/hledger/test/errors/commodities.test @@ -12,4 +12,4 @@ commodity A1000.00 commodity 1.000,00 A / ->>>= 1 \ No newline at end of file +>>>= 1 diff --git a/hledger/test/errors/ordereddates.test b/hledger/test/errors/ordereddates.test index ee53796b0..281a522e5 100644 --- a/hledger/test/errors/ordereddates.test +++ b/hledger/test/errors/ordereddates.test @@ -9,6 +9,5 @@ $$$ hledger check ordereddates -f ordereddates.j Ordered dates checking is enabled, and this transaction's date \(2022-01-01\) is out of order with the previous transaction. -.* -/ ->>>= 1 \ No newline at end of file +Consider/ +>>>= 1 diff --git a/hledger/test/errors/parseable-dates.test b/hledger/test/errors/parseable-dates.test index 08d46dea0..3bda3832a 100644 --- a/hledger/test/errors/parseable-dates.test +++ b/hledger/test/errors/parseable-dates.test @@ -7,4 +7,4 @@ $$$ hledger check -f parseable-dates.j This date is invalid, please correct it: 2022\/1\/32 / ->>>= 1 \ No newline at end of file +>>>= 1 diff --git a/hledger/test/errors/parseable-regexps.test b/hledger/test/errors/parseable-regexps.test index 344d32721..cb7aa167f 100644 --- a/hledger/test/errors/parseable-regexps.test +++ b/hledger/test/errors/parseable-regexps.test @@ -8,4 +8,4 @@ This regular expression is malformed, please correct it: \( / ->>>= 1 \ No newline at end of file +>>>= 1 diff --git a/hledger/test/errors/parseable.test b/hledger/test/errors/parseable.test index b639ea8f8..648860fd0 100644 --- a/hledger/test/errors/parseable.test +++ b/hledger/test/errors/parseable.test @@ -7,4 +7,4 @@ unexpected newline expecting date separator or digit / ->>>= 1 \ No newline at end of file +>>>= 1 diff --git a/hledger/test/errors/payees.test b/hledger/test/errors/payees.test index b891ac1e3..5dc44aeed 100644 --- a/hledger/test/errors/payees.test +++ b/hledger/test/errors/payees.test @@ -11,4 +11,4 @@ Consider adding a payee directive. Examples: payee p / ->>>= 1 \ No newline at end of file +>>>= 1 diff --git a/hledger/test/errors/tcclockouttime.test b/hledger/test/errors/tcclockouttime.test new file mode 100644 index 000000000..8cba9b284 --- /dev/null +++ b/hledger/test/errors/tcclockouttime.test @@ -0,0 +1,9 @@ +$$$ hledger check -f tcclockouttime.timeclock +>>>2 /hledger: Error: .*tcclockouttime.timeclock:5:1: + \| i 2022-01-01 00:01:00 +5 \| o 2022-01-01 00:00:00 + +This clockout time \(2022-01-01 00:00:00\) is earlier than the previous clockin. +Please adjust it to be later than 2022-01-01 00:01:00. +/ +>>>= 1 diff --git a/hledger/test/errors/tcclockouttime.timeclock b/hledger/test/errors/tcclockouttime.timeclock index c59585d94..0aafd29de 100755 --- a/hledger/test/errors/tcclockouttime.timeclock +++ b/hledger/test/errors/tcclockouttime.timeclock @@ -1,4 +1,4 @@ -#!/usr/bin/env -S hledger check -f +#!/usr/bin/env -S hledger check -f # Clockout time before previous clockin. i 2022/01/01 00:01:00 diff --git a/hledger/test/errors/tcorderedactions.test b/hledger/test/errors/tcorderedactions.test new file mode 100644 index 000000000..48d373d0b --- /dev/null +++ b/hledger/test/errors/tcorderedactions.test @@ -0,0 +1,8 @@ +$$$ hledger check -f tcorderedactions.timeclock +>>>2 /hledger: Error: .*tcorderedactions.timeclock:8:1: +8 \| i 2022-01-01 00:01:00 + +Expected timeclock o entry but got i. +Only one session may be clocked in at a time, so please alternate i and o. +/ +>>>= 1 diff --git a/hledger/test/errors/uniqueleafnames.test b/hledger/test/errors/uniqueleafnames.test index 8ac8dd66e..d6d2161e2 100644 --- a/hledger/test/errors/uniqueleafnames.test +++ b/hledger/test/errors/uniqueleafnames.test @@ -9,6 +9,5 @@ $$$ hledger check uniqueleafnames -f uniqueleafnames.j Checking for unique account leaf names is enabled, and account leaf name "c" is not unique. -.* -/ ->>>= 1 \ No newline at end of file +It appears in these account names, which a/ +>>>= 1