Last modified: 2013-09-20 16:23:25 UTC
OAuth extension should have Jenkins jobs, and currently doesn't. There is a tests/ folder at the moment, but it's unclear to me if those are PHPUnit tests. PHPUnit tests may be advisable.
Change 76272 had a related patch set uploaded by Hashar: jobs for mw/ext/Oauth https://gerrit.wikimedia.org/r/76272
Change 76273 had a related patch set uploaded by Hashar: triggers for mw/ext/Oauth https://gerrit.wikimedia.org/r/76273
Change 76273 merged by jenkins-bot: triggers for mw/ext/Oauth https://gerrit.wikimedia.org/r/76273
Change 76272 merged by Hashar: jobs for mw/ext/Oauth https://gerrit.wikimedia.org/r/76272
Change 76274 had a related patch set uploaded by Hashar: Oauth -> OAuth https://gerrit.wikimedia.org/r/76274
Change 76275 had a related patch set uploaded by Hashar: Oauth -> OAuth https://gerrit.wikimedia.org/r/76275
Change 76274 merged by Hashar: Oauth -> OAuth https://gerrit.wikimedia.org/r/76274
Change 76275 merged by Hashar: Oauth -> OAuth https://gerrit.wikimedia.org/r/76275
Change 76276 had a related patch set uploaded by Hashar: Jenkins job validation (DO NOT SUBMIT) https://gerrit.wikimedia.org/r/76276
Build succeeded. https://integration.wikimedia.org/ci/job/mwext-OAuth-jslint/1/console : SUCCESS in 0s (non-voting) https://integration.wikimedia.org/ci/job/mwext-OAuth-lint/1/console : SUCCESS in 0s https://integration.wikimedia.org/ci/job/mwext-OAuth-testextensions-master/1/console : SUCCESS in 1s Tests are not tests registered via the UnitTestsList hook though.
The two tests under /tests/ seems to be manual integration tests, they always exit 0 which is not ideal :] The tests should be converted to PHPUnit tests and registered via UnitTestsList hook. From there they will be running in Jenkins which I have setup with the above changes. Redirecting bug to MediaWiki extensions -> OAuth
Change 76276 abandoned by Hashar: Jenkins job validation (DO NOT SUBMIT) https://gerrit.wikimedia.org/r/76276
Closing this since Gerrit change #79298 added PHPUnit tests a month ago already.
They are even triggered by Zuul :-] https://integration.wikimedia.org/ci/job/mwext-OAuth-testextensions-master/ \O/
