fix test rendering and lint

client-src/elements/chromedash-guide-intent-preview.js

@@ -21,7 +21,7 @@ class ChromedashGuideIntentPreview extends LitElement {
       stage: {type: Object},
       gate: {type: Object},
       loading: {type: Boolean},
-      subject: {type:String},
+      subject: {type: String},
       intentBody: {type: String},
       displayFeatureUnlistedWarning: {type: Boolean},
     };

client-src/elements/chromedash-intent-template_test.js

@@ -3,7 +3,6 @@ import {assert, fixture} from '@open-wc/testing';
 import {ChromedashIntentTemplate} from './chromedash-intent-template';
 
 describe('chromedash-intent-template', () => {
-
   it('renders with fake data', async () => {
     const component = await fixture(
       html`<chromedash-intent-template

client-src/js-src/cs-client.js

@@ -647,7 +647,7 @@ export class ChromeStatusClient {
 
   // Intents API
   async getIntentBody(featureId, stageId) {
-    return this.doGet(`/features/${featureId}/${stageId}/intent`)
+    return this.doGet(`/features/${featureId}/${stageId}/intent`);
   }
   async postIntentToBlinkDev(featureId, stageId, body) {
     return this.doPost(`/features/${featureId}/${stageId}/intent`, body);

internals/reminders_test.py

@@ -174,7 +174,7 @@ class FunctionTest(testing_config.CustomTestCase):
     self.assertEqual('feature_owner@example.com', task['to'])
     self.assertEqual('[Action requested] Update feature one', task['subject'])
     self.assertEqual(None, task['reply_to'])
-    TESTDATA.make_golden(task['html'], 'test_build_email_tasks_feature_accuracy_enterprise.html')
+    # TESTDATA.make_golden(task['html'], 'test_build_email_tasks_feature_accuracy_enterprise.html')
     self.assertMultiLineEqual(
         TESTDATA['test_build_email_tasks_feature_accuracy_enterprise.html'],
         task['html'])

internals/testdata/reminders_test/test_build_email_tasks_escalated_feature_accuracy.html

@@ -63,27 +63,11 @@
     
     
 
-  </table>
-
-  <table>
-
     
     
     
-
-  </table>
-
-
-  <table>
-
     
     
-
-  </table>
-
-
-  <table>
-
     
     
 

internals/testdata/reminders_test/test_build_email_tasks_feature_accuracy.html

@@ -58,27 +58,11 @@
     
     
 
-  </table>
-
-  <table>
-
     
     
     
-
-  </table>
-
-
-  <table>
-
     
     
-
-  </table>
-
-
-  <table>
-
     
     
 

internals/testdata/reminders_test/test_build_email_tasks_feature_accuracy_enterprise.html

@@ -58,27 +58,11 @@
     
     
 
-  </table>
-
-  <table>
-
     
     
     
-
-  </table>
-
-
-  <table>
-
     
     
-
-  </table>
-
-
-  <table>
-
     
     
 

pages/testdata/intentpreview_test/test_html_rendering.html

@@ -139,33 +139,32 @@ limitations under the License.
 
 
 <section>
-<h3>Copy and send this text for your "Intent to ..." email</h3>
+  <h3>Copy and send this text for your "Intent to ..." email</h3>
-<p>Email to</p>
+  <p>Email to</p>
-<div class="subject">
+  <div class="subject">
-  blink-dev@chromium.org
+    blink-dev@chromium.org
-</div>
+  </div>
 
-<p>Subject</p>
+  <p>Subject</p>
-<div class="subject">
+  <div class="subject">
-  Intent to Prototype:
+    Intent to Prototype:
-  feature one
+    feature one
-</div>
+  </div>
 
+  
 
+  <p>Body
+    <span class="tooltip copy-text" style="float:right"
+          title="Copy text to clipboard">
+        <a href="#" data-tooltip>
+          <iron-icon icon="chromestatus:content_copy"
+                    id="copy-email-body"></iron-icon>
+        </a>
+    </span>
+  </p>
 
-<p>Body
+  <div class="email">
-  <span class="tooltip copy-text" style="float:right"
+      <h4>Contact emails</h4>
-        title="Copy text to clipboard">
-      <a href="#" data-tooltip>
-        <iron-icon icon="chromestatus:content_copy"
-                   id="copy-email-body"></iron-icon>
-      </a>
-  </span>
-</p>
-
-<div class="email">
-
-  <h4>Contact emails</h4>
   
   
     <a href="mailto:user1@google.com">user1@google.com</a>
@@ -311,14 +310,11 @@ False
 
 
 
-
+  <br><br><div><small>
-
+    This intent message was generated by
-<br><br><div><small>
+    <a href="https://chromestatus.com">Chrome Platform Status</a>.
-  This intent message was generated by
+  </small></div>
-  <a href="https://chromestatus.com">Chrome Platform Status</a>.
+  </div> <!-- end email body div -->
-</small></div>
-
-</div> <!-- end email body div -->
 </section>