[stable9] Add support to know where the storage test comes from #25166

Merged
merged 1 commit into from Jun 21, 2016

Projects

None yet

4 participants

@jvillafanez
Contributor

(manual) backport of #25008 to stable9 . Minor changes have been added (mainly boolean conversions that weren't required for master)

Required for https://github.com/owncloud/windows_network_drive/pull/408

@jvillafanez jvillafanez added this to the 9.0.3-current-maintenance milestone Jun 17, 2016
@mention-bot

By analyzing the blame information on this pull request, we identified @PVince81, @Xenopathic and @icewind1991 to be potential reviewers

@DeepDiver1975 DeepDiver1975 changed the title from Add support to know where the storage test comes from to [stable9] Add support to know where the storage test comes from Jun 20, 2016
@DeepDiver1975 DeepDiver1975 commented on an outdated diff Jun 20, 2016
...iles_external/controller/globalstoragescontroller.php
) {
+ $testOnly = filter_var($testOnly, FILTER_VALIDATE_BOOLEAN); // boolean conversion required
@DeepDiver1975
DeepDiver1975 Jun 20, 2016 Member

as discussed in #25173 - use PHPDoc

@DeepDiver1975
Member

@jvillafanez please squash - thx

@jvillafanez @DeepDiver1975 jvillafanez dd support to know where the storage test comes from
cd900b8
@DeepDiver1975
Member

took care about the squash

@icewind1991
Member

Code looks good 👍

@DeepDiver1975
Member

👍

@DeepDiver1975 DeepDiver1975 merged commit bf7a08f into stable9 Jun 21, 2016

20 of 21 checks passed

server-master-linux-externals-ci/database=sqlite,external=swift-ceph,label=SLAVE Build #11256 found unstable in 7 min 46 sec
Details
cla-bot-core Build #4929 succeeded in 20 sec
Details
continuous-integration/php-5.4 Build #5132 succeeded in 4 min 15 sec
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
core-ci-linux-jsunit/database=sqlite,label=SLAVE Build #63015 succeeded in 27 sec
Details
core-ci-linux/database=mysql,label=SLAVE Build #31931 succeeded in 17 min
Details
core-ci-linux/database=oci,label=SLAVE Build #31932 succeeded in 1 hr 8 min
Details
core-ci-linux/database=pgsql,label=SLAVE Build #31932 succeeded in 44 min
Details
core-ci-linux/database=sqlite,label=SLAVE Build #31932 succeeded in 7 min 55 sec
Details
ocs-api-integration-tests-ci Build #11751 succeeded in 1 hr 4 min
Details
server-master-linux-externals-ci/database=sqlite,external=smb-silvershell,label=SLAVE Build #11256 succeeded in 58 sec
Details
server-master-linux-externals-ci/database=sqlite,external=webdav-ownCloud,label=SLAVE Build #11256 succeeded in 9 min 33 sec
Details
server-master-linux-externals-smb-windows-ext-ci/database=sqlite,external=smb-windows,label=master Build #22445 succeeded in 2 min 31 sec
Details
server-master-linux-php7-ci/database=sqlite,label=SLAVE Build #40246 succeeded in 2 min 30 sec
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=litmus,mirallBranch=v2.0.2,slave=SMASH Build #15571 succeeded in 9 min 4 sec
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_basicSync@0,mirallBranch=v2.0.2,slave=SMASH Build #15571 succeeded in 52 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_basicSync@1,mirallBranch=v2.0.2,slave=SMASH Build #15571 succeeded in 52 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_shareLink,mirallBranch=v2.0.2,slave=SMASH Build #15571 succeeded in 20 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePermissions,mirallBranch=v2.0.2,slave=SMASH Build #15571 succeeded in 48 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePropagationGroups,mirallBranch=v2.0.2,slave=SMASH Build #15571 succeeded in 24 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePropagationInsideGroups,mirallBranch=v2.0.2,slave=SMASH Build #15571 succeeded in 29 min
Details
@DeepDiver1975 DeepDiver1975 deleted the external_test_with_origin branch Jun 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment