NPM Resource Changed ? ? ? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
28 Feb 2019

This PR just bings back the javascript tests

Pull Request for Issue # .

Summary of Changes

  • Tests are back in the main repo
  • npm test runs the tests locally

Testing Instructions

  • npm test still performs the js tests { run npm install before!!!, deps have changed!!}

screenshot 2019-02-28 at 11 14 11

Documentation Changes Required

This needs some adjustments in the drone files. Pinging @rdeutz here as we already discussed the idea of utilising Sauce Labs joomla/test-javascript#11

@dneukirchen

avatar dgrammatiko dgrammatiko - open - 28 Feb 2019
avatar dgrammatiko dgrammatiko - change - 28 Feb 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Feb 2019
Category NPM Change Unit Tests Repository JavaScript
avatar dgrammatiko dgrammatiko - change - 28 Feb 2019
Labels Added: NPM Resource Changed ? ?
avatar dgrammatiko dgrammatiko - change - 28 Feb 2019
The description was changed
avatar dgrammatiko dgrammatiko - edited - 28 Feb 2019
302d8b3 28 Feb 2019 avatar dgrammatiko upd
avatar wilsonge
wilsonge - comment - 28 Feb 2019

You can't delete the package lock. sorry!

avatar Hackwar
Hackwar - comment - 28 Feb 2019

Please bring back an updated package.lock file. Otherwise this looks good. Please also see @dneukirchen s PR for the unittests and how he described the tests in the Readme.md.

4b7dbac 28 Feb 2019 avatar dgrammatiko .md
avatar dgrammatiko
dgrammatiko - comment - 28 Feb 2019

You can't delete the package lock. sorry!

I wasn't deleting that file, it just removed a lot of lines due to the fact that we removed on dependency...
The .lock is still there

avatar wilsonge
wilsonge - comment - 28 Feb 2019

LGTM will wait for a testing team member to actually do the merge

avatar ghazal ghazal - test_item - 6 Mar 2019 - Tested successfully
avatar ghazal
ghazal - comment - 6 Mar 2019

I have tested this item successfully on d4095f8

Finally, node works ie Terminal found karma.conf.js.
Tks.
Docs should be updated too.
Like this one : https://docs.joomla.org/JavaScript_Tests_for_Joomla4


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24044.
avatar ghazal
ghazal - comment - 6 Mar 2019

I have tested this item successfully on d4095f8

Finally, node works ie Terminal found karma.conf.js.
Tks.
Docs should be updated too.
Like this one : https://docs.joomla.org/JavaScript_Tests_for_Joomla4


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24044.
avatar dneukirchen dneukirchen - test_item - 6 Mar 2019 - Tested successfully
avatar dneukirchen
dneukirchen - comment - 6 Mar 2019

I have tested this item successfully on d4095f8


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24044.

avatar franz-wohlkoenig franz-wohlkoenig - change - 6 Mar 2019
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Mar 2019

Status "Ready To Commit".

avatar rdeutz rdeutz - change - 10 Mar 2019
Labels Added: ?
avatar rdeutz rdeutz - change - 10 Mar 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-03-10 14:48:00
Closed_By rdeutz
avatar rdeutz rdeutz - close - 10 Mar 2019
avatar rdeutz rdeutz - merge - 10 Mar 2019

Add a Comment

Login with GitHub to post a comment