JS-550 Optimize the deduplication logic

[vdiez]

JS-550

[zglicz]

Please add a test to show that deduping works

[sonarqube-next]

Quality Gate passed

Issues
0 New issues
2 Fixed issues
2 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

[zglicz]

Is there a test showing that the deduplication actually worked?

[ericmorand-sonarsource]

Is there a test showing that the deduplication actually worked?

@zglicz , yes, there is one:

SonarJS/sonar-plugin/sonar-javascript-plugin/src/test/java/org/sonar/plugins/javascript/analysis/JsTsSensorTest.java

Line 195 in 94bb2ae

/**

As you can see, it is not located in the test suite where it should be.

[zglicz]

Ahh, I missed the first reply, as it auto closed the conversation as it was outdated code. I'll try to get better filters on my github notifs.