From 39a420f13e8000e2dfa53777b9e51594ba03e5d4 Mon Sep 17 00:00:00 2001
From: Christopher Peplin <chris.peplin@rhubarbtech.com>
Date: Tue, 24 Dec 2013 12:00:20 -0500
Subject: [PATCH] Add test cases from vi-firmware.

---
 .gitignore              |   4 ++++
 Makefile                |   5 +++--
 src/bitfield/bitfield.h |   8 ++++++++
 src/canutil/read.h      |  16 ++++++++++++++++
 src/canutil/write.c     |  12 ++++++------
 src/canutil/write.h     |  16 ++++++++++++----
 tests/read_tests.c      |  36 ++++++++++++++++++++++++++++++++++++
 tests/tests.bin         | Bin 24728 -> 0 bytes
 tests/write_tests.c     |  40 ++++++++++++++++++++++++++++++++++++++++
 9 files changed, 125 insertions(+), 12 deletions(-)
 create mode 100644 .gitignore
 create mode 100644 tests/read_tests.c
 delete mode 100755 tests/tests.bin
 create mode 100644 tests/write_tests.c

diff --git a/.gitignore b/.gitignore
new file mode 100644
index 00000000..b4225c09
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1,4 @@
+*.o
+.DS_Store
+*~
+*.bin
diff --git a/Makefile b/Makefile
index 8385c3ac..49e22f2c 100644
--- a/Makefile
+++ b/Makefile
@@ -19,15 +19,16 @@ SRC = $(wildcard src/**/*.c)
 OBJS = $(SRC:.c=.o)
 TEST_SRC = $(wildcard $(TEST_DIR)/*.c)
 TEST_OBJS = $(TEST_SRC:.c=.o)
+TESTS=$(patsubst %.c,%.bin,$(TEST_SRC))
 
 all: $(OBJS)
 
-test: $(TEST_DIR)/tests.bin
+test: $(TESTS)
 	@set -o $(TEST_SET_OPTS) >/dev/null 2>&1
 	@export SHELLOPTS
 	@sh runtests.sh $(TEST_DIR)
 
-$(TEST_DIR)/tests.bin: $(TEST_OBJS) $(OBJS)
+$(TEST_DIR)/%.bin: $(TEST_DIR)/%.o $(OBJS)
 	@mkdir -p $(dir $@)
 	$(CC) $(LDFLAGS) $(CC_SYMBOLS) $(INCLUDES) -o $@ $^ $(LDLIBS)
 
diff --git a/src/bitfield/bitfield.h b/src/bitfield/bitfield.h
index 27766733..7d9f3995 100644
--- a/src/bitfield/bitfield.h
+++ b/src/bitfield/bitfield.h
@@ -4,6 +4,10 @@
 #include <stdint.h>
 #include <stdbool.h>
 
+#ifdef __cplusplus
+extern "C" {
+#endif
+
 /* Public: Reads a subset of bits from a byte array.
  *
  * data - the bytes in question.
@@ -55,4 +59,8 @@ void setBitField(uint64_t* data, uint64_t value, int startPos, int numBits);
  */
 uint8_t nthByte(uint64_t source, int byteNum);
 
+#ifdef __cplusplus
+}
+#endif
+
 #endif // __BITFIELD_H__
diff --git a/src/canutil/read.h b/src/canutil/read.h
index 815f26b0..6ac4eebe 100644
--- a/src/canutil/read.h
+++ b/src/canutil/read.h
@@ -4,10 +4,26 @@
 #include <stdint.h>
 #include <stdbool.h>
 
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+/* Public: Parse a CAN signal from a message and apply required transformation.
+ *
+ * signal - The details of the signal to decode and forward.
+ * data   - The raw bytes of the CAN message that contains the signal, assumed
+ *      to be in big-endian byte order from CAN.
+ *
+ * Returns the final, transformed value of the signal.
+ */
 float parseFloat(uint64_t data, uint8_t bitPosition, uint8_t bitSize,
         float factor, float offset);
 
 bool parseBoolean(uint64_t data, uint8_t bitPosition, uint8_t bitSize,
         float factor, float offset);
 
+#ifdef __cplusplus
+}
+#endif
+
 #endif // __READ_H__
diff --git a/src/canutil/write.c b/src/canutil/write.c
index fdcba1f5..741778c7 100644
--- a/src/canutil/write.c
+++ b/src/canutil/write.c
@@ -1,18 +1,18 @@
 #include "write.h"
 
