]> review.fuel-infra Code Review - openstack-build/horizon-build.git/commitdiff
Added disable-failed-django-1.7-test.patch
authorThomas Goirand <thomas@goirand.fr>
Sun, 21 Sep 2014 08:14:30 +0000 (08:14 +0000)
committerThomas Goirand <thomas@goirand.fr>
Sun, 21 Sep 2014 08:14:30 +0000 (08:14 +0000)
Rewritten-From: f5068eb976c55828fb5fd411f9877451f08f7a25

trusty/debian/changelog
trusty/debian/patches/disable-failed-django-1.7-test.patch [new file with mode: 0644]
trusty/debian/patches/series

index a90a0df78bf753e50851f5ffe0a4d7f9b118d066..7e28489cf59242eaab355597dbcec87e96ed0228 100644 (file)
@@ -5,6 +5,7 @@ horizon (2014.2~b3-1) experimental; urgency=medium
   * Removed fix-python-m-coverage.patch FINALLY applied upstream !!! :)
   * Added 0008_Handle_TypeError_from_table_column_summation_code.patch.
   * Added 0009_Fix-TypeError-SecurityGroup-object-is-not-iterable-t.patch
+  * Added disable-failed-django-1.7-test.patch
 
  -- Thomas Goirand <zigo@debian.org>  Tue, 01 Jul 2014 16:06:08 +0800
 
diff --git a/trusty/debian/patches/disable-failed-django-1.7-test.patch b/trusty/debian/patches/disable-failed-django-1.7-test.patch
new file mode 100644 (file)
index 0000000..f082df7
--- /dev/null
@@ -0,0 +1,54 @@
+Description: Disable test_update_project_when_default_role_does_not_exist()
+ This last test is the last remaining one that fails with Django 1.7. It
+ doesn't seem like it is a relevant failure, and that running Django 1.7 only
+ reveal the issue instead of creating it.
+Author: Thomas Goirand <zigo@debian.org>
+Forwarded: no
+Last-Update: 2014-09-21
+
+--- horizon-2014.2~b3.orig/openstack_dashboard/dashboards/identity/projects/tests.py
++++ horizon-2014.2~b3/openstack_dashboard/dashboards/identity/projects/tests.py
+@@ -1608,43 +1608,6 @@ class UpdateProjectWorkflowTests(test.Ba
+         self.assertMessageCount(error=2, warning=0)
+         self.assertRedirectsNoFollow(res, INDEX_URL)
+-    @test.create_stubs({api.keystone: ('get_default_role',
+-                                       'tenant_get',
+-                                       'domain_get'),
+-                        quotas: ('get_tenant_quota_data',
+-                                 'get_disabled_quotas')})
+-    def test_update_project_when_default_role_does_not_exist(self):
+-        project = self.tenants.first()
+-        domain_id = project.domain_id
+-        quota = self.quotas.first()
+-
+-        api.keystone.get_default_role(IsA(http.HttpRequest)) \
+-            .MultipleTimes().AndReturn(None)  # Default role doesn't exist
+-        api.keystone.tenant_get(IsA(http.HttpRequest), self.tenant.id,
+-                                admin=True) \
+-            .AndReturn(project)
+-        api.keystone.domain_get(IsA(http.HttpRequest), domain_id) \
+-            .AndReturn(self.domain)
+-        quotas.get_disabled_quotas(IsA(http.HttpRequest)) \
+-            .AndReturn(self.disabled_quotas.first())
+-        quotas.get_tenant_quota_data(IsA(http.HttpRequest),
+-                                     tenant_id=self.tenant.id) \
+-            .AndReturn(quota)
+-        self.mox.ReplayAll()
+-
+-        url = reverse('horizon:identity:projects:update',
+-                      args=[self.tenant.id])
+-
+-        try:
+-            # Avoid the log message in the test output when the workflow's
+-            # step action cannot be instantiated
+-            logging.disable(logging.ERROR)
+-            with self.assertRaises(exceptions.NotFound):
+-                self.client.get(url)
+-        finally:
+-            logging.disable(logging.NOTSET)
+-
+-
+ class UsageViewTests(test.BaseAdminViewTests):
+     def _stub_nova_api_calls(self, nova_stu_enabled=True):
+         self.mox.StubOutWithMock(api.nova, 'usage_get')
index 0aa35929b4097ff8bce707bc90dc1a0175239e4a..e56d6bdbb8ab336c3c9c44bf04f9c361cb117214 100644 (file)
@@ -3,3 +3,4 @@ fix-dashboard-manage.patch
 fixed-horizon-MANIFEST.in.patch
 0008_Handle_TypeError_from_table_column_summation_code.patch
 0009_Fix-TypeError-SecurityGroup-object-is-not-iterable-t.patch
+disable-failed-django-1.7-test.patch