SuccessChanges

Summary

  1. Remove useless code (details)
  2. Star'ing should happen outside of the UI thread. (details)
  3. Set the review flag outside the UI thread (details)
  4. Fix typo (details)
  5. Fix test failure (details)
  6. Allow the address and port of the gerrit test server to be configured by (details)
  7. Configure proxy for running from the eclipse foundation (details)
  8. Topic list does not refresh properly (details)
  9. Make the tests more resilient to slow gerrit servers (details)
  10. Tuleap #725 - Rebase button in the editor doesn't work (details)
  11. Tuleap #707 - Add rcptt test for the FilesDialog functionality (details)
  12. Fix reliability issue in RebaseButton test (details)
  13. Force reload does not cause an update to the last updated date (details)
  14. Tuleap #665 - Current user allowed to remove reviewer when not (details)
  15. revert test needs git server having an initial contribution (details)
  16. Make the wait time in rcptt configurable (details)
  17. Merge all the reviewers tests together (details)
  18. Delete old reviewers test (details)
  19. Make sure to remove test1 server at the end of the reviewer test (details)
  20. make sure there is a git project submitted (details)
  21. [Tuleap #732] Fix NPE in RefreshRelatedEditors.run() (details)
  22. Tweak pom.xml to make sure code coverage data is generated (details)
  23. Add wait in DraftPublish test (details)
  24. Add waits in the reviewer modification test (details)
  25. Add test checking that we open the doc (details)
  26. Make the RCPTT wait time configurable by a system property (details)
  27. Pass RCPTTWaitTime to RCPTT invocation (details)
  28. Fix error in open documentation test (details)
  29. Remove open documentation test because we can't make it work atm. (details)
  30. Fix typo in spelling of Jenkins (details)
  31. add missing dependencies to httpresponse (details)
  32. Update to 1.3 (details)
  33. [Tuleap #706] Add tests for refresh related editors (details)
  34. Remove unnecessary default constructor (details)
  35. Attempt at stabilizing the RefreshReladtedEditors (details)
  36. Tuleap #648 - On Oxygen, when swap side is enabled, the right side (details)
  37. runQuery and runQueryAndOpenEditor should wait (details)
  38. OpenReviewEditor needs to wait for the editor to open (details)
  39. Remove unused code (details)
  40. Tuleap #719 - Persistent information for the files table in compare (details)
  41. Tuleap #709 - ModelLoader needs be more specific to track changeInfos (details)
  42. Tuleap #667 - option to remove all check marks reviewed files from a (details)
  43. Tuleap #740 - Delete reviewer should refresh the review (details)
  44. Apply consistent formatter settings across all the projects (details)
  45. Remove deadcode (details)
  46. cleanup unecessary projects and tweak dependencies among egerrit bundles (details)
  47. Exclude from code coverage two classes with only statics (details)
  48. Tuleap #745 - React to removal of internal method in compare (details)
  49. Use Eclipse platform I-build site as a source for the SDK (details)
  50. Revert accidental release of a formatter change (details)
  51. NPE using reply from the compare editor (details)
  52. Bug 508087 - Stop opening Markers View when activating review markers (details)
  53. Use version of the compare plugin to tell whether swap is supported or (details)
  54. Stop the ActivateMarkerTest temporary (details)
  55. tuleap #747 Fix null pointer when using CTRL + 6 (details)
  56. Clarify navigation in compare editor (details)
  57. tuleap #722 use thread when performing (details)
  58. Tuleap #531 - In the plain text compare editor, adding a comment cause (details)
  59. Tweak the comment coloration logic. (details)
  60. Comments can't be empty (details)
  61. Improve test readability (details)
  62. Tuleap #737 - Compare editor should show the number of comments for the (details)
  63. Persistent storage added to fix the issue with review 89144 (details)
  64. tuleap #748 Add test for the ActivateMarkers (details)
  65. Fix formatting of the tycho-surefire-invocation parameters (details)
  66. tuleap #721 Activate comment Markers outside the UI thread. (details)
  67. re-adjust the test time when activating the markers. (details)
  68. Tuleap #754 - Comment edition is broken when swapping is done (details)
Commit e93880bbdc57db907f5ae792e5e494f01cbb7c9c by Pascal Rapicault
Remove useless code

Change-Id: I28eab8bc9e2a601480d39a904bc2b5c89eb2047a
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.dashboard.ui/src/org/eclipse/egerrit/internal/dashboard/ui/commands/table/AdjustMyStarredHandler.java
Commit 708a65a5ae189af587e34dd8019a54b98c57a630 by Pascal Rapicault
Star'ing should happen outside of the UI thread.

- The star and unstar happens outside of the UI thread.
- The Star/UnStartCommand now owns the responsibility of updating the
model.
- Model change is done pre-emptively to ensure maximum responsiveness.


Change-Id: I4462c177fcda663e98f129cf2f62369020fdde02
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/command/UnstarChangeCommand.java
The file was modifiedorg.eclipse.egerrit.dashboard.ui/src/org/eclipse/egerrit/internal/dashboard/ui/views/GerritTableView.java
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/UnstarCommandTest.java
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/GerritClient.java
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/command/StarChangeCommand.java
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/StarCommandTest.java
Commit d89f28221158e9b65ce3a0c7becfb44911ebdeb4 by Pascal Rapicault
Set the review flag outside the UI thread

- Change the invocation of the QueryHelpers methods
- Move the logic to set the flag into the command

Change-Id: Ifa333ee2c14bc5bbb1277d8dac7f7c4e8ad0ff61
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/CommandTest.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CompareUpperSection.java
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/command/SetReviewedCommand.java
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/UnstarCommandTest.java
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/DeleteReviewedCommandTest.java
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/GerritClient.java
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/SetReviewedCommandTest.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/UIFilesTable.java
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/command/DeleteReviewedCommand.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/QueryHelpers.java
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/GetReviewedCommandTest.java
Commit 3bb7258f840065578b71632d05b5b48a9b0621f3 by Pascal Rapicault
Fix typo

Change-Id: I8d771a003863902fbcdd6a9a35e94188ab8e1dbf
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/GerritDiffNode.java
Commit a66aa6549d9bbdd0a8a5f303c91034dc1052adc5 by Pascal Rapicault
Fix test failure

To test the status of a review in the dashboard, we acquire the row by
the value of the first column. This value changed when we improved the
presentation of the star.


Change-Id: I1eb70e6843d9433225b956a4f2c55fed48fd9b55
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
Commit dd13a7aa21529e84f9b66f78172e69f2927e0ff2 by Pascal Rapicault
Allow the address and port of the gerrit test server to be configured by
a system property

Change-Id: Ic08136435932219b0470bc0505d8f966c5836108
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/tests/Common.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/pom.xml
The file was modifiedpom.xml
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/SetGlobalVariables.ctx
The file was modifiedorg.eclipse.egerrit.core.tests/pom.xml
Commit 367455822e029a8822edd72073e2fdc47995d9e5 by Pascal Rapicault
Configure proxy for running from the eclipse foundation

Change-Id: I7ea0d5112c0831485a01ac866a7d4ff221e16af6
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/tests/Common.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/SubmitRevertReview.test
The file was modifiedorg.eclipse.egerrit.core.tests/pom.xml
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/tests/GerritRepositoryTest.java
The file was addedorg.eclipse.egerrit.releng/eclipseFoundation-proxy-settings.ini
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/GerritRepository.java
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/tests/Gerrit_2_11Test.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/SetProxyToDirect.ctx
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/CommandTest.java
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/tests/Gerrit_2_9Test.java
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/tests/GerritTest.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/dashboard/ValidateStar.test
The file was modifiedorg.eclipse.egerrit.ui.tests/src/org/eclipse/egerrit/ui/tests/DashboardCompletionTest.java
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/GerritCredentials.java
The file was modifiedorg.eclipse.egerrit.ui.tests/pom.xml
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ChangeCommitMessage.test
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/GerritHttpClient.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ReplyToReview.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/drafts/ReviewWithDraftPatchSet.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/pom.xml
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/tests/GerritFactoryTest.java
Commit 2c47f184222e1d5519f2dd660f35559174f26eec by Jacques Bouthillier
Topic list does not refresh properly

Change-Id: I33e451f70d7707cc2fe7848d8b932a787ce6b21f
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/AddRemoveTopic.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/QueryHelpers.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
The file was addedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/DeleteTopicCommandTest.java
The file was addedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/command/DeleteTopicCommand.java
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/GerritClient.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/tabs/DetailsTabView.java
The file was removedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/AddTopic.test
Commit f65e77ce4ef227ee87f86b11b0c8880c81d2c9bd by Pascal Rapicault
Make the tests more resilient to slow gerrit servers

The tests assumed the test server was running locally, and expected the server response to have been received
when the test was checking the assertions. However when running against a remote server (e.g. when we are running the tests
from the foundation server, the test server is hosted on a google machine), this is no longer true, and we need to introduce delays
before checking the assertions.

Change-Id: I5d405956db625587f314683d1258e9aa52bb113f
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/AbandonRestore.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/AmendReview.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/OpenReviewEditor.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/AddRemoveTopic.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ReplyToReview.test
The file was removedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/tests/Gerrit_2_9Test.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/SubmitRevertReview.test
The file was modifiedorg.eclipse.egerrit.dashboard.ui/src/org/eclipse/egerrit/internal/dashboard/ui/commands/table/AdjustMyStarredHandler.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/dashboard/ValidateStar.test
Commit aca71d15b93b80394ef6e6bb262ff9223340a91c by Pascal Rapicault
Tuleap #725 - Rebase button in the editor doesn't work

Change-Id: I6bcf02454e4cb2cd12552d010436b443402a7763
Signed-off-by: Patrick-Jeffrey Pollo Guilbert <patrick.pollo.guilbert@ericsson.com>
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/InitialSubmit.ctx
The file was removedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/RebaseCommandTest.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/ChangeDetailEditor.java
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/GerritClient.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/RebaseButton.test
The file was modifiedorg.eclipse.egerrit.model/src/org/eclipse/egerrit/internal/model/ModifiedRevisionInfoImpl.java
The file was removedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/command/RebaseCommand.java
Commit b0b208e41c8324362da0282bb5e3bc022dd259fb by Pascal Rapicault
Tuleap #707 - Add rcptt test for the FilesDialog functionality

Change-Id: Ic4fe69b155ced372d59aabd0001c7ca1d2e70704
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ActivateMarkerView.test
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/FilesDialog.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
Commit ef64d74deb7ff1d122468255a073e502cc1bb774 by Pascal Rapicault
Fix reliability issue in RebaseButton test

Change-Id: I128fc6113a35ab612a3481f81bf85916d564b35f
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/InitialSubmit.ctx
Commit c6422659587b7a14639d9d1c67646da718fc2c31 by Pascal Rapicault
Force reload does not cause an update to the last updated date

The detection of a change in the review is done by watching the last
updated date.
When we are forcing a reload, this date needs to be forced-modified in
order for the listeners watching this date to be notified.

Also, in order to avoid a number of spurious reloads, the listener
reloading the basic information is disabled when doing a force reload.


Change-Id: I1f2fb6a696ab5650f25023ab15e9b1dede4f9f72
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/ModelLoader.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/QueryHelpers.java
Commit ed28fa7deea61d9d58327ae483064f913f437384 by Pascal Rapicault
Tuleap #665 - Current user allowed to remove reviewer when not
permitted.

Change-Id: I6c5633aba539604f86fe6af1e539056a78f1fc0c
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.model/src-gen/org/eclipse/egerrit/internal/model/ModelPackage.java
The file was modifiedorg.eclipse.egerrit.model/src/org/eclipse/egerrit/internal/model/ModifiedChangeInfoImpl.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/provider/ReviewersTableLabelProvider.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/tabs/DetailsTabView.java
The file was modifiedorg.eclipse.egerrit.model.edit/plugin.properties
The file was modifiedorg.eclipse.egerrit.model/src-gen/org/eclipse/egerrit/internal/model/ReviewerInfo.java
The file was modifiedorg.eclipse.egerrit.model/src-gen/org/eclipse/egerrit/internal/model/impl/ModelPackageImpl.java
The file was modifiedorg.eclipse.egerrit.model.edit/src-gen/org/eclipse/egerrit/internal/model/provider/ReviewerInfoItemProvider.java
The file was modifiedorg.eclipse.egerrit.model/model/egerrit.ecore
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/QueryHelpers.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ReviewerModification.test
The file was modifiedorg.eclipse.egerrit.model/src-gen/org/eclipse/egerrit/internal/model/impl/ReviewerInfoImpl.java
Commit 9feb9160204bbe34d3e624a8a996c194cc2dbad8 by Jacques Bouthillier
revert test needs git server having an initial contribution

Change-Id: I5e972f6a66c2b2cf6f0cce6fc0141d20f42b4d9d
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/RevertCommandTest.java
Commit 5261897e4afe6d1909e39544b043030277a62534 by Pascal Rapicault
Make the wait time in rcptt configurable

Change-Id: I64e2beadda23c86cb647b4227e82b349abc60b71
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ReviewerModification.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CompareEditorComment.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/OpenReviewEditor.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/InitialSubmit.ctx
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/SetGlobalVariables.ctx
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/AbandonRestore.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/ModifyDeleteDraft.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/drafts/ReviewWithDraftPatchSet.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/dashboard/ValidateStar.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ChangeCommitMessage.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/RebaseButton.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/AmendReview.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/AddRemoveTopic.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ReplyToReview.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/SubmitRevertReview.test
Commit 98646e5310c610b39870d618b04e4475e67808c8 by Pascal Rapicault
Merge all the reviewers tests together

Change-Id: I09cbac1394ea883a893d229e3d81ce3fd25f90be
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ReviewerModification.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/AddReviewers.test
Commit 918248f016c1b9b3939deb890595ca9bcbb40faf by Pascal Rapicault
Delete old reviewers test

Change-Id: I303b34a8647254f46faa102c293bc853dabfe28c
The file was removedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/AddReviewers.test
Commit 03d082ef3acc9ab5910f6ac12ed1fd6aef48d819 by Pascal Rapicault
Make sure to remove test1 server at the end of the reviewer test

Change-Id: Ia07582d8216e7a4abb9865f66fc70aff7315b1d2
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ReviewerModification.test
Commit b1bbedd66f13f4b564e381f34df2b97fdf61dd0b by Jacques Bouthillier
make sure there is a git project submitted
before testig the buttons.

Change-Id: I8ffd320899aff2e114447cf9e29007c2b1c8dbac
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/CheckButtons.test
Commit 25401b79afc242c5b5e16374d02c9bc47b510db4 by Patrick-Jeffrey Pollo Guilbert
[Tuleap #732] Fix NPE in RefreshRelatedEditors.run()

Change-Id: I6feea8befe687e096c4ee7079a6b0a47299cb7cb
Signed-off-by: Patrick-Jeffrey Pollo Guilbert <patrick.pollo.guilbert@ericsson.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/RefreshRelatedEditors.java
Commit 4c01a17c052e653efa17f9aeb362ffd7b67c3f98 by Pascal Rapicault
Tweak pom.xml to make sure code coverage data is generated

- Add tycho.testArgLine to the hudson profile
- Movei additional sonar properties in the pom

Change-Id: I688fa8b479cc8611881abd390a950f789f9bf941
The file was modifiedpom.xml
The file was modifiedorg.eclipse.egerrit.ui.tests/pom.xml
The file was modifiedorg.eclipse.egerrit.core.tests/pom.xml
Commit 668ce455664f3e0ce1d632c06ed4c1c23dcc4fb3 by Pascal Rapicault
Add wait in DraftPublish test

Change-Id: I700b7b65704c1f906ae14cbbe2839f9509786ef3
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/drafts/DraftPublish.test
Commit a8c4e84b9f9b2c20006dbbb9e42e72f8e0a5538b by Pascal Rapicault
Add waits in the reviewer modification test

Change-Id: Idc52faad990e0477240c596c271c48581bc8fd11
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ReviewerModification.test
Commit c537404ad102fc775451fc7e7e5930224973ab11 by Pascal Rapicault
Add test checking that we open the doc

Change-Id: Iee2ce82e707791ea7e883ab4880e432a35e4ba86
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/dashboard/OpenDocumentation.test
Commit c356267fd50cfdac8bf7cd17035c5436420591a2 by Pascal Rapicault
Make the RCPTT wait time configurable by a system property

Change-Id: I13184266f97b68fb0f2748dccfd9643b3b0e6538
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/SetGlobalVariables.ctx
Commit 94e854db0a1e8353682cd7c7da29189e30e13157 by Pascal Rapicault
Pass RCPTTWaitTime to RCPTT invocation

Change-Id: I766da7f2a9d7e25523c61ad0b5b8e2317df14133
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/pom.xml
The file was modifiedpom.xml
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/SetGlobalVariables.ctx
Commit 76f62905731b468f8a3cce3f3a38c4d30d67ee58 by Pascal Rapicault
Fix error in open documentation test

Change-Id: Ieaae4ed8ab394d3ce7d57ace2a235c90acd35659
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/dashboard/OpenDocumentation.test
Commit cd67e134f8e867906699bdb635612245fc52fcb5 by Pascal Rapicault
Remove open documentation test because we can't make it work atm.

Change-Id: Ib7900513f2a68ff2e0f871cddfb1a05e73bb76da
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was removedorg.eclipse.egerrit.ui.rcptt.tests/dashboard/OpenDocumentation.test
Commit 6cc7a50fa3760229801377399a76a88916c1f539 by Pascal Rapicault
Fix typo in spelling of Jenkins

Change-Id: Iaed5f144158fabcf3f24315a3c60806f400f8dda
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/filter/AuthorKindFilter.java
Commit 47fab6bc5abae4927994c4fc09dc9f2f55be8a15 by Jacques Bouthillier
add missing dependencies to httpresponse

Change-Id: I4e458fccd19d850bf70d821189340d0bd835e189
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.ui/META-INF/MANIFEST.MF
Commit 11c5677401cc01e8c7b717e0bdfc60a092142a9b by Jacques Bouthillier
Update to 1.3

Change-Id: I4a7a132fa70af8697f9a24ec09c243eb725c9705
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.updatesite.tests/category.xml
The file was modifiedorg.eclipse.egerrit.ui.tests/META-INF/MANIFEST.MF
The file was modifiedorg.eclipse.egerrit.updatesite.tests/pom.xml
The file was modifiedorg.eclipse.egerrit.core/META-INF/MANIFEST.MF
The file was modifiedorg.eclipse.egerrit.feature/pom.xml
The file was modifiedorg.eclipse.egerrit.target/pom.xml
The file was modifiedorg.eclipse.egerrit.core.tests/pom.xml
The file was modifiedorg.eclipse.egerrit.model.edit/pom.xml
The file was modifiedorg.eclipse.egerrit.updatesite/category.xml
The file was modifiedorg.eclipse.egerrit.dashboard.ui/META-INF/MANIFEST.MF
The file was modifiedorg.eclipse.egerrit.ui/pom.xml
The file was modifiedorg.eclipse.egerrit.dashboard.core/pom.xml
The file was modifiedorg.eclipse.egerrit.core.tests/META-INF/MANIFEST.MF
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/pom.xml
The file was modifiedorg.eclipse.egerrit.releng/publication/pom.xml
The file was modifiedorg.eclipse.egerrit.updatesite/pom.xml
The file was modifiedorg.eclipse.egerrit.dashboard.core/META-INF/MANIFEST.MF
The file was modifiedorg.eclipse.egerrit.releng/update-versions.sh
The file was modifiedpom.xml
The file was modifiedorg.eclipse.egerrit.model/META-INF/MANIFEST.MF
The file was modifiedorg.eclipse.egerrit.feature/feature.xml
The file was modifiedorg.eclipse.egerrit.dashboard.ui/pom.xml
The file was modifiedorg.eclipse.egerrit.model.edit/META-INF/MANIFEST.MF
The file was modifiedorg.eclipse.egerrit.model/pom.xml
The file was modifiedorg.eclipse.egerrit.core/pom.xml
The file was modifiedorg.eclipse.egerrit.ui/META-INF/MANIFEST.MF
The file was modifiedorg.eclipse.egerrit.ui.tests/pom.xml
The file was modifiedorg.eclipse.egerrit.model.tests/META-INF/MANIFEST.MF
Commit 6abfd4099c46593539085886399352f90409b387 by Patrick-Jeffrey Pollo Guilbert
[Tuleap #706] Add tests for refresh related editors

Also add the filename attribute to CreateReview in the model so that
reviews with different filenames can be created.

Change-Id: I3f86fce33c67f889b500081023e1a1a689f7c6e5
Signed-off-by: Patrick-Jeffrey Pollo Guilbert <patrick.pollo.guilbert@ericsson.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/process/RebaseProcess.java
The file was modifiedorg.eclipse.egerrit.ui.tests/src-gen/org/eclipse/egerrit/internal/rcptt/support/egerritecl/CreateReview.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/SetGlobalVariables.ctx
The file was modifiedorg.eclipse.egerrit.ui.tests/src/org/eclipse/egerrit/internal/rcptt/support/ReviewFactory.java
The file was modifiedorg.eclipse.egerrit.ui.tests/src/org/eclipse/egerrit/internal/rcptt/support/CreateReviewService.java
The file was modifiedorg.eclipse.egerrit.ui.tests/src-gen/org/eclipse/egerrit/internal/rcptt/support/egerritecl/egerriteclPackage.java
The file was modifiedorg.eclipse.egerrit.ui.tests/src-gen/org/eclipse/egerrit/internal/rcptt/support/egerritecl/impl/CreateReviewImpl.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/RefreshRelatedEditors.test
The file was modifiedorg.eclipse.egerrit.ui.tests/model/egerritecl.ecore
The file was modifiedorg.eclipse.egerrit.ui.tests/src-gen/org/eclipse/egerrit/internal/rcptt/support/egerritecl/impl/egerriteclPackageImpl.java
Commit 3e4aa89648a4f23652f0e45671974d3e6e0c9225 by Pascal Rapicault
Remove unnecessary default constructor

Change-Id: I9e93f500da8a0151980575c9bc42aa76f5c87538
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/provider/BranchListLabelProvider.java
Commit 7e40a2373ab71dde27da7af9240c885a5f1814b1 by Pascal Rapicault
Attempt at stabilizing the RefreshReladtedEditors

The listener on related changes needs to be registered before the call
to refresh is done

Change-Id: Ie0bc06b2d398d145ee53cafd1baedba9a4ecc81c
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/process/RebaseProcess.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/RefreshRelatedEditors.test
Commit 831661ab16c93b40b001c15bf1e118c634768b12 by Pascal Rapicault
Tuleap #648 - On Oxygen, when swap side is enabled, the right side
should be the patchset.

Two additional things in this fix:
- Update the title of the editor as the user selects a new patchset
- Cleanup the translation strings used in the creation of the title (the
title was not properly presented).

Change-Id: I27df451a317e9b34c0a1b95df1e0bf138cf6d4a7
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/messages.properties
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/Messages.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CompareUpperSection.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/OpenCompareEditor.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/GerritMultipleInput.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/UICompareUtils.java
Commit d20b9efc0802e78f7b5c8ecfe36b5f3da9970085 by Pascal Rapicault
runQuery and runQueryAndOpenEditor should wait

Change-Id: I8115d56d0014c3a70f3526f33449e4798358d531
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
Commit fac494ee2b5e93a7d73c0a491e0d036605831ed3 by Pascal Rapicault
OpenReviewEditor needs to wait for the editor to open


Change-Id: If4d9dd52b908d8a7dfe0ef18df4a11b48d6f1a24
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/OpenReviewEditor.test
Commit 192a4a147dc1bd50115b2c1c57dfa6dcf6e0a54a by Jacques Bouthillier
Remove unused code

Change-Id: Ie3c95f3fd8183095467be8d9a4c4dfb02ab1af93
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was removedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/tabs/AlwaysOnLinkManager.java
Commit 8cb8c1698d98c6952909c9c9a56871f7cc474907 by Pascal Rapicault
Tuleap #719 - Persistent information for the files table in compare
and history tab view

Change-Id: I8196da9839dd730f27215233e02a25377940afb5
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CompareUpperSection.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/FilesDialog.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/tabs/HistoryTabView.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/UIFilesTable.java
The file was addedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/PersistentStorage.java
The file was modifiedorg.eclipse.egerrit.ui/META-INF/MANIFEST.MF
Commit fe579e2e4e0e4babadfc351604a5aa96c2989cf5 by Pascal Rapicault
Tuleap #709 - ModelLoader needs be more specific to track changeInfos

The modelloader needs to keep track of its instances by using the
gerritclient and the changinfo instead of just the changeinfo which is
not specific enough to distringuish for cases where the user changes
login.

Change-Id: I4f37ad1f05994c88130903a39cb5c42c888bc284
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/ModelLoader.java
Commit e99c089b049a3bb6f4d6c51a91064888f8fabb82 by Jacques Bouthillier
Tuleap #667 - option to remove all check marks reviewed files from a review.

Change-Id: I051d80f1e272cb77e22a3cb6459de534bba9d671
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/UIFilesTable.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/Messages.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/provider/DynamicMenuBuilder.java
The file was addedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/ClearReviewedFlag.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/messages.properties
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CompareUpperSection.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ClearReviewedFlag.test
Commit 97c684144072b154e7306f96fefc15ea237fa976 by Pascal Rapicault
Tuleap #740 - Delete reviewer should refresh the review

The deletion of reviewers may render the review submittable, so we need
to force the review to be fully refreshed.

Change-Id: I3b0b45c852984e44fbaf15091884e981e188d2aa
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/tabs/DetailsTabView.java
Commit 40ad0286a5f37783c4f0b67b53397cb3b329f1cb by Pascal Rapicault
Apply consistent formatter settings across all the projects

Change-Id: I7fb380da7841636248687cd78c3104f527fe6326
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was addedorg.eclipse.egerrit.model.tests/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/ShowReviewEditorHandler.java
The file was modifiedorg.eclipse.egerrit.dashboard.ui/src/org/eclipse/egerrit/internal/dashboard/ui/views/VoteHandler.java
The file was addedorg.eclipse.egerrit.model/.settings/org.eclipse.jdt.ui.prefs
The file was modifiedorg.eclipse.egerrit.dashboard.ui/.settings/org.eclipse.jdt.ui.prefs
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/ReplyDialog.java
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/command/EMFTypeAdapterFactory.java
The file was modifiedorg.eclipse.egerrit.dashboard.core/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/GetModifiedFilesCommandTest.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/tabs/MessageTabView.java
The file was modifiedorg.eclipse.egerrit.model/src/org/eclipse/egerrit/internal/model/ModifiedEGerritFactory.java
The file was modifiedorg.eclipse.egerrit.core.tests/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.core.tests/.settings/org.eclipse.jdt.ui.prefs
The file was modifiedorg.eclipse.egerrit.model/src/org/eclipse/egerrit/internal/model/ModifiedRevisionInfoImpl.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CommitCompareItem.java
The file was modifiedorg.eclipse.egerrit.ui.tests/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.ui.tests/.settings/org.eclipse.jdt.ui.prefs
The file was addedorg.eclipse.egerrit.model.edit/.settings/org.eclipse.core.resources.prefs
The file was modifiedorg.eclipse.egerrit.core.tests/src/org/eclipse/egerrit/core/command/tests/CommandTest.java
The file was modifiedorg.eclipse.egerrit.core/src/org/eclipse/egerrit/internal/core/ServersStore.java
The file was modifiedorg.eclipse.egerrit.core/.settings/org.eclipse.jdt.ui.prefs
The file was modifiedorg.eclipse.egerrit.dashboard.ui/.settings/org.eclipse.jdt.core.prefs
The file was addedorg.eclipse.egerrit.model.edit/.settings/org.eclipse.jdt.ui.prefs
The file was modifiedorg.eclipse.egerrit.model/src/org/eclipse/egerrit/internal/model/ModifiedChangeInfoImpl.java
The file was modifiedorg.eclipse.egerrit.dashboard.ui/src/org/eclipse/egerrit/internal/dashboard/ui/utils/UIUtils.java
The file was addedorg.eclipse.egerrit.model.tests/.settings/org.eclipse.jdt.ui.prefs
The file was modifiedorg.eclipse.egerrit.model/src/org/eclipse/egerrit/internal/model/ModelHelpers.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/tabs/HistoryTabView.java
The file was modifiedorg.eclipse.egerrit.ui/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.dashboard.core/.settings/org.eclipse.jdt.ui.prefs
The file was modifiedorg.eclipse.egerrit.model/src/org/eclipse/egerrit/internal/model/ModifiedFileInfoImpl.java
The file was modifiedorg.eclipse.egerrit.model/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.ui.tests/pom.xml
The file was addedorg.eclipse.egerrit.model/.settings/org.eclipse.core.resources.prefs
The file was addedorg.eclipse.egerrit.model.edit/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/provider/HistoryTableMenuBuilder.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/AnnotationContributionItems.java
The file was modifiedorg.eclipse.egerrit.core/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/provider/PatchSetHandlerProvider.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/PatchSetCompareItem.java
The file was modifiedorg.eclipse.egerrit.core.tests/pom.xml
The file was modifiedorg.eclipse.egerrit.ui/.settings/org.eclipse.jdt.ui.prefs
Commit 1d13c8247f04918da5a1227c1d3527f32d052d64 by Pascal Rapicault
Remove deadcode

Change-Id: I739e7fe1f5aca15401e0ed245c73c47dcde1d57a
The file was modifiedorg.eclipse.egerrit.dashboard.core/src/org/eclipse/egerrit/internal/dashboard/GerritPlugin.java
Commit 796705dd586e9a87b5e5fec5a0ecedef78145111 by Pascal Rapicault
cleanup unecessary projects and tweak dependencies among egerrit bundles

Change-Id: I640bc781420909747d2e07104de423645f38c4fa
The file was removedorg.eclipse.egerrit.model.tests/META-INF/MANIFEST.MF
The file was modifiedorg.eclipse.egerrit.core/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.core.tests/.settings/org.eclipse.jdt.core.prefs
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ChangeMessageInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/CommitInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ActionInfoTest.java
The file was modifiedorg.eclipse.egerrit.core.tests/.classpath
The file was removedorg.eclipse.egerrit.model.tests/plugin.properties
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/IncludedInInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ProblemInfoTest.java
The file was removedorg.eclipse.egerrit.model.editor/src-gen/org/eclipse/egerrit/internal/model/presentation/ModelEditor.java
The file was modifiedorg.eclipse.egerrit.dashboard.core/.classpath
The file was removedorg.eclipse.egerrit.model.editor/src-gen/org/eclipse/egerrit/internal/model/presentation/ModelModelWizard.java
The file was modifiedorg.eclipse.egerrit.ui/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.ui.tests/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.model.edit/.settings/org.eclipse.jdt.core.prefs
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/StringToRevisionInfoTest.java
The file was modifiedorg.eclipse.egerrit.model.edit/.classpath
The file was removedorg.eclipse.egerrit.model.tests/build.properties
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/BranchInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ProjectAccessInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/.project
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/StringToListOfStringTest.java
The file was removedorg.eclipse.egerrit.model.tests/.settings/org.eclipse.jdt.ui.prefs
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/CommentRangeTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/LabelInfoTest.java
The file was modifiedorg.eclipse.egerrit.dashboard.ui/.settings/org.eclipse.jdt.core.prefs
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/GitPersonInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/StringToFetchInfoTest.java
The file was modifiedorg.eclipse.egerrit.ui.tests/.classpath
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/MergeableInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/StringToLabelInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/.settings/org.eclipse.jdt.core.prefs
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ApprovalInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/StringToActionInfoTest.java
The file was removedorg.eclipse.egerrit.ui.tests.tests/.project
The file was removedorg.eclipse.egerrit.model.editor/src-gen/org/eclipse/egerrit/internal/model/presentation/ModelActionBarContributor.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ReviewInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ReviewerInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/AccountInfoTest.java
The file was modifiedorg.eclipse.egerrit.model.edit/META-INF/MANIFEST.MF
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ChangeInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/FetchInfoTest.java
The file was modifiedorg.eclipse.egerrit.ui.tests/META-INF/MANIFEST.MF
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/RevisionInfoTest.java
The file was removedorg.eclipse.egerrit.ui.tests.edit/.project
The file was removedorg.eclipse.egerrit.ui.tests.editor/.project
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/RelatedChangesInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/SubmitInfoTest.java
The file was modifiedorg.eclipse.egerrit.model/.settings/org.eclipse.jdt.core.prefs
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/StringToStringTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/EgerritAllTests.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/CommentInfoTest.java
The file was modifiedorg.eclipse.egerrit.ui/META-INF/MANIFEST.MF
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/FileInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ModelTests.java
The file was modifiedorg.eclipse.egerrit.dashboard.core/META-INF/MANIFEST.MF
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/StringToFileInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/.classpath
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ModelExample.java
The file was modifiedorg.eclipse.egerrit.dashboard.ui/META-INF/MANIFEST.MF
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/RelatedChangeAndCommitInfoTest.java
The file was removedorg.eclipse.egerrit.model.tests/src-gen/org/eclipse/egerrit/internal/model/tests/ProjectInfoTest.java
The file was modifiedorg.eclipse.egerrit.core.tests/META-INF/MANIFEST.MF
The file was modifiedorg.eclipse.egerrit.dashboard.core/.settings/org.eclipse.jdt.core.prefs
Commit d5adfe85fc5a013e5746cca815ffcfef5a6e53c2 by Jacques Bouthillier
Exclude from code coverage two classes with only statics
The file was modifiedpom.xml
Commit 34677d1d8f7096e0d4993f4ff94e36eda1b4290b by Jacques Bouthillier
Tuleap #745 - React to removal of internal method in compare

Change-Id: I6d4903430932ff670297b21d2692c5a163f5ed50
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/NextPreviousFileHandler.java
Commit 98e84cd3e1ee6f446a882737bd8437ecc7fbe57d by Jacques Bouthillier
Use Eclipse platform I-build site as a source for the SDK

Change-Id: If4af3816ae0028b70a081e9bca7483229c944ee7
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.target/eGerrit-Eclipse_4.7-oxygen.target
Commit e37c0e74c1d4fdf3b191a053f2b4794a12475091 by Pascal Rapicault
Revert accidental release of a formatter change


Change-Id: I4c8d44b7977e877822056bd1a05e02bb15987747
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui.tests/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.model.edit/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.core/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.dashboard.core/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.model/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.ui/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.core.tests/.settings/org.eclipse.jdt.core.prefs
The file was modifiedorg.eclipse.egerrit.dashboard.ui/.settings/org.eclipse.jdt.core.prefs
Commit e71c1182c7344a915b3e25c4f5afa7484e0b2b2a by Pascal Rapicault
NPE using reply from the compare editor

This bug resulted from the handling of the swap button in Oxygen.
GMI.getRightSide() returned what is given as input to the compare
editor, however this does not represent the value shown on the right in
the title bar of the compare editor.
We needed to introduce a new method returning what is actually shown so
the users of the input don't have to worry about whether the editor is
shown in mirror mode or not.

Change-Id: Icd4d732c6c8fa5977d4d9abcacb60b0f7e326a1b
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/SelectNextPatchSetHandler.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/messages.properties
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/GerritMultipleInput.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/ReplyHandler.java
Commit 00fdceb8f854c1f821249f880fc0bee6dd0decc7 by Pascal Rapicault
Bug 508087 - Stop opening Markers View when activating review markers
Also tuleap # 738 do not open the markers view when there is no comment

New behaviour is:
- Don't open the markers view if the problem view is opened
- If the problem view is closed, open the marker view only if we have comments
- The markers view no longer steals the focus

Change-Id: I9937daa9cf23ab383483612195e2f8f48d0115dc
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/ActiveWorkspaceRevision.java
Commit d550f5bd70973e141cc593efa60e08820c7947eb by Pascal Rapicault
Use version of the compare plugin to tell whether swap is supported or
not

Change-Id: I7c446d1002c9fb3837b0dcd94a95447ca6c7a9df
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/OpenCompareEditor.java
Commit 65ffbda07ae82a1cbef4067b9bb05f0748eb4285 by Jacques Bouthillier
Stop the ActivateMarkerTest temporary

Change-Id: I0ec88035483e74d7460c507dc7fe93199fab9621
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ActivateMarkerView.test
Commit 0a2b0f8bc6f3d6c7cc0c6a17eeed2a939f371e3b by Jacques Bouthillier
tuleap #747 Fix null pointer when using CTRL + 6
Reset the Dashboard table to the default value
when the selected setting is empty

Change-Id: I390fa0db9733f67231d9e7e099e1d6f20fa4b7ab
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.dashboard.ui/src/org/eclipse/egerrit/internal/dashboard/ui/commands/FixupTableDefaultHandler.java
The file was modifiedorg.eclipse.egerrit.dashboard.ui/src/org/eclipse/egerrit/internal/dashboard/ui/model/UIReviewTable.java
Commit 56eefbf37244b8ae71562870d73f70b5820cf3ee by Pascal Rapicault
Clarify navigation in compare editor

Before this patch, the presentation used for the selection of patchsets
(ps3 / ps2) lead the user to understand that the editors would show
patchsets in the same order: ps3 on the left and ps2 on the right.
This was right, but with the introduction of the swap button and our
attempt at reacting to the swap action by reordering the title in the
upper section of the editor caused additional troubles and unclear
behaviours. For example, what is the patchset that the reply button acts
on, which patchset does the "next comment" button acts on, etc.
In order to clarify the situation,  we have done the following changes:

- In the upper section, change the presentation of patchsets to explicit
the direction of the comparison (since ps3 vs ps1 is different than ps1
vs ps3).
- No longer swap the patchset in the upper section when the user hits
the swap button. The swap button only affects the editor area of the
compare section.
- The next / previous comment button applies to the editor showing the
patchset shown first in the title.

Change-Id: I9474bb200b487363335cab853e4386baad7362b2
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/GerritMultipleInput.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/OpenCompareFromDashboard.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CompareUpperSection.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CompareEditorDefaultOpen.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CompareEditorChangeInput.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/SelectNextPatchSetHandler.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/Messages.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/messages.properties
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/pom.xml
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CompareEditorComment.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/SwapSide.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentInReplyDialog.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/NextPreviousCommentAnnotationHandler.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/GoToNextFile.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/UICompareUtils.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/ReplyHandler.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/NextPreviousCommentAction.test
Commit 15787b9b84dfa5a7547a7e57e2b035d2c724b02a by Pascal Rapicault
tuleap #722 use thread when performing
abandon/retore commands
submit and rebase commands

Change-Id: I76c33cffa4e11ad49b025905d72fd70e2f426582
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/ChangeDetailEditor.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/process/RebaseProcess.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/process/SubmitProcess.java
Commit 7e6af228de9cca53be1f818e0e7d5df320afcd19 by Pascal Rapicault
Tuleap #531 - In the plain text compare editor, adding a comment cause exceptions.

When asked to apply the presentation for a piece of text, the
CommentAnnotationPainter must take into account the area being
repainted.

Change-Id: Ie3f42f06b679939735551525649a5ee9421ac977
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CommentAnnotationPainter.java
Commit 122486ee931b3afedabf4e6192d4335d9cfaf491 by Jacques Bouthillier
Tweak the comment coloration logic.

Existing problems addressed:
- The first character typed that causes a new line to be added does not
get colored
- Ctrl+D keeps an empty line
- Adding, deleting, then causes generates exceptions

Change-Id: I2f98e92b57f3e833640ccf93e3b4dbbbd67f542b
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/MultipleLineDeletion.test
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/OneCharacterComment.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/EditionLimiter.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/AddCommentBeginningLine.test
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/CommentSavedAtRightPlace.test
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/ConsecutiveComments.test
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/LineDeletion.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CommentExtractor.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/dashboard/ValidateBuiltInQueries.test
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/AddCommentFromPreviousLine.test
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/MultipleLineComment.test
The file was modifiedorg.eclipse.egerrit.ui.tests/src/org/eclipse/egerrit/ui/tests/CommentExtractorTest.java
Commit b7643251aa15e4b2bd2c91acba4e2acea1802c17 by Jacques Bouthillier
Comments can't be empty

Comments that are empty must not be pushed to the server

Change-Id: I02f35e83cf3786b8fc2fa922f18c42a905839541
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CommentExtractor.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/NoEmptyComments.test
Commit 4f3aa50bfa074dc1e59a1c81d0ccccf94ac2ce54 by Pascal Rapicault
Improve test readability

Change-Id: Ida921e0bb2b3620d1e6a488223526f1662ba1db9
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/MultipleLineComment.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/LineDeletion.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/MultipleLineDeletion.test
Commit bf3de1804f1a0f5ef67c5c8dbbab59df1b8d08b6 by Pascal Rapicault
Tuleap #737 - Compare editor should show the number of comments for the
both sides shown

Change-Id: I7a1faa79b54a828dadb74a52ff91d4e67d443726
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CommentableCompareItem.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsOnTwoSide.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/model/FilesTableModel.java
The file was addedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CompareUpperSectionColumn.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CompareUpperSection.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/table/provider/FileInfoCompareCellLabelProvider.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/Messages.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsAgainstBaseInPS.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/ShowFilePathHandler.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/messages.properties
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/FilterCommentedFiles.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CommentedFilesFilter.java
Commit 4e03db99b1886da7a8562f3aa52ca3e77e1b2627 by Jacques Bouthillier
Persistent storage added to fix the issue with review 89144

Change-Id: I18bc6e539a9630e9736082768e6b4e74361c276e
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/PersistentStorage.java
Commit 4c1b091d2915aed3e5a6cb6fcd0d3a11672a3db0 by Jacques Bouthillier
tuleap #748 Add test for the ActivateMarkers

Change-Id: Id6467731af798355601a76276ccffdd00be148dd
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ActivateMarkersInProblemsView.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ActivateMarkerView.test
Commit 97164e96c5c8b5ae29d1827e6ff5247d491c4dff by Pascal Rapicault
Fix formatting of the tycho-surefire-invocation parameters

Change-Id: I1472f2f017fbdd3f5d3359741218d888ccccca5c
The file was modifiedorg.eclipse.egerrit.core.tests/pom.xml
Commit 6875f696d2b87918fc69f1d5f4a8cd07e59fb564 by Jacques Bouthillier
tuleap #721 Activate comment Markers outside the UI thread.

Change-Id: I9117a0577f305704f71d6ef337820cddb09b2754
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ActivateMarkerView.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/FilesDialog.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ActivateMarkersInProblemsView.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/editors/ChangeDetailEditor.java
Commit e3ac9d873ebd0ef76215a69bf14df189ffa6dd62 by Jacques Bouthillier
re-adjust the test time when activating the markers.

Change-Id: I4e48bcf26f71030bc801f40a8500cc87dc84c305
Signed-off-by: Jacques Bouthillier <lmcbout@gmail.com>
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/FilesDialog.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ActivateMarkerView.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/HelperScripts.ctx
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/reviewEditor/ActivateMarkersInProblemsView.test
Commit 281167145a0f70413c78a929f70d82bca0cde8c2 by Pascal Rapicault
Tuleap #754 - Comment edition is broken when swapping is done

Because swap side can change whether the left or right side of the
compare editor is editable, the hooking of the edition limiter and
colorer needs to happen every time the document shown in the sub-editors
changes.

Change-Id: I9c8a4e3f7932a54976b2490c9666c4457513358b
Pascal Rapicault <pascal@rapicorp.com>
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/GerritMultipleInput.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/CannotCommentOnPartsOfFiles.test
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/EditSwapThenSave.test
The file was modifiedorg.eclipse.egerrit.ui.tests/src/org/eclipse/egerrit/ui/tests/CommentExtractorTest.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/EditionLimiter.java
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CommentAnnotationPainter.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/ContinueCommentAfterSwapping.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/LineDeletion.test
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/LineDeletionAfterSave.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CommentExtractor.java
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/OneCharacterComment.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/NoEmptyComments.test
The file was modifiedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/ConsecutiveComments.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/compare/CommentableCompareItem.java
The file was addedorg.eclipse.egerrit.ui.rcptt.tests/compare/CommentsCornerCases/EnterCommentAfterDeletion.test
The file was modifiedorg.eclipse.egerrit.ui/src/org/eclipse/egerrit/internal/ui/utils/messages.properties