[Findit] Add feedback button for manual triage.

appengine/findit/model/crash/test/crash_analysis_test.py

 # Copyright 2016 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 from datetime import datetime
 
 import unittest
 
 from model.crash.crash_analysis import CrashAnalysis
 from model import analysis_status
@@ skipping 50 matching lines @@
     analysis.solution = ''
     analysis.Reset()
     self.assertIsNone(analysis.pipeline_status_path)
     self.assertEqual(analysis_status.PENDING, analysis.status)
     self.assertIsNone(analysis.requested_time)
     self.assertIsNone(analysis.started_time)
     self.assertIsNone(analysis.findit_version)
     self.assertIsNone(analysis.has_regression_range)
     self.assertIsNone(analysis.found_suspects)
     self.assertIsNone(analysis.solution)
-    self.assertEqual(analysis.result, {})
+    self.assertEqual(analysis.result, None)
     self.assertEqual(analysis.regression_range_triage_status,
                      triage_status.UNTRIAGED)
     self.assertEqual(analysis.suspected_cls_triage_status,
                      triage_status.UNTRIAGED)
     self.assertEqual(analysis.suspected_project_triage_status,
                      triage_status.UNTRIAGED)
     self.assertEqual(analysis.suspected_components_triage_status,
                      triage_status.UNTRIAGED)
-    self.assertEqual(analysis.culprit_regression_range, [])
-    self.assertEqual(analysis.culprit_cls, [])
-    self.assertEqual(analysis.culprit_project, '')
-    self.assertEqual(analysis.culprit_components, [])
+    self.assertEqual(analysis.culprit_regression_range, None)
+    self.assertEqual(analysis.culprit_cls, None)
+    self.assertEqual(analysis.culprit_project, None)
+    self.assertEqual(analysis.culprit_components, None)
     self.assertEqual(analysis.triage_history, None)
-    self.assertEqual(analysis.note, '')
+    self.assertEqual(analysis.note, None)
+
+  def testUpdateCrashAnalysis(self):
+    update = {'note': 'dummy'}
+    analysis = CrashAnalysis()
+    analysis.Update(update)
+    self.assertEqual(analysis.note, update['note'])
+
+  def testUpdateCrashAnalysisWithNonExistentProperty(self):
+    update = {'dummy': 'dummy_content'}
+    analysis = CrashAnalysis()
+    analysis.Update(update)
+    self.assertFalse(hasattr(analysis, 'dummy'))