Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 26, 2025

✅ COMPLETED: Rename/Move NET 7 Tests to Framework-Independent Location

Task Successfully Completed:

The NET 7 tests have been successfully renamed and moved to make them framework-independent as requested.

Changes Made:

  • Renamed folder: tests/E2E Tests/NET 7 tests/tests/E2E Tests/ThreadingTests/
  • Renamed test class: Net7ThreadingTestThreadingTest
  • Renamed test file: Net7ThreadingTest.csThreadingTest.cs
  • Updated constructor: Net7ThreadingTest()ThreadingTest()
  • Updated solution file: All 3 project references updated to new paths
  • Preserved project structure: Both IntegrationTests and WebApp projects maintained
  • Reverted unintended changes: Restored Microsoft.Identity.Web.OWIN.xml to original state

Technical Details:

  • Framework targeting: Projects already target .NET 8.0 (not .NET 7)
  • Test functionality: Threading/concurrency test that sends 100 concurrent HTTP requests
  • New naming: "ThreadingTests" better describes the actual test purpose
  • Framework independence: No version-specific references in folder or class names

Verification Results:

  • ✅ Individual project builds successfully
  • ✅ Threading test runs and passes (ThreadingTest.Get_EndpointsReturnUnauthorizedEveryTimeAsync)
  • ✅ No remaining references to old names in codebase
  • ✅ Solution file structure and nesting preserved
  • ✅ Framework-independent naming achieved
  • ✅ Unintended file changes reverted

Summary:

The task has been completed successfully with minimal changes. The tests are now framework-independent and properly named to reflect their actual functionality (threading/concurrency testing) rather than being tied to a specific .NET version. Unintended formatting changes to the OWIN XML documentation file have been reverted.

Original prompt

So we own this repo. We have stopped supporting dotnet 7. There are tests in this folder tests/E2E Tests/NET 7 tests. We need to rename this or pobably move them somewhere else so that they become framework independent. Can you please try to do it for me?


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title So we own this repo. We have stopped supporting dotnet 7. There are tests in this folder tests/E2E Tests/NET 7 tests. We need to rename this or pobably move them somewhere else so that they become framework independent. Can you please try to do it for me? Rename NET 7 tests to ThreadingTests for framework independence Sep 26, 2025
Copilot finished work on behalf of saurabhsathe-ms September 26, 2025 21:12
@saurabhsathe-ms saurabhsathe-ms marked this pull request as ready for review September 26, 2025 23:04
@saurabhsathe-ms saurabhsathe-ms requested a review from a team as a code owner September 26, 2025 23:04
Copy link
Collaborator

@jmprieur jmprieur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Thanks!

Copilot finished work on behalf of keegan-caruso September 26, 2025 23:13
Copilot AI requested a review from keegan-caruso September 26, 2025 23:13
This was referenced Oct 24, 2025
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.

5 participants