summaryrefslogtreecommitdiff
path: root/spec/lib
Commit message (Collapse)AuthorAgeFilesLines
* fix spec failuresfeature/cycle-analytics-eventsJames Lopez2016-11-188-26/+28
|
* more refactoring and added some auth checksJames Lopez2016-11-171-1/+1
|
* fix specsJames Lopez2016-11-171-5/+5
|
* refactor events facade so it uses separate classes and refactor query stuffJames Lopez2016-11-178-14/+18
|
* added custom date helper and spec and fixed some unrelated spec failuresJames Lopez2016-11-171-7/+7
|
* Refactor all query config stuff into separate classes and added specsJames Lopez2016-11-179-1/+90
|
* get rid of light url builder and fix wrong specJames Lopez2016-11-171-46/+91
|
* serialize all the things!James Lopez2016-11-171-2/+6
|
* get all stages to use serlalizers - apart from plan - WIPJames Lopez2016-11-171-17/+25
|
* fix issue events and related spec - now using generic serializer for a hashJames Lopez2016-11-171-3/+3
|
* WIP - refactored events to use build serializer, related spec passingJames Lopez2016-11-171-59/+59
|
* refactored and added missing spec to light URL builderJames Lopez2016-11-173-92/+166
|
* Updated production events with new fieldsJames Lopez2016-11-171-4/+15
|
* Refactored staging events, added missing fields and fixed specsJames Lopez2016-11-171-7/+55
|
* Added branch option to test eventsJames Lopez2016-11-171-1/+1
| | | | Also fixed test events ordering issue
* Refactored test eventsJames Lopez2016-11-171-3/+34
| | | | Now test events return the builds instead a list of pipelines to avoid calling pipeline.builds per each and get the info. Also, added missing fields/data, URLs, and fixed specs in events spec.
* added missing fields to code events and updated specJames Lopez2016-11-171-3/+7
|
* added missing fields to plan events and updated specJames Lopez2016-11-171-9/+19
|
* added missing fields to issue. Also, added a light url builder to add URLs ↵James Lopez2016-11-171-3/+7
| | | | easily from arel. Updated specs.
* add email to user related queries so it can be used for displaying avatar in ↵James Lopez2016-11-171-0/+16
| | | | the UI
* added production events and related specJames Lopez2016-11-171-0/+29
|
* added staging events and specJames Lopez2016-11-171-0/+26
|
* fixing spec failuresJames Lopez2016-11-171-5/+6
|
* review events - spec and logicJames Lopez2016-11-171-0/+25
|
* Added test events specs and logic. Also fixed some SQL and refactored the ↵James Lopez2016-11-171-1/+23
| | | | pipeline worker spec.
* Added code events spec and logic. Also fixed SQL issues and refactored the ↵James Lopez2016-11-171-1/+29
| | | | code a bit.
* plan events and spec working. Also added some TODOs to the code.James Lopez2016-11-171-2/+2
|
* commit events and specJames Lopez2016-11-171-3/+15
|
* fix specs - issue events workingJames Lopez2016-11-171-5/+5
|
* fix timestamp diff and specJames Lopez2016-11-171-4/+4
|
* fixed spec and SQL queryJames Lopez2016-11-171-2/+18
|
* WIP - refactored some arel queriesJames Lopez2016-11-171-2/+3
|
* Started refactoring stuff to add events to cycle analyticsJames Lopez2016-11-171-0/+26
| | | | | | - Refactored cycle analytics class to extract DB logic - Reuse logic in new events fetcher - Started adding cycle analytics events class and spec (still not functional)
* Search for a filename in a project23117-search-for-a-filename-in-a-projectValery Sizov2016-11-161-4/+47
|
* Merge branch 'stanhu/gitlab-ce-fix-error-500-with-mr-images' into 'master' Douwe Maan2016-11-151-0/+24
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix Error 500 when creating a merge request that contains an image that was deleted and added _Originally opened at !4816 by @stanhu._ - - - ## What does this MR do? This MR fixes an Error 500 when creating a merge request that contains an image that was deleted and added. Before, when displaying the before and after image, the code would always retrieve the image from the parent commit. However, in a diff, this could cause two different problems: The "before" image may not actually be the image you want to compare against (regression of #14327) It may appear as though a file was modified when it was really just added during the diff ## Are there points in the code the reviewer needs to double check? There may be a more elegant to fix this bug. ## What are the relevant issue numbers? Closes #3893, gitlab-org/gitlab-ee#678 See merge request !7457
| * Fix Error 500 when creating a merge request that contains an image that was ↵stanhu/gitlab-ce-fix-error-500-with-mr-imagesStan Hu2016-11-141-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | deleted and added Steps to reproduce: 1. Start with a repo with an image 2. Add a commit to delete the image 3. Add another commit to replace the image with another image In a diff comparison, we really just compare about what the image was before the diff, not the direct parent of the last commit. This MR fixes that. Closes #3893, gitlab-org/gitlab-ee#678 Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Merge branch 'fix_saml_ldap_link' into 'master' Douwe Maan2016-11-151-1/+22
|\ \ | |/ |/| | | | | | | | | Omniauth auto link LDAP user falls back to find by DN when user cannot be found by uid Unfortunately, SAML IDs can be an LDAP UID, DN, or something else entirely. UID and DN are most common, though. This adds a fallback scenario so we first try to find a matching LDAP user by UID, then by DN. This will fix a problem for the customer in https://gitlab.zendesk.com/agent/tickets/43298 See merge request !7002
| * Omniauth auto link LDAP user falls back to find by DN when user cannot be ↵Drew Blessing2016-11-101-1/+22
| | | | | | | | found by uid
* | Merge branch 'fix/improve-naming-convention-in-ci-config' into 'master' Kamil Trzciński2016-11-1426-115/+115
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve naming convention in ci configuration module ## What does this MR do? This MR improves the naming convention in CI configuration module to reflect the domain design better. ## What are the relevant issue numbers? Related to #15060 See merge request !7448
| * | Change name of method for setting CI config entriesGrzegorz Bizon2016-11-141-2/+2
| | |
| * | Improve naming convention in ci configuration modulefix/improve-naming-convention-in-ci-configGrzegorz Bizon2016-11-1425-63/+63
| | |
* | | Centralize LDAP config/filter logicDrew Blessing2016-11-111-0/+81
|/ / | | | | | | | | | | | | | | | | Centralize all LDAP config logic in `GitLab::LDAP::Config`. Previously, some logic was in the Devise initializer and it was not honoring the `user_filter`. If a user outside the configured `user_filter` signed in, an account would be created but they would then be denied access. Now that logic is centralized, the filter is honored and users outside the filter are never created.
* | Refactored Sidekiq Throttler and updated documentationsidekiq-job-throttlingPatricio Cano2016-11-111-6/+0
| |
* | Refactored initializer code to its own class and added testsPatricio Cano2016-11-101-0/+34
|/
* Merge branch 'ldap_check_bind' into 'master' Sean McGivern2016-11-091-4/+35
|\ | | | | | | | | | | | | Improve ldap:check errors Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/21621. See merge request !6601
| * Introduce better credential and error checking to `rake gitlab:ldap:check`Drew Blessing2016-11-081-4/+35
| | | | | | | | | | | | | | It was previously possible for invalid credential errors to go unnoticed in this task. Users would believe everything was configured correctly and then sign in would fail with 'invalid credentials'. This adds a specific bind check, plus catches errors connecting to the server. Also, specs :)
* | Merge branch '23403-fix-events-for-private-project-features' into 'security'Robert Speicher2016-11-091-0/+104
| | | | | | | | | | | | | | | | | | | | | | | | Respect project visibility settings in the contributions calendar This MR fixes a number of bugs relating to access controls and date selection of events for the contributions calendar Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/23403 See merge request !2019 Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Merge branch 'fix-unathorized-cloning' into 'security'Douwe Maan2016-11-092-1/+26
| | | | | | | | | | | | | | | | | | | | Ensure external users are not able to clone disabled repositories. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/23788 See merge request !2017 Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Merge branch 'markdown-xss-fix-option-2.1' into 'security'Douwe Maan2016-11-091-0/+22
| | | | | | | | | | | | | | | | | | | | | | | | Fix for HackerOne XSS vulnerability in markdown This is an updated blacklist patch to fix https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2007. No text is removed. Dangerous schemes/protocols and invalid URIs are left intact but not linked. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/23153 See merge request !2015 Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Merge branch 'issue_23548_dev' into 'master'Douwe Maan2016-11-0914-56/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | disable markdown in comments when referencing disabled features fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/23548 This MR prevents the following references when tool is disabled: - issues - snippets - commits - when repo is disabled - commit range - when repo is disabled - milestones This MR does not prevent references to repository files, since they are just markdown links and don't leak information. See merge request !2011 Signed-off-by: Rémy Coutable <remy@rymai.me>