Fix double import FQN (#1363)

Author: karim-ouerghemmi-sonarsource

python-frontend/src/main/java/org/sonar/python/semantic/Scope.java

@@ -202,7 +202,7 @@ void addImportedSymbol(Name nameTree, @CheckForNull String fullyQualifiedName, S
       globalSymbol = globalSymbol != null ? copySymbol(symbolName, globalSymbol) : null;
     }
     if (globalSymbol == null || isExistingSymbol(symbolName)) {
-      addBindingUsage(nameTree, Usage.Kind.IMPORT, fullyQualifiedName);
+      addBindingUsage(nameTree, Usage.Kind.IMPORT, globalSymbol != null ? globalSymbol.fullyQualifiedName() : fullyQualifiedName);
     } else {
       this.symbols.add(globalSymbol);
       symbolsByName.put(symbolName, globalSymbol);

python-frontend/src/test/java/org/sonar/python/semantic/FullyQualifiedNameTest.java

@@ -610,6 +610,15 @@ public void fqn_of_inherited_method_with_import() {
     assertThat(qualifiedExpression.symbol().fullyQualifiedName()).isNull();
   }
 
+  @Test
+  public void fqn_resolution_works_with_double_import() {
+    FileInput tree = parse(
+            "from flask import request, request",
+            "request.cookies.get('a')"
+    );
+    assertNameAndQualifiedName(tree, "get", "flask.globals.request.cookies.get");
+  }
+
   private void assertNameAndQualifiedName(FileInput tree, String name, @Nullable String qualifiedName) {
     CallExpression callExpression = getCallExpression(tree);
     assertThat(callExpression.calleeSymbol()).isNotNull();