cloudogu / sonar-cas-plugin

CAS Authentication support for SonarQube
14 stars 8 forks source link

Add more documentation and more tests after migration to SonarQube 6.7 #11

Closed ppxl closed 5 years ago

ppxl commented 5 years ago

The inner workings of the sonar-cas-plugin have drastically changed with the migration from SonarQube version 5 to 6. The new code suffered a low line coverage and only very little documentation about how the plugin actually works.

Usually I would create a separate issue for these things, but both parts are closely related. Eating through the code with tests is a good time to aggregate information for the documentation. Furthermore both unit tests and integration tests can be a valuable source of documentation as well.

  1. Please add documentation so other developers can learn about the design choices and the inner choices
  2. Please add more unit tests/integration tests.
ppxl commented 5 years ago

Closed with #12