Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The Axis base class is missing a unit test suite #7745

Closed
mantid-builder opened this issue Apr 22, 2013 · 3 comments
Closed

The Axis base class is missing a unit test suite #7745

mantid-builder opened this issue Apr 22, 2013 · 3 comments
Labels
Framework Issues and pull requests related to components in the Framework Low Priority Things that you don't ever want to be done. Maintenance Unassigned issues to be addressed in the next maintenance period.

Comments

@mantid-builder
Copy link
Collaborator

This issue was originally TRAC 6899

Original Reporter: Russell Taylor

Write one. The class is in API.


Keywords: Maintenance

@mantid-builder
Copy link
Collaborator Author

@NickDraper (2013-07-26T13:55:01):
Moved to backlog at the code freeze for R2.6


Keith Brown (2013-10-21T10:51:20):
As this is for an abstract class, I'll be creating a few new classes that inherit from it in the test class itself.

They'll probably be heavily based on TextAxis, SpectraAxis and NumericAxis


Russell Taylor (2013-10-21T14:22:24):
Consider using gmock for the abstract methods. Owen is probably the best person to talk to about this.


@NickDraper (2014-02-14T11:04:59):
Bulk move to assigned at the introduction of the triage step


Keith Brown (2014-06-26T13:07:14):
not going to get this done before i leave, leaving to anyone

@mantid-builder mantid-builder added Framework Issues and pull requests related to components in the Framework Maintenance Unassigned issues to be addressed in the next maintenance period. labels Jun 3, 2015
@Anders-Markvardsen Anders-Markvardsen added State: Review Required Low Priority Things that you don't ever want to be done. and removed State: Review Required labels Jul 10, 2015
@Anders-Markvardsen
Copy link
Member

Axis is already tested indirectly through child classes therefore changed this to low-priority

@NickDraper NickDraper added this to the Release 3.6 milestone Oct 2, 2015
@NickDraper NickDraper modified the milestones: Release 3.6, Release 3.7 Jan 22, 2016
@NickDraper NickDraper modified the milestones: Release 3.7, Release 3.8 May 20, 2016
@NickDraper NickDraper modified the milestones: Release 3.8, Release 3.9 Oct 3, 2016
@NickDraper NickDraper modified the milestones: Release 3.9, Temporary Holding Oct 14, 2016
@NickDraper
Copy link
Contributor

Archiving

@NickDraper NickDraper removed this from the Temporary Holding milestone Oct 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework Issues and pull requests related to components in the Framework Low Priority Things that you don't ever want to be done. Maintenance Unassigned issues to be addressed in the next maintenance period.
Projects
None yet
Development

No branches or pull requests

3 participants