← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1478504] [NEW] test_admin_version_v3 actually tests public app

 

Public bug reported:

VersionTestCase.test_admin_version_v3
(keystone/tests/unit/test_versions.py) in fact tests public app:

def test_admin_version_v3(self):
    client = tests.TestClient(self.public_app)

It makes sense only in case of V3 eventless setup where public app
handles bot endpoints, but I believe it should be tested by a separate
test like .test_admin_version_v3_eventlets which will be introduced as
part of fix of bug #1381961. Also this behavior was introduced when 2
apps setup was used for eventless.

** Affects: keystone
     Importance: Undecided
     Assignee: Alexey Miroshkin (amirosh)
         Status: New


** Tags: test-improvement

** Changed in: keystone
     Assignee: (unassigned) => Alexey Miroshkin (amirosh)

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

Title:
  test_admin_version_v3 actually  tests public app

Status in Keystone:
  New

Bug description:
  VersionTestCase.test_admin_version_v3
  (keystone/tests/unit/test_versions.py) in fact tests public app:

  def test_admin_version_v3(self):
      client = tests.TestClient(self.public_app)

  It makes sense only in case of V3 eventless setup where public app
  handles bot endpoints, but I believe it should be tested by a separate
  test like .test_admin_version_v3_eventlets which will be introduced as
  part of fix of bug #1381961. Also this behavior was introduced when 2
  apps setup was used for eventless.

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


Follow ups