[Bugzilla] [Bug 14261] Samuel GOUGEON has changed bug_status, assigned_to

[bugzilla.admin at scilab.org]

http://bugzilla.scilab.org/show_bug.cgi?id=14261
--------- Bug Summary -----------
Running tests for the "atoms" modules changes the repository used by atoms, makes it unusable

---------- Changed by ------------
sgougeon at free.fr

---------- What changed ----------
bug_status, assigned_to


Samuel GOUGEON <sgougeon at free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |NEW
           Assignee|sgougeon at free.fr            |bugzilla.admin at scilab.org

--- Comment #5 from Samuel GOUGEON <sgougeon at free.fr> ---
Looking at this bug to fix it, here are some additional comments:

>the issue happens when running just one test:

* "http://scene10.6.0.test.atoms.scilab.org" is set in all the following tests :
atomsAutoloadAdd.tst
atomsAutoloadDel.tst
atomsDepTreeShow.tst
atomsInstall.tst
atomsInstallList.tst
atomsRemoveList.tst
atomsRepositoryAdd.tst
atomsRepositoryDel.tst
atomsShow.tst
atomsUpdate.tst

* atomsSaveConfig() and atomsRestoreConfig() should handle both repositories lists (user and allusers), in addition configuration parameters.
 These lists should be seen as part of the configuration.

* atomsRepositorySetOfl() that sets "http://scene10.6.0.test.atoms.scilab.org" is an internal and so is not documented.
  It puzzlingly looks like a weak duplicate of atomsRepositoryAdd().
  Anyway, it makes no backup of the repository list that it overwrites.

* Presently, in the tests, atomsRepositorySetOfl() is used at the end to restore the former repositories list.
  But this list is a "factory" one taken from SCI, not from SCIHOME. So if the contributor running tests has priorly set other repositories than only
the official Scilab one, they are not restored.

* Finally, "http://scene10.6.0.test.atoms.scilab.org" is not reachable from outside. So tests can't be run by a contributor external to S/E.
  And this bug can't be fixed by a contributor.

IMO, 
* atomsSaveConfig() and atomsRestoreConfig() should be upgraded in order to manage lists of repositories. This will likely require a SEP.
* A public sandbox should be created on the ATOMS server, in order to replaced "http://scene10.6.0.test.atoms.scilab.org" or to make it public and
usable by atoms tests.
* Then, files.tst should be upgraded

-- 
You are receiving this mail because:
You are watching all bug changes.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.scilab.org/pipermail/bugzilla/attachments/20180315/583bde9a/attachment.htm>