-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Fixed multi parameter bug #2743
Fixed multi parameter bug #2743
Conversation
cmd_def = CmdDef() | ||
cmd_def.add('test', 0, par_sched) | ||
|
||
actual = cmd_def.get('test', 0, 0.01, 0.02, 0.03) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closes #2741
self.assertEqual(actual.start_time, expected.start_time) | ||
self.assertEqual(actual.stop_time, expected.stop_time) | ||
|
||
self.assertEqual(cmd_def.get_parameters('test', 0), ('x', 'y', 'z')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closes #2742
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for jumping on this George! Looks good to me.
* Remove repeated parameters * Updated changelog * Update CHANGELOG.md * Adding test * Adding test of CmdDef arguments issue being fixed * Formatting * Formatting * Formatting * Formatting * Formatting
Summary
Closes #2742, and
possiblycloses #2741.Details and comments
I don't have a concise tests of this to add yet, so this is just a draft for now until I can find some.