Makes analyzer always output static_libraries that have changed

test/analyzer/gyptest-analyzer.py

 #!/usr/bin/env python
 # Copyright (c) 2014 Google Inc. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Tests for analyzer
 """
 
 import json
 import TestGyp
 
 found = 'Found dependency'
 found_all = 'Found dependency (all)'
 not_found = 'No dependencies'
 
 
 def _CreateConfigFile(files, targets):
-  """Creates the analyzer conflig file, which is used as the input to analyzer.
+  """Creates the analyzer config file, which is used as the input to analyzer.
   See description of analyzer.py for description of the arguments."""
   f = open('test_file', 'w')
   to_write = {'files': files, 'targets': targets }
   json.dump(to_write, f)
   f.close()
 
 
 def _CreateBogusConfigFile():
   f = open('test_file','w')
   f.write('bogus')
@@ skipping 108 matching lines @@
 
 def EnsureInvalidTargets(expected_invalid_targets):
   """Verifies output contains invalid_targets."""
   result = _ReadOutputFileContents()
   actual_invalid_targets = set(result['invalid_targets'])
   if actual_invalid_targets != expected_invalid_targets:
     print 'actual invalid_targets:', actual_invalid_targets, \
         '\nexpected :', expected_invalid_targets
     test.fail_test()
 
+
+# Two targets, A and B (both static_libraries) and A depends upon B. If a file
+# in B changes, then both A and B are output. It is not strictly necessary that
+# A is compiled in this case, only B.
+_CreateConfigFile(['b.c'], [])
+test.run_gyp('static_library_test.gyp', *CommonArgs())
+EnsureContains(matched=True, build_targets={'a' ,'b'})
+#test.pass_test()

[Nico, 2015/06/08 17:53:31]

is this intentionally commented out? probably not?

[sky, 2015/06/08 18:01:17]

Ya, sorry, I removed this line.

test.pass_test() is effectively an early out. I do that some times when I only
care about this test and not any other future failures. I forgot to remove it.

+
 # Verifies config_path must be specified.
 test.run_gyp('test.gyp')
 EnsureStdoutContains('Must specify files to analyze via config_path')
 
 # Verifies config_path must point to a valid file.
 test.run_gyp('test.gyp', '-Gconfig_path=bogus_file',
              '-Ganalyzer_output_path=analyzer_output')
 EnsureError('Unable to open file bogus_file')
 
 # Verify 'invalid_targets' is present when bad target is specified.
@@ skipping 187 matching lines @@
 _CreateConfigFile(['f.c'], [])
 run_analyzer4()
 EnsureContains(matched=True, build_targets={'e', 'f'})
 
 _CreateConfigFile(['d.c'], [])
 run_analyzer4()
 EnsureContains(matched=True, build_targets={'a', 'b', 'c', 'd'})
 
 _CreateConfigFile(['i.c'], [])
 run_analyzer4()
-EnsureContains(matched=True, build_targets={'h'})
+EnsureContains(matched=True, build_targets={'h', 'i'})
 
 test.pass_test()