Mglaser has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/350400 )

Change subject: Tests: clarified usage of addDBDataOnce
......................................................................

Tests: clarified usage of addDBDataOnce

The assignement in this function was misleading and unusable. In order
to state the total number, please use protected iFixtureTotal in your
tests

Change-Id: I412d267f37fdc6cde7222ce3eac8bc0e1bf732cf
---
M tests/BSApiExtJSStoreTestBase.php
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceFoundation 
refs/changes/00/350400/1

diff --git a/tests/BSApiExtJSStoreTestBase.php 
b/tests/BSApiExtJSStoreTestBase.php
index ae458a4..d93ba8a 100644
--- a/tests/BSApiExtJSStoreTestBase.php
+++ b/tests/BSApiExtJSStoreTestBase.php
@@ -23,7 +23,8 @@
        }
 
        public function addDBDataOnce() {
-               $this->iFixtureTotal = $this->createStoreFixtureData();
+               // Caution: this is only called once per testsuite, not per 
test.
+               $this->createStoreFixtureData();
        }
 
        public function testSchema() {

-- 
To view, visit https://gerrit.wikimedia.org/r/350400
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I412d267f37fdc6cde7222ce3eac8bc0e1bf732cf
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceFoundation
Gerrit-Branch: master
Gerrit-Owner: Mglaser <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to