chore(tests): add unit tests to contract functions

[atticusofsparta]

No description provided.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (441cf11) 7.90% compared to head (e4cbab8) 81.25%.

Additional details and impacted files

@@            Coverage Diff             @@
##            main      #18       +/-   ##
==========================================
+ Coverage   7.90%   81.25%   +73.34%     
==========================================
  Files         11       11               
  Lines        177      176        -1     
  Branches      42       41        -1     
==========================================
+ Hits          14      143      +129     
+ Misses       144       33      -111     
+ Partials      19        0       -19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dtfiedler]

small comments r.e. test setup - but looking good!

[dtfiedler]

add a number, boolean, object, etc

[dtfiedler]

broad check, refine to controllers array

[dtfiedler]

you can just spread baselineAntState in your tests rather than do this and the beforeEach

[dtfiedler]

i would create a function and use it when you want the ANT state. you can then remove these lines.

e.g.

export functon generateAntBaseState = (params) => {
   return {
       ....baselineAntState
   }
}

and instead of

const _state = { ...state, controllers: [target] };

use the function

const initState = { ...generateBaselineAntState, controllers: [target] };
const result = await removeContrller(initState);
...

you can then remove these lines

Suggested change

	let state = { ...baselineAntState };
	beforeEach(() => {
	state = { ...baselineAntState };
	});

[dtfiedler]

the intent is to always guarantee a fresh state object that wasn't manipulated in other tests, so a function or spreading the base state would achieve that

[dtfiedler]

🚀