summaryrefslogtreecommitdiff
path: root/spec/models/issue_spec.rb
Commit message (Collapse)AuthorAgeFilesLines
* Do not allow to move issue if it has not been persistedmove-issue-section-should-not-be-displayed-in-the-new-issue-form-14489Grzegorz Bizon2016-03-231-0/+5
|
* Merge branch 'master' into 2489-soft-delete-issuesZeger-Jan van de Weg2016-03-211-2/+52
|\
| * Merge branch 'issues-show-performance' into 'master' Robert Speicher2016-03-211-2/+2
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve performance of viewing individual issues This MR does two things: 1. `Issue#related_branches` no longer performs Git operations that aren't needed 2. The output of `Repository#exists?` is now cached and flushed properly Combined these two changes should further cut down the amount of Git operations performed when viewing individual issues (and possibly other pages). See merge request !3296
| | * Tweaked performance of Issue#related_branchesYorick Peterse2016-03-191-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Requesting the branch names of a repository works even when it's empty, thus there's no need to explicitly check for an empty repository. Removing this check cuts down the amount of Git operations which in turn cuts down request timings a bit. The regular expression used to compare branches was also moved out of the loop so it's created only once.
| * | Merge branch 'master' into feature/issue-moveGrzegorz Bizon2016-03-201-3/+3
| |\ \ | | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: Fix bug where wrong commit ID was being used in a merge request diff to show old image Remove CHANGELOG item that was added during merge resolution Improve the "easy WIP & un-WIP from link" feature Fix specs \#to_branch_name now uses the iid as postfix Add label description in tooltip to labels in issue index and sidebar Easily (un)mark merge request as WIP using link Use specialized system notes when MR is (un)marked as WIP another attempt to fix oauth issue attempting to fix omniauth problem Conflicts: app/assets/javascripts/issuable_form.js.coffee
| * | Merge branch 'master' into feature/issue-moveGrzegorz Bizon2016-03-191-0/+17
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (121 commits) Dedupe labels in labels selector in Dashboard pages Refactor colors and lists Add a safeguard in MergeRequest#compute_diverged_commits_count Fix an issue when the target branch of a MR had been deleted Add avatar to issue and MR pages header Cleanup somce css colors Re-group scss variables Refactor `Todo#target` Fixes issue with filter label missing on labels & milestones Rename `Todo#to_reference` to `Todo#target_reference` Fixed failing tests Updated controller with before_action Fixed other issues based on feedback Fixes issue on dashboard issues Full labels data in JSON Fixed issue with labels dropdown getting wrong labels Update CHANGELOG Use `Note#for_project_snippet?` to skip notes on project snippet Use `Commit#short_id` instead of `Commit.truncate_sha` Reuse `for_commit?` on conditional validations Update schema info comment on todo related files ... Conflicts: app/models/issue.rb db/schema.rb spec/models/issue_spec.rb
| * | | Prevent issue move if issue has been already movedGrzegorz Bizon2016-03-171-0/+50
| | | |
* | | | Soft delete issuablesZeger-Jan van de Weg2016-03-191-0/+5
| |_|/ |/| |
* | | \#to_branch_name now uses the iid as postfixissue-branch-iid-postfixZeger-Jan van de Weg2016-03-181-3/+3
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | Given the branch name 'mep-mep' with an iid being 1, the current way, master's way, would yield a branch name of 1-mep-mep. The problem for larger projects however would be that a developer might forget what iid the issue was. When this developer would try to tab complete it would: - Or result in 20+ branches possibly - Or start with the wrong digit, try again with digit++ - Would see 20 branches, repeat Thus the obvious way of solving this is letting the dev tab complete on the issue title, which is easier to remember.
* | Incorporate reviewZeger-Jan van de Weg2016-03-151-0/+9
| |
* | Enhance new branch button on an issueZeger-Jan van de Weg2016-03-151-4/+0
| |
* | new-branch-buttonZeger-Jan van de Weg2016-03-151-0/+12
|/
* Added specs for Issue#referenced_merge_requestsYorick Peterse2016-03-091-0/+25
|
* Tag model specsDouwe Maan2015-12-091-1/+1
|
* Only accept open issues and merge requestsZeger-Jan van de Weg2015-10-161-0/+37
|
* Fix mentionable specsDouwe Maan2015-10-121-1/+1
|
* Re-annotate modelsStan Hu2015-09-061-13/+14
|
* Use to_reference for issue and merge request mentionable specsRobert Speicher2015-06-221-1/+1
|
* Minor model spec cleanupsRobert Speicher2015-05-261-5/+2
| | | | Snippet model was missing project association
* Add `to_reference` for models that support referencesRobert Speicher2015-05-261-3/+18
| | | | | Now there is a single source of information for which attribute a model uses to be referenced, and its special character.
* Correct usage of `subject` in specsRobert Speicher2015-04-161-1/+2
|
* Updated rspec to rspec 3.x syntaxJeroen van Baarsen2015-02-121-5/+5
| | | | Signed-off-by: Jeroen van Baarsen <jeroenvanbaarsen@gmail.com>
* Add specs for tasksVinnie Okada2014-10-051-0/+4
| | | | | Add tests for the new task list functionality in the Markdown parser and in issues and merge requests.
* Remove mass assgnment specsDmitriy Zaporozhets2014-06-261-2/+0
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Re-annotate modelsDmitriy Zaporozhets2014-04-091-2/+2
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Link issues from comments and automatically close themash wilson2013-08-251-0/+6
| | | | | | | | | | 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.
* ReannotateDmitriy Zaporozhets2013-08-211-2/+3
|
* Fix typosJohannes Schleifenbaum2013-07-291-1/+1
|
* AnnotatedDmitriy Zaporozhets2013-06-191-2/+2
|
* reannotatedDmitriy Zaporozhets2013-03-151-1/+1
|
* Additional tests added to IssueAndrew8xx82013-02-181-0/+12
|
* Issue uses StateMachine nowAndrew8xx82013-02-181-1/+1
|
* Issuable consern uses StateMachine nowAndrew8xx82013-02-181-31/+0
|
* fix testsValeriy Sizov2013-01-221-4/+0
|
* Increase Issue description size to 10k. Fixes #2545Dmitriy Zaporozhets2013-01-191-1/+1
|
* Fixed tests. added test for gitlab_ci_service modelDmitriy Zaporozhets2013-01-031-2/+1
|
* Annotated. schema updatedDmitriy Zaporozhets2012-11-191-5/+5
|
* Fix specsRiyad Preukschas2012-11-141-1/+0
|
* Remove backward compatibility of factories.Vincent Bonmalais2012-11-131-4/+4
|
* AnnotatedDmitriy Zaporozhets2012-10-091-0/+18
|
* set activerecord whitelist_attributes to trueNihad Abbasov2012-09-261-0/+5
|
* Rename Upvote role to VotesRiyad Preukschas2012-09-081-1/+1
|
* add validations for boolean attributesNihad Abbasov2012-09-051-0/+1
|
* Add more coverage for model validations and associationsRobert Speicher2012-08-291-11/+1
|
* Remove annotations from specsRobert Speicher2012-08-291-19/+0
|
* Move IssueCommonality and Upvote specs out of models and into their own specsRobert Speicher2012-08-291-34/+5
|
* Simple model spec changes made possible by new factoriesRobert Speicher2012-08-281-28/+9
|
* Fully embrace Ruby 1.9 hash syntaxRobert Speicher2012-08-101-16/+16
| | | | Didn't bother with files in db/, config/, or features/
* Fix `search` class method for IssueCommonalityRobert Speicher2012-08-091-0/+8
| | | | | Also adds specs to the two affected classes that would have caught my dumb mistake.
* Issues can be unassigned nowrandx2012-06-261-1/+0
|