summaryrefslogtreecommitdiff
path: root/spec/observers
Commit message (Collapse)AuthorAgeFilesLines
* Remove rails observers from applicationDmitriy Zaporozhets2014-06-171-63/+0
|
* Remove user observerDmitriy Zaporozhets2014-06-171-27/+0
|
* Remove NotesObserverDmitriy Zaporozhets2014-06-171-56/+0
|
* Remove UsersGroup observerDmitriy Zaporozhets2014-06-171-32/+0
|
* Fix user_project observer test.Marin Jankovski2014-04-031-1/+1
|
* Remove MergeRequest observerDmitriy Zaporozhets2014-04-021-131/+0
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Remove KeysObserverDmitriy Zaporozhets2014-04-021-23/+0
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Remove email observer specsDmitriy Zaporozhets2014-04-021-17/+0
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Remove issue observer testsDmitriy Zaporozhets2014-04-021-99/+0
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Merge branch 'event-create-service' into 'master'Dmitriy Zaporozhets2014-03-261-61/+0
|\ | | | | | | | | | | | | EventCreateService class The goal is to collect all event creation logic in one place called EventCreateService. Because now its placed in observers, controllers, services etc
| * Remove activity observer specsDmitriy Zaporozhets2014-03-251-61/+0
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Merge pull request #6374 from zzet/patch-10Dmitriy Zaporozhets2014-03-251-1/+1
|\ \ | |/ |/| Update merge_request_observer_spec.rb
| * Update merge_request_observer_spec.rbAndrew Kumanyaev2014-02-211-1/+1
| |
* | Fix specsDmitriy Zaporozhets2014-03-141-24/+0
|/ | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Add email aliases for usersJason Hollingsworth2014-02-131-0/+17
| | | | | | | | Emails are used to associate commits with users. The emails are not verified and don't have to be valid email addresses. They are assigned on a first come, first serve basis. Notifications are sent when an email is added.
* Make changes to testsDmitriy Zaporozhets2014-01-221-18/+12
| | | | | | | * project_with_code -> project * project -> ermpty_project Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* fix most of warningsskv2013-12-151-1/+1
|
* fix observer specsDmitriy Zaporozhets2013-12-105-24/+17
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix UserProject obserse specDmitriy Zaporozhets2013-12-101-4/+4
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Avoid duplicate group membership emailsJacob Vosmaer2013-12-091-0/+5
|
* Modify test to pass new project hooksDmitriy Zaporozhets2013-12-041-1/+1
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Updated missed stylesIzaak Alpert2013-10-101-2/+5
| | | | Change-Id: I05e02ff7e36e2350cdf5d2b94095ada9760a204e
* Fix style issuesIzaak Alpert2013-10-101-1/+1
| | | | Change-Id: I1ffe18414b7f6eae38d7860a8e3b64a9a185273f
* Brings this fix inline with updates to masterIzaak Alpert2013-10-101-0/+23
| | | | Change-Id: Ia50c3ac043b4c927ca59cbbdc2bba42f73e550e4
* Add UsersGroup observerDmitriy Zaporozhets2013-09-121-0/+27
| | | | Trigger notification service when new user-group relation was created
* Link issues from comments and automatically close themash wilson2013-08-254-12/+114
| | | | | | | | | | Any mention of Issues, MergeRequests, or Commits via GitLab-flavored markdown references in descriptions, titles, or attached Notes creates a back-reference Note that links to the original referencer. Furthermore, pushing commits with commit messages that match a (configurable) regexp to a project's default branch will close any issues mentioned by GFM in the matched closing phrase. If accepting a merge request would close any Issues in this way, a banner is appended to the merge request's main panel to indicate this.
* Fix passign full user object instead of id to Event.createDmitriy Zaporozhets2013-08-141-1/+0
|
* Style changes from review with @randxIzaak Alpert2013-07-171-1/+0
| | | | | | -Some changes around calling origional methods for !for_fork? merge requests. Other changes to follow Change-Id: I009c716ce2475b9efa3fd07aee9215fca7a1c150
* MR on Fork multiple fixesIzaak Alpert2013-07-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | -Disable observers post test run -Allow db:seed_fu RAILS_ENV=test to be run more than once without error -fix diffs_in_between, was passing in the default_options for grit, but grit in this case doesn't take options, fixed the test to actually fail if the incorrect diffs are returned -make notes/commits render against proper project -MR discussion file links should reference note's project -Added tests for commit links on edit merge request -fixes edit issues (canceling an edited mr, updating an edited mr) -updates tests with checks for source code updates -still forked_merge_requests.feature (project_forked_merge_requests) test not passing (commented out -- "stable" not being set) MR API: error on bad target_project -If the target project id is specified and it is not the same as the project the request is being made on (the source), and the it isn't a fork of that project, error out, otherwise use it as the target -Fixes some busted (but hidden) test cases Conflicts: app/views/merge_requests/show/_diffs.html.haml spec/features/notes_on_merge_requests_spec.rb Change-Id: I20e595c156d0e8a63048baaead7be6330c738a05
* Merge Request on forked projectsIzaak Alpert2013-07-175-28/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The good: - You can do a merge request for a forked commit and it will merge properly (i.e. it does work). - Push events take into account merge requests on forked projects - Tests around merge_actions now present, spinach, and other rspec tests - Satellites now clean themselves up rather then recreate The questionable: - Events only know about target projects - Project's merge requests only hold on to MR's where they are the target - All operations performed in the satellite The bad: - Duplication between project's repositories and satellites (e.g. commits_between) (for reference: http://feedback.gitlab.com/forums/176466-general/suggestions/3456722-merge-requests-between-projects-repos) Fixes: Make test repos/satellites only create when needed -Spinach/Rspec now only initialize test directory, and setup stubs (things that are relatively cheap) -project_with_code, source_project_with_code, and target_project_with_code now create/destroy their repos individually -fixed remote removal -How to merge renders properly -Update emails to show project/branches -Edit MR doesn't set target branch -Fix some failures on editing/creating merge requests, added a test -Added back a test around merge request observer -Clean up project_transfer_spec, Remove duplicate enable/disable observers -Ensure satellite lock files are cleaned up, Attempted to add some testing around these as well -Signifant speed ups for tests -Update formatting ordering in notes_on_merge_requests -Remove wiki schema update Fixes for search/search results -Search results was using by_project for a list of projects, updated this to use in_projects -updated search results to reference the correct (target) project -udpated search results to print both sides of the merge request Change-Id: I19407990a0950945cc95d62089cbcc6262dab1a8
* Fix key observer specDmitriy Zaporozhets2013-06-251-2/+2
|
* Replace old hashes with new 1.9 ruby hashes (rebase)Andrey Kumanyaev2013-05-051-1/+1
|
* Update all gems!Dmitriy Zaporozhets2013-05-011-0/+2
|
* Specs refactoring to reduce test time. Disabled observers by default for specsDmitriy Zaporozhets2013-04-104-53/+14
|
* refactor observers test since email logic moved to serviceDmitriy Zaporozhets2013-03-264-161/+10
|
* use NotificationService for handle notify logic when MR createdDmitriy Zaporozhets2013-03-261-14/+5
|
* refactor issue observer specDmitriy Zaporozhets2013-03-261-65/+16
|
* fix key observer testDmitriy Zaporozhets2013-03-201-6/+1
|
* fixed omniauth email specDmitriy Zaporozhets2013-03-051-1/+1
|
* Merge branch 'omniauth_no_email_1' of https://github.com/gliptak/gitlabhq ↵Dmitriy Zaporozhets2013-03-051-0/+6
|\ | | | | | | | | | | | | | | into gliptak-omniauth_no_email_1 Conflicts: app/observers/user_observer.rb spec/observers/user_observer_spec.rb
| * Don't email omniauth created usersGabor Liptak2012-09-191-2/+10
| |
* | Issue uses StateMachine nowAndrew8xx82013-02-181-100/+59
| |
* | Merge Request uses StateMachine nowAndrew8xx82013-02-181-82/+37
| |
* | Constants in Events looks good nowAndrew8xx82013-02-131-3/+3
| |
* | fixing tests after refactoringDmitriy Zaporozhets2013-02-111-2/+2
| |
* | Fixed: post-receive, project remove, testsDmitriy Zaporozhets2013-02-051-3/+3
| |
* | fix key observer testsDmitriy Zaporozhets2013-01-281-8/+2
| |
* | more tests fixedDmitriy Zaporozhets2013-01-151-1/+2
| |
* | sidekiq with green testsDmitriy Zaporozhets2013-01-094-36/+26
| |
* | fix testsDmitriy Zaporozhets2013-01-031-3/+3
| |