Skip to content
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

Testing improvement #16

Open
7 of 12 tasks
ChacaS0 opened this issue Feb 23, 2018 · 0 comments
Open
7 of 12 tasks

Testing improvement #16

ChacaS0 opened this issue Feb 23, 2018 · 0 comments
Assignees
@ChacaS0
Labels
enhancement Improvement of existing feature suggestion proposal for a new functionality or whatever test stuff to do about testing (unit tests, live tests, etc) todo when it needs to be done
Milestone
v0.1

Comments

@ChacaS0
Copy link
Owner

ChacaS0 commented Feb 23, 2018 •
edited
Loading

Write unit tests

For cmd package:

  • add.go
  • get.go
  • init.go
  • list.go
  • rm.go
  • set.go
  • start.go
  • purge.go
  • update.go
  • version.go
  • doc.go
  • root.go

This might even get followed by a new command like tempest debug or something to see what's wrong. In that case a new issue or task will be created.
@ChacaS0 ChacaS0 added enhancement Improvement of existing feature todo when it needs to be done test stuff to do about testing (unit tests, live tests, etc) suggestion proposal for a new functionality or whatever labels Feb 23, 2018
@ChacaS0 ChacaS0 added this to In progress in v0.2B Feb 26, 2018
@ChacaS0 ChacaS0 moved this from In progress to To do in v0.2B Feb 26, 2018
@ChacaS0 ChacaS0 added this to the v0.2B milestone Feb 26, 2018
@ChacaS0 ChacaS0 moved this from To do to In progress in v0.2B Mar 1, 2018
ChacaS0 added a commit that referenced this issue Mar 1, 2018
@ChacaS0
🏁 Added test file for cmd/set.go
53528df 
 Issues related:
 	-> #16

 On branch devel
 Changes to be committed:
	modified:   cmd/init.go
	new file:   cmd/set_test.go
ChacaS0 added a commit that referenced this issue Mar 1, 2018
@ChacaS0
✔️ Update travis.yaml to handle test files
ddcfc3a 
 Issues related:
 	-> #16

 On branch devel
 Changes to be committed:
	modified:   .travis.yml
ChacaS0 added a commit that referenced this issue Mar 1, 2018
@ChacaS0
🏁✔️ Added test file for get.go
eae24e5 
 Issues related:
 	-> #16

 On branch devel
 Changes to be committed:
	new file:   cmd/get_test.go
	modified:   cmd/init.go
	modified:   cmd/set_test.go
ChacaS0 added a commit that referenced this issue Mar 4, 2018
@ChacaS0
✔️🎨📝 Improved a few visual stuff; Added add_test.go
d9ec05b 
 Issues related:
 	-> #16
 	-> #19

 On branch devel
 Changes to be committed:
	modified:   README.md
	modified:   cmd/add.go
	new file:   cmd/add_test.go
	modified:   cmd/get.go
	modified:   cmd/init.go
	modified:   cmd/purge.go
	modified:   cmd/root.go
	modified:   cmd/set_test.go
ChacaS0 added a commit that referenced this issue Mar 5, 2018
@ChacaS0
🎨🏁 Added test for add_test.go; small user improvements
76041e3 
 Issues related:
 	-> #19
 	-> #16

 On branch devel
 Changes to be committed:
	modified:   README.md
	modified:   cmd/add.go
	modified:   cmd/add_test.go
	modified:   cmd/purge.go
	modified:   cmd/root.go
	modified:   cmd/version.go
	modified:   list.todo
ChacaS0 added a commit that referenced this issue Mar 5, 2018
@ChacaS0
🏁✔️ Improved a few stuff; Completed test for add_test.go
640ee42 
 Issues related:
 	-> #9
 	-> #16

 On branch devel
 Changes to be committed:
	modified:   cmd/add_test.go
	modified:   cmd/init.go
	modified:   cmd/list.go
ChacaS0 added a commit that referenced this issue Mar 10, 2018
@ChacaS0
🏁 Added test for initTempestCf for init_test.go
90a8b22 
 Issues related:
 	-> #16

 On branch devel
 Changes to be committed:
	new file:   cmd/init_test.go
	deleted:    cmd/test_init.go
ChacaS0 added a commit that referenced this issue Mar 11, 2018
@ChacaS0
💊❔💻 Ajusted some stuff in init, root and added test for rm.go
e5cfeff 
 Issues related:
 	-> #16
 	-> #17

 On branch devel
 Changes to be committed:
	modified:   .gitignore
	modified:   cmd/add.go
	modified:   cmd/init.go
	modified:   cmd/rm.go
	new file:   cmd/rm_test.go
	modified:   cmd/root.go
ChacaS0 added a commit that referenced this issue Mar 11, 2018
@ChacaS0
🏁💻 Made rm_test.go; Fixed some path to use PathSeparator
4288769 
 Issues related:
 	-> #16
 	-> #9

 On branch devel
 Changes to be committed:
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go
ChacaS0 added a commit that referenced this issue Mar 12, 2018
@ChacaS0
🏁 Made tests for list.go (list_test.go)
65a509a 
 Issues related:
 	-> #16

 On branch devel
 Changes to be committed:
	modified:   cmd/list_test.go
	modified:   cmd/rm_test.go
ChacaS0 added a commit that referenced this issue Mar 15, 2018
@ChacaS0
🚧🏁 Added test for start.go + Refractored some test files
bf1ab1d 
 Issues related:
 	-> #16

 On branch devel
 Changes to be committed:
	modified:   cmd/add_test.go
	modified:   cmd/init_test.go
	modified:   cmd/list_test.go
	modified:   cmd/rm_test.go
	new file:   cmd/start_test.go
ChacaS0 added a commit that referenced this issue Mar 27, 2018
@ChacaS0
✔️🏁 processArgsRm added + tests
fb436d1 
 Issues related:
 	-> #21
 	-> #16
 	-> #9

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/init_test.go
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go
ChacaS0 added a commit that referenced this issue Mar 28, 2018
@ChacaS0
✔️🏁 New feature for rm
b35ca71 
	-> Added & adapted tests only for the ones impacted by those changes

 Issues related:
 	-> #21
 	-> #16

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go
	modified:   cmd/root.go
ChacaS0 added a commit that referenced this issue Mar 28, 2018
@ChacaS0
💻🏁 Added test for rm.go:simpleDelAllString()
856d8d8 
	-> Few changes (variadic params instead of slice)
	-> Some update for the ``$HOME`` on Windows

 Issues related:
 	-> #21
 	-> #16
 	-> #9

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/purge.go
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go
	modified:   cmd/root.go