-uint64_t encodeFloat(float value, float offset, float factor, uint8_t bitPosition,
-        uint8_t bitSize) {
+uint64_t encodeFloat(float value, uint8_t bitPosition, uint8_t bitSize,
+        float factor, float offset) {
     float rawValue = (value - offset) / factor;
     if(rawValue > 0) {
         // round up to avoid losing precision when we cast to an int
         rawValue += 0.5;
     }
     uint64_t result = 0;
-    setBitField(&result, rawValue, bitPosition, bitSize);
+    setBitField(&result, (uint64_t)rawValue, bitPosition, bitSize);
     return result;
 }
 
-uint64_t encodeBoolean(bool value, float offset, float factor,
-                uint8_t bitPosition, uint8_t bitSize) {
-        return encodeFloat(value, offset, factor, bitPosition, bitSize);
+uint64_t encodeBoolean(bool value, uint8_t bitPosition, uint8_t bitSize,
+        float factor, float offset) {
+    return encodeFloat(value, offset, factor, bitPosition, bitSize);
 }
diff --git a/src/canutil/write.h b/src/canutil/write.h
index 85a5c1a0..1b086dbe 100644
--- a/src/canutil/write.h
+++ b/src/canutil/write.h
@@ -4,10 +4,18 @@
 #include <stdint.h>
 #include <stdbool.h>
 
-uint64_t encodeFloat(float value, float offset, float factor, uint8_t bitPosition,
-        uint8_t bitSize);
+#ifdef __cplusplus
+extern "C" {
+#endif
 
-uint64_t encodeBoolean(bool value, float offset, float factor,
-                uint8_t bitPosition, uint8_t bitSize);
+uint64_t encodeFloat(float value, uint8_t bitPosition, uint8_t bitSize,
+                float factor, float offset);
+
+uint64_t encodeBoolean(bool value, uint8_t bitPosition, uint8_t bitSize,
+                float factor, float offset);
+
+#ifdef __cplusplus
+}
+#endif
 
 #endif // __WRITE_H__
diff --git a/tests/read_tests.c b/tests/read_tests.c
new file mode 100644
index 00000000..f5f0f0c3
--- /dev/null
+++ b/tests/read_tests.c
@@ -0,0 +1,36 @@
+#include <check.h>
+#include <stdint.h>
+#include <canutil/read.h>
+
+const uint64_t BIG_ENDIAN_TEST_DATA = __builtin_bswap64(0xEB00000000000000);
+
+START_TEST (test_parse_float)
+{
+    float result = parseFloat(BIG_ENDIAN_TEST_DATA, 2, 4, 1001.0, -30000.0);
+    float correctResult = 0xA * 1001.0 - 30000.0;
+    fail_unless(result == correctResult,
+            "parse is incorrect: %f but should be %f", result, correctResult);
+}
+END_TEST
+
+Suite* canreadSuite(void) {
+    Suite* s = suite_create("read");
+    TCase *tc_core = tcase_create("core");
+    tcase_add_checked_fixture(tc_core, NULL, NULL);
+    tcase_add_test(tc_core, test_parse_float);
+    suite_add_tcase(s, tc_core);
+
+    return s;
+}
+
+int main(void) {
+    int numberFailed;
+    Suite* s = canreadSuite();
+    SRunner *sr = srunner_create(s);
+    // Don't fork so we can actually use gdb
+    srunner_set_fork_status(sr, CK_NOFORK);
+    srunner_run_all(sr, CK_NORMAL);
+    numberFailed = srunner_ntests_failed(sr);
+    srunner_free(sr);
+    return (numberFailed == 0) ? 0 : 1;
+}
diff --git a/tests/tests.bin b/tests/tests.bin
deleted file mode 100755
index 5fd0916e8c57bdc25dd7bcbd2ea25ce90be0e75f..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 24728
zcmcJ13w%`7x$oXHc}x<R2~R;#MjnbHAv{EUpc7zVA_=5G1RppVGLvL9nMvn?!HVRZ
zpk;2P$9O%~+tLH=wf?ZxE4{Xyz|lOwTd?+b8+$mC_SiOBv=gPLu~1Dp*t!32J@)Lq
zX9hgy^e)I=>wACeTWhbq_F8N9E_c;xi>fLzS(G~zf<_8m48^aC(&u5Iz%5g3ic6WL
zIF$lm^6}?l6v}x?=6yo7%x!|tiad|X7J;z1ctPaTIi$#3dPq6hODC5o@_f}i4$54C
zkCaG`eDw&rXI>p3f_eUw@Z&k@kB}pf<m!Z6o#2=G5Gh8IxvU@6jsB(!ebX&2Km@xe
zQ&zESI%lx}uI7W7pUGa!8S<`mv4WNhdwDYFY|57)M}B_MB>h~_9Ne~a$%1D8{N`XN
z-ZsB&+0yw-my|`rWs8{qrBFzw)~u^#s%1Q!fXL5@_@lNY`u$hDddv43`kr5w+&ujB
zHy2!8x1pexWaS@<UpO5D=lCq#4&a<w@J+yF<KLSD|E(PO_8j<s%YmPb%4M_rc@BQ+
zMo#>hQ*jPGzXoGA{jcW0-wM1Ff9CWy5ZUys1Kw$7CK&yqz9~@u5Osxe#ap?l+UpNQ
z0*%3FED+gLy|OtR3T*OiYYr%0Z(~b1<c-FBk(k%3c<<Wo-5~f@Hv6K{K$MGT6^=&Y
zp->>=jRs=ghH&H|QWJ|um1sN|3wY}z0beYjNG9-lea+1p9Et^^v8cDf7i<prHLMVd
z)%&6WZ-a15;h<E=wEO&iFGDNd`iH#aDJswsZPYY1L;|QDmt@Q^#{xq^S&E8NkVM5>
ze8G^irmAwqO7EhwMP)Z-#8eCfCcLVs%9Ut)y3Z)ZuM8H^ptuw`O0M~JET?<Bn3edY
zPT&gx3RXsF(&6d8l&MvRmtzgl+H`pRxZ1A6>&KNnIy~8y=_wsvjx9ucPKQUuv~)m+
z*VpTy4qu{iE6O1qevA&^t;3Ji;rn!W8Z%__bey$$I`iGvIu+03eX;!HH$m}qykR@e
zIZ`*>3Q}s$?YKH-xKJXPMi*sxD1|g<5y2Gl;Q<EUKrl7#a36zb5=@~VKE&WF38s(_
zA7Jo!f+^I)PcgWdU<&bYJA*9*7ZTjc;0tpAQ%HyF82m?qDU`!C4E~5<3gNJe!S54H
zp&Kq|@Gl9bkPSNlFZ_LF;_U}KiT6DnXGUr^Ri5Z8M=d=k`unb?>_o?vE+yFr#`6tN
zQT@ArgFx1L<_{wCbd1_Pi4PvOY(3r$eJp_IxAqb#HRPDlPPlG@B!B;?75tvWF;D+T
zcY4$T&)cJM2p=Uz&*MLjo!{z7{FuwXFXY>oSHNE-KCTv$$;aTx)*t7W0i~WhPX2g0
zmLHf2hS=4b3lnR5x6Ffgi4Rf(TZg{_5yuQ=ILdQUgEn_2cAh;-LGSTENv{`|<TZau
zrFx-CNzTB?=t;OoyCIYE+~OW}Jk#em+*iR&9BBjdQ3@!NO`KrZ<YGv_d0elax*1o$
zpVS}w9J)@pN85q+bh<~6AXko3$LXPxCvhVA`8Z~G#M7~JM0NbV&Sp2+oIt$fue8|<
zm72|Rs?rsbrS~1OxsGfy|A23&D&?@;VX(|<>*=f>^_*BWo87&h&e{=AV&d$|#G4h}
zNvJ+D#R-q5OLNkr`Ff9BiJk8rbxM}rTOg5q?IL-k=G|Lcz1fmy^oX(2z(=w81O<>s
z5V9mO!|@T%ZSEv*3h5jhCgEJtlh`yHH?q#l8kjrMPF`LoU65XW6;5bzV_wo@n77sU
z$;-*iOW>5$#p-iLFUe$&^m47v%hW}9{Ga!7k#s?N`QE?lykxDrRI>EGPF}WxL2KO$
zMlbV-6@M(2-$hqsu|&Ue7z3Eux&st9<r>88Hfc_Jw4MxVU6+}l3pLjYor&VhnV1Xh
zN4AiOJIp4|UeKE$@3dO{#x82{H#<izfMB#Dd%|dDPiQA|hsa!o+1yT;(|Q7nv0JkA
z{u~m?G31Z9gD#A}M2st>cZiRtKY6<+(a&#VelumgkyxnNjkH!;m7zZKXMG$Dp+1f0
z8f#5%X<5gRLm0D&kYqKMq$o_8i72c$8+$m@7)74i^+BVtC23=_BV{$_f*sz8HkyqU
zXBwlY5gY$d(ABJ*s&zE6hbK?sr028u@#MfWWWduul;^pAz;k`SC-D=Cv+Q$eZ>lk1
zjlqERc13q9dBghKX0tbsj_Nu98w&1|EWP_6mMj57t`0y?U9Ee%($2%OJBfj7Xbu=~
z!z3|q4QbsKl=P@Nq>ZYFVC_f}m!n=%s~_i2e`a>FUb6HS&w`T;VBk((5Rb0(ZOZN;
z8R2i@xg{d3IZ2P8FY4r6SmRDMN)Kd@U8HxiF0u2qqYp}!-a2x!|5G@5!rcd~;@Ru&
zJ3@xpv%iPAfZjGh?sbAhJjQ3^dzjL|bM^pZHNbO$iJ3=SJTcPY#dASCv|8Z;_dE>U
zhVZs!b&Qm2j@_K;nDp6WbZiJ!92AZXq1^|yb_XTx*kBgNqGrc_^CwfpXdF0dB{mGV
z2*-)!Kah1TVr-lsI+0#M_tZsf0FIqxVx8^*ICfHV43xBECpE_oY7uKAi~0_^8jcx<
z%||6m?-0b2|8ib-$aB>fQ>?xAx<`)CAbpfBJtr{;k(0kBC((qw#Tkh%MEV1IF83d|
z^55(nC6V+1M_dZ|z5XO=sKvN1TG}=nE$zbPXa`x+`~5Y|El}Dux2U;}P9kR3aIn3G
zxrNyT^{)P;6Z29`IYz`lkO$dY`qRG9Ee2P~{wEdPO=SNd*-wy4y?s0EYjI<v4XsBo
zmfl}LBKfUzrnn8^P8m8v29DA_+(Yl|6r2|*IGrE?79@F(|07-&bnVV&Q1ekSRu}yY
zD#x8NO8zHj!vCkC+u;8?xTpEg9+%Hbmfl5k;Qz1wX!3sq{*N4?nKqv?OaDJ4C$)BG
zGjZHvtZJVXNdFP=j-8|9(}C6+ovG0`dJU`}X~*U0x1>jMVEP}I^T{F%Z^-Ray~!ut
zlUDPIsmbt(J#n8m`;-r#WFX&YPwd=x^q^$v{SIo8eCCr>s#`z#)SfeQ;N>3oXFqm-
zsyGh2KSN6=k$;MO2m$?Kj~ZCuE&Q^yDudMwyHGRQLqiF5w8N-5H+N2$8x-aSz&D72
zWI!882K58UfObP3(#-w9Z0_?BVUBtLnOmcos}|-?3UhrhcM|6MG;=5Q=K8Xj`;pn)
z*Q7aEVde&5m)Gv1FxM^29fG-Tm^-AI>(-k)l*Js~i-y}}hBSvJW9EXIxs)(>P?$Ra
za|dDWfM)KX-rRvK=3Y0OJM@Qir#kEG9YNTWPXb4SLGPal`U{8Huy!Jk)+&_>HiC#(
zTfmrp;PtL&$mU`m?;{D^r#ZRlJ+tT4BF-JU95=lQ_uMJCU6<pgx164?60Upv<+$n1
zt>@!n&VA<JF5@@7jrO<&_u<QN)4M>=pT=<AdoIVlZzi}uwR7%!FUL(YxSnr1IQNa0
z<EE)pkE@h(|K<0WsU<y(du|-hxd$%CO^=`+{{+tcoy&34fZsEFBIk}>j+;ico|Z|R
zdo|-`k5@dm-=Q0SC08&iN5!+w<RV(V(58GRlI6rgO9JSRJ#4gv+*i)>5NL=;1cH=4
zhH}DvmIMfV1z^X{vkG5Yc@qmrSei<FfQ6GpJc+Thv-YfdEHThOI-@^%m3pk>{le=3
zCfpZ0c3xCs`JL{IALn=D!K-X#3q6UQ7lC5aRJ^GC^ZWE((vej4#Jx3zkM}t|$FTCH
zp%d{@46vQG7u5l9XK-M3YMCdob1)G<m8cy^oZRF7X{Y-Xyng|UQK)@q?Et~=0P}r@
zIg26WryV2e?!H(FYC7m~U=D|DFaGdQpt{!&fG~s=H^*T-H2Ns0K;Py*8(WOLqfbq^
z&wuRn^an<5o$m7)>_zU;4)=K#B#1mVsPo`;ta=@#h_FB*2rW=N0N36H!86M(nsXy6
zH5Zi{@+9JeiP}?%>Va+-q~^I#%>%ppJXBK%1B1-Z&M92U*GWm@m>{zFem3B_=C~*E
zOU9Qd<?M>5gz9=sB0Jugdmj%sY>p=4=erA;vGe^yR+=i3sgKZFTaP~_7VOHwm`r?<
z7&zw`=YAKL>cKtLXZGN&?Qkvr;%A^b$zE8fAJsCy#JkAjeQky3JdPXsKAuD_y!}zf
zkm@NqMyO-w99#O%Iqo5iTik=B`xLdE`_wr{sr!`H)h}UvC$rvJJJ{)d7c<Z#f(O}Z
zO-BOjK0iw?MMp?$zv$T9=O}YubbRAjHx2O;Ty-pMk+{!NJ(IZ9o=IFtjO?i$+C!d}
z65)O~2oJc=5Z^_@LFt>0vi|Lp$vk25-$qbTuYSgYmx~dYLhq`5*KxRdh>P6YN7KK=
z|3%Q4C&@q1EsZXxIpQSgQCzCeQ}|tbc3#|5J+jA*P94Wz^$0pQtliMpS&L3Gn)q3w
z7LJwHCNb?HPm;+SP<OhOcawRgVsk|zRdIJ^;?s)S%EZ4^Y^vHjr*(`I_xaxWly9uM
z{)2`8>goU7N-O`#&wmGh9{+vpYAIip_^2xJ=_=?*O?=nWaa{G>@^Snl+7P&JYsLK)
zTPq%@@E&hyD3ko>jyF6-n&g&&DmJj$PSk>J!B|5u(2Q**+L$V<S19*3`y!12FK8{k
z=tInFMOlqKt9T^fye<-m#+zf#YnD69+Zsf=&Ka(+2i4EzvC~G<-cUH^jXvyaZ3V$B
zzkIoKVRkWYaCI;ejX9wu>I{ZpYR(qt!@ejl1Vh-<^ZA|O2EYqBe+AnO3x-3Qhp+(Z
zbxyI{#%i{B5p%577l{U>9aP^L3B)47!1jRulE&uv(SF(N1ln2y^|65eZeMdeV6?b!
zsm>zvfkkHrb_rSH^<45rYPy;Pvf3|iTdi5;YL)+h-ik};tuX7YXl^$7icQum%`#^T
zcO;j;X~irkE$CB0?=~$g%Emx_IOKO`3CtX7BwYt!?&pfjBD4QF<6>4T`qdZrV-fHC
zjo290)h46<%a@n8ExJj%bt}3a-A~Jx&w0>)BjUK|CRWtfhc<iLoaoL}*dZm;*1`=9
zEOOp$;dsdJi|okSo@BRNyB#v@mM`RXS!=M(Z5kJ{^f%IzTaD0?L#{zOpJBd%TgN^b
z4M1ey9}YxAF>ifyc-uDYw+Gs=ksk~-di}nb4;5TZ?tQ`gmPQZW8?*RDfh})a(onxl
zI>%Vu;+A&Shg-G<Ll_yh)6EUmW#vdq=)#k8TRi4$49D0_pc&G&J9*5lm_@6qYBi%;
z#j`f2zCXwwgLYWtm+~1A1<ll=VH_U_#ak$vVe}Xbty&xra5?hL*+T!=U~9Z3X!Oe&
zzlG@=ja4KRYvOmaE)LRfIa=m$U6Q%?aFacul=^T)44HST)2z4AA%Yq3XUu!4R29l+
zkxxhd?)#}!4Eb%pO{Jbe{tEIRBCj};N}WRfJLKn)@BJW^nut02Cm*I#OOXFDnM!R!
z{`FzVBftG4$RnSNDfthP*I{md3b_;SKj)DL@wPV+E15l5k640y7go+TAwP>ehI}?w
zAfG{Af&7Qaw;(@-{1N2mk*~v=?!-K-5F)=F`B~)mBYzO9)Q=*67x_Nq0W9yohWs_;
z?;^huyA!kVmQLrGD5;NZP}H_kb=sIh+f!;`DdB0?^W;z}bptVol${@bf=>M1ze%Nr
zh}T}a+CJqjN6EvscID11Z@Xdf?3sino>lm(L!4>-U#YN{K4Doow!m^PEOWXa=>4F_
z4D_%@-wpgZ&}rE&Ex%WzKM!;O?e~I#{%wu^2JjzWf_?_{FQdKvXkSgg)vJ;M@}JJ2
z{Rr)^qW#kJiyEEQ8=ggf_%6sA-Eu&)zY6qen2_b7lidTLpGB_8TYsnN-wk{-`qwf8
zeX&M=9{AnpcYOx?|D@S}1N7M+r&5&$`Bj?y8Q`nYUuzBZtFdVH2mWu-fBOye*EITU
z;Af#fKWMNY(Cn`Qy%qFjdU}+RS$zGV?*aWPJ$*}>z8mz%L8n6iT6{L7>Cc1yDCjfw
z^sQ<78=yZ1`YAo#@-xOT8@Mx|FT%nle*lX9>F%&vco!ro=I0pPe;wc>8O;xG+Rlxj
z?*aWWJ>9Z{F;G9M0{uhKBL;elMsEgv8CIfcd0o?Q*{#u^1pV8fW4}O0zl+gV0re8-
zgP^nVZiT({?<{Wnl&@LccIVzaw|(xD`5t@u69pdovW~)PyQ|s0tioPiVV}Fg?p$G?
z0{#_t8yg>JoKY|yAuJi`5Z2EqJtcAm;6h!Ck~ya{UzBJOpVCNyi{)e4RZ*5;;f&H$
zc2%@<s?!8a@8FcK<e5@4ua1={o|Gj0Pk%{;39hraa7~|`os!4l0*1xrh=s$&0$w5f
zqeXg3Q+TG(u8BTTNGN%5?1z$!4Z9}+%W;xk$SJYDio84DMW7<_2LvqrmiCGmSK2<_
z#94jX1wUQ}87$+A6<7w}E9gS}|9d9S1Dg$9Tj*k_r6OM~^1DUeAoA@Ze?sKXiu?tU
z9~SwKMgES+KNPvy?wXY=Z*|Vasod+GOUjnvFz=1v;ElsWObfo0#?sbGmc;p!VCMWI
zSGpKiN=Diw!}$-PzX<;6oS&@{n0N`~{MVwt3xBTS{DWC|33Uc+d5mULhv9>IEr3^1
zqihwi*z$+KW3d(PL1xX%4~Aj{{hW}N!h0ZUQ40zPQVUW9S@P!s%KHYP%3B%T_BmyF
zg?owc2$gSMf@|JR7_~expAfdf4id2C|10tQ6c_80FN2Uz#k{-0X#M-w38rEJ-OG7a
zzj_{d;fug1g@43FEqsDB7M75BVI4^px`EGEZ1jY(ewogQD*03{s0KdII;744P{{7p
z!sSc_E@<AO`=LqAmmy&j3nK4(9Q?)JAZT>R$I^9wNlLW;9VCe~rjbO8XpD}7c`WfM
z_V1vm78BCa)ubpzbWC1#(1Jdt7`u}GUrYTCss91NL+3##*`LQn9Vb+A#p5|t2gTD!
z@fEbMK#9$0kgNM{2A(;7g;e}=!SiLtDHTsOD84eScrw+%Nelm!*i;C)y6+#PVkcKz
zRpJ*sk26lGc&0(|ETP!mOiE^RXbb$mo&29iyAhPqX7VvWGXLjEGsW27pky>N-(aT9
zU}izuOqk3pr7aRle$<VUy6*vLW~mI*A;Gg!@bLIAGnlzWn1TOqlNvYecu-<9I?DI`
zqg3pciYJbB5sUVV2d;RHL9r*TcrJN-A8n3M`YyYY;yZ9Nv%uUZ12f6tV$|J&hb!J{
zQ2c;UT(UwiZ<fq2qM&*cxLUfl*k92>H7FOksT<VOt_!8K8~;bTQ7_#%n$?Yf!Hov#
z#_zJa(P(f(K+KI}Fs?pG+c}ij#1^@_@6~(}jR(mMYu8J=O1y&UEkgq~lVxkyp<VnA
z3JJFy-xl059yL&CXE6zLXX<!N1hhqQ+&az$Rf-}d{#KKSa)}x)As~-Z^3Ns-uD^|Q
zyEr$4tX<u^pt-067xhuvj-n)|1nRyqn02tWe^hp=wSwmgLuDTmVdiyh7j=c=7pXP(
z(#{p7K6a%r{&%T(uWZc@!LwiR@J{u2TvZKM+$9v3yd{`rNO%RGHUv;WxaXCkMsnj>
z+7hG0UL%mJ`_4)?o|W}mDR}51Ly5cboS}Z-NY`%>DLzQMXO!6c2Xb{^jZFmZAnztm
z?kagwFfGA-Mu}_wp3ucB`27F~maazoFmm-J4mCJl7NWAcyD=H#ReqUQS=>hgs!<%%
zC%d2_-DeJK5<lP)9^MNC<Wd}+CW((Ui64;!^%O25z(czf#|4wj)u=W1>j;-QC}ae9
zV3$%7yuw9}X}y{Azs~ue6Z`^viaKZMIvSYzpM>sXLYBS9fu!yu?6|0+9-{)DeD?PU
zef((#)hXj`{~Pdl+jmHhi`wcmwFjM@r}QAZlAiY<V5d_F>Y(Ter1cj<>$qq{QGZE1
zmM&6yp5*AIEKf<HB<4Bo7=dY?W1fI@!me@lqRn&I<VVb9HbIj`<#QP^^D#_mVnpZj
zq<mV4QElX@G`0+7O6tB7xE7Sc7tVkwL;<~*NoY%95kW<~n$1c+n@)@kLTc_b7p{tO
z6}ytI&rsshY?@)|0)Himk0Wun;J-rSzYbO9wiyHCpGy3bi2r`UKU3omX#9);{M?#=
zniSh}Bt4b9drU~5(xfLN3I%Mo!Wb-F9&URouiGlFOF*uIa;A!etX;3|Dx|kV^-9iK
zS9q47X&k!0Fh;D-tAMj~)f8cqL{YCHAJD^!KM{T`7Jd|AvNoFw`l$sik3hJ@RtfT$
z*`#OjhUqTGyF*B*`@Raz-1Eg$gpq;|$$`b(h^WV6Lp=oKD!iMNE|C^XY$P+*Psm%z
z=v%_tY|UE13P$9*n+k4akd#*<XoEcEJzPaU{t$H5kA<QUp{Rm9HO#{Q%l-H*a>e=y
z42%e4k7-^LmdD7-J>OFBJ;qLzv~+DMIIM$e3f>_okAy5;RRw?6L7u{25tJ|Lk2i(9
z=Yh!YA#8C76%SqIf}Em6*cI4)gyqJI4aSQN#swsdml%u-$WzD~b_}OA%l@J4Vo~3K
zkcz0=F~4DNNy3*w;mgZ^{ix4`DU}6vNEl0K6(2(tXD(Z*NTUafrGU2ZI16Z#p+#zV
zi{$YTRf(u`EzO1e1%(H*&c<$x66%9RFd`a+UK>>hu>`MnF^|@>D>cYhG`}2Fq>+O8
z<>308xL*SDbH7TtCw1H}0o|`PfjUl|W1moBwNJB;Q^yyUVi>c`R!U0l019J-C}Pmh
zo2*#vx0c*iQaJ(LW&(iQry@#us?}an0(`y-eBoT+r;kN@3hsR3E-C{PiC#CAY`CU8
zyJ9MF7w0Qh8&_$Wg5jbB&Wy(scTAaj=Ts?CIpKE3JT_k;UG^oK8zkf4GNi54C}prT
z4pKTx;~g+Y+AxOyr>dXL)Yo~X3Q#E#CvvdR=&voCqzFfm(aB;NjuPpuAufjcjm@SO
z4*hj&DEz#14gZ=U{XzzHA*H^8?uM`{%2(+IQNBbGQ2*UN7FQJ*TANU5k1DDxh({v}
zf}#56xF1tYxznVbg#cqT=Eq7*Sre^9@n09SA}m<sn#JUoY!#<1<rB+avfa^PJ8ij<
zl}@9yY7*g4>af&I&MaD1O(Z6|)JRUt+DQP{X_v5+vuxJXZl1tR?FZ-P2^}_;Pn0>g
zL%8*mPuuoeHVX}$D^0B;D!8f`z00<IEllLPmUo;!Z7G}Z1^?U1*Ou?@I&Yg`oAALu
z?YCGEc$$7%lr`C!A{wdJj-lje5u8ggwFNbRobpZGZ+jVIr&`c}t7Qu=)`EM=<F@7H
z@zdq|ZTFPh8sD_deEENM*cu-v4q~?4Ll>JLSGC}-f<3l3PcN}Gp6;?Nm|$DG{bgGt
z^SX(7y&{*_^}bL%7Hl>-jbDKKb2u!X?UQUD*j}^TJmK564{)`0Fk7@fV^LQA$80?u
z!H>?f8<UT;Nw#yg<v+3A+hLhCVeLX#oIiotTe;S|ZvE=@8`df)QSM#i-sE0)x6&Fx
zkN1Y-vDSEu?Z!v7=F#p9Awr^66&p7xh9AN=g(ER%JQNK!(ylozWXc^~c05Lj1GRLC
zmn*;gRzExaz<%()L*cvEbd*RQHA<gUVB5~RgBP)SFWWMf#>v_0%5|016;+DnU44@;
zLWH~B8&<5}=;o&n42OiGtU6Q+<Gwcg`8-3>fsf2Rdnl!7G7fSw%jP@2NYLnH7tJ5C
zL-ARL@bK+&ora8K9ve5XZxJY)R^rP6N;LRLAhx45prrMZIxML>SMoJ==lqsfJQP^o
z7{C_{g7pA>k@}|P{1Y0^`HlGB=-<XIiZfgsFr8^(l`<S`l4Iva`c46tF&-jgBHGC(
z_|@WTLmd&Nb?Y}!lcX6;2gks&v9ij&Zj-X0DcllR&>CpPmp>Nx1KSs%+xZu456*80
zMp_>BMFI<GEWsxZq6_4$$<AyjfslU%oh)hyhWu`ngE0*3`1k`u5>Z2-02s$i7zynL
z5<6*vr$Zw|gT9bLhm_XETa;)x9;rvzNs}hg6l{pC!bufh2qYYWV0zGPp(n~D@ok9}
z;V^v?B0cPBHyCbZgj%!N&>Z%~=($uAjs{~m+eYP$Xl8{SU?V*NIK+U}jJLs89}7p6
zi0@&3uaOlIiv?m}LRUq-P|f<VKOj};`Q)I&Ik;Y>EV`p5=G%rm7U6l5ETJ(2kyfP)
zr~LwD6)P&|$9#<fY7E88w#9?^SWD2alo8bAi#91`{v9Fc<9RH?Dcb{)C{>ZM3BU_l
zB+%?50a0pgjwxlq5E`m17HGp2jT!_k%+3Xs1)97K5noHd+vJC=gt&y)7m4_Ga0yv{
zupVsC?rRCw!*m!5aVcX~xW;YKs8WVQKrI2>OlAJSws<3ieW6BtCPaY2P(xUjecQH0
z0^22oo*Iy4#0#VQe>F{m0-D8OLc){9C32q0{ZtNdURkDm7XC27*5s#KT!64IE)+T$
zl)m6y#E=#O(#zj&aWOjWCs3NrGwo`YUj+(|y3oD=OLZ2P$XUkr=c$CnU6%0PVMyx$
zmEcdxw_99*6zm7HOnHuigM<}L7AGmsW`w}qjX%8bYVvYld5UmU?jwVRC817&$^$6T
z9<H=6_igVL4$l;v(tpV(^M^pEJz)vA=ew9hyAW_PDARv|R%l<3T$S<%g!};^FY77o
zo9n;ZBro^1yM=&U^d@i&{><`U1CC-s@ueis0gMQQ+~4O^?yD?Qp1_}8zO>LqXr+`o
z719v9rZ1(UDD!6k>*Ys;{D_cWC0F50a#G_rO!D&F!jc)B1MlI6B)Wx$JAe||m+^Pa
zZ~;;jmymx$28LnN@e`nO5Cy%w_MNP2IY%B5b;)f1Flc&t+jTC+ZM%*O-EI<)?Z>Hn
z?*Xs%>>ePT{GOQn&X}3s4F41|4xPL_$5Atnv}M-UEI$Yy@}KTOO7tZZp0iBrZyd_h
z_bU)6#8O_KW02oF8xrx8%^>Z|{_`PZ_4Zx!>wrP~&1YFo`%=%xVAIRXb0i)i-_F%&
zS1Bj+^CtO|!v0AiFa4AXq@08=pg=fTfBJk4rNJEjo4NyAs3-r+nN<2B<;SH_tR%w8
z>P!e55E7?n9TxWc^hlJ=ytH9v#*xU2WnAz@JJ03S%sVTG{02H!fb<J_V{$OV*1{a}
z^ITluDjC=u^2>$1Iewm%T%gPpaOuCyDZVBXOmO=eF7J_y9L(^a$fhbX&URYoKw@83
zLOv6JZv4sqBcBU2zGbtYyblDVLpl8C)JtEj<gLY(!DNNUa-N|lTH%SDKN#_O+W9Xd
zK3_ZcWyBXK@;PqA7b^1kY{c6X`P?<)ixl}hHR6l4^It}MiT3<6;>X~5=F+2B*{Y1y
zzEi0uS{1u?e$I$@;JM<`qgdIBudm7HhMs6u#wqf7V8oBt#(5)tf_4tih@Ys9%SQYp
zd{?qgk78x3GFkiXo1SRJ={Y$*8u3#UIqn(pQ!$R!=~1j~)lLl>Nmhks!<osd%_5C7
ztG1<Q#$Rnj2*}D#m>WrS|K%y4q>NY&Wy90`mkrOBo^jexKL4aXCkhty6Q?-tB=I*1
ze2oe35qR@CPMW8xIE^XeDTkF7hR<wwnpcrsc`g(nO9yaOmCXLr3w*YE4FFGi<npK&
zFs0w(kMQQBSQiD}e3Gh&h7$0b4?;~7c=O36`hW!Sn~xgZCh+EyLTd%ye9*@$@aB^@
zj|jZ^C`>}&&4)%_5qR^NkDmy<`4Gm30&hM;LBCbV7H2xvm<?ZmheI}eu6}3-zlw3g
zB_Dv)SV;6fsAlP(M9Ze1j!of+LB@D;3)7Q1Zmbaea{fuul<4e;nq}O02>9`6U;6qV
zB_@viD-6$2hRc5g-f6&a;01=y9Djbm^o-3oThS}<=FRK37(Ua$Z-Fnx>0^1{Nsf<l
z@L$M*r;`igvpg3jGrZKA&S&MoFUo=cQVu*F%p?EJ<MTs|e{9AcaU1Z?v>RIKtH6)L
z_Sb-EJb#Apc!QA7H96h*pF+P}w@wI6uLykK)h>n!2z)Q_r5N9aO#0st{PMj}uq(91
zPW{KcrTPnomt5(5Sm-ym<DY~c^VyjTIrJ2xf@IfxsAN3Dr|rqoHG<!KNaRL=Hy;qV
z8F;4@*5zw+=-&c7wTro4^bzN5dLCnX#%3I+ctYUK$5EaZc=Mr{{S2QuU;3`#m%BO3
zWF2UY59^`zkytbq#}At8@%h@jR&MZCRc_pbulA<D?dy%T(60nTSdWum?A4*X_#vUU
z5kDgIHGBP78;yE>@izRJtfjR%fM4vEEnU2HAv9-U^3uv8Unlfp?GHbz<SUE*cuUI;
zi0B|MIcs3WLhWULSe<jpTTkE%<AyB>Mm8+Jrv+%4n?d@W5c)}Fwk7=RI~Ppcw68+o
zC4Z2q31#ktq)C~-KV&rRw}LWgXlt=pE!U#G-qjl_s@>R9Sw&xLW*tG64YBtt-c|Rk
ztEjGA2`(Llt<o#rHC5|ZR8)D_uU@^;y~(?&Vnvmk)UnSZ*GJ>5$;FqKwGT00>QmCR
zM9jaZtZg6jEfD?If&3aYi>dCD(cS=CVV2}ee=M52X86jq=_Ak?Uw3x7^q+T@X81Rv
z+0ST&=Z4J$^T)Bh{%F|S<O})f!`YSVA%g7@Z#;?(IQ<5JE`}6!1$PXZf=AuB)nNWS
zwbzRkcEQOu)p#A#-?~PxVBaB-6%e1q*6q{iHbAm{%-irKZLb&K0+;=WJ`yg+48teF
T<<5<MH^#8h!}^wfL+SqkE9#4Y

