Support building launch assets for NET6 projects

Author: JoeRobich

src/assets.ts

@@ -161,7 +161,7 @@ export class AssetGenerator {
         const startupProjectDir = path.dirname(this.startupProject.Path);
         const relativeProjectDir = path.join('${workspaceFolder}', path.relative(this.workspaceFolder.uri.fsPath, startupProjectDir));
         const configurationName = 'Debug';
-        const targetFramework = protocol.findNetCoreAppTargetFramework(this.startupProject) ?? protocol.findNet5TargetFramework(this.startupProject);
+        const targetFramework = protocol.findNetCoreTargetFramework(this.startupProject);
         const result = path.join(relativeProjectDir, `bin/${configurationName}/${targetFramework.ShortName}/${this.startupProject.AssemblyName}.dll`);
         return result;
     }

src/omnisharp/protocol.ts

@@ -865,27 +865,32 @@ export function findNetFrameworkTargetFramework(project: MSBuildProject): Target
     return project.TargetFrameworks.find(tf => regexp.test(tf.ShortName));
 }
 
-export function findNet5TargetFramework(project: MSBuildProject): TargetFramework {
-    const targetFramework = project.TargetFrameworks.find(tf => tf.ShortName.startsWith('net5'));
-    // Temprorary workaround until changes to support the net5.0 TFM is settled. Some NuGet
-    // builds report the shortname as net50.
-    if (targetFramework !== undefined) {
-        targetFramework.ShortName = "net5.0";
-    }
-    return targetFramework;
+export function findNetCoreTargetFramework(project: MSBuildProject): TargetFramework {
+    return findNetCoreAppTargetFramework(project) ?? findModernNetFrameworkTargetFramework(project);
 }
 
 export function findNetCoreAppTargetFramework(project: MSBuildProject): TargetFramework {
     return project.TargetFrameworks.find(tf => tf.ShortName.startsWith('netcoreapp'));
 }
 
+export function findModernNetFrameworkTargetFramework(project: MSBuildProject): TargetFramework {
+    let regexp = new RegExp('^net[5-6]');
+    const targetFramework = project.TargetFrameworks.find(tf => regexp.test(tf.ShortName));
+
+    // Shortname is being reported as net50 instead of net5.0
+    if (targetFramework !== undefined && targetFramework.ShortName.charAt(4) !== ".") {
+        targetFramework.ShortName = targetFramework.ShortName.substr(0, 4) + "." + targetFramework.ShortName.substr(4);
+    }
+
+    return targetFramework;
+}
+
 export function findNetStandardTargetFramework(project: MSBuildProject): TargetFramework {
     return project.TargetFrameworks.find(tf => tf.ShortName.startsWith('netstandard'));
 }
 
 export function isDotNetCoreProject(project: MSBuildProject): Boolean {
-    return findNet5TargetFramework(project) !== undefined ||
-        findNetCoreAppTargetFramework(project) !== undefined ||
+    return findNetCoreTargetFramework(project) !== undefined ||
         findNetStandardTargetFramework(project) !== undefined ||
         findNetFrameworkTargetFramework(project) !== undefined;
 }
@@ -928,8 +933,7 @@ export function findExecutableMSBuildProjects(projects: MSBuildProject[]) {
     let result: MSBuildProject[] = [];
 
     projects.forEach(project => {
-        const projectIsNotNetFramework = findNetCoreAppTargetFramework(project) !== undefined
-            || findNet5TargetFramework(project) !== undefined
+        const projectIsNotNetFramework = findNetCoreTargetFramework(project) !== undefined
             || project.IsBlazorWebAssemblyStandalone;
 
         if (project.IsExe && projectIsNotNetFramework) {

test/featureTests/assets.test.ts

@@ -86,6 +86,19 @@ suite("Asset generation: csproj", () => {
         segments.should.deep.equal(['${workspaceFolder}', 'bin', 'Debug', 'net5.0', 'testApp.dll']);
     });
 
+    test("Create launch.json for NET 6 project opened in workspace", () => {
+        let rootPath = path.resolve('testRoot');
+        let info = createMSBuildWorkspaceInformation(path.join(rootPath, 'testApp.csproj'), 'testApp', 'net60', /*isExe*/ true);
+        let generator = new AssetGenerator(info, createMockWorkspaceFolder(rootPath));
+        generator.setStartupProject(0);
+        let launchJson = parse(generator.createLaunchJsonConfigurations(ProgramLaunchType.Console), undefined, { disallowComments: true });
+        let programPath = launchJson[0].program;
+
+        // ${workspaceFolder}/bin/Debug/net6.0/testApp.dll
+        let segments = programPath.split(path.posix.sep);
+        segments.should.deep.equal(['${workspaceFolder}', 'bin', 'Debug', 'net6.0', 'testApp.dll']);
+    });
+
     test("Create launch.json for nested project opened in workspace", () => {
         let rootPath = path.resolve('testRoot');
         let info = createMSBuildWorkspaceInformation(path.join(rootPath, 'nested', 'testApp.csproj'), 'testApp', 'netcoreapp1.0');