Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

improve unit test for 'core' module 'handlers' part #1134

Closed
hrishin opened this issue Dec 12, 2017 · 1 comment
Closed

improve unit test for 'core' module 'handlers' part #1134

hrishin opened this issue Dec 12, 2017 · 1 comment
Assignees
@piyush-garg
Labels
cat/techdebt Technical debt, like missing unit tests or tests
Milestone
Sprint 142

Comments

@hrishin
Copy link
Member

hrishin commented Dec 12, 2017 •
edited
Loading

Implement the unit tests for classes falling under the handlers directory.
@hrishin hrishin added test cat/techdebt Technical debt, like missing unit tests or tests labels Dec 12, 2017
@hrishin hrishin added this to the Sprint 142 milestone Dec 12, 2017
@hrishin hrishin changed the title Improve unit test for 'core' module 'handelers' part Improve unit test for 'core' module 'handlers' part Dec 12, 2017
@hrishin
Copy link
Member Author

hrishin commented Dec 12, 2017

part of #1102 #1107

@hrishin hrishin changed the title Improve unit test for 'core' module 'handlers' part improve unit test for 'core' module 'handlers' part Dec 12, 2017
piyush-garg added a commit to piyush-garg/fabric8-maven-plugin that referenced this issue Dec 12, 2017
@piyush-garg
Added Test for Container Handler
7e6bc20 
 fabric8io#1134
@piyush-garg piyush-garg mentioned this issue Dec 12, 2017
Merged
piyush-garg added a commit to piyush-garg/fabric8-maven-plugin that referenced this issue Dec 13, 2017
@piyush-garg
Added Test for Container Handler
7052d9a 
 fabric8io#1134

Added more strong assertion

Removed unused import
piyush-garg added a commit to piyush-garg/fabric8-maven-plugin that referenced this issue Dec 20, 2017
@piyush-garg
Added Test for Container Handler
7393505 
 fabric8io#1134

Added more strong assertion

Removed unused import
piyush-garg added a commit to piyush-garg/fabric8-maven-plugin that referenced this issue Dec 27, 2017
@piyush-garg
Added Test for Container Handler
fd9bdfe 
 fabric8io#1134

Added more strong assertion

Removed unused import
@hrishin hrishin closed this as completed Jan 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@piyush-garg piyush-garg

Labels
cat/techdebt Technical debt, like missing unit tests or tests
Projects
None yet
Milestone
Sprint 142
Development

No branches or pull requests
2 participants
@piyush-garg @hrishin