summaryrefslogtreecommitdiffstats
path: root/cvp
diff options
context:
space:
mode:
authorchenjiankun <chenjiankun1@huawei.com>2017-12-28 01:39:07 +0000
committerLeo wang <grakiss.wanglei@huawei.com>2018-01-22 01:37:30 +0000
commit4e7a72bba9a0d7ca95478b298e056025421e221d (patch)
tree844ca1ab74d8adc76f2ae9466e74c1a306174007 /cvp
parentdb295b41a4da176863c7be60ead3828c024cadbd (diff)
[ovp-web] Add SUT version number to OVP portal
JIRA: DOVETAIL-572 We currently have a label column in the OVP test result view. This allows users to attach a custom label field to their results. It is at the users discretion to fill out this field. We need a separate SUT version column to track the product version number being submitted for official review. The field can be left blank when users upload and share results in a private manner. However, the OVP portal must check that this new 'SUT version' field is populated before results are officially submitted for review. An error message prompting the user to fill out this field if it is blank must be implemented when users submit results for review. Change-Id: I1ca915423a56e8173a3a13f1b84b066cf91d9291 Signed-off-by: chenjiankun <chenjiankun1@huawei.com> (cherry picked from commit af1debe11cf4793bae049ff3ffd63dc906bbc1c9)
Diffstat (limited to 'cvp')
-rw-r--r--cvp/3rd_party/static/testapi-ui/components/results/results.html4
-rw-r--r--cvp/3rd_party/static/testapi-ui/components/results/resultsController.js6
-rw-r--r--cvp/opnfv_testapi/resources/test_handlers.py5
3 files changed, 11 insertions, 4 deletions
diff --git a/cvp/3rd_party/static/testapi-ui/components/results/results.html b/cvp/3rd_party/static/testapi-ui/components/results/results.html
index 8d7e4482..f53023d9 100644
--- a/cvp/3rd_party/static/testapi-ui/components/results/results.html
+++ b/cvp/3rd_party/static/testapi-ui/components/results/results.html
@@ -29,6 +29,7 @@
<th>Status</th>
<th>Log</th>
<th>SUT</th>
+ <th>SUT Version</th>
<th class="col-md-2">Operation</th>
<th class="col-md-2">Share List</th>
</tr>
@@ -40,10 +41,11 @@
<td><a uib-tooltip="{{ result.id }}" tooltip-placement="top" tooltip-append-to-body="true" ng-click="ctrl.gotoResultDetail(result.id, result._id)">{{ result.id | limitTo:8 }}</a></td>
<td>{{ result.owner }}</td>
<td>{{ result.filename || "None"}}</td>
- <td><div class="popover-wrapper"><a editable-theme="bs3" onbeforesave="ctrl.changeLabel(result, $data)" editable-text="result.label"> {{ result.label || "None" }}</a></div></td>
+ <td><div class="popover-wrapper"><a editable-theme="bs3" onbeforesave="ctrl.changeLabel(result, 'label', $data)" editable-text="result.label"> {{ result.label || "None" }}</a></div></td>
<td>{{ result.status }}</td>
<td><a ng-click="ctrl.downloadLogs(result.id)">logs</a></td>
<td><a ng-click="ctrl.gotoSUT(result.id)">info</a></td>
+ <td><div class="popover-wrapper"><a editable-theme="bs3" onbeforesave="ctrl.changeLabel(result, 'sut_label', $data)" editable-text="result.sut_label"> {{ result.sut_label || "None" }}</a></div></td>
<td>
<div class="btn-group" uib-dropdown>
<a id="single-button" type="button" class="btn btn-success cvp-btn medium accent-color regular-button" uib-dropdown-toggle>
diff --git a/cvp/3rd_party/static/testapi-ui/components/results/resultsController.js b/cvp/3rd_party/static/testapi-ui/components/results/resultsController.js
index 0b0bbbc8..aa593dc0 100644
--- a/cvp/3rd_party/static/testapi-ui/components/results/resultsController.js
+++ b/cvp/3rd_party/static/testapi-ui/components/results/resultsController.js
@@ -191,12 +191,12 @@
});
}
- function changeLabel(result, data){
- toggleCheck(result, 'label', data);
+ function changeLabel(result, key, data){
+ toggleCheck(result, key, data);
}
function toReview(result, value){
- var resp = confirm('Once you submit a test result for review, it will become readable to all CVP reviewers. Do you want to proceed?');
+ var resp = confirm('Once you submit a test result for review, it will become readable to all OVP reviewers. Do you want to proceed?');
if(resp){
toggleCheck(result, 'status', value);
}
diff --git a/cvp/opnfv_testapi/resources/test_handlers.py b/cvp/opnfv_testapi/resources/test_handlers.py
index 2b5f28c2..b28a15d3 100644
--- a/cvp/opnfv_testapi/resources/test_handlers.py
+++ b/cvp/opnfv_testapi/resources/test_handlers.py
@@ -218,6 +218,11 @@ class TestsGURHandler(GenericTestHandler):
self.finish_request({'code': 403, 'msg': msg})
return
+ if not test['sut_label']:
+ msg = 'Please fill out SUT version before submission'
+ self.finish_request({'code': 403, 'msg': msg})
+ return
+
query['owner'] = curr_user
db_keys.append('owner')