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

Put tests execution out of Excel Application Scope #34

Open
jestinga opened this issue Jun 28, 2021 · 3 comments
Open

Put tests execution out of Excel Application Scope #34

jestinga opened this issue Jun 28, 2021 · 3 comments

Comments

@jestinga
Copy link

Hello,

I had some issues using the tests features with workflows using Excel applications (for example testing a KillAllProcesses workflow that kills Excel processes)

I edited the workflow to only put "Read Range" and "Write Range" activity into "Excel Application Scope" and it seems to work perfectly.

Does the executions of the tests needs to be in the Excel Application Scope?

@Richard5Chan
Copy link

Richard5Chan commented Jun 28, 2021 via email

@jestinga
Copy link
Author

Thanks for your answer.

I may have not been very clear.

The issue I encountered was that all the executions of tests are done within an Excel Application Scope (by default). So during the execution of tests the file "Tests.xlsx" remains open. This can produce errors if - for example - the workflows I am testing kills all Excel processes (as a KillAllProcesses workflow would do).

As the Excel Application Scope are only used to read workflows to test at the begining and write tests result at the end: Wouldn't it be better to close the file "Tests.xlsx" during the execution of tests to prevent this kind of errors ?

Here is how I think the workflow "RunAllTests" should look like
(instead of wrapping the whole tests execution within one "Excel Application Scope")

@Richard5Chan
Copy link

Richard5Chan commented Jul 1, 2021 via email

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

No branches or pull requests

2 participants