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

Clean up step definitions #640

Closed
mvz opened this issue May 31, 2019 · 1 comment · Fixed by #666
Closed

Clean up step definitions #640

mvz opened this issue May 31, 2019 · 1 comment · Fixed by #666

Comments

@mvz
Copy link
Contributor

mvz commented May 31, 2019

Summary

Some of Aruba's step definitions are very complex and have a lot of options. We should clean them up.

Expected Behavior

Step definitions should be clear easy to follow.

Current Behavior

Some step definitions have an awful lot of complex logic, making it hard to see what's going on. Especially when indirection is used to activate matchers.

Possible Solution

We can at least split up some of the cases where the actions taken depend on some conditional based on a string literal.

Context & Motivation

This came up in a pull request. See #638 (comment).

@mvz mvz mentioned this issue Jun 10, 2019
8 tasks
@luke-hill
Copy link
Contributor

I'm gonna have a partial tackle at this, I saw a couple of places we could get some quick wins.

It won't close this issue out, but it'll definitely make some of the triaging a bit easier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants