Tests on slice

[henriquemoraeszup]

Description

There were some changes on this PR besides adding testify. Some of the methods from the utils package were used in a single file. Such methods were moved to their respective files. Also to avoid folder nesting, slice_util.go was moved up one folder

How to verify it

Run slice tests and check if they pass

Changelog

Test refactor on slice util

[codecov-io]

Codecov Report

Merging #808 (10a453f) into master (13b1f44) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #808      +/-   ##
==========================================
+ Coverage   82.52%   82.63%   +0.10%     
==========================================
  Files         110      110              
  Lines        3806     3806              
==========================================
+ Hits         3141     3145       +4     
+ Misses        481      479       -2     
+ Partials      184      182       -2     

Impacted Files	Coverage Δ
pkg/autocomplete/generator.go	50.00% <ø> (ø)
pkg/cmd/root.go	75.83% <ø> (ø)
pkg/env/setter.go	89.47% <ø> (ø)
pkg/formula/formula.go	75.00% <ø> (ø)
pkg/sliceutil/slice_util.go	100.00% <ø> (ø)
pkg/cmd/delete_formula.go	74.68% <100.00%> (+0.48%)	⬆️
pkg/cmd/formula.go	85.71% <100.00%> (+2.52%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 13b1f44...10a453f. Read the comment docs.

[kaduartur]

Keep the sliceutil/ package inside the slice/ so we don't have "util" packages lost inside pkg/, this is a very common pattern in Golang. An example is the ioutil package.

[kaduartur]

I liked the approach of removing functions that are specific to their respective packages! 🚀

[kaduartur]

/merge qa

[ritchie-bot]

👌 Merged branch hm/tests-on-slice into qa