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.

Sign up for GitHub

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

Jump to bottom

Go through behave tests and make sure they are valid/useful #498

Closed
Mletter1 opened this issue Sep 30, 2015 · 7 comments
Closed

Go through behave tests and make sure they are valid/useful #498

Mletter1 opened this issue Sep 30, 2015 · 7 comments
Assignees
@Mletter1
Labels
enhancement Medium Priority

Comments

@Mletter1
Copy link
Collaborator

currently we do not know the validity of the unit tests as they were written by someone who was out of scope of the project as a whole.
@Mletter1
Copy link
Collaborator Author

this closes #490

@Mletter1 Mletter1 mentioned this issue Sep 30, 2015
Closed
@Mletter1 Mletter1 self-assigned this Sep 30, 2015
@Mletter1
Copy link
Collaborator Author

slycat-project.feature was review and does test creation of a model, nothing to remove

@Mletter1 Mletter1 reopened this Nov 10, 2015
@Mletter1
Copy link
Collaborator Author

slycat-wed-server.feature was reviewed and tests crud operation from the main page of the slycat, nothing needs to be removed tests look good

@Mletter1
Copy link
Collaborator Author

not these are the tests being reviewed

  • agent
  • hyperchunks
  • rest-api
  • slycat-web-server
  • slycat-project

@Mletter1
Copy link
Collaborator Author

how to run test bed

behave -i "(agent|hyperchunks|rest-api|slycat-web-server|slycat-project)"

@Mletter1
Copy link
Collaborator Author

review so far

  • agent
  • hyperchunks
  • rest-api
  • slycat-web-server
  • slycat-project

@Mletter1
Copy link
Collaborator Author

the tests themselves are good looking just need to build a new docker image now that runs them

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Mletter1 Mletter1

Labels
enhancement Medium Priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Mletter1