cu-ecen-aeld / assignment-autotest

Auto test code for assignments, using the Unity automated test framework
MIT License
3 stars 20 forks source link

Update Sockettest.sh to send multiple packets per connection #26

Closed DeekshithPatil closed 1 month ago

DeekshithPatil commented 1 year ago

Created a new branch and started a pull request here: https://github.com/cu-ecen-aeld/assignment-autotest/pull/25

Need to update the script as per Dan's suggestions made on the pull-request.

DeekshithPatil commented 1 year ago

Updated branch as per Dan's comments. Ready for review.

dwalkes commented 1 year ago

Thanks @DeekshithPatil

I thought we needed changes to the assignment private source code to correctly handle this? Are these changes on a branch on assignments-complete-private as well?

I'd like to see a passing unit test for a branch on assignments-complete-private with assignment-autotest submodule updated to point to this branch and source code updated to pass this test.