[bindings] remove unique_external test failure

[frutiger]

Summary

In SVN r314037, linkage determination of a symbol was significantly
refactored. This resulted in extern anonymous namespace declarations to
no longer be considered 'unique_external'. This ultimately broke a test
in the Python bindings.

This commit removes the 'unique_external' test case.

[frutiger]

I was unable to produce a simple test case that would still result in a cursor with 'unique external' linkage. @rsmith I would appreciate if you can think of a symbol that may still have this kind of linkage. Thanks!

[frutiger]

Friendly poke @rsmith @compnerd

[mgorny]

Let's try some more reviewers.

[mgorny]

I've rewritten the tests to use unittest, so this patch needs to be updated now. However, you may want to wait a while since @compnerd promised to look at it and he might have a way to get uniqueExternal() linkage.

[frutiger]

Thanks for the note, I'll wait.

[frutiger]

Fixed by r317986.

bindings/python/tests/cindex/test_linkage.py

     tu = get_tu("""
 void foo() { int no_linkage; }
 static int internal;
-namespace { extern int unique_external; }
 extern int external;
 """, lang = 'cpp')
 

     internal = get_cursor(tu.cursor, 'internal')
     assert internal.linkage == LinkageKind.INTERNAL
 
-    unique_external = get_cursor(tu.cursor, 'unique_external')
-    assert unique_external.linkage == LinkageKind.UNIQUE_EXTERNAL
-
     external = get_cursor(tu.cursor, 'external')
     assert external.linkage == LinkageKind.EXTERNAL