Commit graph

13 commits

Author SHA1 Message Date
Umherirrender f18dd9e52d tests: Make PHPUnit data providers static
Initally used a new sniff with autofix (T333745)

Bug: T332865
Change-Id: Ibd68b67915fdcd1a5af5c8aedc7235c0a655434d
2023-05-21 16:35:54 +02:00
Umherirrender fd816a2a7c tests: Replace assert(Not)Empty with assert(Not)Same
assertSame avoids use of loose comparisons and
allows to check the expected type

Change-Id: Ia6f0cc449f42ef078eb69667d0ce1706022de19b
2022-11-25 10:29:22 +00:00
Universal Omega 03d6d20179 tests: replace deprecated phpunit methods
Bug: T243600
Change-Id: Ib7bb66a1e853a413aa3cb02b45b2e7f5ef2e20e2
2022-10-10 16:10:24 +00:00
Jayprakash12345 37f46abdbb Use new Hook system
Change-Id: I9f45c6fafabe6d6f8dde7e894772d4f1b1c5bcd3
2021-12-04 19:18:52 +05:30
libraryupgrader 8e712c4e2a build: Updating dependencies
composer:
* mediawiki/mediawiki-codesniffer: 36.0.0 → 37.0.0

npm:
* postcss: 7.0.35 → 7.0.36
  * https://npmjs.com/advisories/1693 (CVE-2021-23368)
* glob-parent: 5.1.0 → 5.1.2
  * https://npmjs.com/advisories/1751 (CVE-2020-28469)
* trim-newlines: 3.0.0 → 3.0.1
  * https://npmjs.com/advisories/1753 (CVE-2021-33623)

Change-Id: I85d018a0837ca36a024187539bd334ffd99a2f4a
2021-07-22 15:33:56 +00:00
Ostrzyciel edd0a7d949 Move the warning outside sitenotice, tidy up code
There's a lot of things going on here, so a quick summary:
* Moved the warning outside sitenotice to avoid reflows and all
  kinds of other issues, including those with dismissable notices.
* Placed the warning on the bottom of the page, on all skins, for
  consistency and to avoid obstructing vital UI elements such as
  site name, user tools, search box.
* Changed the UI to use OOUI for, again, consistency and to avoid
  reinventing the wheel with button styling.
* Removed the cookie emoji as it was only taking up valuable space
  on mobile and making designing a sensible layout much harder.
* Merged the mobile and desktop RL modules into a single, responsive
  one. The warning will work correctly for any skin and screen width.
* The integration tests are truly horrible, I tried to at least
  make them work. They deserve a proper rewrite, but that should be
  done after the extension switches to the new hook system.

I will post screenshots of this in action in the first of the linked
tasks.

Bug: T271047
Bug: T173335
Change-Id: I6e2a3d7aeccc0d4df1b3238e52c67e72099d27d8
2021-02-01 11:23:53 +01:00
Paladox 0dc0e795a1 Replace deprecated hook
Bug: T254302
Change-Id: I140af90b5f175974eb20873f2c1c6449be5a6669
2020-11-22 15:48:59 +00:00
Kunal Mehta 2729a661c3 Unbreak tests
Use an anonymous class to implement QuickTemplate instead of relying on a
specific implementation that no longer exists.

Change-Id: Ifbd47e6fa3aff8c1969e8510edbfd16507025ed2
2020-07-13 12:54:21 -07:00
Florian Schmidt 6634e8c829 Enable GeoLocation service being replaced by other providers
Making GeoLocation an interface makes it easier to replace the underlying implementation
from the current Http backed method.

Change-Id: I2beb97772fd74ab08b2214c08d82dbc1ebfcdcd2
2019-11-21 17:31:23 +01:00
Paladox 6ecc3fd4fc Fix tests to be compatible with master again
Also fix tests relying on MessageCache. MessageCache is enabled in setup,
however, as setMwGlobals is resetting services, it gets disabled again.
Instead of setting the configuration to enable the use of database
messages, let's enable the message cache after the last reset of services.

Change-Id: Iff670def50ab15054099f7a1ca3adc119bd877ce
2019-11-19 00:40:51 +01:00
Fomafix 42fca811c6 Use \u{00A0} instead of   or  
Directly use the UTF-8 encoding of the 'NO-BREAK SPACE' (U+00A0) instead of
the HTML/XML entitiy  .

Also use \u{1F36A} instead of 🍪 for Unicode Character 'COOKIE'
(U+1F36A).

Bug: T154300
Change-Id: I13536589a4c183215785fe3352f4fc081571451b
2018-08-10 20:58:11 +02:00
Florian Schmidt 1a2dd83c15 Use wgUseMediaWikiUIEverywhere true in tests
This seems to be the default for Jenkins CI.

Bug: T201359
Change-Id: Ie436c601a95957b3c3fc2bfcce6d12bdef69e76a
2018-08-06 22:22:32 +02:00
Florian Schmidt 90d2e58096 Move classes into CookieWarning namespace
This should make the code a bit more structured and easier to navigate
through. It also removes some lines from extension.json as the autoload
classes list is replaced by an autoload namespaces property.

This also changes the minimum requirement of MediaWiki to 1.31 for this
extension to work.

Change-Id: I249ccc3035297c99202a5f83fcc8d3fda68b682c
2018-07-19 12:57:52 +02:00
Renamed from tests/phpunit/includes/CookieWarningHooksTest.php (Browse further)