Ticket #2172 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Ensure classnames of unit tests match the filename

Reported by: Stuart Campbell Owned by: Peter Peterson
Priority: major Milestone: Iteration 27
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Janik Zikovsky

Description

As we get more and more tests it is difficult to find a test that has a different name than the filename it is in.

Change History

comment:1 Changed 10 years ago by Stuart Campbell

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Peter Peterson

  • Owner changed from Stuart Campbell to Peter Peterson

comment:3 Changed 10 years ago by Peter Peterson

(In [8567]) Renamed tests. Refs #2172 and #1881.

comment:4 Changed 10 years ago by Peter Peterson

(In [8571]) Renamed tests. Refs #2172 and #1881.

comment:5 Changed 10 years ago by Peter Peterson

(In [8573]) Renamed tests. Refs #2172 and #1881.

comment:6 Changed 10 years ago by Peter Peterson

(In [8574]) Renamed tests. Refs #2172 and #1881.

comment:7 Changed 10 years ago by Peter Peterson

(In [8575]) Renamed tests. Refs #2172 and #1881.

comment:8 Changed 10 years ago by Stuart Campbell

(In [8577]) Made test filename match classname. refs #2172 & #1881 & #2180

comment:9 Changed 10 years ago by Peter Peterson

(In [8588]) Renaming more tests so they run. Refs #1881 and #2172.

comment:10 Changed 10 years ago by Peter Peterson

(In [8591]) Refs #1881 and #2172.

comment:11 Changed 10 years ago by Peter Peterson

(In [8599]) Refs #1881 and #2172.

comment:12 Changed 10 years ago by Peter Peterson

(In [8621]) Refs #1881 and #2172.

comment:13 Changed 10 years ago by Peter Peterson

(In [8627]) Putting more tests in. Refs #1881 and #2172.

comment:14 Changed 10 years ago by Peter Peterson

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

The last commit finished this.

comment:15 Changed 10 years ago by Janik Zikovsky

  • Status changed from verify to verifying
  • Tester set to Janik Zikovsky

comment:16 Changed 10 years ago by Janik Zikovsky

  • Status changed from verifying to closed

My TestViewer gives warnings about this automatically. All are good except for the following two tests have bad names, but they are in MD* so I guess we should expect that. 

Warning! Source file for test testCPcomponents not found: /home/8oz/Code/Mantid/Code/Mantid/Framework/MDAlgorithms/test/testCPcomponents.h
Warning! Source file for test MDFileTestDataGeneratorTest not found: /home/8oz/Code/Mantid/Code/Mantid/Framework/MDDataObjects/test/MDFileTestDataGeneratorTest.h

Pass!

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​3019

Note: See TracTickets for help on using tickets.