Skip to content

Initial tests#10

Open
kkotenko wants to merge 1 commit intomentalisttraceur:mainfrom
kkotenko:tests/initial_tests
Open

Initial tests#10
kkotenko wants to merge 1 commit intomentalisttraceur:mainfrom
kkotenko:tests/initial_tests

Conversation

@kkotenko
Copy link

This adds some tests to the dependency functionality (mostly because I wanted to see how the dependency on suites behaves, exactly.)

Unfortunately, I was unable to figure out how to let multiple suites run programmatically, so I am delegating that to a CLI execution, which only gives me the return code (i.e. number of failed tests, effectively). I hope someone can find a better way.

I was also unable to verify the issued warnings, as I am not that deeply familiar with the internals of robot.

@kkotenko kkotenko force-pushed the tests/initial_tests branch from 676ed08 to 1d14544 Compare April 24, 2024 10:34
@mentalisttraceur
Copy link
Owner

mentalisttraceur commented Jun 1, 2024

Thank you for this! We definitely should add some tests, and this is a great start!

Sorry it's taken so long for me to get to this. I've now taken a closer look and first impressions are good.

Now I just gotta make some time to pull this down and play with it a bit - I'm not sure yet when that will happen, but I'll go ahead and merge it as soon as I've had a chance to do that, and then we can improve on it as time+motivation+priority permits.

@kkotenko
Copy link
Author

kkotenko commented Jun 2, 2024

No worries!

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 this pull request may close these issues.

2 participants