Skip to content

Conversation

@JosephPotapenko
Copy link

Worked on this with @steeley21.

JosephPotapenko and others added 6 commits November 8, 2025 20:01
…hods. Made sure to make sure the copy is set to the newest in the csproj of Assignment. Wrote tests for CSVRows and SampleData and implemented TDD as I worked.
…and contains here and there to try to fulfill that requirement. Tried to ensure generics. Visual studion was upset at me for using .ToArray initially, giving suggestions to use [.. } notation, but since Mark mentioned in class to not use it yet I did it back to what I coded at first.
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements LINQ-based data processing exercises (Assignments 7+8), focusing on CSV file manipulation, LINQ query operations, and implementing IEnumerable<T> on a custom Node<T> class.

  • Implements LINQ-based CSV parsing and filtering operations in SampleData class
  • Adds Node<T> class with ICollection<T> implementation and ChildItems method
  • Upgrades to .NET 9.0 with comprehensive test coverage using MSTest v4

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
README.md Updated assignment completion status with check marks and identified spelling errors
Assignment/Directory.Build.props Upgraded to net9.0 and added LangVersion=latest
Assignment/Directory.Build.targets Added empty targets file for future shared items
Assignment/Assignment/SampleData.cs Implemented CSV parsing and LINQ operations for data filtering and aggregation
Assignment/Assignment/Node.cs Implemented circular linked list with ICollection and ChildItems method
Assignment/Assignment/Assignment.csproj Configured People.csv to copy to output directory
Assignment/Assignment.Tests/SampleDataTests.cs Added comprehensive tests for SampleData LINQ operations
Assignment/Assignment.Tests/NodeTests.cs Added tests for Node class functionality including enumeration
Assignment/Assignment.Tests/CsvRowsTests.cs Added tests verifying CSV file loading and resource disposal
Assignment/Assignment.Tests/Assignment.Tests.csproj Added project references and package dependencies with invalid SDK version

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@thigiang16 thigiang16 left a comment

Choose a reason for hiding this comment

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

Implement the ISampleData.CsvRows property, loading the data from the People.csv file and returning each line as a single string. ✔

Change the "Copy to" property on People.csv to "Copy if newer" so that the file is deployed along with your test project. ❌
Using LINQ, skip the first row in the People.csv. ✔
Be sure to appropriately handle resource (IDisposable) items correctly if applicable (and it may not be depending on how you implement it). ✔
Implement IEnumerable GetUniqueSortedListOfStatesGivenCsvRows() to return a sorted, unique list of states. ✔

Use ISampleData.CsvRows for your data source. ✔
Don't forget the list should be unique. ✔
Sort the list alphabetically. ✔
Include a test that leverages a hardcoded list of addresses. ✔
Include a test that uses LINQ to verify the data is sorted correctly (do not use a hardcoded list). ✔
Implement ISampleData.GetAggregateSortedListOfStatesUsingCsvRows() to return a string that contains a unique, comma separated list of states. ✔

Use ISampleData.GetUniqueSortedListOfStatesGivenCsvRows() for your data source. ✔
Consider "selecting" only the states and calling ToArray() to retrieve an array of all the state names. ✔
Given the array, consider using string.Join to combine the list into a single string.✔
Implement the ISampleData.People property to return all the items in People.csv as Person objects ✔

Use ISampleData.CsvRows as the source of the data. ✔
Sort the list by State, City, and Zip. ✔
Be sure that Person.Address is also populated. ✔
Adding null validation to all the Person and Address properties is optional.
Consider using ISampleData.CsvRows in your test to verify your results. ✔
Implement ISampleDate.FilterByEmailAddress(Predicate filter) to return a list of names where the email address matches the filter. ✔

Use ISampleData.People for your data source. ✔
Implement ISampleData.GetAggregateListOfStatesGivenPeopleCollection(IEnumerable people) to return a string that contains a unique, comma-separated list of states. ✔

