summaryrefslogtreecommitdiffstats
path: root/.drone.yml
Commit message (Collapse)AuthorAgeFilesLines
* Exclude build/.phan/ from PHP 5.6 syntax checkLukas Reschke2017-07-201-1/+1
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Add Phan plugin to check for SQL injectionsLukas Reschke2017-07-201-0/+1
| | | | | | | | | | This adds a phan plugin which checks for SQL injections on code using our QueryBuilder, while it isn't perfect it should already catch most potential issues. As always, static analysis will sometimes have false positives and this is also here the case. So in some cases the analyzer just doesn't know if something is potential user input or not, thus I had to add some `@suppress SqlInjectionChecker` in front of those potential injections. The Phan plugin hasn't the most awesome code but it works and I also added a file with test cases. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Merge pull request #5800 from ↵Morris Jobke2017-07-201-9/+9
|\ | | | | | | | | nextcloud/enable-acceptance-tests-again-on-drone-0.7 Enable acceptance tests again on Drone 0.7
| * Enable acceptance tests again on Drone 0.7Daniel Calviño Sánchez2017-07-201-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Running the acceptance tests on Drone relied on the pod-style networking used by services (service containers were available at 127.0.0.1 from the build containers). However, in Drone 0.7 service and build containers must be accessed from each other using their domain name instead. Thus, acceptance tests had to be disabled on Drone. Now that the acceptance test system supports setting a different domain for the Selenium server and for the Nextcloud test server the acceptance tests can be enabled again on Drone. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Update PHP 7.1 containersLukas Reschke2017-07-191-6/+6
|/ | | | | | Previously this container used a very old CentOS version. It has been migrated to Debian Jessie now using the deb.sury.org repositories. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Install phan manuallyLukas Reschke2017-07-191-0/+2
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Use new Drone imageLukas Reschke2017-07-191-2/+2
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Run phan over code baseLukas Reschke2017-07-191-0/+9
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Disable postgres temporarilyMorris Jobke2017-07-181-3/+3
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Temporarily disable acceptance and object storage testsMorris Jobke2017-07-141-10/+10
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Fix droneMorris Jobke2017-07-141-0/+3
| | | | | | | | | Fix service container host name check current folder fix redis for integration test Fix more hostnames Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Update integration tests composer cacheMorris Jobke2017-06-141-30/+30
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #4767 from nextcloud/app-code-checkerMorris Jobke2017-05-161-12/+1
|\ | | | | Check language files and database schema with app code checker
| * Run the app checker on all appsJoas Schilling2017-05-161-12/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #4666 from nextcloud/enable-redis-clusterMorris Jobke2017-05-161-0/+18
|\ \ | |/ |/| Add redis cluster tests to our CI jobs
| * Make cache tests a bit more clearMorris Jobke2017-05-111-0/+1
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
| * Add redis cluster tests to our CI jobsMorris Jobke2017-05-111-0/+17
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Run sign-off check as last checkMorris Jobke2017-05-151-1/+1
|/ | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #4682 from ↵Morris Jobke2017-05-041-3/+3
|\ | | | | | | | | nextcloud/try-to-start-browser-sessions-again-when-they-fail-in-acceptance-tests Try to start browser sessions again when they fail in acceptance tests
| * Set timeout multiplier to 10 for acceptance tests run by DroneDaniel Calviño Sánchez2017-05-031-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sometimes, acceptance tests run by Drone fail due to a timeout when starting the web browser sessions. Increasing the timeout should minimize the possibility of the failure happening, although it can not guarantee that it will not happen. A timeout multiplier of 10 was set just because it looks like a reasonable margin of time, although it is not based on any hard data. The timeout multiplier affects too the timeout used when finding elements. Like when starting a session, increasing the find timeout simply gives the acceptance tests more time to find the objects before giving up, so it does not change their behaviour when successful and can also prevent failures due to default timeouts being too low for a strained system. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Merge pull request #4667 from nextcloud/split-up-more-integration-testsMorris Jobke2017-05-021-0/+10
|\ \ | | | | | | Split up sharing-v1-part2.feature to avoid timeouts
| * | Split up sharing-v1-part2.feature to avoid timeoutsMorris Jobke2017-05-021-0/+10
| |/ | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #4669 from nextcloud/checkersMorris Jobke2017-05-021-40/+5
|\ \ | | | | | | Consolidate all the code checkers into one job
| * | Consolidate all the code checkers into one jobMorris Jobke2017-05-021-40/+5
| |/ | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* / Add redis support to our CI jobsMorris Jobke2017-05-021-0/+16
|/ | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #4485 from nextcloud/translation-checkerMorris Jobke2017-04-251-0/+8
|\ | | | | Check whether we can json decode the translations
| * Loop over the apps directory and add the taskJoas Schilling2017-04-251-0/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Check whether we can json decode the translationsJoas Schilling2017-04-251-0/+7
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Add acceptance tests for sharing password protected linksDaniel Calviño Sánchez2017-04-241-0/+9
|/ | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Move acceptance tests from build/acceptance to tests/acceptanceDaniel Calviño Sánchez2017-04-211-2/+2
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add safety parameterDaniel Calviño Sánchez2017-04-211-2/+2
| | | | | | | | As the script modifies the Git repository a safety parameter was added to prevent running it by mistake and messing with the local copy of the repository. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Generalize names and descriptionsDaniel Calviño Sánchez2017-04-211-2/+2
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Use PHP built-in web server instead of Apache in DroneDaniel Calviño Sánchez2017-04-211-13/+0
| | | | | | | | | | | | | | | Instead of running an additional Drone service with the Nextcloud server now the Nextcloud server is run in the same Drone step as the acceptance tests themselves using the PHP built-in web server. Thanks to this, the Nextcloud server control is no longer needed, as the acceptance tests can now directly reset, start and stop the Nextcloud server. Also, the "nextcloudci/php7.0:php7.0-7" image provides everything needed to run and manage the Nextcloud server (including the Git command used to restore the directory to a saved state), so the custom image is no longer needed either. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add acceptance test steps to DroneDaniel Calviño Sánchez2017-04-191-0/+40
| | | | | | | | | | | | | | | Each acceptance test feature is run in its own Drone step. The container of the step runs the acceptance tests themselves, but they require two additional Drone services. One service provides the Selenium server that performs the web browser actions specified by the tests, and the other service provides the Nextcloud server that the tests will be run against (due to security concerns the acceptance tests themselves can not create Docker containers for the Nextcloud server as done when running them in a local system, as if Drone containers had access to Docker a malicious pull request could be used to take over the Drone server). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Enable testing app in CI stepLukas Reschke2017-04-131-0/+1
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Fix .drone.ymlLukas Reschke2017-04-131-1/+1
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Add support for ratelimiting via annotationsLukas Reschke2017-04-131-0/+12
| | | | | | | | | | | | | This allows adding rate limiting via annotations to controllers, as one example: ``` @UserRateThrottle(limit=5, period=100) @AnonRateThrottle(limit=1, period=100) ``` Would mean that logged-in users can access the page 5 times within 100 seconds, and anonymous users 1 time within 100 seconds. If only an AnonRateThrottle is specified that one will also be applied to logged-in users. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Add drone.yml configMorris Jobke2017-04-111-0/+10
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Run maintenance mode tests on DroneLukas Reschke2017-04-111-0/+10
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Split long running features/sharing-v1.feature into two smaller partsMorris Jobke2017-03-281-0/+10
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #4013 from nextcloud/bundle_vendor_jsLukas Reschke2017-03-241-0/+8
|\ | | | | Bundle vendor js
| * Add CI step to verify merged vendor jsRoeland Jago Douma2017-03-231-0/+8
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
| * Revert "Bundle vendor js"Roeland Jago Douma2017-03-221-8/+0
| |
| * Add CI step to verify merged vendor jsRoeland Jago Douma2017-03-221-0/+8
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | add drone configMorris Jobke2017-03-201-0/+10
|/ | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Revert "Add integration test for trashbin"Morris Jobke2017-03-171-10/+0
|
* Add drone.yml configMorris Jobke2017-03-161-0/+10
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Run drone only on master/stable branchesMorris Jobke2017-03-061-0/+2
| | | | | | * fixes #3729 Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* enable user_ldap app for testsArthur Schiwon2017-01-201-0/+1
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* fix indentationMorris Jobke2017-01-191-9/+9
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>