diff --git a/tests/write_tests.c b/tests/write_tests.c
new file mode 100644
index 00000000..da5ee89c
--- /dev/null
+++ b/tests/write_tests.c
@@ -0,0 +1,40 @@
+#include <canutil/write.h>
+#include <check.h>
+#include <stdint.h>
+
+START_TEST (test_encode_can_signal)
+{
+    uint64_t value = encodeFloat(0, 1, 3, 1, 0);
+    ck_assert_int_eq(value, 0);
+}
+END_TEST
+
+START_TEST (test_encode_can_signal_rounding_precision)
+{
+    uint64_t value = encodeFloat(50, 2, 19, 0.001, 0);
+    ck_assert_int_eq(value, 0x061a800000000000LLU);
+}
+END_TEST
+
+Suite* canwriteSuite(void) {
+    Suite* s = suite_create("write");
+    TCase *tc_core = tcase_create("core");
+    tcase_add_checked_fixture(tc_core, NULL, NULL);
+    tcase_add_test(tc_core, test_encode_can_signal);
+    tcase_add_test(tc_core, test_encode_can_signal_rounding_precision);
+    suite_add_tcase(s, tc_core);
+
+    return s;
+}
+
+int main(void) {
+    int numberFailed;
+    Suite* s = canwriteSuite();
+    SRunner *sr = srunner_create(s);
+    // Don't fork so we can actually use gdb
+    srunner_set_fork_status(sr, CK_NOFORK);
+    srunner_run_all(sr, CK_NORMAL);
+    numberFailed = srunner_ntests_failed(sr);
+    srunner_free(sr);
+    return (numberFailed == 0) ? 0 : 1;
+}
-- 
2.16.6