Devhub 3.0.0

[jwgmeligmeyling]

Finally this is the Devhub 3.0.0 pull request. I managed to get all the existing integration tests to work. However, it involved some quite deep nested mocking, so we should probably have a look at this. I also did a smoke test after the rebase. We should however, first do some more testing / review before deployment.

Changelog

• The current User and Group are now injected into the @RequestScoped resources. This deduplicates logic checking if the user is allowed to view project information for the current group.
• The Selenium Firefox driver is updated to the newest version

Features

• Ability to create Pull Requests for branches
• Ability to merge pull requests into the master branch
• Ability to create/update courses
• Ability to create/update assignments for courses
• A student can submit deliveries for assignments
• Assistants and administrators can review assignment deliveries
• Assistants can be added to / removed from courses
• Ability to add comments to pieces of code for review (files, diffs, pull requests)

Deployment notes

• This pull request requires changes on the Git server: Devhub 3.0.0 git-server#41 (This is also why the CI test fails)
• This pull request requires changes on the build server: Optional branch name build-server#15
• Please also merge Reject invalid keys Java-Gitolite-Manager#18 to prevent invalid ssh keys
• In the configuration file, a folder should be specified where Devhub should store file attachments (for me /etc/devhub-server/storage/

[jwgmeligmeyling]

Finally I have managed to get through all the small tweaks, bugs and new feature request. Not only can we add inline comments, we can also add comments to the scope of a pull request or commit. Grades can now be downloaded. The tests pass and the test environment has been redeployed.

Ready for deployment! @michaeldejong

[TimvdLippe]

Change to "No closed/merged pull requests in this repository." as you can have unmerged pull requests, but they don't show up in this table.

[jwgmeligmeyling]

Fixing this atm.

[jwgmeligmeyling]

On hold for two small fixes (estimate 2 hours for fix + full pass)

[jwgmeligmeyling]

Fixed the issues and redeployed the test environment.

[jwgmeligmeyling]

Added the @Type(type = "org.hibernate.type.TextType") annotations to the entity classes to work properly with Postgres. I say properly because it already worked under Postgres, but after querying the database through psql I found that the TEXT fields were stored as blobs and showed up as numbers. I have tried various variants on the web, with/without @Lob, @Basic, and the newer @Type(type="org.hibernate.type.StringClobType"). Even more difficult was getting it to work under H2.

For now this seems to be the only way to go:

• Annotate TEXT types with @Type(type = "org.hibernate.type.TextType")
• Run H2 under PostgreSQL mode and use a Postgres dialect for Hibernate.

Redeployed the test environment, but dropped the existing schema.

[jwgmeligmeyling]

Fixed merge conflict for @Type(type = "org.hibernate.type.TextType").