Ticket #7292 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

LoadLotsOfFiles test is broken with .peaks files

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.6
Component: Diffraction Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description (last modified by Nick Draper) (diff)

Message from RHEL6: 

Loading '/home/mantidlog/jenkins/workspace/is_test_rhel6_develop/Data/TOPAZ_3007.peaks'
FAILED TO LOAD '/home/mantidlog/jenkins/workspace/is_test_rhel6_develop/Data/TOPAZ_3007.peaks' WITH ERROR:
'IPeaksWorkspace' object has no attribute 'getNumberHistograms'

Change History

comment:1 Changed 7 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 7 years ago by Martyn Gigg

Fix workspace type checks in LoadLotsOFiles. Refs #7292

It now checks whether a method exists on an object first before trying to use it.

Changeset: 25d056b93755a70bb16a208d24e3621525174242

comment:3 Changed 7 years ago by Martyn Gigg

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

THIS IS IN THE SYSTEMTESTS REPOSITORY

Branch: bugfix/7292_fix_load_lots_files_peaks

Tester: The LoadLotsOfFiles test should now be working again on develop. Either checkout develop locally and try and run the test or run a job manually here: ​http://download.mantidproject.org/jenkins/view/Manually%20Run%20Tests/job/isis_systemtests_rhel6/

comment:4 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:5 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

This looks to be working properly.

comment:6 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:7 Changed 7 years ago by Nick Draper

  • Component changed from Framework to Diffraction
  • Description modified (diff)

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8138

Note: See TracTickets for help on using tickets.