Ticket #5257 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

System test installer script needs to uninstall at the end

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.2
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

We now have system tests builds for both master and the release branch. One issue that has cropped up is that if, say for example, the release branch tests run first and then master tries to run it can't install the package because the version number is lower so it says a newer is already installed.

The system test script should uninstall the package afterwards to leave the machine in a cleaner state.

Change History

comment:1 Changed 8 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Martyn Gigg

InstallerTests now uninstalls at the end. Refs #5257

Only if it did the installing though. This is required so that the release & dev package system tests wil run properly while the names are not correctly changed.

Changeset: 74101f063ba9a38a14a3bfffbc346e8f93715514

comment:3 Changed 8 years ago by Martyn Gigg

Put back commented code. Refs #5257

Changeset: 55e6d11bd78646e57ab2dc69f1b3b5fc90b627be

comment:4 Changed 8 years ago by Martyn Gigg

Fix for attribute name. Refs #5257

Changeset: fe79b44598891b9c716ffd495531b77fee0ccec4

comment:5 Changed 8 years ago by Martyn Gigg

Track errors more easily in installertests script. Refs #5257

Changeset: caeffe9f3f71e8cb7ea07b1e6691b1fd03b5c1ca

comment:6 Changed 8 years ago by Martyn Gigg

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:7 Changed 8 years ago by Martyn Gigg

Add self argument to do_uninstall for Windows. Refs #5257

Changeset: d907c3cd5ad6dfcfd444ddee7e31cd72048ed3c7

comment:8 Changed 8 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:9 Changed 8 years ago by Russell Taylor

  • Status changed from verifying to closed

Tested on Mac & Windows and by observing that Mantid is not being left installed on mantidlx1

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6103

Note: See TracTickets for help on using tickets.