OpenStack Dashboard (Horizon)

Change JS coverage report dir to match CTI

Bug #1588389 reported by Matt Borland on 2016-06-02

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	OpenStack Dashboard (Horizon)	Fix Released	Low	Matt Borland	OpenStack Dashboard (Horizon) newton-2 "n2"

Bug Description

Krotscheck has pointed out that to follow the CTI definition, we should place all our JS coverage reports in ./cover. This means we need subdirs in there for 'horizon' and 'openstack_dashboard'.

https://governance.openstack.org/reference/cti/javascript-cti.html#executing-tests-and-code-coverage

Tags:

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-06-02: Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/324558

Changed in horizon:
assignee:	nobody → Matt Borland (palecrow)
status:	New → In Progress

Matt Borland (palecrow) on 2016-06-02

tags:

added: angularjs

Rob Cresswell (robcresswell-deactivatedaccount) on 2016-06-03

Changed in horizon:
importance:	Undecided → Low
milestone:	none → newton-2

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-06-06: Fix merged to horizon (master)

Reviewed: https://review.openstack.org/324558
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=bf7c5908fbe46d322bb1842ff659206d8fdd3583
Submitter: Jenkins
Branch: master

commit bf7c5908fbe46d322bb1842ff659206d8fdd3583
Author: Matt Borland <email address hidden>
Date: Thu Jun 2 08:25:14 2016 -0600

Output JS Karma test coverage reports to ./cover/

Per CTI direction (see bug filing), this patch moves the JS Karma test coverage
reports to ./cover/ so they may be collected in the gate for archiving.

The two runs each have their own directory so they don't overlap/conflict.

Change-Id: I0b1536c115a516f4b6ab80b5ebab5337b308c210
Closes-Bug: 1588389