Paladox has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/277614

Change subject: Revert "testsuitegenerator: Always test empty values for 
'string' type, not just for 'label'"
......................................................................

Revert "testsuitegenerator: Always test empty values for 'string' type, not 
just for 'label'"

This reverts commit df5cc372507ddf73a51263f14892a5f48a572fb6.

Bug: T130063
Change-Id: Ife4940b9f5c3528c058021052dc7cd952cc1ee58
---
M bin/testsuitegenerator.rb
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/oojs/ui refs/changes/14/277614/1

diff --git a/bin/testsuitegenerator.rb b/bin/testsuitegenerator.rb
index 26229e0..05c8cee 100644
--- a/bin/testsuitegenerator.rb
+++ b/bin/testsuitegenerator.rb
@@ -26,7 +26,7 @@
                'null' => [nil],
                'number' => [0, -1, 300],
                'boolean' => [true, false],
-               'string' => ['Foo bar', '<b>HTML?</b>', '', ' '],
+               'string' => ['Foo bar', '<b>HTML?</b>'],
        }
 
        # values to test for names
@@ -52,6 +52,7 @@
                'icon' => ['image'],
                'indicator' => ['down'],
                'flags' => %w[constructive primary],
+               'label' => expandos['string'] + ['', ' '],
                # these are defined by Element and would bloat the tests
                'classes' => true,
                'id' => true,

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ife4940b9f5c3528c058021052dc7cd952cc1ee58
Gerrit-PatchSet: 1
Gerrit-Project: oojs/ui
Gerrit-Branch: master
Gerrit-Owner: Paladox <[email protected]>

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

Reply via email to