Use the people parameter from ISampleData.People property for your data source. ✔
At a minimum, use the System.Linq.Enumerable.Aggregate` LINQ method to create your result. ✔
Don't forget the list should be unique. ✔
It is recommended that, at a minimum, you use ISampleData.GetUniqueSortedListOfStatesGivenCsvRows to validate your result.
Given the implementation of Node in Assignment5

Implement IEnumerable to return all the items in the "circle" of items. ✔
Add an IEnumberable ChildItems(int maximum) method to Node that returns the remaining items with a maximum number of items returned less than maximum.
Extra Credit
Implement the homework using async/await and multi-threading by defining a new SampleDataAsync class that implements IAsyncSampleData). Refactor your SampleData and SampleDataAsync classes with minimal duplication. Be sure to refactor your tests to re-use a significant amount of the test code for both implementations. ❌
Fundamentals
Place all shared project properties into a Directory.Build.Props file.
Place all shared project items into a Directory.Build.targets file.
Ensure nullable reference types is enabled ✔
Ensure that you turn on code analysis for all projects(EnableNETAnalyzers) ✔
Set LangVersion and the TargetFramework to the latest released versions available (preview versions optional) ✔
and enabled .NET analyzers for both projects ✔
For this assignment, consider using Assert.AreEqual() (the generic version) ✔
All of the above should be unit tested ❌
Choose simplicity over complexity ✔

@github-actions
Copy link

Summary

Summary
Generated on: 11/20/2025 - 02:57:27
Coverage date: 11/20/2025 - 02:57:25
Parser: MultiReport (2x Cobertura)
Assemblies: 1
Classes: 4
Files: 4
Line coverage: 88.3% (159 of 180)
Covered lines: 159
Uncovered lines: 21
Coverable lines: 180
Total lines: 363
Branch coverage: 80% (48 of 60)
Covered branches: 48
Total branches: 60
Method coverage: Feature is only available for sponsors
Tag: 464_19523951453

Coverage

Assignment - 88.3%
Name Line Branch
Assignment 88.3% 80%
Assignment.Address 100%
Assignment.Node`1 84% 81.4%
Assignment.Person 100%
Assignment.SampleData 93.3% 66.6%

Copy link
Collaborator

@Joshua-Lester3 Joshua-Lester3 left a comment

Choose a reason for hiding this comment

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

Instructions

Throughout, consider using the System.Linq.Enumerable methods Zip, Count, Sort and Contains methods for testing collections.. (Preferably avoid using Microsoft.VisualStudio.TestTools.UnitTesting.CollectionAssert although that might be easier, to get a firmer grasp on additional LINQ API.)

  1. Implement the ISampleData.CsvRows property, loading the data from the People.csv file and returning each line as a single string. ✔

    • Change the "Copy to" property on People.csv to "Copy if newer" so that the file is deployed along with your test project. ✔
    • Using LINQ, skip the first row in the People.csv. ✔
    • Be sure to appropriately handle resource (IDisposable) items correctly if applicable (and it may not be depending on how you implement it). ✔
  2. Implement IEnumerable<string> GetUniqueSortedListOfStatesGivenCsvRows() to return a sorted, unique list of states. ✔

    • Use ISampleData.CsvRows for your data source. ✔
    • Don't forget the list should be unique. ✔
    • Sort the list alphabetically. ✔
    • Include a test that leverages a hardcoded list of addresses. ✔
    • Include a test that uses LINQ to verify the data is sorted correctly (do not use a hardcoded list). ✔
  3. Implement ISampleData.GetAggregateSortedListOfStatesUsingCsvRows() to return a string that contains a unique, comma separated list of states. ✔

    • Use ISampleData.GetUniqueSortedListOfStatesGivenCsvRows() for your data source. ✔
    • Consider "selecting" only the states and calling ToArray() to retrieve an array of all the state names. ✔
    • Given the array, consider using string.Join to combine the list into a single string. ✔
  4. Implement the ISampleData.People property to return all the items in People.csv as Person objects ✔

    • Use ISampleData.CsvRows as the source of the data. ✔
    • Sort the list by State, City, and Zip. ✔
    • Be sure that Person.Address is also populated. ✔
    • Adding null validation to all the Person and Address properties is optional.
    • Consider using ISampleData.CsvRows in your test to verify your results. ✔
  5. Implement ISampleDate.FilterByEmailAddress(Predicate<string> filter) to return a list of names where the email address matches the filter. ✔

    • Use ISampleData.People for your data source. ✔
  6. Implement ISampleData.GetAggregateListOfStatesGivenPeopleCollection(IEnumerable<IPerson> people) to return a string that contains a unique, comma-separated list of states. ✔

    • Use the people parameter from ISampleData.People property for your data source. ✔
    • At a minimum, use the System.Linq.Enumerable.Aggregate` LINQ method to create your result. ✔
    • Don't forget the list should be unique. ✔
    • It is recommended that, at a minimum, you use ISampleData.GetUniqueSortedListOfStatesGivenCsvRows to validate your result.
  7. Given the implementation of Node in Assignment5

  • Implement IEnumerable<T> to return all the items in the "circle" of items. ✔
  • Add an IEnumberable<T> ChildItems(int maximum) method to Node that returns the remaining items with a maximum number of items returned less than maximum.

Fundamentals

  • Place all shared project properties into a Directory.Build.Props file.
  • Place all shared project items into a Directory.Build.targets file.
  • Ensure nullable reference types is enabled ✔
  • Ensure that you turn on code analysis for all projects(EnableNETAnalyzers) ✔
  • Set LangVersion and the TargetFramework to the latest released versions available (preview versions optional) ✔
  • and enabled .NET analyzers for both projects ✔
  • For this assignment, consider using Assert.AreEqual<T>() (the generic version) ✔
  • All of the above should be unit tested ✔
  • Choose simplicity over complexity ✔

Looks good!

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.

4 participants