Ticket #4175 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Add systemtest for UserAlgorithms build script

Reported by: Anders Markvardsen Owned by: Karl Palmen
Priority: critical Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Anders Markvardsen

Description

Add a system test which tests that the build script in the UserAlgorithms install directory compiles successfully (if Mantid is installed in c:\MantidInstall, this is the script in the c:\MantidInstall\UserAlgorithms directorty)

For example this build script will be broken when the Third_Party libraries structure updated (as e.g. happenned end of last week) and, which is easy to forget, the installer directory is not updated to reflect any such change.

Change History

comment:1 Changed 9 years ago by Nick Draper

  • Owner set to Anyone
  • Status changed from new to assigned

comment:2 Changed 9 years ago by Anders Markvardsen

Martyn's suggestion for this one is:

1) Restrict this systemtest to Windows

2) A systemtest compare itself against a reference file. Have this reference file to be blank (i.e. empty) and if the UserAlgorithm build script is broken then those error messages are compared with the blank file, and this errors with be visible from jenkins

comment:3 Changed 9 years ago by Anders Markvardsen

  • Owner changed from Anyone to Karl Palmen

comment:4 Changed 9 years ago by Karl Palmen

Added User Algorithm Build Test re #4175

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: 93ef608c150b8ff0682ec0924409338a4094c786

comment:5 Changed 9 years ago by Karl Palmen

Modified Build.bat file for system test re #4175

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: e753e545717ea2e9074a9fcef83b56e020446d9d

comment:6 Changed 9 years ago by Karl Palmen

Enable Windows installer to install all features re #4175

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: e106c17547b9e86564cbf976f99c8ed60258e000

comment:7 Changed 9 years ago by Karl Palmen

  • Status changed from assigned to accepted

comment:8 Changed 9 years ago by Karl Palmen

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

comment:9 Changed 9 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:10 Changed 9 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Looks good

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5022

Note: See TracTickets for help on using tickets.