Skip to content
This repository was archived by the owner on Aug 26, 2024. It is now read-only.

Seeding Report: Test Tray Seeding Summary Table #34 - #173

Open
theahegde wants to merge 3 commits into
DickinsonCollege:mainfrom
teamanaf:thea
Open

Seeding Report: Test Tray Seeding Summary Table #34#173
theahegde wants to merge 3 commits into
DickinsonCollege:mainfrom
teamanaf:thea

Conversation

@theahegde

Copy link
Copy Markdown

Pull Request Description

addresses issue #34


Licensing Certification

FarmData2 is a Free Cultural Work and all accepted contributions are licensed as described in the LICENSE.md file. This requires that the contributor holds the rights to do so. By submitting this pull request I certify that I satisfy the terms of the Developer Certificate of Origin for its contents.

@anlai46

anlai46 commented May 3, 2024

Copy link
Copy Markdown

Overall our team's code successfully checks whether the summary table appears when it's supposed to. Is there any part of the code that can be refactored or improved in any way to make it more maintainable? Also, I think it's still possible that the test might not handle everything because there is possibilities of network or server issues occurring that mess up the testing process. So in situations like that, how can we prepare for those unexpected errors? -Team Anaf

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants