fix(ds_edit): dont show warning when data source does not support testing, fixes #4811

public/app/features/plugins/ds_edit_ctrl.ts

@@ -98,9 +98,7 @@ export class DataSourceEditCtrl {
 
 
       this.datasourceSrv.get(this.current.name).then(datasource => {
       this.datasourceSrv.get(this.current.name).then(datasource => {
         if (!datasource.testDatasource) {
         if (!datasource.testDatasource) {
-          this.testing.message = 'Data source does not support test connection feature.';
-          this.testing.status = 'warning';
-          this.testing.title = 'Unknown';
+          delete this.testing;
           return;
           return;
         }
         }
 
 
@@ -118,7 +116,9 @@ export class DataSourceEditCtrl {
           }
           }
         });
         });
       }).finally(() => {
       }).finally(() => {
+        if (this.testing) {
           this.testing.done = true;
           this.testing.done = true;
+        }
       });
       });
     }
     }
 
 

public/app/features/plugins/partials/ds_edit.html

@@ -53,7 +53,7 @@
 			</plugin-component>
 			</plugin-component>
 			</rebuild-on-change>
 			</rebuild-on-change>
 
 
-			<div ng-if="ctrl.testing" style="margin-top: 25px">
+			<div ng-if="ctrl.testing" class="gf-form-group">
 				<h5 ng-show="!ctrl.testing.done">Testing.... <i class="fa fa-spiner fa-spin"></i></h5>
 				<h5 ng-show="!ctrl.testing.done">Testing.... <i class="fa fa-spiner fa-spin"></i></h5>
 				<div class="alert-{{ctrl.testing.status}} alert">
 				<div class="alert-{{ctrl.testing.status}} alert">
 					<div class="alert-title">{{ctrl.testing.title}}</div>
 					<div class="alert-title">{{ctrl.testing.title}}</div>