ChacaS0 added a commit that referenced this issue Mar 31, 2018
@ChacaS0
Improvement of the rm command (#21) (#25)
adee1cc 
* ✔️ Mispells fixes

 On branch devel
 Changes to be committed:
	modified:   cmd/init_test.go
	modified:   cmd/list.go
	modified:   cmd/list_test.go
	modified:   cmd/rm_test.go
	modified:   cmd/start_test.go

* 🚧 Work on rework of rm command starts -- bad

 Issues related:
 	-> #21 Improve ``rm`` command

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/rm.go

* 💊 Fixed missspelling

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/init_test.go
	modified:   cmd/list.go
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go

* ✔️🏁 processArgsRm added + tests

 Issues related:
 	-> #21
 	-> #16
 	-> #9

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/init_test.go
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go

* ✔️🏁 New feature for rm
	-> Added & adapted tests only for the ones impacted by those changes

 Issues related:
 	-> #21
 	-> #16

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go
	modified:   cmd/root.go

* 💻🏁 Added test for rm.go:simpleDelAllString()
	-> Few changes (variadic params instead of slice)
	-> Some update for the ``$HOME`` on Windows

 Issues related:
 	-> #21
 	-> #16
 	-> #9

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/purge.go
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go
	modified:   cmd/root.go

* ✔️🏁 Added handler for ``*`` wildcard as argument
	-> Tests, were adapted for that

 Issues related:
 	-> #21

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go

* 📝 Update of usage for ``rm`` and deletion of deprecated stuff

 Issues related:
 	-> #21
 	-> #19

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/rm.go
@ChacaS0 ChacaS0 self-assigned this Apr 1, 2018
ChacaS0 added a commit that referenced this issue Apr 4, 2018
@ChacaS0
🏁 Added tests for shutup mode handler (start.go)
fc433f8 
	-> Issues related: #15, #16
ChacaS0 added a commit that referenced this issue Apr 4, 2018
@ChacaS0
✔️🏁 Added logs flag for get command
276a657 
	[+] Added the corresponding test
	--> Issues related: #15, #16
ChacaS0 added a commit that referenced this issue Apr 7, 2018
@ChacaS0
🏁 Test for getAllLogs() (--logs)
6d264b7 
	--> Issues related: #15, #16
	[*] Should be the end of the Linux part for this one
ChacaS0 added a commit that referenced this issue Apr 8, 2018
@ChacaS0
🏁 Test only for error checks
b2144f8 
	[I] #15, #16
	[T] More elaborated tests for setAutoStart()
	[#] Fixes #15
ChacaS0 added a commit that referenced this issue Apr 8, 2018
@ChacaS0
🔨 Auto mode for TEMPest (#28)
deec00c 
* 🚧 Moved configs files for TEMpest into a directory
	[+] Added .sh for auto-mode
	--> Issues related: #24, #15
* Handler for shutup mode made, need merge
* Stash pop
* 🏁 Added tests for shutup mode handler (start.go)
	-> Issues related: #15, #16
* ✔️🏁 Added logs flag for get command
	[+] Added the corresponding test
	--> Issues related: #15, #16
* ✔️ Added handler for relative paths with ``add``
* 🏁 Added tests for treatRelativePath()
* 🏁 Test for getAllLogs() (--logs)
	--> Issues related: #15, #16
	[*] Should be the end of the Linux part for this one
* 💻 Windows autostart kinda done for theoric part
	[#] Issues related: #15, #9
	[T] Unit tests for set.go:setAutoStart()
	[T] Runtime testing for Windows
* 📛 Hot fix for codecov chmod
* 🏁 Test only for error checks
	[I] #15, #16
	[T] More elaborated tests for setAutoStart()
	[#] Fixes #15
ChacaS0 added a commit that referenced this issue Apr 9, 2018
@ChacaS0
➕ New flag for get (auto-mode)
fccc3e1 
	[+] Also implemented test func for it
	[#] Issues: #15, #16
ChacaS0 added a commit that referenced this issue May 1, 2018
@ChacaS0
Apply patch v0.2B (#26)
19ca504 
* ✔️ Mispells fixes

* Improvement of the ``rm`` command (#21) (#25)

* ✔️ Mispells fixes

* 🚧 Work on rework of rm command starts -- bad

* 💊 Fixed missspelling

* ✔️🏁 processArgsRm added + tests

* ✔️🏁 New feature for rm
	-> Added & adapted tests only for the ones impacted by those changes

* 💻🏁 Added test for rm.go:simpleDelAllString()
	-> Few changes (variadic params instead of slice)
	-> Some update for the ``$HOME`` on Windows

* ✔️🏁 Added handler for ``*`` wildcard as argument
	-> Tests, were adapted for that

* 📝 Update of usage for ``rm`` and deletion of deprecated stuff

* Implementation of ``tempest list --fix`` flag (#27)

* 🚧 Added the functionality, now needs the tests

* 🏁 Tests for list.go (``--fix`` flag related)
	+ Begin of migration to the use of Target instead of simple strings

-----------------------
Merge Fixes #20

* 🔨 Auto mode for TEMPest (#28)

* 🚧 Moved configs files for TEMpest into a directory
	[+] Added .sh for auto-mode
	--> Issues related: #24, #15
* Handler for shutup mode made, need merge
* Stash pop
* 🏁 Added tests for shutup mode handler (start.go)
	-> Issues related: #15, #16
* ✔️🏁 Added logs flag for get command
	[+] Added the corresponding test
	--> Issues related: #15, #16
* ✔️ Added handler for relative paths with ``add``
* 🏁 Added tests for treatRelativePath()
* 🏁 Test for getAllLogs() (--logs)
	--> Issues related: #15, #16
	[*] Should be the end of the Linux part for this one
* 💻 Windows autostart kinda done for theoric part
	[#] Issues related: #15, #9
	[T] Unit tests for set.go:setAutoStart()
	[T] Runtime testing for Windows
* 📛 Hot fix for codecov chmod
* 🏁 Test only for error checks
	[I] #15, #16
	[T] More elaborated tests for setAutoStart()
	[#] Fixes #15

* ➕ New flag for get (auto-mode)
	[+] Also implemented test func for it
	[#] Issues: #15, #16

* 📝 Update of README.md

* ⬆️ dep ensure

* Manage temp files (#30)

* 🚧 Started to refractor purge to purge files
	[T] Keep refratoring each functions affected
	[#] Issues: #17
	[N] Some comments were deleted (cobra related)

* New func to display size + units

* 📄🚽 Implemented the file purging
	[T] Tests
	[#] #30, #17

* 🏁 Tests for purge
👉 Apply TreatLastChar() to conf.Gobin & conf.Gopath
@ChacaS0 ChacaS0 removed this from In progress in v0.2B May 28, 2018
@ChacaS0 ChacaS0 modified the milestones: v0.2B, v0.1 May 28, 2018
ChacaS0 added a commit that referenced this issue Jun 8, 2018
@ChacaS0
Patch v0.1 (#36)
76f2192 
* ✔️ Mispells fixes

 On branch devel
 Changes to be committed:
	modified:   cmd/init_test.go
	modified:   cmd/list.go
	modified:   cmd/list_test.go
	modified:   cmd/rm_test.go
	modified:   cmd/start_test.go

* Improvement of the ``rm`` command (#21) (#25)

* ✔️ Mispells fixes

 On branch devel
 Changes to be committed:
	modified:   cmd/init_test.go
	modified:   cmd/list.go
	modified:   cmd/list_test.go
	modified:   cmd/rm_test.go
	modified:   cmd/start_test.go

* 🚧 Work on rework of rm command starts -- bad

 Issues related:
 	-> #21 Improve ``rm`` command

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/rm.go

* 💊 Fixed missspelling

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/init_test.go
	modified:   cmd/list.go
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go

* ✔️🏁 processArgsRm added + tests

 Issues related:
 	-> #21
 	-> #16
 	-> #9

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/init_test.go
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go

* ✔️🏁 New feature for rm
	-> Added & adapted tests only for the ones impacted by those changes

 Issues related:
 	-> #21
 	-> #16

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go
	modified:   cmd/root.go

* 💻🏁 Added test for rm.go:simpleDelAllString()
	-> Few changes (variadic params instead of slice)
	-> Some update for the ``$HOME`` on Windows

 Issues related:
 	-> #21
 	-> #16
 	-> #9

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/purge.go
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go
	modified:   cmd/root.go

* ✔️🏁 Added handler for ``*`` wildcard as argument
	-> Tests, were adapted for that

 Issues related:
 	-> #21

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/rm.go
	modified:   cmd/rm_test.go

* 📝 Update of usage for ``rm`` and deletion of deprecated stuff

 Issues related:
 	-> #21
 	-> #19

 On branch dev-enhancement-rm
 Changes to be committed:
	modified:   cmd/rm.go

* Implementation of ``tempest list --fix`` flag (#27)

* 🚧 Added the functionality, now needs the tests

 Issues related:
 	-> #20

* 🏁 Tests for list.go (``--fix`` flag related)
	+ Begin of migration to the use of Target instead of simple strings

 Issues related:
 	-> #20

-----------------------
Merge Fixes #20

* 🔨 Auto mode for TEMPest (#28)

* 🚧 Moved configs files for TEMpest into a directory
	[+] Added .sh for auto-mode
	--> Issues related: #24, #15
* Handler for shutup mode made, need merge
* Stash pop
* 🏁 Added tests for shutup mode handler (start.go)
	-> Issues related: #15, #16
* ✔️🏁 Added logs flag for get command
	[+] Added the corresponding test
	--> Issues related: #15, #16
* ✔️ Added handler for relative paths with ``add``
* 🏁 Added tests for treatRelativePath()
* 🏁 Test for getAllLogs() (--logs)
	--> Issues related: #15, #16
	[*] Should be the end of the Linux part for this one
* 💻 Windows autostart kinda done for theoric part
	[#] Issues related: #15, #9
	[T] Unit tests for set.go:setAutoStart()
	[T] Runtime testing for Windows
* 📛 Hot fix for codecov chmod
* 🏁 Test only for error checks
	[I] #15, #16
	[T] More elaborated tests for setAutoStart()
	[#] Fixes #15

* ➕ New flag for get (auto-mode)
	[+] Also implemented test func for it
	[#] Issues: #15, #16

* 📝 Update of README.md

* ⬆️ dep ensure

* Manage temp files (#30)

* 🚧 Started to refractor purge to purge files
	[T] Keep refratoring each functions affected
	[#] Issues: #17
	[N] Some comments were deleted (cobra related)

* New func to display size + units

* 📄🚽 Implemented the file purging
	[T] Tests
	[#] #30, #17

* 🏁 Tests for purge
👉 Apply TreatLastChar() to conf.Gobin & conf.Gopath

* Targets automation (#34)

* ✔️ Func to find dirs from exact pattern added

* ✔️ Flag on ``add`` command added
🏁 Test for the research func
	[T]	Test the other funcs created

* ✔️ Better error handling for that
👉 TODO? More interaction for this command/flag

* ✔️ command ``new`` implemented to generated new target dirs
🏁 Tests passed

*  Added fallback to cmd.Help() by default

* ✔️ add

* ✔️ init

* ✔️ new
✅ fix init

* ✔️ purge

* ✔️ rm

* ✔️ root

* ✔️ set

* ✔️ start
👉 the rest can wait :(

* 👍 Small update to README.md

* messing

* Update documentation (#37)

*  Added fallback to cmd.Help() by default

* ✔️ add

* ✔️ init

* ✔️ new
✅ fix init

* ✔️ purge

* ✔️ rm

* ✔️ root

* ✔️ set

* ✔️ start
👉 the rest can wait :(

* 👍 Small update to README.md

* messing

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

@ChacaS0 ChacaS0

Labels
enhancement Improvement of existing feature suggestion proposal for a new functionality or whatever test stuff to do about testing (unit tests, live tests, etc) todo when it needs to be done
Projects
None yet
Milestone
v0.1
Development

No branches or pull requests
1 participant
@ChacaS0