New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add unit tests for all exported functions #18

Open

l50 opened this issue Oct 6, 2019 · 2 comments

Labels

good first issue

Contributor

l50 commented Oct 6, 2019 •

edited

Loading

Exported functions (functions that start with a capital letter) should have unit tests.

l50 self-assigned this

l50 added the enhancement label

l50 added good first issue and removed enhancement labels

l50 removed their assignment

tlabruyere commented Jan 13, 2020 •

edited

Loading

list of function that are exported:

TestBackupManifest
TargetAgents
CreateFolders
FileExists
GrepFile
GetFileAndDirList
File2lines
InsertStringToFile
LinesFromReader
TarFiles
ReplLineInFile
TrackChanges
RemoveTracker
GetHostname
GetLocalIP
StartServer
LoadSettings
SliceUniqMap
StringInSlice
CpFile
Cd
FindFiles
FindBin
AskUserQuestion
in files:
cmd/mose/puppet/main_test.go
pkg/chefutils/chefutils.go
pkg/moseutils/fileutils.go
pkg/moseutils/netutils.go
pkg/moseutils/settings.go
pkg/moseutils/sliceutils.go
pkg/moseutils/sysutils.go
pkg/moseutils/ui.go

Contributor Author

l50 commented Jan 13, 2020

Yeah, with the exception of those in cmd/mose/puppet/main_test.go, all of those functions need unit tests implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment