Public bug reported:

I see direct values for protocols are used while testing listener and pool.
It would be good to use constants which are already defined. in functional and 
unit tests.

I was trying to implement its case insensitivity but since protocol value is 
hardcoded
in most tests so all the test were breaking.

The changes will be easy if we use already defined constants.

** Affects: neutron
     Importance: Undecided
     Assignee: Manjeet Singh Bhatia (manjeet-s-bhatia)
         Status: New


** Tags: lbaas

** Changed in: neutron
     Assignee: (unassigned) => Manjeet Singh Bhatia (manjeet-s-bhatia)

** Tags added: lbaas

** Description changed:

- I see direct values for protocols are used while testing listener and pool. 
+ I see direct values for protocols are used while testing listener and pool.
  It would be good to use constants which are already defined. in functional 
and unit tests.
  
- 
- I was trying to implement its case insensitivity but since protocol value is 
hardcoded 
- in most tests so all the test were breaking. 
+ I was trying to implement its case insensitivity but since protocol value is 
hardcoded
+ in most tests so all the test were breaking.
  
  The changes will be easy if we use already defined constants.

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1516296

Title:
  Use constants for protocol values in tests in load balancer comonents

Status in neutron:
  New

Bug description:
  I see direct values for protocols are used while testing listener and pool.
  It would be good to use constants which are already defined. in functional 
and unit tests.

  I was trying to implement its case insensitivity but since protocol value is 
hardcoded
  in most tests so all the test were breaking.

  The changes will be easy if we use already defined constants.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1516296/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to