DEV: Fix template linting issues (#124)

* Update test
* Remove the empty line in table

assets/javascripts/discourse/templates/admin/plugins-explorer.hbs

@@ -67,7 +67,7 @@
                 </span>
                 {{#if runDisabled}}
                   {{#unless editing}}
-                  <span class='setting-controls'>
+                    <span class="setting-controls">
                       {{d-button class="ok" action=(action "save") icon="check"}}
                       {{d-button class="cancel" action=(action "discard") icon="times"}}
                     </span>
@@ -75,6 +75,7 @@
                 {{/if}}
               </div>
             </div>
+
             <div class="clear"></div>
 
             {{! the SQL editor will show the first time you }}
@@ -93,9 +94,11 @@
                     </div>
                   </div>
                 </div>
+
                 <div class="grippie">
                   {{d-icon "discourse-expand"}}
                 </div>
+
                 <div class="clear"></div>
               </div>
             {{else}}
@@ -119,6 +122,7 @@
                 {{d-button action=(action "showHelpModal") label="explorer.help.label" icon="question-circle"}}
               {{/if}}
             </div>
+
             <div class="pull-right">
               {{#if selectedItem.destroyed}}
                 {{d-button action=(action "recover") class="" icon="undo" label="explorer.recover"}}
@@ -131,6 +135,7 @@
                 {{/unless}}
               {{/if}}
             </div>
+
             <div class="clear"></div>
           {{/if}}
         </div>
@@ -143,6 +148,7 @@
               {{/each}}
             </div>
           {{/if}}
+
           {{#if runDisabled}}
             {{#if saveDisabled}}
               {{d-button label="explorer.run" disabled="true" class="btn-primary"}}
@@ -152,12 +158,15 @@
           {{else}}
             {{d-button action=(action "run") icon="play" label="explorer.run" disabled=runDisabled class="btn-primary" type="submit"}}
           {{/if}}
+
           <label class="query-plan">{{input type="checkbox" checked=explain name="explain"}} {{i18n "explorer.explain_label"}}</label>
         </form>
 
         <hr>
       {{/unless}}
+
       {{conditional-loading-spinner condition=loading}}
+
       {{#unless selectedItem.fake}}
         {{#if results}}
           <div class="query-results">
@@ -177,27 +186,26 @@
           <table class="recent-queries">
             <thead class="heading-container">
               <th class="col heading name">
-              <div class="heading-toggle" {{action "sortByProperty" "name"}}>
+                <div role="button" class="heading-toggle" {{action "sortByProperty" "name"}}>
                   {{table-header-toggle field="name" labelKey="explorer.query_name" order=order asc=asc automatic=true}}
                 </div>
               </th>
               <th class="col heading created-by">
-              <div class="heading-toggle" {{action "sortByProperty" "username"}}>
+                <div role="button" class="heading-toggle" {{action "sortByProperty" "username"}}>
                   {{table-header-toggle field="username" labelKey="explorer.query_user" order=order asc=asc automatic=true}}
                 </div>
               </th>
-            <th class='col heading group-names'>
+              <th class="col heading group-names">
-              <div class='group-names-header'>
+                <div class="group-names-header">
                   {{i18n "explorer.query_groups"}}
                 </div>
               </th>
               <th class="col heading created-at">
-              <div class="heading-toggle" {{action "sortByProperty" "last_run_at"}}>
+                <div role="button" class="heading-toggle" {{action "sortByProperty" "last_run_at"}}>
                   {{table-header-toggle field="last_run_at" labelKey="explorer.query_time" order=order asc=asc automatic=true}}
                 </div>
               </th>
             </thead>
-          <tr></tr>
             <tbody>
               {{#each filteredContent as |query|}}
                 <tr class="query-row">
@@ -241,7 +249,6 @@
       {{/if}}
 
       <div class="explorer-pad-bottom"></div>
-
     {{/if}}
   {{/if}}
 {{/explorer-container}}

assets/javascripts/discourse/templates/components/share-report.hbs

@@ -1,7 +1,8 @@
-<div onclick={{action "open"}}>
+<div role="button" onclick={{action "open"}}>
   {{d-icon "link"}}
   {{group}}
 </div>
+
 {{#if visible}}
   <div class="popup">
     <label>{{i18n "explorer.link"}} {{group}}</label>

test/javascripts/acceptance/list-queries-test.js.es6

@@ -1,8 +1,8 @@
 import {
   acceptance,
+  count,
   exists,
   query,
-  queryAll,
 } from "discourse/tests/helpers/qunit-helpers";
 import { clearPopupMenuOptionsCallback } from "discourse/controllers/composer";
 import I18n from "I18n";
@@ -162,8 +162,9 @@ acceptance("Data Explorer Plugin | List Queries", function (needs) {
       "the import button was rendered"
     );
 
-    assert.ok(
+    assert.equal(
-      queryAll("div.container table.recent-queries tbody tr").length === 2,
+      count("div.container table.recent-queries tbody tr"),
+      2,
       "the list of queries was rendered"
     );