The inspectIT team has defined on the meeting on the ideas for the defining when a task is done. This page summarizes the identified ideas.
Continue or Start Right Away
| Description | Automated Checks |
|---|---|
| Develop | |
| Problem/feature/bug has been solved/implemented/fixed. | |
| All methods and classes are check if they are used (Dead Code). | |
| The code-style matches the standards. | |
| Needed migration scripts for the server has been written. | |
| The code can be re-based to the master brunch. |  |
| The oomph installer is still working and generates a proper inspectIT work-space setup. | |
| When its code can be pushed in all conscience. | |
| The commit message has a useful commit message that is in correct format. |  |
| When readme.md is updated (otherwise we never publish new info on the GitHub). | |
| The non functional requirements (performance, stability,...) were taken into account during implementation. | |
| Build | |
| Everything compiles and is executable. | |
| The changed source code is available in the stable branch | |
| Q&A | |
| For each public method a JUnit Test exists. | |
| All implemented functionality is tested by unit tests. | |
| All necessary unit / component tests are existing. If tests are missing or the coverage of the class under test is too low, it has to be discussed between the developer and the integration manager. | |
| All the unit and static code tests are working. | |
| The whole build (releaseAndAnalyze) executes without any errors and warnings. | |
| The regression test is executed and a small explorative test has been done. | |
| The integration test has been updated (if required for a new feature). | |
| Integration | |
| The integration manager has looked upon the realization (in terms of functionality, architecture, performance, ...) and approved it | |
| At least one other person has checked and tested (e.g. integration manager) the code. | |
| Documentation | |
| 4-eyes see the documentation. | |
| Documentation has been created/updated for the ticket. | |
| Process | |
| The related JIRA tasks have been closed. | |
| Documenting tasks is done in detail on JIRA. | |
| Other | |
| A blog post has been released (for big and important features). |
We want in Future
| Description | Automated Checks |
|---|---|
| Demo application | |
| When (if possible) feature "is presented" on the demo app as well. | |
| The developed feature has been verified on the central AWS instance. | |
| When stuff is deployed on AWS and some kind of automatic test run. | |
| Performance | |
| When the new feature is tested with JMH. | |
| When at least once a profiler is used to check the new feature. | |
| Have a performance test for some core features. Perform performance regression tests using jenkins plugin. | |
| Q&A | |
| When we know it runs on different type of JVMs (think providers, versions) and/or application/web servers. | |
| when some level of code coverage is reached (excluding getters/setters/constructors). | |