? ? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
25 Sep 2020

Summary of Changes

move com_finder from cli to console command

Testing Instructions

php cli/joomla.php finder:index purge to index and rebuilds
php cli/joomla.php finder:index to index only
php cli/joomla.php finder:index --pause=10 to index with a pause of 10 seconds

Actual result BEFORE applying this Pull Request

php finder_indexer.php --purge

Expected result AFTER applying this Pull Request

php cli/joomla.php finder:index purge

image

Documentation Changes Required

yes

avatar alikon alikon - open - 25 Sep 2020
avatar alikon alikon - change - 25 Sep 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Sep 2020
Category Libraries
avatar alikon alikon - change - 25 Sep 2020
The description was changed
avatar alikon alikon - edited - 25 Sep 2020
6960de8 25 Sep 2020 avatar alikon cs
avatar alikon alikon - change - 25 Sep 2020
Labels Added: ?
2b42e5e 26 Sep 2020 avatar alikon cs
avatar ceford ceford - test_item - 26 Sep 2020 - Tested successfully
avatar ceford
ceford - comment - 26 Sep 2020

I have tested this item successfully on 2b42e5e

In the testing instructions the before test is incomplete - it should be php cli/finder_indexer.php --purge
And the first of the Testing Instructions has a stray p - php cli/joomla.php finder:indexp purge
In the second test I got a skipping pause message so I guess there is a default pause

  • Skipping pause, as previous batch had a very low processing time (0.134s < 1s)

But all worked as described. I did not understand the PHPCS link.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30768.
avatar richard67
richard67 - comment - 26 Sep 2020

I did not understand the PHPCS link.

@ceford For every pull request, automatic tests like e.g. unit tests or system tests are run, and also code style tests (PHP, javasript, scss, ...). The results to the tests can be found on GitHub at the bottom of each PR in a section for the checks. If all checks were successfukll, that box is hidden, but it can be expanded with the link "Show all checks". There you see all checks, failed ones marked with a red mark. The link "Details" right beside each check leads to a page with a log file, where you can see at the left hand side which step of a test has failed, in this case it was "PHPCS". By clicking on the failed test step you can see then details about the failures. I had linked the log of the PHPCS test for Nicola so he can check and fix. Proper code style is necessary for good maintenance, and passing the PHPCS test is also necessary for later running the (more important) unit and system tests.

avatar alikon alikon - change - 26 Sep 2020
The description was changed
avatar alikon alikon - edited - 26 Sep 2020
avatar sandewt
sandewt - comment - 26 Sep 2020

Skipping pause, as previous batch had a very low processing time (0.134s < 1s)

Pause = 10

#30768

Pause = 0.1

#30768-2

Pause is skipping. How should I interpret this?

avatar alikon
alikon - comment - 26 Sep 2020

pause is skipping cause the elab time is already very low (0.1 sec)

avatar sandewt
sandewt - comment - 26 Sep 2020

pause is skipping cause the elab time is already very low (0.1 sec)

#30768-3

Indeed, works so properly.

avatar sandewt sandewt - test_item - 26 Sep 2020 - Tested successfully
avatar sandewt
sandewt - comment - 26 Sep 2020

I have tested this item successfully on 2b42e5e

In the testing instructions the before test is incomplete - it should be php cli/finder_indexer.php --purge

Tested before with:
php cli/finder_indexer.php --purge

Joomla! 4.0.0-beta5-dev Development [ Mañana ] 15-September-2020 19:15 GMT


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

avatar alikon alikon - change - 26 Sep 2020
Status Pending Ready to Commit
avatar alikon
alikon - comment - 26 Sep 2020

RTC


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

avatar HLeithner HLeithner - change - 26 Sep 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-09-26 11:42:01
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 26 Sep 2020
avatar HLeithner HLeithner - merge - 26 Sep 2020
avatar HLeithner
HLeithner - comment - 26 Sep 2020

Thanks

avatar HLeithner
HLeithner - comment - 26 Sep 2020

@alikon can you update the documentation? and the upgrade site for J4, should be too much I think it's updating the filename and parameters in the original documentation and in upgrade path.

avatar ceford
ceford - comment - 26 Sep 2020

I did not understand the PHPCS link.

@ceford For every pull request, automatic tests like

@richard67 Once again, thank you for the explanation - I see it now and have read a few more articles on testing.

Add a Comment

Login with GitHub to post a comment