Opened 6 years ago

Closed 6 years ago

#1705 closed defect (done)

Remove solution folders

Reported by: ascheibe Owned by: ascheibe
Priority: medium Milestone: HeuristicLab 3.3.6
Component: General Version: 3.3.6
Keywords: Cc:

Description

The HeuristicLab solutions shouldn't contain solution folders because Visual Studio Express Edition can't open them.

Change History (9)

comment:1 Changed 6 years ago by ascheibe

  • Status changed from new to assigned

comment:2 Changed 6 years ago by ascheibe

r7170 removed all solution folders except the Solution Items folder

comment:3 Changed 6 years ago by ascheibe

  • Owner changed from ascheibe to mkommend
  • Status changed from assigned to reviewing

I have tested the solutions with Visual Studio Express. When opening the HeuristicLab 3.3 solution the Express edition shows an error dialog because it can't handle the test project as well as the Solution Items Folder. But after the warning the solution can be built and everything works fine. The reason why we can't get grid of the Solution Items folder is that the .vsmdi file is added automatically to solutions which contain unit tests.

The build command also fails for the HeuristicLab 3.3 solution because it tries to build the test project, which fails, because the Visual Studio Test assemblies aren't included in the Express edition.

comment:4 Changed 6 years ago by swagner

Should we move the test project into a separate solution? Maybe this leads to some complications on the build server, but I would prefer a clean separation of the tests.

comment:5 Changed 6 years ago by mkommend

  • Owner changed from mkommend to ascheibe
  • Status changed from reviewing to assigned

I would prefer to leave the test project in the trunk solution, to have an easy access to all the unit tests. It should be possible to unload the test project, when working with Visual Studio Express and so the build command should also succeed.

Andreas could you please test, if this would be a solution.

comment:6 Changed 6 years ago by mkommend

Further more we should discuss a way to have the unit tests available for the developed branches, but this is another issue that should be handled in a separate ticket.

comment:7 Changed 6 years ago by ascheibe

  • Status changed from assigned to reviewing

Using the build cmd after removing the test project from the solution works with VS Express edition. Extracting the unit test from the main solution to an own solution and using the build cmd to run the tests will be further discussed for the next release.

comment:8 Changed 6 years ago by ascheibe

  • Status changed from reviewing to readytorelease

comment:9 Changed 6 years ago by swagner

  • Resolution set to done
  • Status changed from readytorelease to closed
  • Version changed from 3.3.5 to 3.3.6
Note: See TracTickets for help on using tickets.