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

Create tests for TEST_FINISHED_NAMED in verifier #74

Open
sagatowski opened this issue Aug 6, 2019 · 4 comments
Open

Create tests for TEST_FINISHED_NAMED in verifier #74

sagatowski opened this issue Aug 6, 2019 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@sagatowski
Copy link
Member

According to issue #38 in TcUnit: #38
a new function has been added called TEST_FINISHED_NAMED.

Add tests in the TcUnit-Verifier to verify that this works as expected

@slacAWallace
Copy link
Contributor

slacAWallace commented Aug 12, 2019

@sagatowski I tried using this today, and I am afraid to say I ran into issues. Are there any installation requirements, packages, or other things I need to add to an environment before this will work?

When I run this in power-shell I get:
.\TcUnit-Verifier.exe -v "C:\Users\...\Documents\Source\TcUnit-Verifier\TcUnit-Verifier_TwinCAT\TcUnit -Verifier_TwinCAT.sln" 2019-08-12 23:57:52 - Starting TcUnit-Verifier... 2019-08-12 23:57:52 - Loading the Visual Studio Development Tools Environment (DTE)... 2019-08-12 23:57:52 - Error loading VS DTE. Is the correct version of Visual Studio installed? 2019-08-12 23:57:52 - Closing the Visual Studio Development Tools Environment (DTE), please wait... 2019-08-12 23:58:12 - Exiting application... PS C:\Users\...\Documents\Source\TcUnit-Verifier\TcUnit-Verifier_DotNet\TcUnit-Verifier\bin\Release>

@sagatowski
Copy link
Member Author

sagatowski commented Aug 13, 2019

@slacAWallace the problem was that TcUnit-Verifier_TwinCAT required you to have VS2015 installed (as this was the version used to develop it). The visual studio DTE (development tools environment) couldn't find it in your machine and thus you got the error.
An issue was recently created regarding this topic, Issue #44, which I just solved.
Now all software in TcUnit (TcUnit + TcUnit-Verifier_TwinCAT and TcUnit-Verifier_DotNet) is developed using VS2013. The contributing.MD has been updated and should answer any remaining questions regarding this.

@slacAWallace
Copy link
Contributor

@sagatowski I am worried about this. I just tried a bit to get VS 2013 community and had trouble.

Do we simply need VS-shell 2013 to make this work and we can continue to work in 2017/19 or whatever?

@sagatowski
Copy link
Member Author

@slacAWallace What was the problem with the download? I downloaded it yesterday and it went smooth.

The problem is that it's not possible to mix versions of Visual Studio for the three different projects. It's not possible to open a project made in a newer version in an older. It should work to open in a newer, but the risk is that VS overwrites the solution version information (xyz.sln), and then it won't be possible to open in an older again. You can try to see if this happens though. If VS leaves the sln-file alone it should be fine.

@sagatowski sagatowski transferred this issue from another repository Dec 31, 2019
@sagatowski sagatowski added the enhancement New feature or request label Jan 8, 2020
@sagatowski sagatowski changed the title Create tests for TEST_FINISHED_NAMED Create tests for TEST_FINISHED_NAMED in verifier Jan 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants